Skip to content

Commit 7ae560f

Browse files
k-s-deanmarkgoddard
authored andcommitted
manila: add glance section in manila-share.conf
Since the Victoria release, manila-share.conf requires a glance section for some drivers. This change adds the missing section. It also uses the correct cinder_keystone_user variable to reference the cinder user. Closes-Bug: #1921935 Change-Id: Ib7ce4ed79c28456281087eb4156577f910c072e7 (cherry picked from commit 2e4f51f)
1 parent 28ca1ae commit 7ae560f

File tree

3 files changed

+19
-1
lines changed

3 files changed

+19
-1
lines changed

ansible/group_vars/all.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -704,6 +704,7 @@ nova_keystone_user: "nova"
704704
placement_keystone_user: "placement"
705705
murano_keystone_user: "murano"
706706
cinder_keystone_user: "cinder"
707+
glance_keystone_user: "glance"
707708

708709
# Nova fake driver and the number of fake driver per compute node
709710
enable_nova_fake: "no"

ansible/roles/manila/templates/manila-share.conf.j2

Lines changed: 13 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,18 @@ enabled_share_backends = {{ manila_enabled_backends|map(attribute='name')|join('
55

66
default_share_type = default_share_type
77

8+
[glance]
9+
auth_url = {{ keystone_admin_url }}
10+
auth_type = password
11+
project_domain_id = {{ default_project_domain_id }}
12+
user_domain_id = {{ default_user_domain_id }}
13+
region_name = {{ openstack_region_name }}
14+
endpoint_type = internalURL
15+
project_name = service
16+
username = {{ glance_keystone_user }}
17+
password = {{ glance_keystone_password }}
18+
cafile = {{ openstack_cacert }}
19+
820
[cinder]
921
auth_uri = {{ keystone_internal_url }}
1022
auth_url = {{ keystone_admin_url }}
@@ -14,7 +26,7 @@ user_domain_id = {{ default_user_domain_id }}
1426
region_name = {{ openstack_region_name }}
1527
endpoint_type = internalURL
1628
project_name = service
17-
username = cinder
29+
username = {{ cinder_keystone_user }}
1830
password = {{ cinder_keystone_password }}
1931
cafile = {{ openstack_cacert }}
2032

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
---
2+
fixes:
3+
- |
4+
Fixes an issue with Manila configuration which was missing a ``[glance]``
5+
section, preventing some drivers from operating.

0 commit comments

Comments
 (0)