Add glance section required by Cinder to use new glance location-api#631
Conversation
| region_name = {{ .Region }} | ||
| {{ end -}} | ||
|
|
||
| [glance] |
There was a problem hiding this comment.
@rajathere @konan-abhi does this block eventually makes sense based on [1]?
[1] https://review.opendev.org/c/openstack/devstack/+/958718
[2] https://review.opendev.org/c/openstack/cinder/+/958716
[3] https://docs.openstack.org/cinder/latest/_static/cinder.conf.sample
|
Looks good to me!! |
We need to register glance user in keystoneauth plugin to allow service to service communication and let Cinder access to the location api glance calls. [1] [1] https://review.opendev.org/c/openstack/cinder/+/958716 Signed-off-by: Francesco Pantano <fpantano@redhat.com>
|
just rebased. |
|
Build failed (check pipeline). Post ✔️ openstack-k8s-operators-content-provider SUCCESS in 2h 17m 39s |
|
recheck |
|
ok tests look good, @stuggi to re-approve (thanks a lot for the help!) |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: fmount, stuggi 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 |
openshift-merge-bot
Bot
merged commit d2dc795
into
openstack-k8s-operators:main
|
Sorry for being late but this is a exact replica of the [nova] section which is desired. |
5 participants
We need to register
glanceuser inkeystoneauthplugin to allow service to service communication and letCinderaccess to the location-api glance calls. [1].Jira: OSPRH-28617
[1] https://review.opendev.org/c/openstack/cinder/+/958716