Skip to content

Commit 9e9cafe

Browse files
stubbiclaude
andauthored
fix(operatorhub): ship ci.yaml to redhat submission + add reviewers (#82)
## Summary - Adds `reviewers: [stubbi]` to `bundle/ci.yaml`, which already had `index_images:` and `updateGraph: semver-mode`. Preserves both existing fields. - Mirrors the k8s job's ci.yaml staging (`cp bundle/ci.yaml submission/operators/hermes-operator/`) and seed-if-absent logic into the `submit-redhat` (community-operators-prod) job, which previously omitted `ci.yaml` entirely. - Result: both submission jobs now ship a complete `operators/hermes-operator/ci.yaml` containing `updateGraph: semver-mode` + `reviewers: stubbi`. - Clears the `check_ci_upgrade_graph` warning on both OperatorHub and community-operators-prod. ## Test plan - [ ] Trigger `OperatorHub Submission` workflow dispatch — verify both jobs include `operators/hermes-operator/ci.yaml` with `updateGraph: semver-mode` and `reviewers: [stubbi]`. - [ ] Confirm `bundle/ci.yaml` still has `index_images:` preserved: `yq '.index_images' bundle/ci.yaml`. - [ ] Confirm workflow YAML parses: `yq '.jobs|keys' .github/workflows/operatorhub-submit.yaml`. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
1 parent 8378640 commit 9e9cafe

2 files changed

Lines changed: 9 additions & 0 deletions

File tree

.github/workflows/operatorhub-submit.yaml

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -170,6 +170,8 @@ jobs:
170170
cp bundle/manifests/hermes.agent_hermesselfconfigs.yaml "${BUNDLE_DIR}/manifests/"
171171
cp bundle/manifests/hermes.agent_hermesclusterdefaults.yaml "${BUNDLE_DIR}/manifests/"
172172
cp bundle/metadata/annotations.yaml "${BUNDLE_DIR}/metadata/"
173+
# Per community-operators convention, copy ci.yaml alongside.
174+
cp bundle/ci.yaml "submission/operators/hermes-operator/" || true
173175
174176
- name: Fork and submit to redhat community-operators-prod
175177
env:
@@ -192,6 +194,10 @@ jobs:
192194
git checkout -B "${BRANCH}" origin/main
193195
mkdir -p operators/hermes-operator
194196
cp -r ../submission/operators/hermes-operator/${VERSION} operators/hermes-operator/${VERSION}
197+
# Only seed ci.yaml when the catalog does not already have one. Overwriting
198+
# it on every release is a privileged change that withholds the
199+
# authorized-changes label and blocks auto-merge.
200+
[ -f operators/hermes-operator/ci.yaml ] || cp ../submission/operators/hermes-operator/ci.yaml operators/hermes-operator/ci.yaml 2>/dev/null || true
195201
git add "operators/hermes-operator"
196202
git commit -m "operator hermes-operator (${VERSION})"
197203
git push --force origin "${BRANCH}"

bundle/ci.yaml

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,2 +1,5 @@
11
index_images: registry.redhat.io/redhat/community-operator-index:v4.15
22
updateGraph: semver-mode
3+
# A PR authored by a listed reviewer auto-gets the `authorized-changes` label (enables auto-merge).
4+
reviewers:
5+
- stubbi

0 commit comments

Comments
 (0)