Skip to content

chore: Add release-v0.9 to ubi-bump-prs.sh default branches - #3636

Open
cuipinghuo wants to merge 1 commit into
conforma:mainfrom
cuipinghuo:fix-ubi-bump-default-branches
Open

cuipinghuo wants to merge 1 commit into
conforma:mainfrom
cuipinghuo:fix-ubi-bump-default-branches

Conversation

@cuipinghuo

Copy link
Copy Markdown
Contributor

What:

Add release-v0.9 to the default branch list in hack/ubi-bump-prs.sh.

-  BRANCHES=(release-v0.7 release-v0.8 main)
+  BRANCHES=(release-v0.7 release-v0.8 release-v0.9 main)

Why:

Running hack/ubi-bump-prs.sh without arguments only produced UBI bump PRs for release-v0.7, release-v0.8, and main, silently skipping release-v0.9 — an active, maintained release branch (within Renovate's baseBranchPatterns release-v0.7+). This caused the v0.9 bump to be missed on a manual run (see EC-2343, where the v0.9 PR had to be created separately). Adding it keeps the default run aligned with the maintained release window.

Tickets:

Ref: https://redhat.atlassian.net/browse/EC-2343

🤖 Generated with Claude Code

@cuipinghuo
cuipinghuo requested a review from a team as a code owner October 23, 2026 19:17
@coderabbitai

coderabbitai Bot commented Oct 23, 2026 •

Copy link
Copy Markdown

Warning

Review limit reached

This review ran on the open-source allowance, not this organization's plan, because the pull request author doesn't have an assigned seat. Waiting won't change this — ask an organization admin to assign them a seat, or add seats in Billing if every seat is already assigned, then retry.

Next included review available in 59 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

Learn how review limits work.

Review configuration:

⚙️ Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: d7c2176b-c584-4605-bc14-177b0a33b5d8
📥 Commits

Reviewing files that changed from the base of the PR and between 5d22a0b and 11a4960.

📒 Files selected for processing: 1
  • hack/ubi-bump-prs.sh

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

@fullsend-ai-review fullsend-ai-review Bot added the risk/low PR risk: low label Oct 23, 2026
@fullsend-ai-review

fullsend-ai-review Bot commented Oct 23, 2026 •

Copy link
Copy Markdown

<!-- fullsend:risk-assessment -->
Risk Assessment: low (1/5)

Details

Single-line addition to a utility shell script in hack/ with no protected paths, security-sensitive files, CI workflow changes, or dependency changes; zero churn in 30 days and single authorship; Tier 1 signals identical to prior run, anchoring preserves prior score of 1.

Previous run

<!-- fullsend:risk-assessment -->
Risk Assessment: low (1/5)

Details

Single-line addition to a shell script in hack/ with no protected paths, security sensitivity, or dependency changes; low churn and single authorship; composite 1.

@fullsend-ai-review

fullsend-ai-review Bot commented Oct 23, 2026 •

Copy link
Copy Markdown

Looks good to me

<!-- fullsend:review-findings-v2:eyJ2ZXJzaW9uIjoyLCJmaW5kaW5ncyI6W119 -->

Previous run

Looks good to me

<!-- fullsend:review-findings-v2:eyJ2ZXJzaW9uIjoyLCJmaW5kaW5ncyI6W119 -->

fullsend-ai-review[bot]

This comment was marked as outdated.

@fullsend-ai-review fullsend-ai-review Bot added the ready-for-merge All reviewers approved — ready to merge label Oct 23, 2026
The default branch list in hack/ubi-bump-prs.sh was missing release-v0.9,
so running the script without arguments only produced UBI bump PRs for
release-v0.7, release-v0.8, and main — silently skipping release-v0.9,
which is an active, maintained release branch (covered by Renovate's
baseBranchPatterns). Add it so the default run matches the maintained
release window.

Ref: https://redhat.atlassian.net/browse/EC-2343

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Signed-off-by: cuipinghuo <chuo@redhat.com>
@cuipinghuo
cuipinghuo force-pushed the fix-ubi-bump-default-branches branch from 55e74dc to 11a4960 Compare October 23, 2026 20:16
@fullsend-ai-review
fullsend-ai-review Bot dismissed their stale review October 23, 2026 20:29

Superseded by updated review

@codecov

codecov Bot commented Oct 23, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

Flag Coverage Δ
acceptance 54.61% <ø> (ø)
generative 12.22% <ø> (ø)
integration 23.62% <ø> (ø)
unit 72.29% <ø> (ø)

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

This branch has not been deployed

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

Labels

ready-for-merge All reviewers approved — ready to merge risk/low PR risk: low size: XS

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants