Skip to content
This repository was archived by the owner on Sep 9, 2026. It is now read-only.

NO-ISSUE: Bump osac-operator to b613d8d - #221

Merged
openshift-merge-bot[bot] merged 1 commit into
osac-project:mainfrom
DakCrowder:bump-osac-operator-b613d8d
Jun 4, 2026
Merged

openshift-merge-bot[bot] merged 1 commit into
osac-project:mainfrom
DakCrowder:bump-osac-operator-b613d8d

Conversation

@DakCrowder

@DakCrowder DakCrowder commented Jun 4, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Bumps osac-operator submodule from cc4e026 to b613d8d
  • Updates image tag to sha-b613d8d

Included PRs

Test plan

  • CI kustomize build passes for all overlays
  • e2e vmaas test job passes
  • Manual deployment updates osac-operator

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Chores
    • Updated operator component to reference a newer build.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@openshift-ci
openshift-ci Bot requested review from eranco74 and tzvatot June 4, 2026 13:27
@coderabbitai

coderabbitai Bot commented Jun 4, 2026

Copy link
Copy Markdown

Review Change Stack

Walkthrough

This PR updates the osac-operator dependency to a new commit hash (b613d8d). Both the git submodule pointer in base/osac-operator and the Kustomize image tag override in base/kustomization.yaml are updated synchronously to maintain consistency.

Changes

Operator Version Update

Layer / File(s) Summary
Synchronized operator version references
base/osac-operator, base/kustomization.yaml
The base/osac-operator submodule commit reference and the ghcr.io/osac-project/osac-operator image tag in Kustomize are updated together from sha-cc4e026 to sha-b613d8d.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Security note: This is a dependency version bump. Reviewers should verify that the new commit hash represents an intentional, tested upgrade and confirm whether this commit includes any security patches or breaking changes for the operator.

Possibly related PRs

  • osac-project/osac-installer#163: Updates both the osac-operator submodule pin and the corresponding Kustomize image tag override in a similar synchronized manner.
  • osac-project/osac-installer#97: Also updates base/osac-operator submodule and the matching ghcr.io/osac-project/osac-operator newTag in Kustomize to keep versions in sync.
  • osac-project/osac-installer#170: Modifies the ghcr.io/osac-project/osac-operator image tag override in base/kustomization.yaml to update the operator version.

Suggested labels

lgtm, approved

Suggested reviewers

  • adriengentil
  • trewest

Poem

🚀 A version bump, so clean and neat,
Two refs aligned in sync complete,
From cc4e to b613d,
The operator stands on steady ground,
Security and consistency found! ✨


Caution

Pre-merge checks failed

Please resolve all errors before merging. Addressing warnings is optional.

  • Ignore

❌ Failed checks (2 errors, 1 warning)

Check name Status Explanation Resolution
No-Injection-Vectors ❌ Error Sed metacharacter injection found in scripts/sync-image-tags.sh line 61: unescaped variables in sed replacement string allow injection of special characters like & that sed interprets. Escape sed metacharacters in variables before use in replacement: replace ampersands and backslashes in ${current} and ${expected} before passing to sed.
No-Sensitive-Data-In-Logs ❌ Error OSAC-CLI-HOWTO.md logs sensitive tokens: line 558 echoes $HUB_TOKEN, line 762 logs token expiry, and line 1714 outputs raw tokens to stdout, exposing authentication credentials. Remove or redact sensitive logging: replace 'echo "Hub token: $HUB_TOKEN"' with a comment, replace token logging with documentation-only comments, and remove 'echo "$token"' that outputs raw credentials.
Ai-Attribution ⚠️ Warning Commit uses Co-Authored-By trailer for Claude AI tool, which violates the check. Missing required Assisted-by or Generated-by trailers for Red Hat attribution. Update commit trailer from Co-Authored-By to Assisted-by or Generated-by format with appropriate Red Hat attribution per contribution guidelines.
✅ Passed checks (8 passed)
Check name Status Explanation
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-Hardcoded-Secrets ✅ Passed No hardcoded secrets found. PR only updates image tags and submodule pointers with public commit SHAs. All sensitive references use GitHub Actions ${{ secrets.* }} syntax.
No-Weak-Crypto ✅ Passed PR only updates osac-operator submodule pointer and image tag in kustomization.yaml. Pre-existing MD5/SHA1 usage in scripts/realm.json were not introduced by this PR; no new weak crypto added.
Container-Privileges ✅ Passed No privileged containers, host namespace access, SYS_ADMIN capabilities, or allowPrivilegeEscalation settings found. NFS SCC explicitly disables all privilege flags.
Title check ✅ Passed The title 'NO-ISSUE: Bump osac-operator to b613d8d' clearly and specifically describes the main change: updating the osac-operator submodule and image tag to commit b613d8d.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@DakCrowder DakCrowder changed the title Bump osac-operator to b613d8d NO-ISSUE: Bump osac-operator to b613d8d Jun 4, 2026
@openshift-ci-robot

Copy link
Copy Markdown

@DakCrowder: This pull request explicitly references no jira issue.

Details

In response to this:

Summary

  • Bumps osac-operator submodule from cc4e026 to b613d8d
  • Updates image tag to sha-b613d8d

Included PRs

Test plan

  • CI kustomize build passes for all overlays
  • vmaas-ci e2e tests pass with new operator image

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Chores
  • Updated operator component to reference a newer build.

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 openshift-eng/jira-lifecycle-plugin repository.

@akshaynadkarni akshaynadkarni left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@openshift-ci

openshift-ci Bot commented Jun 4, 2026

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: akshaynadkarni, DakCrowder

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-ci openshift-ci Bot added the approved label Jun 4, 2026
@DakCrowder

Copy link
Copy Markdown
Contributor Author

/retest

@openshift-merge-bot
openshift-merge-bot Bot merged commit 09515dc into osac-project:main Jun 4, 2026
9 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants