Add gangway token for hypershift-pool-admins#80420
Conversation
Request a permanent gangway API token for the hypershift-pool-admins group to enable automated triggering of Prow periodic e2e jobs from the HyperShift Operator release gating pipeline in Konflux. Jira: CNTRLPLANE-3434 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
WalkthroughThis PR establishes RBAC configuration for the ChangesGangway Token Access for HyperShift Pool Admins
🎯 1 (Trivial) | ⏱️ ~3 minutes 🚥 Pre-merge checks | ✅ 15✅ Passed checks (15 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
|
[REHEARSALNOTIFIER] Note: If this PR includes changes to step registry files ( |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@clusters/app.ci/gangway-tokens/hypershift-pool-admins/admin_rbac.yaml`:
- Around line 10-13: The ServiceAccount resource named "periodic-job-bot" in
namespace "hypershift-pool-admins" currently allows automatic token mounting;
add the field automountServiceAccountToken: false to the ServiceAccount manifest
(under the ServiceAccount spec/metadata block) so pods using this SA do not
automatically get a token mounted.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Repository YAML (base), Central YAML (inherited)
Review profile: CHILL
Plan: Enterprise
Run ID: e95116de-2956-41b0-9b29-83cec8650ebf
📒 Files selected for processing (2)
clusters/app.ci/gangway-tokens/hypershift-pool-admins/admin_rbac.yamlclusters/app.ci/prow/03_deployment/gangway.yaml
|
/retest |
|
/approve |
|
/lgtm |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: bear-redhat, Nirshal The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
|
@Nirshal: all tests passed! Full PR test history. Your PR dashboard. DetailsInstructions 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. |
Summary
Request a permanent gangway API token for the
hypershift-pool-adminsgroup (rover group:hypershift-team) to enable automated triggering of Prow periodic e2e jobs from the HyperShift Operator release gating pipeline in Konflux (CNTRLPLANE-3434 / OCPSTRAT-3250).Generated with
./hack/gangway_token.py --group hypershift-pool-admins.Discussed with DPTP on
#forum-ocp-testplatform: https://redhat-internal.slack.com/archives/CBN38N3MW/p1781080323781869Jira: https://issues.redhat.com/browse/CNTRLPLANE-3434
Summary by CodeRabbit
This PR adds Gangway API token infrastructure for the HyperShift team to enable automated triggering of Prow periodic e2e jobs from their Konflux release gating pipeline.
Changes made:
New namespace and service account setup (
clusters/app.ci/gangway-tokens/hypershift-pool-admins/admin_rbac.yaml):hypershift-pool-adminsnamespace on the app.ci clusterperiodic-job-botservice account to hold the Gangway API tokenapi-token-secret)hypershift-pool-adminsgroup to manage the token secretGangway authorization (
clusters/app.ci/prow/03_deployment/gangway.yaml):periodic-job-botservice account from thehypershift-pool-adminsnamespace to thecluster-gangway-submitClusterRoleBindingThe configuration follows the established pattern used by other teams in the repository (such as
aro-hcp-prow-ci,hp-sre-rosa-ci, and others). Once deployed, the HyperShift team can retrieve the token from the secret and use it in their Konflux pipeline to automatically trigger OpenShift e2e tests, as part of their HyperShift Operator release process.