Skip to content

ci: add RHCOS version confirmation from job artifacts - #549

Merged
openshift-merge-bot[bot] merged 1 commit into
openshift-eng:mainfrom
petr-muller:trt-2612-rhcos-version-confirmation
Jun 16, 2026
Merged

openshift-merge-bot[bot] merged 1 commit into
openshift-eng:mainfrom
petr-muller:trt-2612-rhcos-version-confirmation

Conversation

@petr-muller

@petr-muller petr-muller commented Jun 16, 2026

Copy link
Copy Markdown
Member

Document how to confirm actual RHCOS version by inspecting .status.nodeInfo.osImage from nodes.json in gather-extra artifacts. Add Step 3b to test-failure and install-failure analysis skills with real examples from CI jobs.

🤖 Generated with Claude Code

Summary by CodeRabbit

Release Notes

  • Documentation

    • Added guidance for confirming the RHCOS variant and version used by a cluster through available job artifacts.
    • Enhanced prow job analysis workflows with new steps for identifying RHCOS versions and detecting heterogeneous clusters.
  • Chores

    • CI plugin version updated: 0.0.49 → 0.0.50

Document how to confirm actual RHCOS version by inspecting
.status.nodeInfo.osImage from nodes.json in gather-extra artifacts.
Add Step 3b to test-failure and install-failure analysis skills.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@openshift-ci
openshift-ci Bot requested review from bentito and cblecker June 16, 2026 12:46
@openshift-ci openshift-ci Bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jun 16, 2026
@coderabbitai

coderabbitai Bot commented Jun 16, 2026

Copy link
Copy Markdown
Contributor

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: 05ae0199-4710-4b4b-a268-ffcdc82772a2

📥 Commits

Reviewing files that changed from the base of the PR and between c6d52ea and 6ea65c2.

📒 Files selected for processing (6)
  • .claude-plugin/marketplace.json
  • docs/index.html
  • plugins/ci/.claude-plugin/plugin.json
  • plugins/ci/references/jobs.md
  • plugins/ci/skills/prow-job-analyze-install-failure/SKILL.md
  • plugins/ci/skills/prow-job-analyze-test-failure/SKILL.md

Walkthrough

The ci plugin gains a new "Step 3b: Confirm RHCOS Version" in both the install-failure and test-failure analysis skills, along with a matching reference section in jobs.md. The step fetches nodes.json from gather-extra, parses osImage fields to classify RHCOS 9 vs 10, and detects heterogeneous clusters. The plugin version is bumped from 0.0.49 to 0.0.50 across three manifest files.

Changes

CI Plugin RHCOS Confirmation and Version Bump

Layer / File(s) Summary
RHCOS version confirmation guidance
plugins/ci/references/jobs.md, plugins/ci/skills/prow-job-analyze-install-failure/SKILL.md, plugins/ci/skills/prow-job-analyze-test-failure/SKILL.md
Adds a "Confirming RHCOS version from artifacts" reference section and Step 3b to both analysis skills: fetch nodes.json from gather-extra, parse .status.nodeInfo.osImage to determine RHCOS 9 vs 10, flag heterogeneous clusters with mixed values, and skip non-blocking if the file is absent.
Plugin version bump to 0.0.50
plugins/ci/.claude-plugin/plugin.json, .claude-plugin/marketplace.json, docs/index.html
Bumps the ci plugin version field from 0.0.49 to 0.0.50 in the plugin manifest, marketplace registry, and the embedded docs index.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

Possibly related PRs

  • openshift-eng/ai-helpers#416: Modifies prow-job-analyze-install-failure skill documentation, directly overlapping with the install-failure skill extended in this PR.
  • openshift-eng/ai-helpers#341: Adds analysis steps to prow-job-analyze-test-failure/SKILL.md, the same file extended with Step 3b in this PR.
  • openshift-eng/ai-helpers#283: Touches plugins/ci/.claude-plugin/plugin.json and .claude-plugin/marketplace.json for the ci plugin version metadata, the same files bumped here.

Suggested labels

approved, lgtm

Suggested reviewers

  • brandisher
  • stbenjam
🚥 Pre-merge checks | ✅ 10
✅ Passed checks (10 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and specifically summarizes the main change: adding documentation on confirming RHCOS version from job artifacts, which is the core objective of this pull request.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
No Real People Names In Style References ✅ Passed The PR includes "Plow" and "Coughlan" only as codenames within example osImage strings (e.g., "Red Hat Enterprise Linux CoreOS 9.8...-(Plow)"), showing actual system output. These are not used as s...
No Assumed Git Remote Names ✅ Passed PR contains no hardcoded git remote names like "origin" or "upstream" in commands, skills, or documentation. New documentation uses prow-job-artifact-search without assuming specific git remote n...
Git Push Safety Rules ✅ Passed PR contains only static documentation and configuration files (JSON, Markdown, HTML) with no git push commands, force pushes, or pushes to main/master branches.
No Untrusted Mcp Servers ✅ Passed The PR contains only documentation updates and version bumps. No MCP server installations from untrusted sources, npm packages, or suspicious configurations were introduced.
Ai-Helpers Overlap Detection ✅ Passed No overlapping ai-helpers functionality detected. PR adds NEW Step 3b to existing skills for RHCOS version confirmation from artifacts; no duplicate commands/skills exist.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Comment @coderabbitai help to get the list of available commands and usage tips.

@cblecker

Copy link
Copy Markdown
Collaborator

/lgtm

@openshift-ci openshift-ci Bot added the lgtm Indicates that a PR is ready to be merged. label Jun 16, 2026
@openshift-ci

openshift-ci Bot commented Jun 16, 2026

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: cblecker, petr-muller

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-merge-bot
openshift-merge-bot Bot merged commit ca0a4bf into openshift-eng:main Jun 16, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants