openstackgerrit | yang yawei proposed openstack/stackviz master: setup.cfg: Replace dashes with underscores https://review.opendev.org/c/openstack/stackviz/+/791341 | 01:24 |
---|---|---|
*** ysandeep|away is now known as ysandeep | 01:35 | |
*** amodi has quit IRC | 01:53 | |
openstackgerrit | Min Li proposed openstack/tempest master: test_server_multinode.TestServerMultinode.test_schedule_ to_all_nodes failed. https://review.opendev.org/c/openstack/tempest/+/791349 | 02:46 |
openstackgerrit | Min Li proposed openstack/tempest master: test_server_multinode.TestServerMultinode.test_schedule_to _all_nodes failed. https://review.opendev.org/c/openstack/tempest/+/788417 | 02:49 |
*** rcernin has quit IRC | 03:04 | |
*** rcernin has joined #openstack-qa | 03:22 | |
*** brinzhang0 has quit IRC | 03:46 | |
*** hemanth_n has joined #openstack-qa | 03:48 | |
*** hemanth_n has quit IRC | 04:08 | |
*** brinzhang0 has joined #openstack-qa | 04:25 | |
openstackgerrit | Merged openstack/devstack master: Revert "Change Neutron's default ML2 driver to OVN" https://review.opendev.org/c/openstack/devstack/+/791104 | 04:33 |
*** psahoo has joined #openstack-qa | 04:48 | |
*** rcernin has quit IRC | 05:55 | |
*** rcernin has joined #openstack-qa | 05:57 | |
*** slaweq has joined #openstack-qa | 06:06 | |
*** brinzhang_ has joined #openstack-qa | 06:31 | |
*** whoami-rajat_ has joined #openstack-qa | 06:32 | |
*** brinzhang0 has quit IRC | 06:34 | |
*** jpena|off is now known as jpena | 07:30 | |
*** tosky has joined #openstack-qa | 07:47 | |
openstackgerrit | yang yawei proposed openstack/grenade master: setup.cfg: Replace dashes with underscores https://review.opendev.org/c/openstack/grenade/+/791399 | 07:55 |
*** rcernin has quit IRC | 07:56 | |
*** rcernin has joined #openstack-qa | 07:56 | |
*** lucasagomes has joined #openstack-qa | 08:00 | |
*** yoctozepto6 is now known as yoctozepto | 08:03 | |
*** ysandeep is now known as ysandeep|lunch | 08:08 | |
*** gfidente has joined #openstack-qa | 08:23 | |
*** rcernin has quit IRC | 08:32 | |
*** gcheresh has joined #openstack-qa | 08:43 | |
*** ysandeep|lunch is now known as ysandeep | 09:12 | |
openstackgerrit | Lucas Alvares Gomes proposed openstack/devstack master: DO NOT REVIEW https://review.opendev.org/c/openstack/devstack/+/791436 | 09:36 |
openstackgerrit | Lucas Alvares Gomes proposed openstack/devstack master: DO NOT REVIEW https://review.opendev.org/c/openstack/devstack/+/791436 | 10:38 |
*** ysandeep is now known as ysandeep|brb | 11:17 | |
*** jpena is now known as jpena|lunch | 11:34 | |
*** whoami-rajat_ is now known as whoami-rajat | 11:51 | |
*** ysandeep|brb is now known as ysandeep | 12:13 | |
*** brinzhang0 has joined #openstack-qa | 12:13 | |
*** brinzhang_ has quit IRC | 12:16 | |
*** psahoo_ has joined #openstack-qa | 12:29 | |
*** jpena|lunch is now known as jpena | 12:31 | |
*** psahoo has quit IRC | 12:33 | |
lucasagomes | hi, 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 run | 12:35 |
*** nweinber has joined #openstack-qa | 13:04 | |
*** brinzhang_ has joined #openstack-qa | 13:06 | |
*** brinzhang0 has quit IRC | 13:09 | |
*** ysandeep is now known as ysandeep|away | 13:16 | |
*** brinzhang0 has joined #openstack-qa | 13:18 | |
*** brinzhang_ has quit IRC | 13:21 | |
gmann | lucasagomes: which one you are trying to test 791436 or 791085? | 13:34 |
gmann | lucasagomes: 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#9 | 13:34 |
gmann | this job will clone what you have in ensure_devstack_git_refspec and try to install devstack | 13:35 |
lucasagomes | gmann, ah sorry thanks yeah! I thought the depends-on would be enough cheers will try with that | 13:35 |
gmann | lucasagomes: yeah as this job is for 'using devstack' of what we mention in config depends-on is kind of no use here | 13:36 |
openstackgerrit | Lucas Alvares Gomes proposed openstack/devstack master: DO NOT REVIEW https://review.opendev.org/c/openstack/devstack/+/791436 | 13:36 |
lucasagomes | gmann, gotcha | 13:36 |
lucasagomes | thanks for the help | 13:36 |
gmann | lucasagomes: sorry for revert but that made devstack failing as default (because this 735097 mixed up the default setting in devstack + job side) | 13:37 |
lucasagomes | gmann, it's all good | 13:39 |
lucasagomes | it's understandable, it's something that I wasn't aware of that was failing cause the gate was green | 13:39 |
gmann | lucasagomes: yeah that is actually we missing as testing in devstack so not your fault. | 13:40 |
*** ysandeep|away is now known as ysandeep | 13:56 | |
*** nweinber has quit IRC | 14:29 | |
*** nweinber has joined #openstack-qa | 14:29 | |
openstackgerrit | Lucas Alvares Gomes proposed openstack/devstack master: DO NOT REVIEW https://review.opendev.org/c/openstack/devstack/+/791436 | 15:07 |
*** tkajinam has quit IRC | 15:14 | |
*** gcheresh has quit IRC | 15:34 | |
openstackgerrit | Lucas Alvares Gomes proposed openstack/devstack master: Change default network backend driver to ML2/OVN https://review.opendev.org/c/openstack/devstack/+/791436 | 16:10 |
clarkb | lucasagomes: 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 swap | 16:13 |
lucasagomes | clarkb, yes | 16:13 |
clarkb | lucasagomes: 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 reproduce | 16:14 |
lucasagomes | clarkb, actually I just got it passing :-) | 16:14 |
lucasagomes | zuul-jobs-test-ensure-devstack https://zuul.opendev.org/t/zuul/build/7b492d6dfefa4f718d2c3fc728b2a550 : SUCCESS in 22m 58s | 16:14 |
clarkb | cool | 16:14 |
lucasagomes | clarkb, with the previous patch-set https://review.opendev.org/c/openstack/devstack/+/791436 | 16:14 |
clarkb | basically the key thing is that any person can clone devstack and run stack.sh and it works | 16:14 |
lucasagomes | this new patch-set is mostly cosmetics so we should be good there | 16:14 |
gmann | yeah testing patch are able to catch those | 16:14 |
lucasagomes | clarkb, exactly yeah | 16:14 |
clarkb | the defaults are not set by CI they are set by devstack and we need to respect that | 16:14 |
lucasagomes | I think there was two problems, this one that you mentioned and the fix for that was to change the ENABLED_SERVICES variable in stackrc | 16:15 |
lucasagomes | so if no enable_service is specified in local.conf it will use those | 16:15 |
lucasagomes | and the other problem is regarding the precloned repositories | 16:15 |
lucasagomes | in the lib/neutron_plugins/ovn_agent we were always trying to import a module from the neutron repository | 16:16 |
lucasagomes | this module is only used when we set the OVN_BUILD_FROM_SOURCE variable to True (it's False by default) | 16:16 |
lucasagomes | because by default we install OVN from the distro packages | 16:16 |
clarkb | how did nodepool jobs hit it then as they don't set any non default vars for ovn? | 16:16 |
lucasagomes | but that second problem was only seein in this zuul gate there cause it didn't have all the repos precloned | 16:17 |
clarkb | I guess we loaded the module either way | 16:17 |
lucasagomes | clarkb, yes before it tried to load it regardless | 16:17 |
clarkb | well anyone who also just clones devstack would fail too unless they precloned | 16:17 |
lucasagomes | now I changed it, I will only load it if OVN_BUILD_FROM_SOURCE is set to True | 16:17 |
lucasagomes | which is not the default so we should be good there on nodepool | 16:17 |
gmann | we can remove these service enable/disable in job now - https://review.opendev.org/c/openstack/devstack/+/791436/5/.zuul.yaml | 16:19 |
clarkb | gmann: I don't think so as they replaced the test amtrix stuff in devstack-gate | 16:19 |
lucasagomes | gmann, we can't, because the stackrc ENABLED_SERVICES only get used if no other service has been disabled/enabled | 16:20 |
lucasagomes | gmann, see L64 from https://review.opendev.org/c/openstack/devstack/+/791436/5/stackrc | 16:20 |
clarkb | lucasagomes: its only if no other service is enabled I think it is used if you disabled things | 16:20 |
gmann | ah yeah, for overridden config yes we need it there, got it | 16:21 |
lucasagomes | clarkb, gotcha | 16:21 |
gmann | disable services has separate var | 16:21 |
lucasagomes | right | 16:21 |
lucasagomes | clarkb, gmann I need to run but, I think we are good now. We can resume this on Monday. Thanks for all the help | 16:22 |
lucasagomes | I learned something a few new things doing it :D | 16:22 |
lucasagomes | would be good to have that ensure-devstack job in devstack itself tho | 16:23 |
lucasagomes | but that's a separated thing | 16:23 |
gmann | lucasagomes: thanks for tests and all. will review it today. | 16:23 |
lucasagomes | gmann, thank you! Appreciate it | 16:23 |
gmann | yeah i am in favor of adding that job in devstack side | 16:23 |
gmann | but yeah separate patch | 16:24 |
lucasagomes | sounds good | 16:25 |
*** lucasagomes has quit IRC | 16:26 | |
*** jpena is now known as jpena|off | 16:28 | |
*** psahoo_ has quit IRC | 16:41 | |
*** rh-jlabarre has quit IRC | 17:16 | |
*** amodi has joined #openstack-qa | 18:07 | |
*** ysandeep is now known as ysandeep|away | 18:10 | |
*** whoami-rajat has quit IRC | 19:21 | |
openstackgerrit | Monty Taylor proposed openstack/devstack master: DNM add feature pragma https://review.opendev.org/c/openstack/devstack/+/791541 | 19:52 |
*** nweinber has quit IRC | 20:19 | |
*** rcernin has joined #openstack-qa | 20:39 | |
*** amodi has quit IRC | 21:34 | |
*** rcernin has quit IRC | 21:40 | |
*** rcernin has joined #openstack-qa | 22:06 | |
*** rcernin has quit IRC | 22:11 | |
*** rcernin has joined #openstack-qa | 22:23 | |
*** rcernin has quit IRC | 22:27 | |
*** rcernin has joined #openstack-qa | 22:28 | |
*** rcernin has quit IRC | 22:33 | |
*** rcernin has joined #openstack-qa | 23:05 | |
*** rcernin has quit IRC | 23:09 | |
*** tosky has quit IRC | 23:11 | |
*** rcernin has joined #openstack-qa | 23:35 | |
*** rcernin has quit IRC | 23:40 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!