Friday, 2021-06-18

*** abhishekk is now known as akekane|away05:37
*** akekane|away is now known as abhishekk05:37
*** jpena|off is now known as jpena07:18
*** rpittau|afk is now known as rpittau08:17
opendevreviewLee Yarwood proposed openstack/devstack master: os_vif: Add support for configuring os_vif_ovs plugin  https://review.opendev.org/c/openstack/devstack/+/79682608:44
*** raukadah is now known as chandankumar09:26
*** jpena is now known as jpena|lunch11:41
*** bhagyashris_ is now known as bhagyashris11:50
*** pojadhav- is now known as pojadhav11:51
opendevreviewGregory Thiemonge proposed openstack/devstack master: WIP Fix updating setuptools in Centos  https://review.opendev.org/c/openstack/devstack/+/79706911:54
opendevreviewRadosław Piliszek proposed openstack/devstack master: Revert edk2 workaround  https://review.opendev.org/c/openstack/devstack/+/79707112:07
*** jpena|lunch is now known as jpena12:38
fungirosmaita: looking closer at that error you posted in #opendev, it refers to something not being defined for https://opendev.org/openstack/devstack/src/branch/master/roles/write-devstack-local-conf/tasks/main.yaml and since most of those are either falling back to "omit" or are internal zuul vars, i think that means devstack_local_conf_path isn't being supplied by the job12:46
rosmaitaQA team: can someone help me with https://review.opendev.org/c/openstack/python-brick-cinderclient-ext/+/796835 ?  The issue I'm seeing is here: https://zuul.opendev.org/t/openstack/build/fc3c9eed53d24157983e35cac1eb5ad912:47
rosmaitaas fungi points out, i need to define something else, but i'm not sure what or what its value should be12:48
rosmaitai'm trying to write a new job that has devstack-tox-functional as a parent12:49
fungimight be a regression in the role, https://opendev.org/openstack/devstack/raw/branch/master/roles/write-devstack-local-conf/README.rst indicates it's supposed to have a default value of {{ devstack_base_dir }}/devstack/local.conf12:51
fungibut it's also possible i'm jumping to conclusions about what it is that's undefined, since ansible doesn't seem to say exactly12:51
rosmaitafungi: thanks, also the job seems to work fine for py38 without setting that option12:52
fungistrange12:53
rosmaitai wonder if I should just recheck to verify that it's not some weird spurious error12:57
fungioh, i wonder if it could be the nodeset override causing this?12:57
fungii think maybe the relevant bit of the message may actually be: "hostvars['controller']" is undefined12:58
fungimaybe there is no "controller" node because the parent job makes assumptions about node names in a custom nodeset it defines12:58
rosmaitathat could be12:59
fungiyeah, so if you follow the parentage all the way back you'll see it's ultimately inheriting from the "devstack-minimal" job definition, which includes a nodeset of openstack-single-node-focal13:01
fungirosmaita: so maybe try setting your nodeset to openstack-single-node-bionic instead of ubuntu-bionic?13:02
rosmaitafungi: i did not trace far enough back ... i will try your suggestion13:02
fungithe "job documentation" link from the build report is helpful to do that quickly13:02
fungiit takes you to an interlinked view of the job definition in zuul, where you can click on the parent job and see that job's page, ad infinitum13:03
rosmaitafungi: that is an excellent tip! thank you13:03
rosmaitaTIL :D13:03
fungiso i just quickly kept clicking on parent, parent, parent until i saw one adding a nodeset13:04
rosmaitathat is way more efficient than my (former) way of doing it13:04
rosmaitaok, i pushed an update ... i'll let you know what happens13:09
*** raukadah is now known as chandankumar13:13
rosmaitafungi: didn't work :(13:18
fungidifferent error?13:19
rosmaitastack.sh died due to a check saying "'WARNING: this script has not been tested on bionic'13:19
rosmaitabut you know, isn't py 3.6 a supported distro for Xena?13:19
fungiahh, you probably need an older devstack branch13:19
fungichecking the pti now13:19
fungiyeah, 3.6 is but for centos not for ubuntu: https://governance.openstack.org/tc/reference/runtimes/xena.html13:20
fungii guess the assumption is that if you want to test python 3.6 with current devstack you need to do it on centos-8(-stream)13:21
fungihopefully the devstack maintainers in here can confirm13:21
rosmaitaok, thanks13:22
fungirosmaita: looks like the rationale is included in https://review.opendev.org/78875413:24
opendevreviewsean mooney proposed openstack/devstack master: os_vif: Add support for configuring os_vif_ovs plugin  https://review.opendev.org/c/openstack/devstack/+/79682613:24
sean-k-mooneyyoctozepto: fyi i think ^ adresses your concerns but if not just let me know13:26
opendevreviewGregory Thiemonge proposed openstack/devstack master: WIP Fix updating setuptools in Centos  https://review.opendev.org/c/openstack/devstack/+/79706913:41
opendevreviewRadosław Piliszek proposed openstack/devstack master: os_vif: Add support for configuring os_vif_ovs plugin  https://review.opendev.org/c/openstack/devstack/+/79682614:05
opendevreviewRadosław Piliszek proposed openstack/devstack master: os_vif: Add support for configuring os_vif_ovs plugin  https://review.opendev.org/c/openstack/devstack/+/79682614:06
*** rpittau is now known as rpittau|afk14:09
yoctozeptosean-k-mooney: ack, see my update; also, I need someone from neutron to look at it; I pinged them yesterday but had no response14:11
slaweqyoctozepto I have meeting now but will check it after14:12
yoctozeptoslaweq: thanks!14:13
*** jpena is now known as jpena|out14:48
*** pojadhav- is now known as pojadhav14:48
* pojadhav out, network fluctuating badly(bad weather)..14:49
opendevreviewVictoria Martinez de la Cruz proposed openstack/devstack-plugin-ceph master: Add CentOS8 Stream support  https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/79712415:17
opendevreviewVictoria Martinez de la Cruz proposed openstack/devstack-plugin-ceph master: Add CentOS8 Stream support  https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/79712415:29
opendevreviewVictoria Martinez de la Cruz proposed openstack/devstack-plugin-ceph master: Remove redundant Fedora version check  https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/79713415:44
*** dviroel is now known as dviroel|busy15:46
rosmaitafungi: https://review.opendev.org/c/openstack/python-brick-cinderclient-ext/+/796835 is finally passing!  Thanks for your help15:46
fungirosmaita: aha, so you switched to using centos-8-stream for the 3.6 job?15:53
fungiglad to know that's actually succeeding15:53
rosmaitayes, had to find the single-node nodepool definition15:54
clarkbmy only word of caution on that is I don't think you'll find the same long term stability on that platform as with bionic. But in practice for a small lib like that it probably doesn't matter much15:57
rosmaitaclarkb: thanks, i will keep that in mind16:07
*** jpena|out is now known as jpena16:22
*** jpena is now known as jpena|off16:48
opendevreviewGregory Thiemonge proposed openstack/devstack master: WIP Fix updating setuptools in Centos  https://review.opendev.org/c/openstack/devstack/+/79706919:03
opendevreviewMerged openstack/devstack-gate master: Keep only legacy jobs which actually test the d-g code path  https://review.opendev.org/c/openstack/devstack-gate/+/79542622:45

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