Skip to content

konflux: Replace release-bot service account with konflux-bot-0 in release pipelines#1783

Open
raptorsun wants to merge 1 commit into
openshift:mainfrom
raptorsun:konflux-bot
Open

konflux: Replace release-bot service account with konflux-bot-0 in release pipelines#1783
raptorsun wants to merge 1 commit into
openshift:mainfrom
raptorsun:konflux-bot

Conversation

@raptorsun

Copy link
Copy Markdown
Contributor

Description

As requested from Konflux team, we should name our service accounts as "konfluxt-bot-[0-9]+"

This PR changes service account name from release-bot to konflux-bot-0 .

The related resources in our Konflux Openshift Cluster have been created.

Type of change

  • Refactor
  • New feature
  • Bug fix
  • CVE fix
  • Optimization
  • Documentation Update
  • Configuration Update
  • Bump-up dependent library

Related Tickets & Documents

  • Related Issue #
  • Closes #

Checklist before requesting a review

  • I have performed a self-review of my code.
  • PR has passed all pre-merge test jobs.
  • If it is a core feature, I have added thorough tests.

Testing

  • Please provide detailed steps to perform tests related to this code change.
  • How were the fix/results from this change verified? Please provide relevant screenshots or results.

…lease pipelines

Signed-off-by: Haoyu Sun <hasun@redhat.com>
@openshift-ci openshift-ci Bot requested review from blublinsky and bparees July 3, 2026 15:40
@openshift-ci

openshift-ci Bot commented Jul 3, 2026

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign joshuawilson 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 Jul 3, 2026

Copy link
Copy Markdown

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Enterprise

Run ID: d311c887-eab4-43be-a6f5-e70a9ca775a3

📥 Commits

Reviewing files that changed from the base of the PR and between a1bbf9c and f37427d.

📒 Files selected for processing (10)
  • .tekton/release/bundle-update/pipeline.yaml
  • .tekton/release/bundle-update/pipelinerun.yaml
  • .tekton/release/catalog-update/pipeline.yaml
  • .tekton/release/catalog-update/pipelinerun.yaml
  • .tekton/release/push-to-release/bundle-to-release-pipeline.yaml
  • .tekton/release/release-bot/role.yaml
  • .tekton/release/release-bot/rolebinding.yaml
  • .tekton/release/release-bot/service-account.yaml
  • .tekton/release/secrets/secret-github-token.yaml
  • .tekton/release/secrets/secret-konflux-token.yaml

Walkthrough

This PR renames the release-bot Kubernetes identity to konflux-bot-0 across Tekton release manifests. ServiceAccount, Role, RoleBinding, and Secret resources are renamed, and pipeline/pipelinerun manifests are updated to reference the new secretKeyRef names and serviceAccountName.

Changes

Bot identity rename across manifests

Layer / File(s) Summary
ServiceAccount, Role, and RoleBinding rename
.tekton/release/release-bot/service-account.yaml, .tekton/release/release-bot/role.yaml, .tekton/release/release-bot/rolebinding.yaml
RBAC resource metadata names, subject name, and roleRef name are changed from release-bot to konflux-bot-0.
Secret manifest rename
.tekton/release/secrets/secret-github-token.yaml, .tekton/release/secrets/secret-konflux-token.yaml
Secret metadata.name values are updated to konflux-bot-0-github-token and konflux-bot-0-konflux-token.
Pipeline and PipelineRun secret/service account references
.tekton/release/bundle-update/pipeline.yaml, .tekton/release/bundle-update/pipelinerun.yaml, .tekton/release/catalog-update/pipeline.yaml, .tekton/release/catalog-update/pipelinerun.yaml, .tekton/release/push-to-release/bundle-to-release-pipeline.yaml
Task step secretKeyRef.name values and PipelineRun spec.serviceAccountName are updated to reference the renamed konflux-bot-0 secrets and service account.

Estimated code review effort: 1 (Trivial) | ~5 minutes

Related Issues: None mentioned in the provided information.

Related PRs: None mentioned in the provided information.

Suggested labels: ok-to-test, tekton

Suggested reviewers: None specified based on the provided information.

Poem

A rabbit hopped through YAML fields so neat,
Swapped every bot's name, tidy and complete.
release-bot retired, gone off to rest,
konflux-bot-0 now does its best.
Secrets renamed, tokens realigned,
Pipelines run smooth, credentials aligned. 🐇

✨ 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.

@openshift-ci

openshift-ci Bot commented Jul 3, 2026

Copy link
Copy Markdown

@raptorsun: all tests passed!

Full PR test history. Your PR dashboard.

Details

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 kubernetes-sigs/prow repository. I understand the commands that are listed here.

@blublinsky

Copy link
Copy Markdown
Contributor

Nice-to-have (non-blocking): the manifests now use konflux-bot-0, but the directory is still .tekton/release/release-bot/. Consider renaming it to something like .tekton/release/konflux-bot-0/ for consistency — purely cosmetic, no functional impact.

@blublinsky

Copy link
Copy Markdown
Contributor

/lgtm

@openshift-ci openshift-ci Bot added the lgtm Indicates that a PR is ready to be merged. label Jul 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm Indicates that a PR is ready to be merged.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants