Thursday, 2021-03-11

*** tosky has quit IRC00:26
openstackgerritGoutham Pacha Ravi proposed openstack/manila master: Implement secure RBAC for shares  https://review.opendev.org/c/openstack/manila/+/76346900:32
openstackgerritDouglas Viroel proposed openstack/manila-tempest-plugin master: Add functional tests for add/update security service feature  https://review.opendev.org/c/openstack/manila-tempest-plugin/+/77503400:32
openstackgerritGoutham Pacha Ravi proposed openstack/manila master: Implement secure RBAC for shares  https://review.opendev.org/c/openstack/manila/+/76346900:37
*** LinPeiWen has joined #openstack-manila00:50
*** LinPeiWen has quit IRC01:09
openstackgerritGoutham Pacha Ravi proposed openstack/manila master: Implement secure RBAC for shares  https://review.opendev.org/c/openstack/manila/+/76346901:15
openstackgerritMerged openstack/manila master: Clean up some policy code  https://review.opendev.org/c/openstack/manila/+/77772502:53
*** LinPeiWen has joined #openstack-manila02:54
*** ivan_lin has quit IRC02:57
*** carloss has quit IRC03:14
*** openstackgerrit has quit IRC03:25
*** LinPeiWen89 has joined #openstack-manila04:23
*** rratnaka has joined #openstack-manila04:27
*** felipe_rodrigues has quit IRC06:03
*** lpetrut has joined #openstack-manila07:41
*** LinPeiWen89 has quit IRC07:43
*** ivan_lin has joined #openstack-manila07:54
*** LinPeiWen has quit IRC07:56
*** rratnaka14 has joined #openstack-manila08:21
*** rratnaka has quit IRC08:24
*** tosky has joined #openstack-manila08:33
*** openstackgerrit has joined #openstack-manila08:43
openstackgerritGoutham Pacha Ravi proposed openstack/manila master: Part 6 - Add RBAC protection tests  https://review.opendev.org/c/openstack/manila/+/77993808:43
*** tosky_ has joined #openstack-manila08:52
*** tosky is now known as Guest8681408:53
*** tosky_ is now known as tosky08:53
*** Guest86814 has quit IRC08:55
*** LinPeiWen has joined #openstack-manila10:27
*** carloss has joined #openstack-manila10:52
openstackgerritMerged openstack/manila master: [NetApp] Fix security service configuration for LDAP servers  https://review.opendev.org/c/openstack/manila/+/77935012:49
*** openstack has joined #openstack-manila13:17
*** ChanServ sets mode: +o openstack13:17
openstackgerritDouglas Viroel proposed openstack/manila master: Add security service update for in-use share networks  https://review.opendev.org/c/openstack/manila/+/77472813:35
openstackgerritDouglas Viroel proposed openstack/manila-tempest-plugin master: Add functional tests for add/update security service feature  https://review.opendev.org/c/openstack/manila-tempest-plugin/+/77503413:37
*** disap has joined #openstack-manila14:09
openstackgerritVictoria Martinez de la Cruz proposed openstack/manila master: Update cephfs drivers to use ceph-mgr client  https://review.opendev.org/c/openstack/manila/+/77961914:10
openstackgerritDouglas Viroel proposed openstack/manila master: [NetApp] Add support for FPolicy native mode  https://review.opendev.org/c/openstack/manila/+/77503814:15
*** rratnaka has joined #openstack-manila14:23
*** rratnaka14 has quit IRC14:27
*** rratnaka14 has joined #openstack-manila14:39
*** rratnaka has quit IRC14:43
*** rratnaka has joined #openstack-manila14:46
*** rratnaka14 has quit IRC14:49
*** rratnaka14 has joined #openstack-manila14:56
*** rratnaka has quit IRC14:58
*** disap has quit IRC14:59
*** felipe_rodrigues has joined #openstack-manila15:00
*** haixin has joined #openstack-manila15:01
openstackgerritVictoria Martinez de la Cruz proposed openstack/manila master: Update cephfs drivers to use ceph-mgr client  https://review.opendev.org/c/openstack/manila/+/77961915:02
*** carthaca has joined #openstack-manila15:04
*** disap has joined #openstack-manila15:09
*** chuan137 has joined #openstack-manila15:28
*** lpetrut has quit IRC15:29
*** jmlowe has quit IRC15:30
*** jmlowe has joined #openstack-manila15:32
*** chuan137 has quit IRC15:40
*** chuan137 has joined #openstack-manila15:48
*** disap has quit IRC16:01
*** rratnaka14 has quit IRC16:03
gouthamrvkmc: o/ so these patches won't be ready today, likely?16:04
vkmcgouthamr, yes, they will be ready today16:04
*** disap has joined #openstack-manila16:04
vkmcgouthamr, is that ok to submit them and send a ffe email afterwards?16:04
gouthamrvkmc: yes. FFE is a good plan since these improvements are necessary for the longevity of the driver16:05
vkmcs/that/it/g16:06
gouthamrvkmc: by submit you mean upload to review?16:06
vkmcgouthamr, yes16:06
gouthamrvkmc: oh, nothing stopping you from doing that :D16:06
*** haixin has quit IRC16:06
gouthamra couple of feature patches in manila-ui: https://review.opendev.org/c/openstack/manila-ui/+/774438 https://review.opendev.org/c/openstack/manila-ui/+/76701716:09
gouthamrvkmc: i know you're both strapped for time - what is your feeling about these ^16:10
vkmcgouthamr, do we have until EOD today to review those?16:16
gouthamryes we do16:16
gouthamrpast dinner time for disap though - if we find any issues, lets see if we can do without a new PS - we can file bugs and fix them by RC16:17
vkmcthe resize share one looks fine to me16:17
vkmcthe sleep instruction is still there16:17
vkmcbut that shouldn't impact negatively on horizon16:18
vkmcat least, that's what we could understand from discussing with the horizon team16:18
gouthamr+116:18
gouthamri'll take a look now before stepping into dviroel security service land16:18
gouthamrcarloss: any chance you can help with https://review.opendev.org/c/openstack/python-manilaclient/+/77895516:19
*** disap has left #openstack-manila16:25
carlossyeah, sure gouthamr16:27
gouthamrTy carloss !16:28
carlossyw16:34
openstackgerritGoutham Pacha Ravi proposed openstack/manila master: [ci] Temporarily set docs job to non-voting  https://review.opendev.org/c/openstack/manila/+/77982317:22
*** chuan137 has quit IRC17:24
openstackgerritMerged openstack/python-manilaclient master: Correctly pass mTLS certificate/key to HTTPClient  https://review.opendev.org/c/openstack/python-manilaclient/+/77895517:31
openstackgerritGoutham Pacha Ravi proposed openstack/manila-ui master: Add example of logging setting for Manila  https://review.opendev.org/c/openstack/manila-ui/+/77550117:45
* gouthamr ironic that the docs job passes on a patch that seeks to set it to non-voting because it fails ¯\_(ツ)_/¯17:54
dviroellol18:02
dviroelit must fails to get a W+118:02
openstackgerritVictoria Martinez de la Cruz proposed openstack/manila master: Update cephfs drivers to use ceph-mgr client  https://review.opendev.org/c/openstack/manila/+/77961918:52
openstackgerritVictoria Martinez de la Cruz proposed openstack/devstack-plugin-ceph master: Enable testing Ceph Pacific  https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/77215719:38
openstackgerritVictoria Martinez de la Cruz proposed openstack/devstack-plugin-ceph master: [DNM] Enable testing Ceph Pacific  https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/77215719:39
openstackgerritVictoria Martinez de la Cruz proposed openstack/manila master: Update cephfs drivers to use ceph-mgr client  https://review.opendev.org/c/openstack/manila/+/77961919:41
openstackgerritMerged openstack/manila-ui master: Add example of logging setting for Manila  https://review.opendev.org/c/openstack/manila-ui/+/77550119:48
openstackgerritMerged openstack/manila master: [ci] Temporarily set docs job to non-voting  https://review.opendev.org/c/openstack/manila/+/77982320:35
gouthamrpraise be to zuul20:43
openstackgerritMerged openstack/devstack-plugin-ceph master: Stable jobs: run the tests on victoria, remove stein  https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/77898120:47
tbarrongouthamr: :D20:47
openstackgerritDouglas Viroel proposed openstack/manila master: [NetApp] Implement security service update  https://review.opendev.org/c/openstack/manila/+/77503220:52
openstackgerritVictoria Martinez de la Cruz proposed openstack/manila master: Add create share from snapshot in CephFS  https://review.opendev.org/c/openstack/manila/+/74156020:52
openstackgerritDouglas Viroel proposed openstack/manila master: [NetApp] Implement security service update  https://review.opendev.org/c/openstack/manila/+/77503220:54
dviroelgouthamr carloss regarding the bug that carthaca found while testing share server limits (https://review.opendev.org/c/openstack/manila/+/762725_21:21
dviroelhttps://review.opendev.org/c/openstack/manila/+/76272521:21
dviroelthe issue seems to happen when the new host is equal to parent's host21:22
dviroelbut the new share from snapshot ends up on a different share server21:23
dviroelthe driver fails on volume-clone operation21:23
*** rratnaka has joined #openstack-manila21:23
dviroelsince victoria we added more logic to validate share servers in 'choose_share_server_compatible_with_share'21:24
dviroelso, this scenario might happen if the share server is not healthy at the moment21:24
dviroelhttps://github.com/openstack/manila/blob/470ecda876c1f09d392c442c18a19b1b8ee7ee67/manila/share/drivers/netapp/dataontap/cluster_mode/lib_multi_svm.py#L775-L79821:25
dviroelthis is another test that I can do and see if it will fail on create from snapshot21:26
gouthamri see, so its possible today that the driver doesn't include the source share server at all?21:26
dviroellooking at the code, it is possible to happen. But the driver code expects that, if both 'hosts' are equal, the share from snapshot is in the same share server21:27
dviroeli will try to reproduce this behavior here21:29
gouthamrgreat thanks, it'd be nice for it to not be a limitation -> volmove-then-rehost is pretty cool logic; it'd be nice if that was repurposed to volmove-if-necessary-and-rehost-if-necessary21:31
*** rratnaka has quit IRC21:39
openstackgerritMerged openstack/manila master: [NetApp] Add support for FPolicy native mode  https://review.opendev.org/c/openstack/manila/+/77503822:14
dviroel\o/22:18
gouthamrdviroel: o/ too late in your day, but i'm done looking through the core code, it's in pretty good shape, i've suggested some things in line23:00
gouthamrdviroel: i'll request a delay on python-manilaclient and review https://review.opendev.org/c/openstack/python-manilaclient/+/77502023:00
dviroelgouthamr: great, thanks for reviewing it23:01

Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!