Skip to content
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

Set service_user_id from keystone relation #874

Conversation

rodrigogansobarbieri
Copy link
Contributor

This is necessary so we can avoid ambiguities (by using the id vs name) when we want to use the service user.

This is necessary so we can avoid ambiguities (by using the id vs
name) when we want to use the service user.
@ajkavanagh ajkavanagh merged commit 4646435 into juju:master Jan 15, 2024
4 checks passed
openstack-mirroring pushed a commit to openstack/charm-cinder that referenced this pull request Feb 26, 2024
admin_tenant_id and admin_user_id are more explicit than
admin_tenant_name and admin_user as names could duplicate
and cause issues.

Includes sync from Charm-helpers PR #874 below

Charm-helpers-pr: juju/charm-helpers#874

Closes-Bug: #2030755
Change-Id: Idbc2f3d12dcf325b4a53a3dda1ecfa75a199295a
openstack-mirroring pushed a commit to openstack/openstack that referenced this pull request Feb 26, 2024
* Update charm-cinder from branch 'master'
  to 26cc4b30b58abe3a8f53616745e3154993ef3b2b
  - Merge "Use ids for cinder internal tenant and user"
  - Use ids for cinder internal tenant and user
    
    admin_tenant_id and admin_user_id are more explicit than
    admin_tenant_name and admin_user as names could duplicate
    and cause issues.
    
    Includes sync from Charm-helpers PR #874 below
    
    Charm-helpers-pr: juju/charm-helpers#874
    
    Closes-Bug: #2030755
    Change-Id: Idbc2f3d12dcf325b4a53a3dda1ecfa75a199295a
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.

3 participants