Skip to content

Remove srep-infra-cicd from golang-osd-operator update script [SDCICD-1761]#754

Open
christophermancini wants to merge 1 commit into
openshift:masterfrom
christophermancini:remove-srep-infra-cicd-SDCICD-1761
Open

Remove srep-infra-cicd from golang-osd-operator update script [SDCICD-1761]#754
christophermancini wants to merge 1 commit into
openshift:masterfrom
christophermancini:remove-srep-infra-cicd-SDCICD-1761

Conversation

@christophermancini

@christophermancini christophermancini commented Jun 9, 2026

Copy link
Copy Markdown

The srep-infra-cicd team migration is complete. Operator teams should now own their Konflux setup, so we no longer automatically add srep-infra-cicd to .tekton/OWNERS files during boilerplate updates.

This change prevents future bp-update runs from re-adding the srep-infra-cicd team to operator repositories, and removes our team from any existing OWNERS files.

@openshift-ci openshift-ci Bot requested review from BATMAN-JD and iamkirkbater June 9, 2026 14:20
@openshift-ci

openshift-ci Bot commented Jun 9, 2026

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: christophermancini
Once this PR has been reviewed and has the lgtm label, please assign bergmannf for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found 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

@coderabbitai

coderabbitai Bot commented Jun 9, 2026

Copy link
Copy Markdown

Important

Review skipped

Review was skipped due to path filters

⛔ Files ignored due to path filters (1)
  • boilerplate/openshift/golang-osd-operator/update is excluded by !boilerplate/**

CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including **/dist/** will override the default block on the dist directory, by removing the pattern from both the lists.

⚙️ Run configuration

Configuration used: Repository: openshift/coderabbit/.coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: 0621dcac-b4a1-499b-a7e6-40198912eade

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ 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.

…-1761]

The srep-infra-cicd team migration is complete. Operator teams should
now own their Konflux setup.

This update:
1. Removes automatic creation of .tekton/OWNERS with srep-infra-cicd
2. Adds cleanup logic to remove srep-infra-cicd from existing .tekton/OWNERS:
   - Deletes the file if it only contains srep-infra-cicd
   - Removes only srep-infra-cicd if other owners are present

Repos will get this cleanup automatically on their next bp-update run.

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
@christophermancini christophermancini force-pushed the remove-srep-infra-cicd-SDCICD-1761 branch from bf221e7 to 78ca4f4 Compare June 9, 2026 16:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant