Skip to content

CherryPicked: [cnv-4.20] CherryPicked: [4.21] Fix _sc_name_for_storage_api#5437

Merged
rnetser merged 1 commit into
cnv-4.20from
CherryPicked-4.21-fix-storage-config-7qraY
Jul 2, 2026
Merged

CherryPicked: [cnv-4.20] CherryPicked: [4.21] Fix _sc_name_for_storage_api#5437
rnetser merged 1 commit into
cnv-4.20from
CherryPicked-4.21-fix-storage-config-7qraY

Conversation

@cnv-tests-github-webhook-dollierp

@cnv-tests-github-webhook-dollierp cnv-tests-github-webhook-dollierp Bot commented Jun 30, 2026

Copy link
Copy Markdown

Cherry-pick from cnv-4.21 branch, original PR: #5424, PR owner: hmeir

@hmeir
CherryPicked: [4.21] Fix _sc_name_for_storage_api (#5424)
cf290f2 
Backport of
#3416

PR #5228 was merged recently, but its prerequisite fix PR #3416 (Fix
_sc_name_for_storage_api) was not.
Causing upgrade failures in setup in multiple branches.

NONE

Signed-off-by: Harel Meir <hmeir@redhat.com>
Merged
@openshift-virtualization-qe-bot-5

openshift-virtualization-qe-bot-5 commented Jun 30, 2026

Copy link
Copy Markdown

⚠️ This cherry-pick had conflicts resolved by AI (claude/claude-opus-4-6-1m)

@hmeir — AI automatically resolved merge conflicts for this cherry-pick. Please review the changes carefully and verify correctness before merging.

@openshift-virtualization-qe-bot-6

openshift-virtualization-qe-bot-6 commented Jun 30, 2026

Copy link
Copy Markdown

Report bugs in Issues

Welcome! 🎉

This pull request will be automatically processed with the following features:

🔄 Automatic Actions

  • Reviewer Assignment: Reviewers are automatically assigned based on the OWNERS file in the repository root
  • Size Labeling: PR size labels (XS, S, M, L, XL, XXL) are automatically applied based on changes
  • Issue Creation: A tracking issue is created for this PR and will be closed when the PR is merged or closed
  • Branch Labeling: Branch-specific labels are applied to track the target branch
  • Auto-verification: Auto-verified users have their PRs automatically marked as verified
  • Labels: Enabled categories: branch, can-be-merged, cherry-pick, has-conflicts, hold, needs-rebase, size, verified, wip

📋 Available Commands

PR Status Management

  • /wip - Mark PR as work in progress (adds WIP: prefix to title)
  • /wip cancel - Remove work in progress status
  • /hold - Block PR merging (approvers only)
  • /hold cancel - Unblock PR merging
  • /verified - Mark PR as verified
  • /verified cancel - Remove verification status
  • /reprocess - Trigger complete PR workflow reprocessing (useful if webhook failed or configuration changed)
  • /regenerate-welcome - Regenerate this welcome message
  • /security-override - Set security check runs to pass (maintainers only)
  • /security-override cancel - Re-run security checks

Review & Approval

  • /lgtm - Approve changes (looks good to me)
  • /approve - Approve PR (approvers only)
  • /assign-reviewers - Assign reviewers based on OWNERS file
  • /assign-reviewer @username - Assign specific reviewer
  • /check-can-merge - Check if PR meets merge requirements

Testing & Validation

  • /retest tox - Run Python test suite with tox
  • /retest build-container - Rebuild and test container image
  • /retest verify-bugs-are-open - verify-bugs-are-open
  • /retest all - Run all available tests

Container Operations

  • /build-and-push-container - Build and push container image (tagged with PR number)
    • Supports additional build arguments: /build-and-push-container --build-arg KEY=value

Cherry-pick Operations

  • /cherry-pick <branch> - Schedule cherry-pick to target branch when PR is merged
    • Multiple branches: /cherry-pick branch1 branch2 branch3
  • /cherry-pick-retry <branch> - Retry a failed cherry-pick (merged PRs only)

