*** tosky has quit IRC | 00:00 | |
openstackgerrit | Rodrigo Barbieri proposed openstack/manila master: [pylint] Fix Manage-Unmanage with DHSS=True pylint issues https://review.openstack.org/641167 | 00:13 |
---|---|---|
*** hoonetorg has quit IRC | 00:24 | |
*** hoonetorg has joined #openstack-manila | 00:26 | |
*** dviroel_ has quit IRC | 00:37 | |
*** whoami-rajat has joined #openstack-manila | 01:09 | |
openstackgerrit | Merged openstack/manila master: Add manage/unmanage of shares in DHSS=True https://review.openstack.org/635831 | 01:12 |
*** mmethot has joined #openstack-manila | 01:15 | |
openstackgerrit | Merged openstack/manila master: [pylint] Fix Manage-Unmanage with DHSS=True pylint issues https://review.openstack.org/641167 | 03:06 |
*** ganso has quit IRC | 03:27 | |
*** ianychoi_ has joined #openstack-manila | 05:24 | |
*** ianychoi has quit IRC | 05:28 | |
*** hoonetorg has quit IRC | 05:33 | |
*** hoonetorg has joined #openstack-manila | 05:50 | |
*** e0ne has joined #openstack-manila | 06:31 | |
openstackgerrit | zhongshengping proposed openstack/puppet-manila master: Prepare Stein M3 https://review.openstack.org/641221 | 06:56 |
openstackgerrit | Liron Kuchlani proposed openstack/manila-tempest-plugin master: Use an instance method instread of class method https://review.openstack.org/641266 | 07:46 |
*** e0ne has quit IRC | 07:47 | |
*** e0ne has joined #openstack-manila | 07:54 | |
*** kopecmartin|off is now known as kopecmartin | 08:05 | |
*** e0ne has quit IRC | 08:16 | |
*** tosky has joined #openstack-manila | 08:29 | |
*** pcaruana has joined #openstack-manila | 08:29 | |
openstackgerrit | Liron Kuchlani proposed openstack/manila-tempest-plugin master: Use setup_clients classmethod for clients initialization https://review.openstack.org/641280 | 08:44 |
openstackgerrit | Nir Gilboa proposed openstack/manila-tempest-plugin master: Scenario test: Create/extend share and write data https://review.openstack.org/531568 | 09:36 |
*** luizbag has joined #openstack-manila | 10:06 | |
*** ganso has joined #openstack-manila | 10:16 | |
*** e0ne has joined #openstack-manila | 10:35 | |
*** dviroel_ has joined #openstack-manila | 10:43 | |
*** thgcorrea has joined #openstack-manila | 11:00 | |
*** FlorianFa has quit IRC | 11:01 | |
*** FlorianFa has joined #openstack-manila | 11:02 | |
*** pcaruana has quit IRC | 11:04 | |
*** raissa has joined #openstack-manila | 11:05 | |
*** erlon has joined #openstack-manila | 11:09 | |
*** erlon has quit IRC | 11:16 | |
*** erlon has joined #openstack-manila | 11:17 | |
*** ianychoi_ is now known as ianychoi | 11:20 | |
*** pcaruana has joined #openstack-manila | 11:32 | |
*** erlon has quit IRC | 12:03 | |
*** erlon has joined #openstack-manila | 12:04 | |
*** carlos_silva has joined #openstack-manila | 12:08 | |
*** enriquetaso has joined #openstack-manila | 12:09 | |
*** arne_wiebalck has quit IRC | 13:41 | |
*** arne_wiebalck_ has joined #openstack-manila | 13:42 | |
*** pcaruana has quit IRC | 13:51 | |
*** eharney has joined #openstack-manila | 13:51 | |
*** pcaruana has joined #openstack-manila | 14:01 | |
ganso | Ganso proposed openstack/python-manilaclient: Add CLI commands for Manage-Unmanage of Share Servers https://review.openstack.org/635915 | 14:04 |
ganso | openstackgerrit: hey you should have notified of that ^ :P | 14:05 |
*** FlorianFa has quit IRC | 14:09 | |
openstackgerrit | Yajushi Srivastava proposed openstack/manila master: Merge "[pylint] Fix Manage-Unmanage with DHSS=True pylint issues" https://review.openstack.org/641379 | 14:17 |
*** yajushiSri has joined #openstack-manila | 14:23 | |
*** arne_wiebalck_ is now known as arne_wiebalck | 14:35 | |
*** kshitija has joined #openstack-manila | 14:35 | |
*** arne_wiebalck has quit IRC | 14:35 | |
*** yajushiSri has quit IRC | 14:35 | |
*** arne_wiebalck_ has joined #openstack-manila | 14:37 | |
openstackgerrit | jacky06 proposed openstack/python-manilaclient master: Update hacking version https://review.openstack.org/641399 | 14:41 |
*** lseki has joined #openstack-manila | 15:06 | |
openstackgerrit | Yajushi Srivastava proposed openstack/manila master: Adds User Message in case of Shrinking Error https://review.openstack.org/641418 | 15:30 |
*** yajushiSri has joined #openstack-manila | 15:40 | |
*** pcaruana has quit IRC | 15:53 | |
*** raissa has quit IRC | 15:59 | |
*** lseki has quit IRC | 16:00 | |
*** erlon has quit IRC | 16:00 | |
*** luizbag has quit IRC | 16:00 | |
*** thgcorrea has quit IRC | 16:01 | |
*** luizbag has joined #openstack-manila | 16:01 | |
*** erlon has joined #openstack-manila | 16:03 | |
*** pcaruana has joined #openstack-manila | 16:06 | |
*** yajushiSri has quit IRC | 16:29 | |
*** yajushiSri has joined #openstack-manila | 16:31 | |
*** arne_wiebalck_ has quit IRC | 16:32 | |
*** arne_wiebalck_ has joined #openstack-manila | 16:32 | |
*** pcaruana has quit IRC | 16:55 | |
*** arne_wiebalck_ has quit IRC | 16:56 | |
*** yajushiSri has quit IRC | 17:03 | |
*** eharney has quit IRC | 17:06 | |
*** eharney has joined #openstack-manila | 17:13 | |
*** lseki has joined #openstack-manila | 17:22 | |
*** kopecmartin is now known as kopecmartin|off | 17:33 | |
*** e0ne has quit IRC | 17:34 | |
*** mmethot has quit IRC | 17:44 | |
*** mmethot has joined #openstack-manila | 17:45 | |
openstackgerrit | Nir Gilboa proposed openstack/manila-tempest-plugin master: Scenario test: Create/shrink share and write data https://review.openstack.org/531615 | 17:58 |
*** luizbag_ has joined #openstack-manila | 18:21 | |
*** ganso has quit IRC | 18:24 | |
*** luizbag__ has joined #openstack-manila | 18:25 | |
*** luizbag_ has quit IRC | 18:27 | |
*** luizbag has quit IRC | 18:27 | |
*** enriquetaso has quit IRC | 18:54 | |
openstackgerrit | Tom Barron proposed openstack/python-manilaclient master: Add CLI commands for Manage-Unmanage of Share Servers https://review.openstack.org/635915 | 19:54 |
*** tosky has quit IRC | 20:00 | |
*** tosky has joined #openstack-manila | 20:01 | |
*** luizbag__ has quit IRC | 20:01 | |
*** e0ne has joined #openstack-manila | 20:03 | |
*** arne_wiebalck has joined #openstack-manila | 20:06 | |
*** nirg22 has joined #openstack-manila | 20:19 | |
nirg22 | gouthamr: 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 IRC | 20:21 | |
*** ganso has joined #openstack-manila | 20:30 | |
ganso | tbarron, gouthamr: just got home and saw the -1 | 20:31 |
ganso | tbarron: thanks for fixing it | 20:31 |
ganso | gouthamr: I indeed had forgotten that unit test | 20:31 |
ganso | gouthamr: ok I see a -1 in the functional tests patch | 20:33 |
ganso | gouthamr: will work on it now | 20:35 |
openstackgerrit | Tom Barron proposed openstack/manila master: DNM: Test running all CI/CD on Bionic https://review.openstack.org/639140 | 20:36 |
gouthamr | ganso: thanks | 20:41 |
gouthamr | nirg22: 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 |
gouthamr | nirg22: 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 steps | 20:44 |
nirg22 | gouthamr: ok, got it | 20:47 |
openstackgerrit | Nir Gilboa proposed openstack/manila-tempest-plugin master: Scenario test: Create/shrink share and write data https://review.openstack.org/531615 | 20:52 |
*** erlon has quit IRC | 21:05 | |
ganso | gouthamr: 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-manila | 21:09 | |
ganso | gouthamr: it doesn't look appropriate to me | 21:10 |
gouthamr | ganso: looks like i missed a comment there, please add it at the class level then if all tests need to skip | 21:10 |
*** e0ne has joined #openstack-manila | 21:13 | |
*** kshitija has quit IRC | 21:14 | |
*** whoami-rajat has quit IRC | 21:18 | |
*** lseki has quit IRC | 21:21 | |
*** irclogbot_1 has quit IRC | 21:28 | |
*** eharney has quit IRC | 21:29 | |
openstackgerrit | Merged openstack/python-manilaclient master: Add CLI commands for Manage-Unmanage of Share Servers https://review.openstack.org/635915 | 21:56 |
gouthamr | tbarron 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_063167 | 21:57 |
gouthamr | the test failed due to an issue in the test, but, this is good and bad news | 21:58 |
*** e0ne has quit IRC | 21:58 | |
tbarron | ? | 21:59 |
gouthamr | i'm assuming you can't objectively determine when a backend's asynchronous ops will free up space after deleting files from a mounted share | 21:59 |
tbarron | right | 21:59 |
tbarron | so can you retry with a backoff? does tempest have a wait with retry already? | 22:00 |
gouthamr | tbarron: that's a good idea... we can write a test that waits for a configurable timeout with backoff | 22:00 |
gouthamr | s/waits/repeats | 22:00 |
tbarron | gouthamr: better than guessing sleep times :) | 22:01 |
tbarron | though that's fine for diagnosis, no criticism intended | 22:01 |
*** e0ne has joined #openstack-manila | 22:01 | |
gouthamr | tbarron: ack, we can use tempest's build_timeout as the limit | 22:01 |
*** e0ne has quit IRC | 22:01 | |
*** carlos_silva has quit IRC | 22:07 | |
*** bal has quit IRC | 22:11 | |
openstackgerrit | Rodrigo Barbieri proposed openstack/manila-tempest-plugin master: Add functional tests for manage/unmanage of shares in DHSS=True https://review.openstack.org/638231 | 22:42 |
openstackgerrit | Merged openstack/manila master: Fix missing size value in snapshot instance https://review.openstack.org/638681 | 23:17 |
openstackgerrit | Rodrigo Barbieri proposed openstack/manila-tempest-plugin master: Add functional tests for manage/unmanage of shares in DHSS=True https://review.openstack.org/638231 | 23:58 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!