opendevreview | Takashi Kajinami proposed openstack/python-manilaclient master: Replace OS_TENANT_NAME https://review.opendev.org/c/openstack/python-manilaclient/+/924170 | 00:44 |
---|---|---|
tkajinam | carloss, gouthamr https://review.opendev.org/c/openstack/python-manilaclient/+/924170 fixes the broken https://review.opendev.org/c/openstack/python-manilaclient/+/924170 job | 15:14 |
tkajinam | I faced a similar problem with heat CI and had to disable openstack-cli-server there, too. I've not yet identified the actual problem with openstack-cli-sever yet but proposed disabling it for now to unblock CI early. | 15:16 |
jayaanand | @ganso @gireesh | 16:26 |
jayaanand | fixed similar issue related to kerberos config. can you please check with him BUG details https://review.opendev.org/c/openstack/manila/+/922896 | 16:28 |
ganso | jayaanand: thanks, sure! is it related to LIF-naming? | 16:34 |
ganso | gireesh: ^ | 16:34 |
gireesh | yes it is, here is the reason | 16:53 |
gireesh | It seems that the customer has a vserver management LIF that is not configured with Kerberos, while other data LIFs are configured but disabled. Before renaming the LIF, it picked up the vserver management LIF and failed because the ZAPI threw an exception when the LIF was not configured with Kerberos. After renaming the LIF, it picked up the first data LIF. Since the data LIF is configured with Kerberos, the ZAPI runs | 16:53 |
gireesh | successfully, but we are returning from there because Kerberos is disabled for the LIF, we skip to check the Kerberos configure for other LIF | 16:53 |
gireesh | applying my patch will fix the issue | 16:54 |
gireesh | @genso | 16:54 |
gireesh | @ganso | 16:54 |
gouthamr | tkajinam: thank you; this makes sense.. i was confused if this was due to the continued usage of keystoneclient within the "manila" CLI.. | 16:55 |
gouthamr | gireesh: when constructing export locations, shouldn't the driver just ignore management LIFs? | 16:58 |
gouthamr | https://bugs.launchpad.net/manila/+bug/2069125/comments/17 | 16:58 |
* gouthamr remembers filtering for data LIFs in some place in the driver for a different issue | 16:59 | |
gireesh | Yes @gouthamr , I think we have to ignore management LIF. But this logic is there from starting, we are checking if Kerberos is configured and enabled for all the LIF. If it is enabled for all the LIF authentication happen through Kerberos if one of the LIF is not configured or configured but disabled, then we return false and use the normal authentication | 17:13 |
gireesh | I am thinking about the configuration, if customer has Kerberos configured and enabled for all the LIF except SVM management LIF. In this case authentication will not happened through Kerberos.We need how is customer configuring the Kerberos and even we need to check with ONTAP SME regarding this | 17:16 |
opendevreview | Merged openstack/python-manilaclient master: Replace OS_TENANT_NAME https://review.opendev.org/c/openstack/python-manilaclient/+/924170 | 18:43 |
opendevreview | Merged openstack/python-manilaclient stable/2024.1: [OSC] Fix output format for osc share subnet create https://review.opendev.org/c/openstack/python-manilaclient/+/911613 | 18:43 |
opendevreview | Merged openstack/manila stable/2024.1: bug fix: error message fix https://review.opendev.org/c/openstack/manila/+/923087 | 18:45 |
opendevreview | Merged openstack/manila stable/2024.1: Ensure password values are not logged https://review.opendev.org/c/openstack/manila/+/917731 | 18:57 |
opendevreview | Merged openstack/manila stable/2024.1: Fix the link to get the next share replica https://review.opendev.org/c/openstack/manila/+/913803 | 19:04 |
ganso | gireesh: ooooooooh it works in that case because Kerberos is disabled, so it returns FIRST without cycling through all the LIFs, which would otherwise reach the management LIF again and raise the error. Awesome! | 19:37 |
ganso | gireesh++ thanks! | 19:39 |
opendevreview | Goutham Pacha Ravi proposed openstack/manila stable/2023.2: Ensure password values are not logged https://review.opendev.org/c/openstack/manila/+/924361 | 20:04 |
opendevreview | Goutham Pacha Ravi proposed openstack/python-manilaclient stable/2023.2: [OSC] Fix output format for osc share subnet create https://review.opendev.org/c/openstack/python-manilaclient/+/924362 | 20:05 |
opendevreview | Goutham Pacha Ravi proposed openstack/python-manilaclient stable/2024.1: Replace OS_TENANT_NAME https://review.opendev.org/c/openstack/python-manilaclient/+/924363 | 20:06 |
opendevreview | Carlos Eduardo proposed openstack/manila-specs master: Follow-up ensure shares spec comments https://review.opendev.org/c/openstack/manila-specs/+/924368 | 21:05 |
opendevreview | Merged openstack/manila stable/2023.2: bug fix: error message fix https://review.opendev.org/c/openstack/manila/+/923088 | 21:06 |
opendevreview | Merged openstack/manila stable/2023.2: Fixing skipped default override for CORS options https://review.opendev.org/c/openstack/manila/+/907271 | 21:12 |
opendevreview | Merged openstack/manila stable/2023.1: [ci] Fix zuul configuration errors https://review.opendev.org/c/openstack/manila/+/923796 | 21:12 |
opendevreview | Merged openstack/manila stable/2023.1: Forbid resetting state of active replicas https://review.opendev.org/c/openstack/manila/+/912744 | 21:12 |
opendevreview | Goutham Pacha Ravi proposed openstack/python-manilaclient stable/2023.1: [OSC] Fix output format for osc share subnet create https://review.opendev.org/c/openstack/python-manilaclient/+/924375 | 21:54 |
opendevreview | Merged openstack/python-manilaclient stable/2023.2: [OSC] Fix output format for osc share subnet create https://review.opendev.org/c/openstack/python-manilaclient/+/924362 | 22:08 |
opendevreview | Volodymyr Boiko proposed openstack/manila master: Add share driver for VastData storage https://review.opendev.org/c/openstack/manila/+/915380 | 22:32 |
VolodymyrBoiko[m] | carloss: Thank you for review. I updated code according to comments. Please check | 23:03 |
opendevreview | Merged openstack/devstack-plugin-ceph stable/2023.1: Bump to Reef https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/915146 | 23:08 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!