Friday, 2020-05-22

*** tosky has quit IRC00:01
*** brinzhang_ has joined #openstack-manila01:07
*** brinzhang has quit IRC01:10
*** andrebeltrami has quit IRC01:58
*** brinzhang has joined #openstack-manila03:03
*** brinzhang_ has quit IRC03:06
*** brinzhang_ has joined #openstack-manila03:18
*** brinzhang has quit IRC03:21
*** tobberydberg has quit IRC03:25
*** tobberydberg has joined #openstack-manila03:35
*** brinzhang has joined #openstack-manila03:36
*** brinzhang_ has quit IRC03:40
*** toabctl has joined #openstack-manila04:46
openstackgerritmelissaml proposed openstack/manila master: Switch to newer openstackdocstheme and reno versions  https://review.opendev.org/73019606:17
openstackgerritmelissaml proposed openstack/manila master: Switch to newer openstackdocstheme and reno versions  https://review.opendev.org/73019606:25
*** tosky has joined #openstack-manila07:34
*** tkajinam_ has quit IRC08:05
*** lpetrut has joined #openstack-manila08:26
*** brinzhang has quit IRC11:09
*** lpetrut has quit IRC13:35
openstackgerritVictoria Martinez de la Cruz proposed openstack/python-manilaclient master: Re-enable shell completion cache  https://review.opendev.org/63942416:23
gouthamrdviroel carloss vkmc: hey o/ needed a cephfs image to test the manila-tempest-plugin change we're targeting for the next tag.. can you folks take a look at this build job please? :) https://review.opendev.org/#/c/729806/16:30
vkmcgouthamr, right away16:31
carlosswill do gouthamr :)16:31
gouthamrthank you vkmc carloss!16:31
vkmcgouthamr, carloss, can I get your feedback on this one? https://review.opendev.org/#/c/729226/16:32
gouthamrvkmc: sure thing..16:32
carlosssure, vkmc :)16:33
vkmcyaaay16:33
vkmcnow carloss, it's your turn to ask us for reviews16:33
carlosshaha16:33
carlossI think I'll break the wheel :/16:35
* carloss realizes that the above phrase makes more sense if you've watched game of thrones 16:35
vkmccarloss, all people here watched game of thrones, it's in the how to zorilla manual16:36
vkmcright?16:36
carlossI think so, and if it isn't yet, it should be :p16:36
vkmcxD16:36
vkmcwe are all in quarantine, no excuses16:37
carlosssure haha16:38
* dviroel is still waiting for book 616:40
* carloss is still taking courage to start reading the 1st book16:43
gouthamrdviroel: "winds of winter" will make it when felipe can read and takes interest in fantasy fiction, fingers crossed16:46
dviroelgouthamr: yeah, probably. at least, I guess that I'll have more time to read, which is impossible nowadays16:53
gouthamr:) grrm sees that, and obliges you16:54
dviroelI still have to read fire & blood too, its in the shelf16:55
openstackgerritAndreas Jaeger proposed openstack/python-manilaclient master: Switch to newer openstackdocstheme and reno versions  https://review.opendev.org/72901917:23
*** tosky has quit IRC17:50
*** tosky has joined #openstack-manila17:51
openstackgerritMerged openstack/manila master: [NetApp] Fix falsely report migration cancelation success  https://review.opendev.org/71823317:52
openstackgerritDouglas Viroel proposed openstack/manila stable/ussuri: [NetApp] Fix falsely report migration cancelation success  https://review.opendev.org/73038317:55
gouthamrvkmc: i dug in a bit more, and found what we were doing wrong20:25
gouthamrvkmc: we have wrong logic in three test classes where we pick up the wrong protocol to test with becuase we're just relying on these enable_*_rules_for_protocols options20:26
gouthamrvkmc: can i submit an update to https://review.opendev.org/#/c/729226/ ?20:27
*** andrebeltrami has joined #openstack-manila20:40
openstackgerritGoutham Pacha Ravi proposed openstack/manila-tempest-plugin master: Cleanup access rule test skip conditions  https://review.opendev.org/72922620:41
gouthamrvkmc: pushed it up, we can discuss and enhance :) i agree this is an annoying test bug20:42
*** lpetrut has joined #openstack-manila20:43
vkmcgouthamr, checking20:44
vkmcgouthamr, awesome!20:45
vkmcyes, I was thinking a different approach but was more complex (which I was trying to avoid20:45
vkmc)20:45
vkmcI like your solution :)20:45
vkmcthat's the only blocker that I have (that I know) for the cephfs native zuul v3 job20:46
vkmcI hit that when running all api tests20:47
vkmcso, if we can get that in, I'll update the native zuulv3 job20:47
*** vhari has quit IRC20:51
vkmcgouthamr, the only detail I see to that is that we still fail to disable those tests20:51
gouthamrvkmc: we do?20:52
vkmcunless I'm missing something20:52
gouthamrvkmc: we won't disable them, but we'll use the right protocol and access rule ...20:52
vkmcgouthamr, yeah, I don't think is harmful20:53
gouthamrvkmc: yes, i added a comment to clarify that these tests don't care about the protocol and actual access rule  - they only care about testing the API fields and behavior --- which should be the same no matter the protocol and the access type/access_to20:54
vkmcgouthamr, cool :)20:54
*** lpetrut has quit IRC21:15
*** lpetrut has joined #openstack-manila21:26
openstackgerritMerged openstack/python-manilaclient master: Switch to newer openstackdocstheme and reno versions  https://review.opendev.org/72901921:42
*** lpetrut has quit IRC22:00
*** eharney has quit IRC22:07
*** jmlowe has quit IRC22:20
*** jmlowe has joined #openstack-manila22:22
*** tosky has quit IRC22:59
*** andrebeltrami has quit IRC23:10

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