Tuesday, 2019-04-30

*** luksky has quit IRC00:08
*** mriedem has joined #openstack-cinder00:09
*** mriedem has quit IRC00:17
*** brinzhang has joined #openstack-cinder00:18
*** jmlowe has joined #openstack-cinder00:58
*** dklyle has joined #openstack-cinder01:07
*** dklyle has quit IRC01:12
openstackgerritzhufl proposed openstack/cinder master: Fix invalid function name  https://review.opendev.org/65625801:22
openstackgerritQian Mingyue proposed openstack/cinder master: Update Fujitsu ETERNUS DX driver  https://review.opendev.org/60964501:28
*** gmann has quit IRC01:39
*** whoami-rajat has joined #openstack-cinder02:01
*** ruffian_sheep has joined #openstack-cinder02:08
*** gmann has joined #openstack-cinder02:09
*** e0ne has joined #openstack-cinder02:20
*** gnufied has quit IRC02:21
*** e0ne has quit IRC02:21
ruffian_sheepwhoami-rajat:I finally ran the tempest, but the use case was only 159, and I skipped the 5 backup functions.02:25
ruffian_sheepwhoami-rajat:I set the  regex pattern with test_*volume* in http://paste.openstack.org/show/750071/ ,using it with the other one file http://paste.openstack.org/show/750072/02:33
*** tejdeep has joined #openstack-cinder02:34
ruffian_sheepsmcginnis |  jungleboyj:Is it enough?02:37
ruffian_sheepsmcginnis |  jungleboyj:I am sorry that my server is temporarily unable to provide a connection. Only html content can be sent. You can paste it into a new html file to check it out.http://paste.openstack.org/show/750073/02:41
whoami-rajatruffian_sheep: the '*test' part seems wrong, it will run private helper methods too.03:02
*** markvoelker has joined #openstack-cinder03:12
ruffian_sheepwhoami-rajat:In fact, there is no file matching *test, which is the beginning of test.http://paste.openstack.org/show/750074/03:25
ruffian_sheepwhoami-rajat:The dir of the cases is /opt/stack/tempest/tempest/api/volume/03:25
*** psachin has joined #openstack-cinder03:25
openstackgerritDingDong proposed openstack/cinder master: [Unity] Retype volume support  https://review.opendev.org/65627103:30
*** enriquetaso has joined #openstack-cinder03:38
*** tejdeep has quit IRC03:45
*** gkadam has joined #openstack-cinder03:49
*** udesale has joined #openstack-cinder04:08
*** enriquetaso has quit IRC04:08
whoami-rajatruffian_sheep: it is because of *volume* keyword, if we will add a new method in tempest named _test_random_volume_helper() then your regex will run it and cause failure04:16
whoami-rajatruffian_sheep: currently there isn't any helper method containing the *volume* keyword04:16
whoami-rajatruffian_sheep: also IIRC, my recommendation of creating the regex was to run all volume tests (in tempest/api/volume) without any regex and apply regex in other folders to run volume related tests Eg in tempest/api/compute ...04:18
*** ruffian_sheep has quit IRC04:19
*** _alastor_ has joined #openstack-cinder04:43
*** pooja_jadhav has joined #openstack-cinder04:49
*** gmann has quit IRC04:49
*** e0ne has joined #openstack-cinder04:55
*** e0ne has quit IRC04:57
*** poojajadhav has joined #openstack-cinder04:59
*** pooja_jadhav has quit IRC05:02
*** e0ne has joined #openstack-cinder05:06
*** gmann has joined #openstack-cinder05:08
*** e0ne has quit IRC05:10
*** pjjadhav has joined #openstack-cinder05:14
*** tejdeep has joined #openstack-cinder05:15
*** poojajadhav has quit IRC05:17
*** boxiang has quit IRC05:22
*** boxiang has joined #openstack-cinder05:22
*** dpawlik has joined #openstack-cinder05:25
*** tejdeep has quit IRC05:40
openstackgerritDingDong proposed openstack/cinder master: [Unity] Retype volume support  https://review.opendev.org/65627105:47
*** ruffian_sheep has joined #openstack-cinder05:54
*** pjjadhav has quit IRC05:54
*** poojajadhav has joined #openstack-cinder05:54
ruffian_sheepwhoami-rajat:Then I set it to test*volume*.py would be better.Right?05:56
*** poojajadhav has quit IRC05:59
*** pooja_jadhav has joined #openstack-cinder05:59
ruffian_sheepwhoami-rajat:I will modify the regex and the path to run it again.06:02
*** tejdeep has joined #openstack-cinder06:03
whoami-rajatruffian_sheep: the main idea behind the regex is to be able to run all volume tests irrespective of their location.06:09
whoami-rajatso the following are my recommendations :06:09
whoami-rajat1) run all tests under tempest/api/volumes06:09
whoami-rajat2) create a regex to be able to fetch tests related to volumes under other directories06:09
whoami-rajatpointers to create the regex :06:09
whoami-rajat1) manually check for volume related tests under other directories compute/identity ... etc06:09
whoami-rajat2) try to create a regex that will cover all the tests that you found in (1)06:09
whoami-rajatthere doesn't seem a direct solution to this and maybe there is that i'm not aware of but i would recommend following the procedure above.06:09
whoami-rajatexceptions for current regex :06:09
whoami-rajatthere may exist a method named test_vol_backend or just test_backend in compute dir which will not be covered with the current regex.06:09
tejdeepLooks like Zuul is having some problem https://review.opendev.org/#/c/621465/ . With every recheck i am getting new failures, older ones are not happening again, initially it timed out for tempest-full check and now it is failing at cinder-tempest-dsvm-lvm-lio-barbican06:11
tejdeepshould i recheck everytime until it succeeds?06:12
*** _alastor_ has quit IRC06:13
*** jiaopengju_1 has quit IRC06:14
*** jiaopengju_1 has joined #openstack-cinder06:14
openstackgerritzhufl proposed openstack/cinder master: Fix invalid function name  https://review.opendev.org/65625806:15
*** jiaopengju_2 has joined #openstack-cinder06:18
*** jiaopengju_1 has quit IRC06:20
*** jiaopengju_2 has quit IRC06:21
*** jiaopengju has joined #openstack-cinder06:22
ruffian_sheepwhoami-rajat:In this case, if I use test*vol*.py will be better?The search results using test*vol*.py and test*volume*.py are the same. For the volume-related tests, there are only those in the compete except for the volume. Is it better for me to add it manually?06:27
ruffian_sheepwhoami-rajat:I did not find the method test_vol_backend or test_backend now.06:28
*** threestrands has joined #openstack-cinder06:46
*** luksky has joined #openstack-cinder07:00
*** sahid has joined #openstack-cinder07:01
*** tesseract has joined #openstack-cinder07:03
*** tejdeep has quit IRC07:03
whoami-rajatruffian_sheep: i was just giving an example, like your regex won't work for this case https://github.com/openstack/tempest/blob/master/tempest/api/volume/admin/test_backends_capabilities.py#L4507:13
*** rcernin has quit IRC07:14
whoami-rajatruffian_sheep: this one too https://github.com/openstack/tempest/blob/25572c3b5512aa3ecc949b926af1e1fd64fddfd9/tempest/scenario/test_stamp_pattern.py#L8007:16
*** gkadam has quit IRC07:17
*** gmann has quit IRC07:19
*** markvoelker has quit IRC07:38
*** pcaruana has joined #openstack-cinder07:40
*** threestrands has quit IRC07:55
ruffian_sheepwhoami-rajat:Then can I use test*vol* as a regular expression first, adding missing use cases after discovery?07:55
whoami-rajatruffian_sheep: i guess, if we ultimately manage to run all volume related tests then any approach is fine.08:39
*** luksky has quit IRC08:39
*** markvoelker has joined #openstack-cinder08:40
*** nwonknu has quit IRC08:51
*** nwonknu has joined #openstack-cinder08:58
*** brinzh has joined #openstack-cinder09:02
*** brinzhang has quit IRC09:04
*** tesseract has quit IRC09:10
*** tesseract has joined #openstack-cinder09:12
*** mvkr has joined #openstack-cinder09:36
*** zul has quit IRC09:48
*** brinzh has quit IRC10:22
*** pooja_jadhav has quit IRC11:04
*** baojg has joined #openstack-cinder11:41
*** henriqueof has joined #openstack-cinder11:48
*** baojg has quit IRC11:58
*** carloss has joined #openstack-cinder12:05
*** udesale has quit IRC12:08
*** udesale has joined #openstack-cinder12:11
*** henriqueof has quit IRC12:18
*** altlogbot_2 has quit IRC12:43
*** altlogbot_3 has joined #openstack-cinder12:46
*** markvoelker has quit IRC12:52
*** markvoelker has joined #openstack-cinder12:52
smcginnisruffian_sheep: Just use 'volume', nothing else.13:10
*** jenglisch_ is now known as jenglisch13:17
*** pooja_jadhav has joined #openstack-cinder13:20
*** poojajadhav has joined #openstack-cinder13:22
*** poojajadhav has quit IRC13:23
*** pooja_jadhav has quit IRC13:25
whoami-rajateharney: hey13:25
openstackgerritMerged openstack/cinder master: Fix invalid function name  https://review.opendev.org/65625813:26
eharneywhoami-rajat: hey13:26
whoami-rajateharney: so i was going through the bug https://bugs.launchpad.net/python-cinderclient/+bug/1696780 again13:27
openstackLaunchpad bug 1696780 in python-cinderclient "supplying multiple list --filters does not work correctly" [Undecided,New] - Assigned to Rajat Dhasmana (whoami-rajat)13:27
whoami-rajatmy patch there seems to resolve i think all of the problems mentioned13:27
whoami-rajati'm unable to think of cases for it to fail (which i think exist), can you spark me up here13:29
eharneywhoami-rajat: i don't know how it would fix the case i documented in the bug if the change is to only pass parameters once13:29
eharneybut maybe i'm misinterpreting what the commit message means, haven't looked at this in a bit13:30
whoami-rajateharney: no, it disallows similar parameters to pass only once, like passing --name twice13:30
whoami-rajateharney: it still allows --filters name=xyz size=213:30
eharneybut that didn't actually work, does it work with this patch?13:31
whoami-rajateharney: yeah, i was trying cases i could think of today, they worked correctly13:31
whoami-rajatalthough the commit msg is a little misleading, *restricting duplicate optional params*13:35
eharneywhoami-rajat: cool, will try it out soon13:35
whoami-rajateharney: ok, thanks13:36
*** markvoelker has quit IRC13:39
*** mchlumsky has joined #openstack-cinder13:47
*** gnufied has joined #openstack-cinder14:02
*** pcaruana has quit IRC14:15
*** markvoelker has joined #openstack-cinder14:27
*** dims has quit IRC14:31
*** mriedem has joined #openstack-cinder14:39
*** dims has joined #openstack-cinder14:44
*** pcaruana has joined #openstack-cinder14:46
*** _alastor_ has joined #openstack-cinder14:52
*** e0ne has joined #openstack-cinder14:55
*** _alastor_ has quit IRC15:09
*** baojg has joined #openstack-cinder15:19
*** baojg has quit IRC15:34
*** mriedem has quit IRC15:35
*** dims has quit IRC15:35
openstackgerritMerged openstack/cinder master: Fix swift backup driver crashing during service startup  https://review.opendev.org/65576515:35
*** e0ne has quit IRC15:40
*** dims has joined #openstack-cinder15:42
*** hemna has joined #openstack-cinder15:45
hemnado we have a topics etherpad yet for the ptg?15:46
*** mriedem has joined #openstack-cinder15:48
smcginnisYeah, there's a list. One sec.15:48
*** enriquetaso has joined #openstack-cinder15:51
*** psachin has quit IRC15:52
*** pcaruana has quit IRC15:55
*** e0ne has joined #openstack-cinder15:56
*** dklyle has joined #openstack-cinder15:57
openstackgerritAlan Bishop proposed openstack/cinder stable/stein: Fix swift backup driver crashing during service startup  https://review.opendev.org/65653715:59
*** jmlowe has quit IRC16:04
*** sahid has quit IRC16:06
*** dklyle has quit IRC16:09
*** _hemna has joined #openstack-cinder16:09
*** dpawlik has quit IRC16:10
*** whoami-rajat has quit IRC16:11
*** hemna has quit IRC16:12
*** lseki has joined #openstack-cinder16:16
*** e0ne has quit IRC16:21
*** mriedem has quit IRC16:21
*** mchlumsky has quit IRC16:22
*** boxiang has quit IRC16:27
*** boxiang has joined #openstack-cinder16:27
*** imacdonn has quit IRC16:30
*** imacdonn has joined #openstack-cinder16:30
*** tesseract has quit IRC16:31
*** whoami-rajat has joined #openstack-cinder16:36
*** mriedem has joined #openstack-cinder16:42
*** enriquetaso has quit IRC16:44
*** markvoelker has quit IRC16:45
*** tejdeep has joined #openstack-cinder16:46
*** enriquetaso has joined #openstack-cinder16:48
*** dklyle has joined #openstack-cinder16:52
*** e0ne has joined #openstack-cinder16:56
*** _hemna has quit IRC16:58
*** dklyle has quit IRC17:03
*** udesale has quit IRC17:14
*** _hemna has joined #openstack-cinder17:17
*** gmann has joined #openstack-cinder17:19
*** _hemna has quit IRC17:22
*** e0ne has quit IRC17:25
*** mriedem has quit IRC17:27
*** enriquetaso has quit IRC17:35
*** markvoelker has joined #openstack-cinder17:37
*** enriquetaso has joined #openstack-cinder17:37
*** _hemna has joined #openstack-cinder17:38
_hemnasmcginnis: ping17:48
_hemnahttps://review.opendev.org/#/c/655612/I wanted to +A that, but it's to enable py37 tests, and the py37 tests are failing....17:49
_hemnaI know it's non-voting, but....should we enable a test that fails?17:49
_hemnajungleboyj: ^^ same17:49
*** tejdeep has quit IRC17:51
jungleboyj_hemna:  That doesn't sound like a good idea to me.17:51
*** tejdeep has joined #openstack-cinder17:52
_hemnayah I didn't think so either17:52
_hemnaI -1'd it17:52
*** dklyle has joined #openstack-cinder17:54
_hemnaseeing taskflow failures with that py3.7 test on that review17:54
_hemnaI'll try a recheck17:55
*** tejdeep has quit IRC17:56
*** tejdeep has joined #openstack-cinder17:56
*** mchlumsky has joined #openstack-cinder17:57
*** enriquetaso has quit IRC17:59
*** dklyle has quit IRC18:02
jungleboyjOk, I was going to look.  _hemna18:03
jungleboyj_hemna:  Looks like smcginnis is purposefully tring to poke here though  Hmm.18:05
*** _alastor_ has joined #openstack-cinder18:13
*** enriquetaso has joined #openstack-cinder18:14
*** markvoelker has quit IRC18:16
*** enriquetaso has quit IRC18:19
*** _hemna has quit IRC18:20
*** _alastor_ has quit IRC18:23
*** _hemna has joined #openstack-cinder18:25
*** _hemna has quit IRC18:30
*** tejdeep has quit IRC18:39
*** tejdeep has joined #openstack-cinder18:41
*** itlinux has joined #openstack-cinder18:52
*** itlinux has quit IRC18:57
*** pcaruana has joined #openstack-cinder19:10
*** _alastor_ has joined #openstack-cinder19:20
*** e0ne has joined #openstack-cinder19:26
whoami-rajathemna_:  jungleboyj  the failure is emc vnx driver specific, i thought of debugging it but IMO it would be better if their team fixes it in the driver.19:33
*** itlinux has joined #openstack-cinder19:34
*** dklyle has joined #openstack-cinder19:34
*** itlinux has quit IRC19:34
*** e0ne has quit IRC19:36
*** e0ne has joined #openstack-cinder19:38
*** _hemna has joined #openstack-cinder19:38
*** david-lyle has joined #openstack-cinder19:40
*** dklyle has quit IRC19:41
*** e0ne has quit IRC19:43
*** e0ne has joined #openstack-cinder19:44
*** e0ne has quit IRC19:44
*** itlinux has joined #openstack-cinder19:44
*** e0ne has joined #openstack-cinder19:45
*** _hemna has quit IRC19:45
*** lseki has quit IRC19:46
*** itlinux has quit IRC19:46
*** e0ne has quit IRC19:48
*** _alastor_ has quit IRC19:48
*** _hemna has joined #openstack-cinder19:53
smcginnis_hemna, jungleboyj, whoami-rajat: Yeah, there are also some library issues. I thought the taskflow one got fixed, but maybe not released yet.20:02
smcginnisMy thought with adding this test now was it would be a constant reminder of issues there.20:02
*** mriedem has joined #openstack-cinder20:02
smcginnisWe could have that switched back to the voting job and just leave the patch out there until the issues are resolved.20:03
smcginnisAnd we could recheck from time to time to check for any issues.20:03
smcginnisReported last October: https://bugs.launchpad.net/cinder/+bug/179595720:04
openstackLaunchpad bug 1795957 in Cinder "VNX unit tests are not Python 3.7 compatible" [Undecided,Confirmed]20:04
whoami-rajatsmcginnis: i think its related to pep479 https://docs.python.org/3/whatsnew/3.7.html#changes-in-python-behavior20:07
whoami-rajatsmcginnis: i'm not sure, does this require changes in cinder codebase or any other place?20:09
*** pcaruana has quit IRC20:23
*** _hemna has quit IRC20:27
smcginniswhoami-rajat: Yep, that's what the issue was when I dug into it. But couldn't find the exact spot in taskflow where the StopIteration was used.20:29
smcginniswhoami-rajat: I thought I had filed a bug there too, but can't recall now.20:29
smcginniswhoami-rajat: It will require taskflow to be fixed, then a new release of taskflow, then Cinder raising our lower-constraints to make sure only the fixed version and later can be used.20:30
openstackLaunchpad bug 1780823 in taskflow "Py3.7 RuntimeError: generator raised StopIteration" [High,Confirmed]20:31
*** _hemna has joined #openstack-cinder20:31
*** e0ne has joined #openstack-cinder20:34
*** david-lyle has quit IRC20:34
whoami-rajatsmcginnis: ok, seems like we can only wait then. Thanks for all the info.20:35
*** jdillaman has quit IRC20:35
*** _hemna has quit IRC20:36
*** _hemna has joined #openstack-cinder20:38
*** david-lyle has joined #openstack-cinder20:40
_hemnasmcginnis: so you think I should +A it anyway?20:43
*** e0ne has quit IRC20:45
*** e0ne has joined #openstack-cinder20:54
smcginnis_hemna: Maybe we can all discuss it quick in the PTG.20:56
_hemnasounds good20:56
*** david-lyle has quit IRC20:56
*** _alastor_ has joined #openstack-cinder21:09
*** itlinux has joined #openstack-cinder21:15
*** e0ne has quit IRC21:15
*** itlinux has quit IRC21:17
openstackgerritMerged openstack/cinder stable/stein: Fix swift backup driver crashing during service startup  https://review.opendev.org/65653721:19
*** itlinux has joined #openstack-cinder21:20
*** _hemna has quit IRC21:23
*** itlinux has quit IRC21:24
*** _hemna has joined #openstack-cinder21:24
*** david-lyle has joined #openstack-cinder21:24
openstackgerritTejdeep Kautharam proposed openstack/cinder master: Cinder RSD Driver  https://review.opendev.org/62146521:25
*** _hemna has quit IRC21:28
*** itlinux has joined #openstack-cinder21:34
*** david-lyle has quit IRC21:34
*** itlinux has quit IRC21:39
*** itlinux has joined #openstack-cinder21:44
*** itlinux has quit IRC21:48
*** itlinux has joined #openstack-cinder21:51
*** markvoelker has joined #openstack-cinder21:52
*** david-lyle has joined #openstack-cinder21:53
*** _hemna has joined #openstack-cinder21:58
*** mriedem has quit IRC22:00
*** _hemna has quit IRC22:03
*** _hemna has joined #openstack-cinder22:04
*** itlinux has quit IRC22:06
*** markvoelker has quit IRC22:09
*** strigazi has joined #openstack-cinder22:09
*** david-lyle has quit IRC22:10
jungleboyjIf anyone wants to walk to the Cinder Dinner together meet in the lobby of the Hyatt by Reception at 6:45.22:15
*** mriedem has joined #openstack-cinder22:16
*** e0ne has joined #openstack-cinder22:20
*** markvoelker has joined #openstack-cinder22:21
*** david-lyle has joined #openstack-cinder22:21
*** mchlumsky has quit IRC22:22
*** carloss has quit IRC22:22
*** itlinux has joined #openstack-cinder22:22
*** dave-mccowan has joined #openstack-cinder22:22
*** jmlowe has joined #openstack-cinder22:26
*** Anticimex has joined #openstack-cinder22:27
*** mriedem has quit IRC22:39
*** Anticimex has left #openstack-cinder22:42
*** david-lyle has quit IRC22:43
*** rcernin has joined #openstack-cinder22:47
*** markvoelker has quit IRC22:57
*** itlinux has quit IRC22:58
*** e0ne has quit IRC22:58
*** ianychoi_ has quit IRC23:03
*** jmlowe has quit IRC23:03
*** ianychoi_ has joined #openstack-cinder23:04
*** dave-mccowan has quit IRC23:06
*** mriedem has joined #openstack-cinder23:10
mriedemsolution: don't use multipath23:15
*** e0ne has joined #openstack-cinder23:20
smcginnisThere were issues fixed from the pike to queens timeframe.23:20
*** e0ne has quit IRC23:21
*** david-lyle has joined #openstack-cinder23:27
*** david-lyle has quit IRC23:33
*** mriedem has quit IRC23:38
*** e0ne has joined #openstack-cinder23:40
*** e0ne has quit IRC23:50
*** e0ne has joined #openstack-cinder23:52
*** e0ne has quit IRC23:55

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