Wednesday, 2019-09-25

*** markvoelker has joined #openstack-qa00:24
*** rh-jelabarre has quit IRC00:49
*** mriedem has quit IRC01:00
*** markvoelker has quit IRC01:08
*** hongbin has joined #openstack-qa01:49
openstackgerritBrin Zhang proposed openstack/tempest master: Add tests for boot instance with the specified volume type  https://review.opendev.org/60935702:04
*** markvoelker has joined #openstack-qa02:35
*** hongbin has quit IRC02:36
*** igordc has quit IRC02:39
*** markvoelker has quit IRC02:40
openstackgerritMasayuki Igawa proposed openstack/tempest master: Set irrelevant-files for bashate job  https://review.opendev.org/68444702:41
openstackgerritJens Harbott (frickler) proposed openstack/devstack master: Revert "Do not use pip 10 or higher"  https://review.opendev.org/56159702:42
openstackgerritMasayuki Igawa proposed openstack/tempest master: Set irrelevant-files for bashate job  https://review.opendev.org/68444702:47
*** dave-mccowan has quit IRC02:53
*** rfolco has quit IRC03:09
openstackgerritMasayuki Igawa proposed openstack/devstack master: Add PDF documentation build  https://review.opendev.org/68445103:26
*** ramishra has joined #openstack-qa03:44
openstackgerritMerged openstack/tempest master: Add 'single core approve' policy for plugins sanity jobs failure  https://review.opendev.org/67095003:55
*** udesale has joined #openstack-qa04:07
*** artom has quit IRC04:41
*** iokiwi has quit IRC05:09
*** iokiwi has joined #openstack-qa05:09
*** soniya29 has joined #openstack-qa05:11
openstackgerritMasayuki Igawa proposed openstack/devstack master: Add PDF documentation build  https://review.opendev.org/68445105:38
openstackgerritSoniya Murlidhar Vyas proposed openstack/tempest master: Moved subunit_describe_calls from argparse to cliff  https://review.opendev.org/68283505:56
*** eandersson has joined #openstack-qa05:57
*** e0ne has joined #openstack-qa06:16
*** kopecmartin|off is now known as kopecmartin06:20
*** tosky has joined #openstack-qa06:35
openstackgerritMerged openstack/tempest master: Add assertNotEmpty to avoid IndexError  https://review.opendev.org/68333006:35
*** gcheresh has joined #openstack-qa06:36
*** pcaruana has joined #openstack-qa06:42
*** gfidente has joined #openstack-qa06:42
*** rpittau|afk is now known as rpittau06:47
*** markvoelker has joined #openstack-qa06:47
*** markvoelker has quit IRC06:52
*** slaweq has joined #openstack-qa06:55
*** e0ne has quit IRC06:56
*** tesseract has joined #openstack-qa07:01
openstackgerritlinanbj proposed openstack/tempest master: Add floating ip pools client  https://review.opendev.org/68454007:21
*** e0ne has joined #openstack-qa07:39
*** tosky_ has joined #openstack-qa07:41
*** tosky has quit IRC07:42
*** jpena|off is now known as jpena07:46
*** Shushuda has joined #openstack-qa07:54
openstackgerritlinanbj proposed openstack/tempest master: Add floating ip pools client  https://review.opendev.org/68454007:57
soniya29masayukig, I have passed 'sys.stdin' value for default parameter to let user know the default value to be passed for 'subunit-describe-calls -s'. I didn't get your comment here:- https://review.opendev.org/#/c/682835/6/tempest/cmd/subunit_describe_calls.py@304. Will you explain a bit more?07:57
*** soniya29 is now known as soniya29|lunch07:58
masayukigsoniya29|lunch: sorry, my bad. ignore  my comment. It looks working with like this command line `$ cat  ./.stestr/45 | subunit-describe-calls`08:01
*** rpittau is now known as rpittau|bbl08:01
masayukigsoniya29|lunch: and, sorry for interrupt your lunch :p08:02
masayukigsoniya29|lunch: ugh, sorry for the confusion. Let me correct myself. The command should work but doesn't work against the patch-set 6. I meant the default value of option '-s' is sys.stdin. So, if we don't specify the option, the command should read subunit stream from stdin. But the patch-set 6 doesn't.08:11
openstackgerritMasayuki Igawa proposed openstack/devstack master: Add PDF documentation build  https://review.opendev.org/68445108:25
*** e0ne has quit IRC08:28
fricklerianw: could you take a quick look at https://review.opendev.org/683977 please? master patch merged earlier, but I think we need this one in order to stabilize grenade jobs, too08:29
*** markvoelker has joined #openstack-qa08:48
*** markvoelker has quit IRC08:53
amotokihi, I am trying to run patrol in my local ubuntu bionic but I am hitting an error. it seems I hit a basic configuration issue.08:55
amotokithis is the error http://paste.openstack.org/show/779229/08:55
amotokimy local.conf is http://paste.openstack.org/show/779227/08:55
*** markvoelker has joined #openstack-qa08:56
amotokithe local.conf is based from the CI result in gmann's comment #1 of https://bugs.launchpad.net/neutron/+bug/1843290.08:56
openstackLaunchpad bug 1843290 in neutron "Remove network flavor profile fails" [High,In progress] - Assigned to Slawek Kaplonski (slaweq)08:56
amotokiany advise would be appreciated.08:56
amotokithis blocks the debug of the policy bug above.08:57
slaweqamotoki: I can reproduce this bug on my local env08:58
slaweqamotoki: I run devstack with patrole plugin enabled08:58
slaweqand than I had to change in tempest/etc/tempest.conf value of role used by patrole to "member"08:58
slaweqI wanted to write comment with my findings there but I didn't had time yet08:59
slaweqI will do it in about 1h, ok for You?08:59
*** markvoelker has quit IRC09:00
*** soniya29|lunch is now known as soniya2909:01
soniya29masayukig:thanks, I get your point now. How can I approach this issue?09:06
*** dtantsur has joined #openstack-qa09:12
*** iurygregory has joined #openstack-qa09:12
*** e0ne has joined #openstack-qa09:31
*** tosky_ is now known as tosky09:44
openstackgerritRodolfo Alonso Hernandez proposed openstack/tempest master: Add a default network when creating a server  https://review.opendev.org/68296409:47
openstackgerritBrin Zhang proposed openstack/tempest master: Add tests for boot instance with the specified volume type  https://review.opendev.org/60935709:52
*** rpittau|bbl is now known as rpittau09:57
fricklerdirk: cmurphy: I'm getting a new error on the pip 10 removal with opensuse now, which didn't happen 3 months ago when I last iterated on that, maybe one of you can take a look? https://review.opendev.org/56159710:16
*** Shushuda has quit IRC10:22
*** pcaruana has quit IRC10:23
*** rcernin has quit IRC10:23
soniya29Hello everyone, Here is the link of my patch:-https://review.opendev.org/openstack/tempest refs/changes/35/682835/6. If I remove 'import cliff', pep8 check throws this error:- https://paste.fedoraproject.org/paste/HOVeYuCFdL8VD2PO1HGaQQ and If I keep 'import cliff' as it is, pep8 throws this error:-https://paste.fedoraproject.org/paste/s8pB~SzUVCHQtIUYwcNk4A. How can I solve this issue?10:31
*** iurygregory has quit IRC10:31
*** pcaruana has joined #openstack-qa10:36
openstackgerritMerged openstack/devstack stable/stein: Enable accept_ra before enabling forwarding  https://review.opendev.org/68397710:52
*** artom has joined #openstack-qa10:53
*** markvoelker has joined #openstack-qa10:57
openstackgerritMerged openstack/tempest master: Remove barbican-tempest-plugin from blacklist  https://review.opendev.org/68422010:57
*** markvoelker has quit IRC11:01
*** sshnaidm is now known as sshnaidm|afk11:12
*** udesale has quit IRC11:15
*** lucasagomes has joined #openstack-qa11:22
*** raildo has joined #openstack-qa11:24
*** jpena is now known as jpena|lunch11:25
*** iurygregory has joined #openstack-qa11:26
*** markvoelker has joined #openstack-qa11:42
*** rh-jelabarre has joined #openstack-qa11:58
*** rfolco has joined #openstack-qa12:12
amotokislaweq: is there any additional configuration more than installing patrol and neutron-tempest-plugin + setting rbac_test_roles to 'member' in tempest.conf?12:18
*** soniya29 has quit IRC12:20
*** dave-mccowan has joined #openstack-qa12:22
*** jpena|lunch is now known as jpena12:26
*** dave-mccowan has quit IRC12:26
*** dave-mccowan has joined #openstack-qa12:28
slaweqamotoki: no, I didn't need anything else12:30
amotokislaweq: thanks. I will check what happens in my bionic VM in detail.12:30
*** dave-mccowan has quit IRC12:42
*** mriedem has joined #openstack-qa12:47
*** nweinber_ has joined #openstack-qa12:49
slaweqamotoki: I know what's going on there12:54
slaweqamotoki: I just updated comments in launchpad with what I found12:59
amotokislaweq: sounds nice :)12:59
slaweqamotoki: and that's why this additional rule for get_flavor_service_profile would be needed IIUC this code13:02
slaweqand probably that was the reason why it was defined in policy.json in e.g. Rocky, no?13:02
amotokislaweq: what I am not sure is why we need get_flavor_service_profile policy. we don't have an API corresponidng to get_flavor_service_profile, so when is it accessed?13:04
slaweqamotoki: look here https://github.com/openstack/neutron/blob/8b50a36341daf16c5d2c01546f6ca264fc83f7aa/neutron/api/v2/base.py#L58813:06
slaweqif enforcing policy for delete_ action fails, it checks get_ also13:06
slaweqand this is what we see in this case13:07
amotokislaweq: yeah, that's what I guess, but what I am not sure is why my test succeeded like http://paste.openstack.org/show/779150/ (around L.257)13:08
amotokiI wonder what's the difference.13:09
slaweqamotoki: hmm, I have no idea13:10
amotokibut I can investigate it later. it should not be a blocker.13:10
slaweqamotoki: IMO for some reason (dunno exactly why) here https://github.com/openstack/neutron/blob/8b50a36341daf16c5d2c01546f6ca264fc83f7aa/neutron/api/v2/base.py#L587 was returned True, than this test is passing13:16
slaweqamotoki: maybe You were doing it as admin?13:18
amotokislaweq: I created a new user/project/role and tested with that user. At least, I got PolicyNotAuthorized error with the default policy for delete_flavor_service_profile....13:21
slaweqamotoki: than I don't know :/13:21
amotokislaweq: so I am trying to know what is the difference.13:21
slaweqamotoki: sure :)13:22
*** udesale has joined #openstack-qa13:22
*** gcheresh has quit IRC13:33
*** nweinber_ has quit IRC13:44
*** sshnaidm|afk is now known as sshnaidm13:45
*** slaweq has quit IRC13:54
*** slaweq has joined #openstack-qa13:57
*** slaweq has quit IRC14:15
*** slaweq has joined #openstack-qa14:16
openstackgerritMatt Riedemann proposed openstack/devstack stable/pike: Centralize setting default branch for stable branching  https://review.opendev.org/68475614:20
openstackgerritMatt Riedemann proposed openstack/devstack stable/pike: Centralize setting default branch for stable branching  https://review.opendev.org/68475614:24
openstackgerritMatt Riedemann proposed openstack/devstack master: Fix target branch for pbr and diskimage-builder  https://review.opendev.org/68476114:32
*** rpittau is now known as rpittau|afk14:34
*** nweinber_ has joined #openstack-qa14:43
openstackgerritMatt Riedemann proposed openstack/devstack stable/pike: Use Tempest v21.0.0 for Pike testing instead of master  https://review.opendev.org/68476914:57
*** e0ne has quit IRC14:59
*** e0ne has joined #openstack-qa14:59
*** dtantsur is now known as dtantsur|afk15:00
*** tosky has quit IRC15:00
*** tosky has joined #openstack-qa15:01
*** artom has quit IRC15:15
*** artom has joined #openstack-qa15:15
*** openstackgerrit has quit IRC15:21
*** sridharg has joined #openstack-qa15:23
*** gfidente has quit IRC15:26
*** dulek has joined #openstack-qa15:26
*** tosky has quit IRC15:27
*** pcaruana has quit IRC15:30
*** markvoelker has quit IRC15:45
*** lbragstad_ has joined #openstack-qa15:54
*** lbragstad has quit IRC15:55
*** lbragstad has joined #openstack-qa15:57
*** lbragstad_ has quit IRC15:58
*** rfolco is now known as rfolco|bbl16:06
cmurphyfrickler: thanks, will take a look16:08
*** mattw4 has joined #openstack-qa16:25
*** markvoelker has joined #openstack-qa16:38
*** ramishra has quit IRC16:43
*** igordc has joined #openstack-qa16:46
*** markvoelker has quit IRC16:48
*** markvoelker has joined #openstack-qa16:53
*** jpena is now known as jpena|off16:57
*** e0ne has quit IRC16:59
*** mattw4 has quit IRC17:04
gmannamotoki: and you do not have that rule in your env policy.json ?17:12
gmannamotoki: slaweq there is no difference of installing patrole via devstack from fresh or in existing env and edit the conf rbac_test_roles = member. it is something on your existing env's neutron setup which might be making test pass17:14
*** sridharg has quit IRC17:14
*** sshnaidm is now known as sshnaidm|afk17:18
*** udesale has quit IRC17:19
dulekgmann: Hi! Regarding https://review.opendev.org/#/c/682531 - so this fails the Zuul config parsing and I don't really know why…17:38
dulekBut besides that with Kuryr we're totally unprepared to run with IPv6, so this will fail miserably.17:39
gmanndulek: yeah some parsing error. I saw your comment. you mean kuryr-kubernetes only or all repo ?17:40
dulekThe closest deadline we can get this done is Ussuri and we actually have that on our plans.17:40
gmannkuryr-libnetwork too ?17:40
dulekgmann: By all repo you mean all kuryr project repos? Uhm, I'm not sure here really, Hongbin would be the person to ask. Let me lookup his IRC handle…17:41
gmanndulek: yeah that is I am thinking. If things are failing and need time to fix or more works like you mentioned for kuryr-kubernetes then we can do it in ussuri early.17:41
gmanndulek: yeah kuryr repo. kuryr-libnetwork is passing seems- https://review.opendev.org/#/c/682532/17:42
dulekgmann: Alright, at least that.17:42
dulekAnd the kuryr repo is mostly dead, only serves as a library for kuryr-libnetwork and kuryr-kubernetes at this point.17:42
dulekSo there's nobody to take care of that.17:42
gmannyeah17:42
gmannkuryr repo does not need this job as it is lib17:43
gmanndulek: let me check the zuul error on kuryr-kubernetes so that we can at least know the failures.17:43
dulekSure.17:43
*** mattw4 has joined #openstack-qa17:44
gmannbut i agree with your plan of doing it on ussuri as per project roadmap.17:44
dulekgmann: Though I really doubt early Ussuri here - Kubernetes and OpenShift versions we're supporting weren't focusing on IPv6, so neither did we.17:44
dulekgmann: Later Ussuri would be what I want.17:44
dulekOr what I see as feasible. ;)17:44
gmanni see. if we can get the job working with failure then we can make it n-v or experimental so that you guys can check the results during fixing17:45
dulekgmann: Sure, let's see how this plays. Thanks!17:48
gmannthanks dulek for following up.17:48
*** igordc has quit IRC17:52
*** tesseract has quit IRC18:08
*** e0ne has joined #openstack-qa18:25
*** lucasagomes has quit IRC18:57
*** e0ne has quit IRC19:23
*** nweinber_ has quit IRC20:35
*** slaweq has quit IRC20:55
*** igordc has joined #openstack-qa21:03
*** raildo has quit IRC21:04
*** openstackgerrit has joined #openstack-qa21:22
openstackgerritColleen Murphy proposed openstack/devstack master: Fix six package on opensuse for pip 10  https://review.opendev.org/68484521:22
*** rfolco|bbl is now known as rfolco22:09
*** ccamposr__ has joined #openstack-qa22:11
*** ccamposr has quit IRC22:13
*** rcernin has joined #openstack-qa22:15
*** ccamposr has joined #openstack-qa22:20
*** ccamposr__ has quit IRC22:23
openstackgerritGhanshyam Mann proposed openstack/patrole master: test stestr 2.5.0  https://review.opendev.org/68485022:44
*** mriedem has quit IRC22:53
*** ccamposr has quit IRC23:02
openstackgerritGhanshyam Mann proposed openstack/os-performance-tools master: Sync Sphinx requirement  https://review.opendev.org/66638323:09
*** igordc has quit IRC23:10
openstackgerritMerged openstack/hacking master: Fix deprecated escape chars in strings  https://review.opendev.org/67860523:11
openstackgerritGhanshyam Mann proposed openstack/os-performance-tools master: Sync Sphinx requirement  https://review.opendev.org/66638323:22
*** mattw4 has quit IRC23:50
openstackgerritMerged openstack/patrole master: Add PDF building  https://review.opendev.org/68279923:56

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