Thursday, 2017-08-03

*** kaisers2 has joined #openstack-manila00:09
*** kaisers1 has quit IRC00:10
*** http_GK1wmSU has joined #openstack-manila00:22
*** http_GK1wmSU has left #openstack-manila00:24
*** baojg has joined #openstack-manila00:28
openstackgerritMerged openstack/manila master: Fix multiple issues with revert to snapshot in LVM  https://review.openstack.org/48856900:32
*** baojg has quit IRC00:33
*** jiaopengju has joined #openstack-manila00:34
openstackgerritMerged openstack/manila master: Fix cannot deny ipv6 access rules  https://review.openstack.org/48893800:43
*** tinwood has quit IRC00:56
*** tinwood has joined #openstack-manila00:57
*** kaisers_ has joined #openstack-manila01:10
*** kaisers has quit IRC01:12
*** baojg has joined #openstack-manila01:29
*** baojg has quit IRC01:34
*** mtanino has quit IRC01:59
openstackgerritzhongshengping proposed openstack/puppet-manila master: Fix unit test for manila::client class  https://review.openstack.org/49028402:09
*** gcb has quit IRC02:15
*** gcb has joined #openstack-manila02:16
*** baojg has joined #openstack-manila02:30
*** baojg has quit IRC02:38
*** ganso has quit IRC02:46
*** gouthamr has quit IRC03:03
*** tuanluong has joined #openstack-manila03:30
zhongjungouthamr: Hi,  we talked about this bug (https://bugs.launchpad.net/python-manilaclient/+bug/1693701) last night. It  is just want to show the 'created_at' and 'updated_at'  columns in echo when we run access_list command,  and it has been fix in the following patch. https://bugs.launchpad.net/manila/+bug/168279503:40
openstackLaunchpad bug 1693701 in python-manilaclient "Allow user to see which access rule is recent created or recent updated" [Wishlist,Invalid] - Assigned to NidhiMittalHada (nidhimittal19)03:40
openstackLaunchpad bug 1682795 in Manila "Allow user to see which access rule is recent created or recent updated." [Undecided,Fix released] - Assigned to yangweiwei (496176919-6)03:40
zhongjunbswartz: above03:41
openstackgerritMerged openstack/python-manilaclient master: Stop skipping ipv6 delete access rule test  https://review.openstack.org/49011303:46
*** vivsoni_ has quit IRC04:20
*** kaisers_ has quit IRC04:30
*** baojg has joined #openstack-manila04:35
*** baojg has quit IRC04:42
*** vivsoni has joined #openstack-manila04:49
*** kaisers has joined #openstack-manila05:00
*** kaisers has quit IRC05:05
*** kaisers has joined #openstack-manila05:05
*** kaisers has quit IRC05:09
openstackgerritjunbo.li proposed openstack/manila master: Enable some off-by-default checks  https://review.openstack.org/48470705:14
*** lpetrut has joined #openstack-manila05:14
*** arnewiebalck_ has joined #openstack-manila05:16
*** dsariel has joined #openstack-manila05:36
*** baojg has joined #openstack-manila05:38
*** baojg has quit IRC05:43
*** dims has quit IRC05:52
*** dims has joined #openstack-manila05:54
*** arnewiebalck_ has quit IRC06:02
*** dsariel has quit IRC06:06
*** kaisers has joined #openstack-manila06:10
*** lpetrut has quit IRC06:11
*** hoonetorg has quit IRC06:12
*** arnewiebalck_ has joined #openstack-manila06:15
*** kaisers has quit IRC06:15
*** arnewiebalck_ has quit IRC06:20
*** dsariel has joined #openstack-manila06:21
*** hoonetorg has joined #openstack-manila06:25
*** baojg has joined #openstack-manila06:39
*** baojg has quit IRC06:44
*** pcaruana has joined #openstack-manila06:48
*** gcb has quit IRC07:05
*** kaisers has joined #openstack-manila07:11
*** baojg has joined #openstack-manila07:40
*** baojg has quit IRC07:44
*** kaisers has quit IRC07:58
openstackgerritOpenStack Proposal Bot proposed openstack/manila master: Imported Translations from Zanata  https://review.openstack.org/49038008:05
*** lpetrut has joined #openstack-manila08:07
openstackgerritjunbo.li proposed openstack/manila master: Enable some off-by-default checks  https://review.openstack.org/48470708:14
*** openstackgerrit has quit IRC08:18
*** baojg has joined #openstack-manila08:41
*** baojg has quit IRC08:48
*** kaisers has joined #openstack-manila08:55
*** dims has quit IRC09:04
*** arnewiebalck has joined #openstack-manila09:06
*** dims has joined #openstack-manila09:08
*** kaisers has quit IRC09:16
*** baojg has joined #openstack-manila09:43
*** baojg has quit IRC09:48
*** ganso has joined #openstack-manila09:56
*** dsariel has quit IRC10:00
*** kaisers has joined #openstack-manila10:13
*** dsariel has joined #openstack-manila10:13
*** jiaopengju has quit IRC10:14
*** dsariel has quit IRC10:19
*** huyang has joined #openstack-manila10:27
*** dsariel has joined #openstack-manila10:32
*** dsariel has quit IRC10:38
*** kaisers has quit IRC10:40
*** baojg has joined #openstack-manila10:44
*** baojg has quit IRC10:49
*** dsariel has joined #openstack-manila10:50
*** kaisers has joined #openstack-manila11:05
tommylikehuhey bswartz , could you share the reason why you removed the lvchange command when reverting the share in lvm driver? https://review.openstack.org/#/c/488569/7/manila/share/drivers/lvm.py11:07
*** openstackgerrit has joined #openstack-manila11:09
openstackgerritTommyLike proposed openstack/manila master: Fix: [LVM] Can't extend share when snapshots exist  https://review.openstack.org/43131511:09
*** kaisers has quit IRC11:16
*** dsariel has quit IRC11:37
*** baojg has joined #openstack-manila11:45
*** baojg has quit IRC11:49
*** baojg has joined #openstack-manila11:49
*** dsariel has joined #openstack-manila11:52
*** eharney has joined #openstack-manila11:55
*** vivsoni has quit IRC11:57
*** zengyingzhe has joined #openstack-manila12:08
*** rraja has joined #openstack-manila12:09
*** kaisers has joined #openstack-manila12:13
*** tuanluong has quit IRC12:14
*** tinwood is now known as tinwood-afk12:30
*** catintheroof has joined #openstack-manila12:36
*** kaisers has quit IRC12:36
*** gcb has joined #openstack-manila12:41
openstackgerritMerged openstack/manila master: Enable some off-by-default checks  https://review.openstack.org/48470713:21
*** kaisers has joined #openstack-manila13:34
*** tinwood-afk is now known as tinwood13:41
openstackgerritMerged openstack/manila master: Use tempest-plugin service client registration  https://review.openstack.org/33459613:42
*** kaisers has quit IRC13:47
*** cknight has joined #openstack-manila13:54
bswartztommylikehu: it was removed because it wasn't needed14:00
bswartztommylikehu: it was only included initially because I was doing revert wrong14:00
*** baojg has quit IRC14:01
bswartzthe bug fix was to do the the revert right, and then the command wasn't needed anymore14:01
*** baojg has joined #openstack-manila14:01
*** xyang1 has joined #openstack-manila14:04
tommylikehubswartz:  hah, thanks!14:05
*** dustins has joined #openstack-manila14:07
openstackgerritBen Swartzlander proposed openstack/manila master: Fix NFSHelper 0-length netmask bug  https://review.openstack.org/48963914:07
openstackgerritMerged openstack/manila master: Imported Translations from Zanata  https://review.openstack.org/49038014:25
*** baojg has quit IRC14:32
*** baojg has joined #openstack-manila14:33
*** lpetrut has quit IRC14:39
*** kaisers has joined #openstack-manila14:44
*** MVenesio has joined #openstack-manila14:47
*** eharney has quit IRC14:53
*** eharney has joined #openstack-manila15:09
openstackgerritMerged openstack/python-manilaclient master: Add rally job manifests  https://review.openstack.org/48943515:09
*** kaisers has quit IRC15:19
*** baojg has quit IRC16:00
*** kaisers has joined #openstack-manila16:16
*** jmlowe has quit IRC16:20
*** rraja has quit IRC16:26
*** kaisers has quit IRC16:27
*** JoseMello has joined #openstack-manila16:29
*** dsariel has quit IRC16:34
*** jmlowe has joined #openstack-manila16:42
*** pcaruana has quit IRC16:54
*** kaisers has joined #openstack-manila17:21
tbarronbswartz: just want to check if it is expected that with synchronous lvm snapshot revert test cases will take 50-70s each17:28
tbarronbswartz: and if we understand the long pole phases in the revert17:29
tbarronbswartz: see for example http://logs.openstack.org/39/489639/7/check/gate-manila-tempest-minimal-dsvm-lvm-centos-7/62f0c3f/console.html#_2017-08-03_15_11_53_72181017:29
bswartztbarron: errr17:30
bswartzI would have guessed 10-15 seconds17:30
tbarronbswartz: note though that the slowest tests in the lvm jobs are host assisted migration, for which I raised: https://bugs.launchpad.net/manila/+bug/170849117:31
openstackLaunchpad bug 1708491 in Manila "tempest api migration tests very long" [Undecided,New]17:31
bswartzI guess the test nodes are under heavy load due to tempest17:31
bswartzk17:31
bswartzI'm in favor of increasing timeouts in the short term17:31
tbarronwhen we get a "slow node" plus these tests running slow ...17:31
bswartznot because of the slow tests, but because of teh slow nodes17:31
bswartzI doubt much can be done about the slowness of lvm revert17:32
bswartzwe could reduce parallelism which would speed up the individual tests but slow down the overall job17:32
tbarronbswartz: k, was just checking.  Having it synchronous shows things better now.17:32
bswartzwe could shuffle tests around to take better advantage of parallelism17:32
tbarronyeah, we could try to put a host assisted migration case in every thread :D17:33
bswartzor we can reduce the number of test cases (possibly reorganizing them to keep good coverage with fewer tests)17:33
tbarronbut increasing timeouts may make sense if it's acceptable to infra17:33
tbarronwe're certainly using more infra resources with all the rechecks than we would by running fewer jobs a bit longer17:34
bswartzwhat jobs other than the centos lvm job are having problems?17:34
bswartzI'd like to do a blanket increase, but if we can pick 1 or 2 that really need it we might get a better response17:35
tbarronxenial lvm has the same issues17:35
tbarrongeneric jobs are slow, if non-voting17:35
bswartzI see17:36
tbarronhttp://logs.openstack.org/39/489639/7/check/gate-manila-tempest-dsvm-mysql-generic-ubuntu-xenial-nv/05efe94/console.html17:36
tbarron^^^ host assisted migration jobs run 10 minutes and some fail because of tempest timeouts17:36
tbarronwe just ignore these failures and move on17:37
bswartzdid ganso write those tests?17:37
tbarrondunno17:37
tbarronthese aren't really rc candidate fixes unless we happen to see a solution right away17:38
bswartzI just haven't had time to dig into them17:38
tbarronbut since we're scrubbing and doing lots of rechecks I wanted to make sure we note and don't lose the issues17:38
tbarronack, me either17:39
tbarronI've been a lazy re-checker.17:39
bswartztimeouts are harder to debug than a failing test17:42
*** cknight1 has joined #openstack-manila17:52
*** cknight has quit IRC17:52
*** gouthamr has joined #openstack-manila17:53
*** kaisers has quit IRC17:58
*** kaisers has joined #openstack-manila18:09
gansotbarron: are those failures consistent across several rechecks? on generic driver it sometimes fails on several steps for unknown reasons18:11
tbarronganso: well those test cases seem to take a long time on generic even when the job passes, e.g. in that same review there's a run with no failed test cases but we see e.g.18:15
tbarronhttp://logs.openstack.org/39/489639/7/check/gate-manila-tempest-dsvm-mysql-generic-ubuntu-xenial-nv/05efe94/console.html18:15
tbarronthat's 9 minutes on the test case instead of 1018:16
gansotbarron: there are no errors on m-dat.log so the migration did not even start18:18
gansotbarron: in m-shr there are errors on cinder-client failing to attach18:18
tbarronganso: interesting18:19
tbarronganso: so there may be other issues with generic driver that the migration tests just happen to trigger18:20
gansotbarron: yes. It has always been like this, those are long-standing issues with the generic driver, not new issues18:21
tbarronganso: would you expect this with lvm job (no cinder involved): http://logs.openstack.org/39/489639/7/check/gate-manila-tempest-minimal-dsvm-lvm-ubuntu-xenial/3497af0/console.html#_2017-08-03_16_44_17_824670 ?18:21
tbarronganso: btw, I'm not claiming any of these issues are new, I'm just trying to analyze why we have to issue rechecks so much.18:22
tbarronganso: so I'm looking at which test cases are long-poles.18:23
gansotbarron: for this test MigrationWithShareExtendingNFSTest.test_extend_on_migrated_share_1_True I think the duration is not that bad18:23
tbarron ganso so you expect it to take 85s even with lvm ...18:24
tbarronganso: and this scenario test to take > 4 min? http://logs.openstack.org/39/489639/7/check/gate-manila-tempest-minimal-dsvm-lvm-ubuntu-xenial/3497af0/console.html#_2017-08-03_16_49_07_83949518:25
tbarronganso: I'm just checking, don't have an informed opinion myself.18:25
gansotbarron: the numbers are slightly high indeed18:25
gansotbarron: maybe this is a slow host, I don't recall the numbers last time I benchmarked this18:26
tbarronor 299s: http://logs.openstack.org/39/489639/7/check/gate-manila-tempest-minimal-dsvm-lvm-ubuntu-xenial/3497af0/console.html#_2017-08-03_17_01_39_61014318:26
tbarronwe may want to run those migration scenarios in a separate job18:27
tbarronI have a feeling there are a lot of valuable scenario tests we should write that18:27
tbarronare going to take too long if just added to current jobs.18:28
*** kaisers has quit IRC18:35
gansotbarron: I remember a single test did not take more than 2 minutes in my lab18:36
gansotbarron: using LVM18:36
gansotbarron: I suggested moving them to a separate scenario job long time ago, but most people did not agree it was necessary back then and wanted to concentrate efforts somewhere else, like further optimizing the tests18:37
gansotbarron: vponomaryov invested some effort on that and reduced the total time of those tests (not by much though)18:38
tbarronganso: ack18:39
*** kaisers has joined #openstack-manila18:39
tbarronganso: bswartz may be getting us some more time for these jobs anyways ....18:39
*** kaisers has quit IRC18:46
*** MVenesio has quit IRC18:47
bswartzganso: hi18:59
bswartzwe missed you at bug squash yesterday18:59
bswartzwe were talking about the migration tests and trying to understand if there are any opportunities for optimization18:59
gansobswartz: sorry I really couldn't be there :(19:00
gansobswartz: regarding tests, I believe we could increase polling rate in the code and in tests... but that's pretty much about it19:01
gansobswartz: if the problem is timeout, then we could separate the tests to a new host19:01
bswartzganso: so the tests aren't doing a lot of I/O and there are long pauses in the waiter functions?19:07
gansobswartz: not long pauses, and not a lot of I/O19:13
gansobswartz: the only ones that do any I/O are the scenario tests19:14
gansobswartz: there are a lot of waiters at each step19:14
bswartzokay I'll need to run some experiments19:14
*** eharney has quit IRC19:22
*** Yogi1 has joined #openstack-manila19:38
*** kaisers has joined #openstack-manila19:43
*** harlowja has quit IRC19:57
bswartzgouthamr: ping20:00
gouthamrbswartz: pong20:00
*** eharney has joined #openstack-manila20:02
*** arnewiebalck_ has joined #openstack-manila20:13
*** kaisers has quit IRC20:15
*** lpetrut has joined #openstack-manila20:23
*** arnewiebalck_ has quit IRC20:35
*** Yogi1 has quit IRC21:05
*** kaisers has joined #openstack-manila21:07
*** harlowja has joined #openstack-manila21:15
*** jmlowe has quit IRC21:18
*** lpetrut has quit IRC21:37
*** kaisers has quit IRC21:51
*** dustins has quit IRC22:21
*** xyang1 has quit IRC22:25
*** JoseMello has quit IRC22:26
*** cknight1 has quit IRC23:02
*** ganso has quit IRC23:15
*** jmlowe has joined #openstack-manila23:27

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