Branch Management

  • /rebase - Rebase this PR branch onto its base branch

Custom Commands

  • /test-plan - Triggers CodeRabbit to analyze the PR's changed files and post a test execution plan
  • /rerun-smoke - Adds `retest-smoke` label on the PR to trigger smoke tests exectuion

Label Management

  • /<label-name> - Add a label to the PR
  • /<label-name> cancel - Remove a label from the PR

✅ Merge Requirements

This PR will be automatically approved when the following conditions are met:

  1. Approval: /approve from at least one approver
  2. LGTM Count: Minimum 2 /lgtm from reviewers
  3. Status Checks: All required status checks must pass
  4. No Blockers: No wip, hold, has-conflicts labels and PR must be mergeable (no conflicts)
  5. Verified: PR must be marked as verified

📊 Review Process

Approvers and Reviewers

Approvers:

  • dshchedr
  • myakove
  • rnetser
  • vsibirsk

Reviewers:

  • RoniKishner
  • dshchedr
  • geetikakay
  • rnetser
  • vsibirsk
Available Labels
  • hold
  • verified
  • wip
  • lgtm
  • approve
AI Features
  • Cherry-Pick Conflict Resolution: Enabled (claude/claude-opus-4-6-1m)
Security Checks
  • Suspicious Path Detection: Monitors paths: .claude/, .vscode/, .cursor/, .devcontainer/, .pi/, .github/workflows/, .github/actions/
  • Committer Identity Check: Verifies last committer matches PR author
  • Mandatory: Security checks block merge (use /security-override to bypass — maintainers only)

💡 Tips

  • WIP Status: Use /wip when your PR is not ready for review
  • Verification: The verified label is removed on new commits unless the push is detected as a clean rebase
  • Cherry-picking: Cherry-pick labels are processed when the PR is merged
  • Container Builds: Container images are automatically tagged with the PR number
  • Permission Levels: Some commands require approver permissions
  • Auto-verified Users: Certain users have automatic verification and merge privileges

For more information, please refer to the project documentation or contact the maintainers.

@josemacassan

josemacassan commented Jun 30, 2026

Copy link
Copy Markdown
Contributor

/lgtm

@hmeir

hmeir commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

/verified

@rlobillo

rlobillo commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

/lgtm

@rnetser

rnetser commented Jul 1, 2026

Copy link
Copy Markdown
Collaborator

/approve

@rnetser

rnetser commented Jul 1, 2026

Copy link
Copy Markdown
Collaborator

/lgtm

@hmeir

hmeir commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

/check-can-merge

@RoniKishner

RoniKishner commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

/lgtm

@rnetser rnetser merged commit d16f963 into cnv-4.20 Jul 2, 2026
27 checks passed
@rnetser rnetser deleted the CherryPicked-4.21-fix-storage-config-7qraY branch July 2, 2026 10:50
@openshift-virtualization-qe-bot-4

openshift-virtualization-qe-bot-4 commented Jul 2, 2026

Copy link
Copy Markdown

New container for quay.io/openshift-cnv/openshift-virtualization-tests:cnv-4.20 published

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

Reviewers

@RoniKishner RoniKishner Awaiting requested review from RoniKishner
@vsibirsk vsibirsk Awaiting requested review from vsibirsk
@dshchedr dshchedr Awaiting requested review from dshchedr
@rnetser rnetser Awaiting requested review from rnetser
@geetikakay geetikakay Awaiting requested review from geetikakay
1 more reviewer
@hmeir hmeir hmeir approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

@hmeir hmeir

@cnv-tests-github-webhook-dollierp cnv-tests-github-webhook-dollierp[bot]

Labels

ai-resolved-conflicts approved-rnetser branch-cnv-4.20 can-be-merged CherryPicked-from-cnv-4.21 lgtm-hmeir lgtm-josemacassan lgtm-rlobillo lgtm-rnetser lgtm-RoniKishner needs-rebase sig-virt size/XS tox:verify-tc-requirement-polarion:passed D/S Testing verified

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.