Friday, 2021-05-14

openstackgerrityang yawei proposed openstack/stackviz master: setup.cfg: Replace dashes with underscores  https://review.opendev.org/c/openstack/stackviz/+/79134101:24
*** ysandeep|away is now known as ysandeep01:35
*** amodi has quit IRC01:53
openstackgerritMin Li proposed openstack/tempest master: test_server_multinode.TestServerMultinode.test_schedule_ to_all_nodes failed.  https://review.opendev.org/c/openstack/tempest/+/79134902:46
openstackgerritMin Li proposed openstack/tempest master: test_server_multinode.TestServerMultinode.test_schedule_to _all_nodes failed.  https://review.opendev.org/c/openstack/tempest/+/78841702:49
*** rcernin has quit IRC03:04
*** rcernin has joined #openstack-qa03:22
*** brinzhang0 has quit IRC03:46
*** hemanth_n has joined #openstack-qa03:48
*** hemanth_n has quit IRC04:08
*** brinzhang0 has joined #openstack-qa04:25
openstackgerritMerged openstack/devstack master: Revert "Change Neutron's default ML2 driver to OVN"  https://review.opendev.org/c/openstack/devstack/+/79110404:33
*** psahoo has joined #openstack-qa04:48
*** rcernin has quit IRC05:55
*** rcernin has joined #openstack-qa05:57
*** slaweq has joined #openstack-qa06:06
*** brinzhang_ has joined #openstack-qa06:31
*** whoami-rajat_ has joined #openstack-qa06:32
*** brinzhang0 has quit IRC06:34
*** jpena|off is now known as jpena07:30
*** tosky has joined #openstack-qa07:47
openstackgerrityang yawei proposed openstack/grenade master: setup.cfg: Replace dashes with underscores  https://review.opendev.org/c/openstack/grenade/+/79139907:55
*** rcernin has quit IRC07:56
*** rcernin has joined #openstack-qa07:56
*** lucasagomes has joined #openstack-qa08:00
*** yoctozepto6 is now known as yoctozepto08:03
*** ysandeep is now known as ysandeep|lunch08:08
*** gfidente has joined #openstack-qa08:23
*** rcernin has quit IRC08:32
*** gcheresh has joined #openstack-qa08:43
*** ysandeep|lunch is now known as ysandeep09:12
openstackgerritLucas Alvares Gomes proposed openstack/devstack master: DO NOT REVIEW  https://review.opendev.org/c/openstack/devstack/+/79143609:36
openstackgerritLucas Alvares Gomes proposed openstack/devstack master: DO NOT REVIEW  https://review.opendev.org/c/openstack/devstack/+/79143610:38
*** ysandeep is now known as ysandeep|brb11:17
*** jpena is now known as jpena|lunch11:34
*** whoami-rajat_ is now known as whoami-rajat11:51
*** ysandeep|brb is now known as ysandeep12:13
*** brinzhang0 has joined #openstack-qa12:13
*** brinzhang_ has quit IRC12:16
*** psahoo_ has joined #openstack-qa12:29
*** jpena|lunch is now known as jpena12:31
*** psahoo has quit IRC12:33
lucasagomeshi, someone knows how can I test https://review.opendev.org/c/zuul/zuul-jobs/+/791117/ ? Apparently the Depends-On is not honored in the "zuul-jobs-test-ensure-devstack" test run12:35
*** nweinber has joined #openstack-qa13:04
*** brinzhang_ has joined #openstack-qa13:06
*** brinzhang0 has quit IRC13:09
*** ysandeep is now known as ysandeep|away13:16
*** brinzhang0 has joined #openstack-qa13:18
*** brinzhang_ has quit IRC13:21
gmannlucasagomes: which one you are trying to test 791436 or 791085?13:34
gmannlucasagomes: if former then you need to change refspec here https://review.opendev.org/c/zuul/zuul-jobs/+/791117/3/zuul-tests.d/cloud-roles-jobs.yaml#913:34
gmannthis job will clone what you have in ensure_devstack_git_refspec and try to install devstack13:35
lucasagomesgmann, ah sorry thanks yeah! I thought the depends-on would be enough cheers will try with that13:35
gmannlucasagomes: yeah as this job is for 'using devstack' of what we mention in config depends-on is kind of no use here13:36
openstackgerritLucas Alvares Gomes proposed openstack/devstack master: DO NOT REVIEW  https://review.opendev.org/c/openstack/devstack/+/79143613:36
lucasagomesgmann, gotcha13:36
lucasagomesthanks for the help13:36
gmannlucasagomes: sorry for revert but that made devstack failing as default (because this 735097 mixed up the default setting in devstack + job side)13:37
lucasagomesgmann, it's all good13:39
lucasagomesit's understandable, it's something that I wasn't aware of that was failing cause the gate was green13:39
gmannlucasagomes: yeah that is actually we missing as testing in devstack so not your fault.13:40
*** ysandeep|away is now known as ysandeep13:56
*** nweinber has quit IRC14:29
*** nweinber has joined #openstack-qa14:29
openstackgerritLucas Alvares Gomes proposed openstack/devstack master: DO NOT REVIEW  https://review.opendev.org/c/openstack/devstack/+/79143615:07
*** tkajinam has quit IRC15:14
*** gcheresh has quit IRC15:34
openstackgerritLucas Alvares Gomes proposed openstack/devstack master: Change default network backend driver to ML2/OVN  https://review.opendev.org/c/openstack/devstack/+/79143616:10
clarkblucasagomes: gmann I think a key thing about the ensure-devstack case was that it uses a very minial localrc specifically one that doesn't set enabled services or add extra vars like were added to the CI jobs in the ovn swap16:13
lucasagomesclarkb, yes16:13
clarkblucasagomes: gmann: I think if you want to test it locally you should be able to run stack.sh in a clean checkout of devstack and that should reproduce16:14
lucasagomesclarkb, actually I just got it passing :-)16:14
lucasagomeszuul-jobs-test-ensure-devstack https://zuul.opendev.org/t/zuul/build/7b492d6dfefa4f718d2c3fc728b2a550 : SUCCESS in 22m 58s16:14
clarkbcool16:14
lucasagomesclarkb, with the previous patch-set https://review.opendev.org/c/openstack/devstack/+/79143616:14
clarkbbasically the key thing is that any person can clone devstack and run stack.sh and it works16:14
lucasagomesthis new patch-set is mostly cosmetics so we should be good there16:14
gmannyeah testing patch are able to catch those16:14
lucasagomesclarkb, exactly yeah16:14
clarkbthe defaults are not set by CI they are set by devstack and we need to respect that16:14
lucasagomesI think there was two problems, this one that you mentioned and the fix for that was to change the ENABLED_SERVICES variable in stackrc16:15
lucasagomesso if no enable_service is specified in local.conf it will use those16:15
lucasagomesand the other problem is regarding the precloned repositories16:15
lucasagomesin the lib/neutron_plugins/ovn_agent we were always trying to import a module from the neutron repository16:16
lucasagomesthis module is only used when we set the OVN_BUILD_FROM_SOURCE variable to True (it's False by default)16:16
lucasagomesbecause by default we install OVN from the distro packages16:16
clarkbhow did nodepool jobs hit it then as they don't set any non default vars for ovn?16:16
lucasagomesbut that second problem was only seein in this zuul gate there cause it didn't have all the repos precloned16:17
clarkbI guess we loaded the module either way16:17
lucasagomesclarkb, yes before it tried to load it regardless16:17
clarkbwell anyone who also just clones devstack would fail too unless they precloned16:17
lucasagomesnow I changed it, I will only load it if OVN_BUILD_FROM_SOURCE is set to True16:17
lucasagomeswhich is not the default so we should be good there on nodepool16:17
gmannwe can remove these service enable/disable in job now - https://review.opendev.org/c/openstack/devstack/+/791436/5/.zuul.yaml16:19
clarkbgmann: I don't think so as they replaced the test amtrix stuff in devstack-gate16:19
lucasagomesgmann, we can't, because the stackrc ENABLED_SERVICES only get used if no other service has been disabled/enabled16:20
lucasagomesgmann, see L64 from https://review.opendev.org/c/openstack/devstack/+/791436/5/stackrc16:20
clarkblucasagomes: its only if no other service is enabled I think it is used if you disabled things16:20
gmannah yeah, for overridden config yes we need it there, got it16:21
lucasagomesclarkb, gotcha16:21
gmanndisable services has separate var16:21
lucasagomesright16:21
lucasagomesclarkb, gmann I need to run but, I think we are good now. We can resume this on Monday. Thanks for all the help16:22
lucasagomesI learned something a few new things doing it :D16:22
lucasagomeswould be good to have that ensure-devstack job in devstack itself tho16:23
lucasagomesbut that's a separated thing16:23
gmannlucasagomes: thanks for tests and all. will review  it today.16:23
lucasagomesgmann, thank you! Appreciate it16:23
gmannyeah i am in favor of adding that job in devstack side16:23
gmannbut yeah separate patch16:24
lucasagomessounds good16:25
*** lucasagomes has quit IRC16:26
*** jpena is now known as jpena|off16:28
*** psahoo_ has quit IRC16:41
*** rh-jlabarre has quit IRC17:16
*** amodi has joined #openstack-qa18:07
*** ysandeep is now known as ysandeep|away18:10
*** whoami-rajat has quit IRC19:21
openstackgerritMonty Taylor proposed openstack/devstack master: DNM add feature pragma  https://review.opendev.org/c/openstack/devstack/+/79154119:52
*** nweinber has quit IRC20:19
*** rcernin has joined #openstack-qa20:39
*** amodi has quit IRC21:34
*** rcernin has quit IRC21:40
*** rcernin has joined #openstack-qa22:06
*** rcernin has quit IRC22:11
*** rcernin has joined #openstack-qa22:23
*** rcernin has quit IRC22:27
*** rcernin has joined #openstack-qa22:28
*** rcernin has quit IRC22:33
*** rcernin has joined #openstack-qa23:05
*** rcernin has quit IRC23:09
*** tosky has quit IRC23:11
*** rcernin has joined #openstack-qa23:35
*** rcernin has quit IRC23:40

Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!