Wednesday, 2019-03-06

*** tosky has quit IRC00:00
openstackgerritRodrigo Barbieri proposed openstack/manila master: [pylint] Fix Manage-Unmanage with DHSS=True pylint issues  https://review.openstack.org/64116700:13
*** hoonetorg has quit IRC00:24
*** hoonetorg has joined #openstack-manila00:26
*** dviroel_ has quit IRC00:37
*** whoami-rajat has joined #openstack-manila01:09
openstackgerritMerged openstack/manila master: Add manage/unmanage of shares in DHSS=True  https://review.openstack.org/63583101:12
*** mmethot has joined #openstack-manila01:15
openstackgerritMerged openstack/manila master: [pylint] Fix Manage-Unmanage with DHSS=True pylint issues  https://review.openstack.org/64116703:06
*** ganso has quit IRC03:27
*** ianychoi_ has joined #openstack-manila05:24
*** ianychoi has quit IRC05:28
*** hoonetorg has quit IRC05:33
*** hoonetorg has joined #openstack-manila05:50
*** e0ne has joined #openstack-manila06:31
openstackgerritzhongshengping proposed openstack/puppet-manila master: Prepare Stein M3  https://review.openstack.org/64122106:56
openstackgerritLiron Kuchlani proposed openstack/manila-tempest-plugin master: Use an instance method instread of class method  https://review.openstack.org/64126607:46
*** e0ne has quit IRC07:47
*** e0ne has joined #openstack-manila07:54
*** kopecmartin|off is now known as kopecmartin08:05
*** e0ne has quit IRC08:16
*** tosky has joined #openstack-manila08:29
*** pcaruana has joined #openstack-manila08:29
openstackgerritLiron Kuchlani proposed openstack/manila-tempest-plugin master: Use setup_clients classmethod for clients initialization  https://review.openstack.org/64128008:44
openstackgerritNir Gilboa proposed openstack/manila-tempest-plugin master: Scenario test: Create/extend share and write data  https://review.openstack.org/53156809:36
*** luizbag has joined #openstack-manila10:06
*** ganso has joined #openstack-manila10:16
*** e0ne has joined #openstack-manila10:35
*** dviroel_ has joined #openstack-manila10:43
*** thgcorrea has joined #openstack-manila11:00
*** FlorianFa has quit IRC11:01
*** FlorianFa has joined #openstack-manila11:02
*** pcaruana has quit IRC11:04
*** raissa has joined #openstack-manila11:05
*** erlon has joined #openstack-manila11:09
*** erlon has quit IRC11:16
*** erlon has joined #openstack-manila11:17
*** ianychoi_ is now known as ianychoi11:20
*** pcaruana has joined #openstack-manila11:32
*** erlon has quit IRC12:03
*** erlon has joined #openstack-manila12:04
*** carlos_silva has joined #openstack-manila12:08
*** enriquetaso has joined #openstack-manila12:09
*** arne_wiebalck has quit IRC13:41
*** arne_wiebalck_ has joined #openstack-manila13:42
*** pcaruana has quit IRC13:51
*** eharney has joined #openstack-manila13:51
*** pcaruana has joined #openstack-manila14:01
gansoGanso proposed openstack/python-manilaclient: Add CLI commands for Manage-Unmanage of Share Servers  https://review.openstack.org/63591514:04
gansoopenstackgerrit: hey you should have notified of that ^ :P14:05
*** FlorianFa has quit IRC14:09
openstackgerritYajushi Srivastava proposed openstack/manila master: Merge "[pylint] Fix Manage-Unmanage with DHSS=True pylint issues"  https://review.openstack.org/64137914:17
*** yajushiSri has joined #openstack-manila14:23
*** arne_wiebalck_ is now known as arne_wiebalck14:35
*** kshitija has joined #openstack-manila14:35
*** arne_wiebalck has quit IRC14:35
*** yajushiSri has quit IRC14:35
*** arne_wiebalck_ has joined #openstack-manila14:37
openstackgerritjacky06 proposed openstack/python-manilaclient master: Update hacking version  https://review.openstack.org/64139914:41
*** lseki has joined #openstack-manila15:06
openstackgerritYajushi Srivastava proposed openstack/manila master: Adds User Message in case of Shrinking Error  https://review.openstack.org/64141815:30
*** yajushiSri has joined #openstack-manila15:40
*** pcaruana has quit IRC15:53
*** raissa has quit IRC15:59
*** lseki has quit IRC16:00
*** erlon has quit IRC16:00
*** luizbag has quit IRC16:00
*** thgcorrea has quit IRC16:01
*** luizbag has joined #openstack-manila16:01
*** erlon has joined #openstack-manila16:03
*** pcaruana has joined #openstack-manila16:06
*** yajushiSri has quit IRC16:29
*** yajushiSri has joined #openstack-manila16:31
*** arne_wiebalck_ has quit IRC16:32
*** arne_wiebalck_ has joined #openstack-manila16:32
*** pcaruana has quit IRC16:55
*** arne_wiebalck_ has quit IRC16:56
*** yajushiSri has quit IRC17:03
*** eharney has quit IRC17:06
*** eharney has joined #openstack-manila17:13
*** lseki has joined #openstack-manila17:22
*** kopecmartin is now known as kopecmartin|off17:33
*** e0ne has quit IRC17:34
*** mmethot has quit IRC17:44
*** mmethot has joined #openstack-manila17:45
openstackgerritNir Gilboa proposed openstack/manila-tempest-plugin master: Scenario test: Create/shrink share and write data  https://review.openstack.org/53161517:58
*** luizbag_ has joined #openstack-manila18:21
*** ganso has quit IRC18:24
*** luizbag__ has joined #openstack-manila18:25
*** luizbag_ has quit IRC18:27
*** luizbag has quit IRC18:27
*** enriquetaso has quit IRC18:54
openstackgerritTom Barron proposed openstack/python-manilaclient master: Add CLI commands for Manage-Unmanage of Share Servers  https://review.openstack.org/63591519:54
*** tosky has quit IRC20:00
*** tosky has joined #openstack-manila20:01
*** luizbag__ has quit IRC20:01
*** e0ne has joined #openstack-manila20:03
*** arne_wiebalck has joined #openstack-manila20:06
*** nirg22 has joined #openstack-manila20:19
nirg22gouthamr: thanks for your reviews :) I'm not sure how to add a waiter for a limited time period.. waiters.py in tempest.common doesn't provide any such methods.20:21
*** e0ne has quit IRC20:21
*** ganso has joined #openstack-manila20:30
gansotbarron, gouthamr: just got home and saw the -120:31
gansotbarron: thanks for fixing it20:31
gansogouthamr: I indeed had forgotten that unit test20:31
gansogouthamr: ok I see a -1 in the functional tests patch20:33
gansogouthamr: will work on it now20:35
openstackgerritTom Barron proposed openstack/manila master: DNM: Test running all CI/CD on Bionic  https://review.openstack.org/63914020:36
gouthamrganso: thanks20:41
gouthamrnirg22: oh, i meant a sleep, this is to just confirm the theory that it is indeed an asynchronous cleanup that's causing the zfs driver to fail your test..20:42
gouthamrnirg22: i don't expect us to merge the test with a sleep statement in there, maybe there's a better approach to implement the test as intended.. or, we can remove those final steps20:44
nirg22gouthamr: ok, got it20:47
openstackgerritNir Gilboa proposed openstack/manila-tempest-plugin master: Scenario test: Create/shrink share and write data  https://review.openstack.org/53161520:52
*** erlon has quit IRC21:05
gansogouthamr: any reason you don't think test_unmanage_share_server_invalid_state isn't appropriate while all other tests in https://review.openstack.org/#/c/638231/24/manila_tempest_tests/tests/api/admin/test_share_servers_manage_negative.py are?21:09
*** irclogbot_1 has joined #openstack-manila21:09
gansogouthamr: it doesn't look appropriate to me21:10
gouthamrganso: looks like i missed a comment there, please add it at the class level then if all tests need to skip21:10
*** e0ne has joined #openstack-manila21:13
*** kshitija has quit IRC21:14
*** whoami-rajat has quit IRC21:18
*** lseki has quit IRC21:21
*** irclogbot_1 has quit IRC21:28
*** eharney has quit IRC21:29
openstackgerritMerged openstack/python-manilaclient master: Add CLI commands for Manage-Unmanage of Share Servers  https://review.openstack.org/63591521:56
gouthamrtbarron nirg22: so, the sleep/waiter in the shrink test worked.... zfs driver reported the correct size http://logs.openstack.org/15/531615/31/check/manila-tempest-dsvm-postgres-zfsonlinux/eb88c51/logs/screen-m-shr.txt.gz#_Mar_06_21_35_05_06316721:57
gouthamrthe test failed due to an issue in the test, but, this is good and bad news21:58
*** e0ne has quit IRC21:58
tbarron?21:59
gouthamri'm assuming you can't objectively determine when a backend's asynchronous ops will free up space after deleting files from a mounted share21:59
tbarronright21:59
tbarronso can you retry with a backoff?  does tempest have a wait with retry already?22:00
gouthamrtbarron: that's a good idea... we can write a test that waits for a configurable timeout with backoff22:00
gouthamrs/waits/repeats22:00
tbarrongouthamr: better than guessing sleep times :)22:01
tbarronthough that's fine for diagnosis, no criticism intended22:01
*** e0ne has joined #openstack-manila22:01
gouthamrtbarron: ack, we can use tempest's build_timeout as the limit22:01
*** e0ne has quit IRC22:01
*** carlos_silva has quit IRC22:07
*** bal has quit IRC22:11
openstackgerritRodrigo Barbieri proposed openstack/manila-tempest-plugin master: Add functional tests for manage/unmanage of shares in DHSS=True  https://review.openstack.org/63823122:42
openstackgerritMerged openstack/manila master: Fix missing size value in snapshot instance  https://review.openstack.org/63868123:17
openstackgerritRodrigo Barbieri proposed openstack/manila-tempest-plugin master: Add functional tests for manage/unmanage of shares in DHSS=True  https://review.openstack.org/63823123:58

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!