-
Notifications
You must be signed in to change notification settings - Fork 34
Application Credential support #495
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: Deydra71 The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
a1a7e6a to
8dbb266
Compare
|
Merge Failed. This change or one of its cross-repo dependencies was unable to be automatically merged with the current state of its repository. Please rebase the change and upload a new patchset. |
|
@Deydra71: The following test failed, say
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. |
8dbb266 to
329f9c7
Compare
|
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/8409bdddf95b405d9c5c526f4bdb10ff ❌ openstack-k8s-operators-content-provider FAILURE in 14m 38s |
5559fc4 to
27a5912
Compare
|
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/a987ce80e4954ea0b80cf0e569bd632e ❌ openstack-k8s-operators-content-provider FAILURE in 12m 10s |
27a5912 to
7c66883
Compare
|
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/47ed39066edd47feb0b57495752fb931 ✔️ openstack-k8s-operators-content-provider SUCCESS in 1h 02m 47s |
7c66883 to
a531048
Compare
|
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/676b75e021e84d3a980506f946831dbb ✔️ openstack-k8s-operators-content-provider SUCCESS in 29m 05s |
a531048 to
90fbee6
Compare
Signed-off-by: Veronika Fisarova <[email protected]>
90fbee6 to
e9a579f
Compare
Jira: OSPRH-20525
This PR adds end-to-end support for consuming Keystone ApplicationCredentials (AC) in the Manila operator, enabling Manila API pods to use AC-based authentication when available.
API changes:
Adds an optional authentication field to the Manila API CR:
spec.auth.applicationCredentialSecret— name of the Secret that contains the Keystone Application Credential ID and Secret (AC_ID and AC_SECRET).Reconcile behavior:
Reads
spec.auth.applicationCredentialSecretAttempts to load AC_ID / AC_SECRET from the referenced Secret (via the Keystone helper).
If the secret is missing or incomplete, it falls back to password authentication (the AppCred auth is optional, not an error).
Depends-On: openstack-k8s-operators/keystone-operator#567