Skip to content

Conversation

@ajkavanagh
Copy link
Collaborator

The python-manilaclient requires passing the cacert file for TLS
endpoints. This commit enables that on the get_manila_session_client
and also, for the manila tests, wraps the client in a retrier that
retries on connection failures (i.e. if the server is not running).

(cherry picked from commit 42afc47)

The python-manilaclient requires passing the cacert file for TLS
endpoints. This commit enables that on the `get_manila_session_client`
and also, for the manila tests, wraps the client in a retrier that
retries on connection failures (i.e. if the server is not running).

(cherry picked from commit 42afc47)
openstack-mirroring pushed a commit to openstack/openstack that referenced this pull request Oct 25, 2024
* Update charm-manila-ganesha from branch 'master'
  to 6bdf6382d879a51d9828fadc5b154c666bd9b063
  - Use client cert for keystone-identity
    
    When manila-ganesha is related to vault it needs a client cert to
    configure the keystone-auth section of manila.conf to communicate with
    keystone. This patch sets that up and removes the broken server cert
    auto configuration which ended up masking the manila-share service.
    
    func-test-pr: openstack-charmers/zaza-openstack-tests#1253
    Change-Id: I55e9aa09b88684517d4052dc56eed0cab05a0262
    Closes-Bug: #2064487
openstack-mirroring pushed a commit to openstack/charm-manila-ganesha that referenced this pull request Oct 25, 2024
When manila-ganesha is related to vault it needs a client cert to
configure the keystone-auth section of manila.conf to communicate with
keystone. This patch sets that up and removes the broken server cert
auto configuration which ended up masking the manila-share service.

func-test-pr: openstack-charmers/zaza-openstack-tests#1253
Change-Id: I55e9aa09b88684517d4052dc56eed0cab05a0262
Closes-Bug: #2064487
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