Skip to content

chore(ci): bump create-demo-clusters.yml to v1.0.45 - #22815

Open
porridge wants to merge 1 commit into
masterfrom
porridge/bump-create-demo-clusters
Open

chore(ci): bump create-demo-clusters.yml to v1.0.45#22815
porridge wants to merge 1 commit into
masterfrom
porridge/bump-create-demo-clusters

Conversation

@porridge

@porridge porridge commented Sep 14, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Description

Bump the two uses: of stackrox/actions/.github/workflows/create-demo-clusters.yml to df8c8f5830fc09d37908be1a9744365a93c5b3cc, which is the commit the floating v1 tag currently resolves to (v1.0.45).

  • .github/workflows/create-clusters.yml: 43bb707a → df8c8f58
  • .github/workflows/cut-rc.yml: 4efb76aa (v1.0.41) → df8c8f58

Slack discussion.

User-facing documentation

Testing and quality

  • the change is production ready: the change is GA, or otherwise the functionality is gated by a feature flag
  • CI results are inspected

Automated testing

  • not modified any existing tests

CI-only change (workflow ref bump); no tests applicable.

How I validated my change

  • Verified df8c8f58 is the commit stackrox/actions v1 tag currently resolves to (== v1.0.45).
  • Confirmed the # ratchet:...@v1 comments stay accurate.

🤖 Generated with Claude Code

@porridge @claude
chore(ci): bump create-demo-clusters.yml to v1.0.45
a752670 
Bump the two `uses:` of stackrox/actions create-demo-clusters.yml from
their previous pins (v1.0.41 / #101) to df8c8f58, which is what the `v1`
floating tag currently resolves to (v1.0.45). The ratchet `@v1` comments
stay accurate. Deliberately not pinning main HEAD, as that commit is not
yet released under a v1.0.x tag.

User prompt: "Bump uses of stackrox/actions/.github/workflows/
create-demo-clusters.yml to latest commit, but in a separate new
workspace." followed by "bump to v1".

Partially generated by an AI agent (Claude).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@openshift-ci

openshift-ci Bot commented Sep 14, 2026

Copy link
Copy Markdown

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@coderabbitai

coderabbitai Bot commented Sep 14, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Central YAML (base), Organization UI (inherited)

Review profile: CHILL

Plan: Enterprise

Run ID: 9220a8c1-3e1f-4de7-b7d2-df073c7c5ceb

📥 Commits

Reviewing files that changed from the base of the PR and between b625dd4 and a752670.

📒 Files selected for processing (2)
  • .github/workflows/create-clusters.yml
  • .github/workflows/cut-rc.yml

Included review availability: Your plan provides up to 12 included reviews per hour; 11 remain after this review.

📝 Summary

Summary by CodeRabbit

  • Chores
    • Updated automated demo cluster creation workflows to use the latest validated configuration.
    • No user-facing behavior or public interface changes.

Walkthrough

The pull request updates the pinned commit for create-demo-clusters.yml in two GitHub Actions workflows. The cluster job inputs and execution conditions remain unchanged.

Changes

Reusable workflow pin updates

Layer / File(s) Summary
Update demo-cluster workflow references
.github/workflows/create-clusters.yml, .github/workflows/cut-rc.yml		 Both cluster jobs now reference commit df8c8f5830fc09d37908be1a9744365a93c5b3cc of create-demo-clusters.yml instead of the previous pinned commit.

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~5 minutes

Change: Other

Suggested reviewers: janisz

Merge Risk: ⚪ Minimal · up to a7526

This PR only updates pinned reusable workflow references, with no identified functional or operational regression.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
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.
Title check ✅ Passed The title clearly identifies the CI workflow reference update and specifies the target version, v1.0.45.
Description check ✅ Passed The description explains both workflow reference changes and the validation performed. It includes the required sections and correctly states that no tests apply. The CI results inspection checkbox re…
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch porridge/bump-create-demo-clusters

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

@porridge
porridge marked this pull request as ready for review September 14, 2026 07:18
@porridge
porridge requested a review from a team as a code owner September 14, 2026 07:18
@codecov

codecov Bot commented Sep 14, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 51.78%. Comparing base (b625dd4) to head (a752670).

Additional details and impacted files 
@@            Coverage Diff             @@
##           master   #22815      +/-   ##
==========================================
- Coverage   51.82%   51.78%   -0.04%     
==========================================
  Files        2901     2901              
  Lines      182788   182788              
==========================================
- Hits        94721    94654      -67     
- Misses      79776    79824      +48     
- Partials     8291     8310      +19
Flag Coverage Δ
go-unit-tests 51.78% <ø> (-0.04%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@github-actions

github-actions Bot commented Sep 14, 2026

Copy link
Copy Markdown
Contributor

🚀 Build Images Ready

Images are ready for commit a752670. To use with deploy scripts:

export MAIN_IMAGE_TAG=5.0.x-292-ga752670870

@openshift-ci

openshift-ci Bot commented Sep 14, 2026

Copy link
Copy Markdown

@porridge: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/ocp-4-22-nongroovy-e2e-tests a752670 link false /test ocp-4-22-nongroovy-e2e-tests
ci/prow/ocp-4-12-nongroovy-e2e-tests a752670 link false /test ocp-4-12-nongroovy-e2e-tests

Full PR test history. Your PR dashboard.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

ai-assisted ai-review area/ci coderabbit-review

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@porridge