Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

cardoe
Copy link
Contributor

@cardoe cardoe commented Sep 8, 2025

Utilize the External Secrets Operator to define the OpenStack service account credentials that OpenStack Helm needs to work with Keystone service accounts for the various OpenStack services. This allows one cluster to run Keystone and another to run the OpenStack services and to utilize the ESO operator to keep the credentials in sync between the two.

@cardoe cardoe force-pushed the openstack-service-users branch 14 times, most recently from 8e67f52 to 431b431 Compare September 10, 2025 23:32
@cardoe cardoe marked this pull request as ready for review September 10, 2025 23:52
@cardoe cardoe requested a review from a team September 10, 2025 23:52
@cardoe cardoe force-pushed the openstack-service-users branch from 431b431 to a9ba646 Compare September 11, 2025 14:37
Copy link
Collaborator

@skrobul skrobul left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Overall this makes sense to me. The docs may also use a section explaining why we cannot immediately remove the secret-openstack.yaml. I have also left a few comments inline.

@cardoe
Copy link
Contributor Author

cardoe commented Sep 11, 2025

Overall this makes sense to me. The docs may also use a section explaining why we cannot immediately remove the secret-openstack.yaml. I have also left a few comments inline.

The commit I just added makes it so we can get rid of part of it.

@cardoe cardoe force-pushed the openstack-service-users branch 7 times, most recently from d5a7979 to 6657a7d Compare September 13, 2025 18:25
@cardoe
Copy link
Contributor Author

cardoe commented Sep 13, 2025

Ah I should have kept the changes separate but I squashed them all in @skrobul. Sorry. But I think I've now addressed all your reviews.

Copy link
Collaborator

@skrobul skrobul left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@cardoe cardoe force-pushed the openstack-service-users branch from 2718cbe to e49857c Compare September 15, 2025 19:56
cardoe and others added 2 commits September 15, 2025 15:22
Utilize the External Secrets Operator to define the OpenStack service
account credentials that OpenStack Helm needs to work with Keystone
service accounts for the various OpenStack services. This allows one
cluster to run Keystone and another to run the OpenStack services and to
utilize the ESO operator to keep the credentials in sync between the
two.

Co-authored-by: Marek Skrobacki <[email protected]>
The change to mount these has not landed in all upstream charts and
we've also not updated to the charts that have this change across the
board. So for now include it in all of our configs until this change can
make its way upstream.
@cardoe cardoe force-pushed the openstack-service-users branch 2 times, most recently from fba2060 to ba214e0 Compare September 15, 2025 20:43
…scripts

The --config-file flag is hardcoded in a number of scripts which results
in preference to that file over the directories which is not the
behavior we want since we want to override. However since the OpenStack
Helm charts provide no way to change this behavior we need to replace
the entire configmap-bin. This one is taken from two environments which
were identical and then had the --config-dir flag added so that it
continues to work.
@cardoe cardoe force-pushed the openstack-service-users branch from ba214e0 to ad14140 Compare September 15, 2025 20:52
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.

2 participants