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

[zed] Enable SSL support on manila client #1251

Open
wants to merge 1 commit into
base: stable/zed
Choose a base branch
from

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)
@ajkavanagh ajkavanagh requested review from freyes and dosaboy August 4, 2024 15:14
openstack-mirroring pushed a commit to openstack/charm-manila-ganesha that referenced this pull request Oct 16, 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#1251
Change-Id: I55e9aa09b88684517d4052dc56eed0cab05a0262
Closes-Bug: #2064487
(cherry picked from commit ddba2d8)
(cherry picked from commit 47be19b260f2434debf57779e5f9110be4afcee1)
(cherry picked from commit f92ffb44a14f132eb47cf49197d1b7fe3fc11ee4)
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.

1 participant