| clarkb | fungi: its late so no rush but did we start the process of lceaning up the lists.katacontainers.io server yet? | 00:03 |
|---|---|---|
| clarkb | no concerns or anything it just occurred to me that maybe that hadn't happened yet and is something we can do at this point | 00:03 |
| fungi | clarkb: not yet, i figured we'd give it a little time, but frankly we copied everything of value off it anyway, so probably doesn't need to be kept around or even snapshot (but i'll likely do that first anyway) | 00:04 |
| fungi | snapshotting it now | 00:10 |
| fungi | i meant to notify the openstack-discuss ml today about upcoming maintenance, but did not get to it yet | 00:12 |
| fungi | image creation is still in progress. i'll just check it in the morning and delete the server if that succeeded | 00:37 |
| opendevreview | Bernhard Berg proposed zuul/zuul-jobs master: prepare-workspace-git: Add ability to define synced pojects https://review.opendev.org/c/zuul/zuul-jobs/+/887917 | 10:48 |
| fungi | #status log Deleted the old lists.katacontainers.io server now that it's been migrated to its new home for over a week | 11:37 |
| opendevstatus | fungi: finished logging | 11:37 |
| opendevreview | Jeremy Stanley proposed openstack/project-config master: Replace 2023.2/Bobcat key with 2024.1/Caracal https://review.opendev.org/c/openstack/project-config/+/896944 | 14:08 |
| opendevreview | daniel.pawlik proposed zuul/zuul-jobs master: WIP Add workaround to accept traffic in/out the bridge; add dns test https://review.opendev.org/c/zuul/zuul-jobs/+/896646 | 14:10 |
| opendevreview | daniel.pawlik proposed zuul/zuul-jobs master: WIP Add workaround to accept traffic in/out the bridge; add dns test https://review.opendev.org/c/zuul/zuul-jobs/+/896646 | 14:46 |
| opendevreview | Jeremy Stanley proposed openstack/project-config master: Temporarily remove release docs semaphores https://review.opendev.org/c/openstack/project-config/+/896948 | 14:50 |
| opendevreview | Jeremy Stanley proposed openstack/project-config master: Revert "Temporarily remove release docs semaphores" https://review.opendev.org/c/openstack/project-config/+/896949 | 14:50 |
| opendevreview | daniel.pawlik proposed zuul/zuul-jobs master: WIP Add workaround to accept traffic in/out the bridge; add dns test https://review.opendev.org/c/zuul/zuul-jobs/+/896646 | 15:08 |
| opendevreview | daniel.pawlik proposed zuul/zuul-jobs master: WIP Add workaround to accept traffic in/out the bridge; add dns test https://review.opendev.org/c/zuul/zuul-jobs/+/896646 | 15:18 |
| opendevreview | daniel.pawlik proposed zuul/zuul-jobs master: WIP Add workaround to accept traffic in/out the bridge; add dns test https://review.opendev.org/c/zuul/zuul-jobs/+/896646 | 15:28 |
| opendevreview | daniel.pawlik proposed zuul/zuul-jobs master: WIP Add workaround to accept traffic in/out the bridge; add dns test https://review.opendev.org/c/zuul/zuul-jobs/+/896646 | 15:39 |
| opendevreview | daniel.pawlik proposed zuul/zuul-jobs master: WIP Add workaround to accept traffic in/out the bridge; add dns test https://review.opendev.org/c/zuul/zuul-jobs/+/896646 | 15:47 |
| opendevreview | daniel.pawlik proposed zuul/zuul-jobs master: WIP Add workaround to accept traffic in/out the bridge; add dns test https://review.opendev.org/c/zuul/zuul-jobs/+/896646 | 16:04 |
| opendevreview | daniel.pawlik proposed zuul/zuul-jobs master: WIP Add workaround to accept traffic in/out the bridge; add dns test https://review.opendev.org/c/zuul/zuul-jobs/+/896646 | 16:39 |
| opendevreview | daniel.pawlik proposed zuul/zuul-jobs master: WIP Add workaround to accept traffic in/out the bridge; add dns test https://review.opendev.org/c/zuul/zuul-jobs/+/896646 | 16:48 |
| opendevreview | daniel.pawlik proposed zuul/zuul-jobs master: WIP Add workaround to accept traffic in/out the bridge; add dns test https://review.opendev.org/c/zuul/zuul-jobs/+/896646 | 16:57 |
| opendevreview | daniel.pawlik proposed zuul/zuul-jobs master: WIP Add workaround to accept traffic in/out the bridge; add dns test https://review.opendev.org/c/zuul/zuul-jobs/+/896646 | 17:09 |
| opendevreview | daniel.pawlik proposed zuul/zuul-jobs master: WIP Add workaround to accept traffic in/out the bridge; add dns test https://review.opendev.org/c/zuul/zuul-jobs/+/896646 | 17:17 |
| slittle | I was wondering if there was someone who could help me understand a zuul complaint with https://review.opendev.org/c/starlingx/utilities/+/895176 | 17:59 |
| slittle | I think I need to install openldap-dev.deb into the ubuntu container running the tests | 18:04 |
| fungi | looking | 18:05 |
| slittle | or perhaps a newer ubuntu release than 'nodeset: ubuntu-bionic' | 18:07 |
| clarkb | definitely stop using ubuntu-bionic if you can | 18:07 |
| fungi | that job says it's using debian-bullseye actually | 18:08 |
| fungi | we do have debian-bookworm images too | 18:08 |
| fungi | yeah, seems like you're installing https://pypi.org/project/python-ldap/ and there's no wheels for that, only sdists | 18:09 |
| slittle | am I misunderstanding the purpose of the 'nodeset' directive? Does it make sense for that to map to 'debian-bullseye' ? | 18:10 |
| fungi | so you'll need libldap-dev to link against probably, checking for the exact package name for that | 18:10 |
| fungi | slittle: not sure i understand what you're asking but this is the job you're talking about i think (the one that's failing on the change you linked): https://zuul.opendev.org/t/openstack/job/stx-utilities-tox-pylint | 18:11 |
| slittle | y | 18:11 |
| fungi | it says the nodeset it's using is debian-bullseye, and the inventory.yaml in the zuul build result confirms that's what it ran on too | 18:11 |
| slittle | sorry, my mouse drifted down to a different job within that zuul file | 18:12 |
| fungi | no worries. so anyway, it looks like on bullseye the distro package you want to install for those headers is libldap2-dev, though if you switch to bookworm it's just libldap-dev instead | 18:13 |
| fungi | https://packages.debian.org/search?keywords=libldap-dev | 18:13 |
| clarkb | you can add both to your bindep file too | 18:14 |
| slittle | lets stick with bullseye. So I need libldap2-dev not libldap-dev. Where do I add that? I'm missing Al right now. He was a wizard with the tox/zuul stuff. | 18:15 |
| fungi | yeah, it looks like that job does include the unittests pre playbook, which installs anything listed in bindep.txt | 18:15 |
| fungi | https://zuul.opendev.org/t/openstack/build/33426388c34c41259ce6d67f89549906/console#1/0/2/debian-bullseye shows it installed relevant packages listed in the src/opendev.org/starlingx/utilities/bindep.txt file | 18:16 |
| slittle | libldap2-dev is already listed | 18:16 |
| fungi | https://opendev.org/starlingx/utilities/src/branch/master/bindep.txt | 18:16 |
| fungi | yeah, looks like it should be installed for the pylint jobs based on the profile it's tagged with | 18:17 |
| fungi | and that's the problem... https://zuul.opendev.org/t/openstack/build/33426388c34c41259ce6d67f89549906/console#1/0/15/debian-bullseye | 18:18 |
| fungi | /usr/bindep-env/bin/bindep -b -f src/opendev.org/starlingx/utilities/bindep.txt compile test | 18:18 |
| fungi | it didn't ask bindep about the pylint profile | 18:18 |
| clarkb | I think you can set bindep profiles to add to the job vars? but ya I think it only does test by default | 18:19 |
| fungi | confirmed: https://zuul-ci.org/docs/zuul-jobs/latest/general-roles.html#rolevar-bindep.bindep_profile | 18:19 |
| clarkb | wow I remembered correctly | 18:20 |
| slittle | as in ... | 18:21 |
| slittle | vars: | 18:21 |
| slittle | bindep_profile: pylint | 18:21 |
| slittle | which is already part of stx-utilities-tox-pylint job definition | 18:22 |
| fungi | yeah, https://zuul.opendev.org/t/openstack/job/stx-utilities-tox-pylint also confirms if you expand the job variables array | 18:23 |
| fungi | clarkb: what could prevent that from getting plumbed through? the role should automatically pick that up, right? or does the unittests grandparent job definition need that to be a settable variable? | 18:25 |
| fungi | searching repos, i find lots of examples of setting that when including the role in various playbooks, but very few cases of setting it in job definitions/variants | 18:26 |
| clarkb | if something sets it on the role inclusion that wuold win precedence | 18:26 |
| clarkb | I see the variable is set in the role and is used by the role so it isn't just a case of bad docs | 18:27 |
| clarkb | I don't see that happening | 18:29 |
| clarkb | tox_envlist is passed into the job but bindep_profile is not : https://zuul.opendev.org/t/openstack/build/33426388c34c41259ce6d67f89549906/log/zuul-info/inventory.yaml#28-33 | 18:30 |
| clarkb | the failure is from a couple of weeks ago. Was this problem fixed in the interim? | 18:31 |
| clarkb | survey says yes https://opendev.org/starlingx/utilities/commit/a04fa768cd4243d2b665f95e4c9b74366dacc755 | 18:31 |
| clarkb | you just need to recheck I think | 18:31 |
| fungi | oh, good eye | 18:31 |
| slittle | k, I'll recheck | 18:31 |
| fungi | Completed at 2023-09-14 14:23:34 | 18:32 |
| slittle | yeah, I see successful runs that are more recently dated. | 18:32 |
| fungi | and yes, fix merged 4 days after that failure | 18:33 |
| slittle | ha, that's what I get for setting this change aside for 2 weeks to work other stuff. Thanks for looking at it. I've definitely learned a little zuul kungfu | 18:40 |
| fungi | no problem, it was a good double-check, and at least we arrived at the correct answer even if it had already been implemented | 18:43 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!