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

Add secrets RBAC to hub-access Role - #45

Merged
openshift-merge-bot[bot] merged 1 commit into
osac-project:mainfrom
tzvatot:feature/hub-access-secrets-rbac-mgmt-23103
Feb 26, 2026
Merged

openshift-merge-bot[bot] merged 1 commit into
osac-project:mainfrom
tzvatot:feature/hub-access-secrets-rbac-mgmt-23103

Conversation

@tzvatot

@tzvatot tzvatot commented Feb 26, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Add secrets create permission to the hub-access Role
  • Required for the userDataSecretRef feature: the fulfillment-service creates a Kubernetes Secret in the hub namespace to store cloud-init user data provided via the API
  • Only create is needed — the Secret uses owner references to the ComputeInstance CR for automatic cleanup, and the user data field is immutable so no update is required

Related

@openshift-ci
openshift-ci Bot requested review from jhernand and trewest February 26, 2026 12:04
@tzvatot
tzvatot force-pushed the feature/hub-access-secrets-rbac-mgmt-23103 branch from 81da855 to 01ed322 Compare February 26, 2026 12:51
The fulfillment-service requires permissions to manage Kubernetes
Secrets in the hub namespace to support the userDataSecretRef field,
which stores cloud-init user data as a Secret rather than requiring
users to create it manually.

Related: https://issues.redhat.com/browse/MGMT-23103

Generated with [Claude Code](https://claude.com/claude-code)
@tzvatot
tzvatot force-pushed the feature/hub-access-secrets-rbac-mgmt-23103 branch from 01ed322 to c30894c Compare February 26, 2026 13:04
@openshift-ci openshift-ci Bot removed the lgtm label Feb 26, 2026
@openshift-ci openshift-ci Bot added the lgtm label Feb 26, 2026
@openshift-ci

openshift-ci Bot commented Feb 26, 2026

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: adriengentil, tzvatot

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:
  • OWNERS [adriengentil,tzvatot]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-merge-bot
openshift-merge-bot Bot merged commit 9f96542 into osac-project:main Feb 26, 2026
5 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.

2 participants