opendevreview | Julia Kreger proposed openstack/ironic-python-agent master: WIP: Guard shared device/cluster filesystems https://review.opendev.org/c/openstack/ironic-python-agent/+/839084 | 01:07 |
---|---|---|
*** mat_fechner is now known as matfechner | 07:28 | |
dtantsur | iury<tab> :( | 07:44 |
dtantsur | morning folks | 07:44 |
opendevreview | Dmitry Tantsur proposed openstack/metalsmith master: DNM test the CI https://review.opendev.org/c/openstack/metalsmith/+/843112 | 07:45 |
dtantsur | folks, please review: https://review.opendev.org/c/openstack/ironic/+/839984 https://review.opendev.org/c/openstack/ironic/+/839985 | 07:46 |
dtantsur | I suspect IPA wallaby may be broken because of it | 07:46 |
dtantsur | arne_wiebalck: hey, if you have a minute ^^^ | 07:46 |
opendevreview | Dmitry Tantsur proposed openstack/ironic-python-agent stable/wallaby: DNM test the CI https://review.opendev.org/c/openstack/ironic-python-agent/+/843113 | 07:49 |
arne_wiebalck | dtantsur: done! | 08:19 |
opendevreview | Riccardo Pittau proposed openstack/ironic-python-agent master: Use json for lsblk output https://review.opendev.org/c/openstack/ironic-python-agent/+/775391 | 08:20 |
dtantsur | thanks! | 08:33 |
dtantsur | arne_wiebalck: another, a bit more interesting one if you have another minute: https://review.opendev.org/c/openstack/python-ironicclient/+/842986 | 09:16 |
opendevreview | Dmitry Tantsur proposed openstack/metalsmith master: Provide a work around to fix the CI https://review.opendev.org/c/openstack/metalsmith/+/843112 | 10:03 |
opendevreview | Merged openstack/python-ironicclient master: CI: Removing ironic job queue https://review.opendev.org/c/openstack/python-ironicclient/+/842600 | 10:23 |
dtantsur | iurygregory: morning! https://review.opendev.org/c/openstack/ironic/+/839985 should fix your wallaby struggles | 11:15 |
iurygregory | dtantsur, \o/ | 11:22 |
iurygregory | YAY! | 11:22 |
iurygregory | I think we will probably need this in the bugfix branches for Ironic... | 11:23 |
iurygregory | good morning Ironic o/ | 11:23 |
opendevreview | Merged openstack/ironic stable/wallaby: Fix ironic-lib from source and branch detection in IPA builds https://review.opendev.org/c/openstack/ironic/+/839985 | 11:31 |
opendevreview | Verification of a change to openstack/ironic stable/xena failed: Fix ironic-lib from source and branch detection in IPA builds https://review.opendev.org/c/openstack/ironic/+/839984 | 11:31 |
iurygregory | ironic-grenade https://zuul.opendev.org/t/openstack/build/2bcc4d187ecf4b02bb5e2e0052785885 : FAILURE in 37m 52s \o/ | 11:33 |
dtantsur | iurygregory: bugfix branches will be.. interesting. we cannot use TARGET_BRANCH for them. | 11:40 |
iurygregory | ohhhh | 11:40 |
iurygregory | well, we can say to use the stable branch | 11:41 |
dtantsur | iurygregory: then we won't be testing IPA patches | 11:41 |
dtantsur | I guess we'll need another iteration of this patch, where we use $something that defaults to TARGET_BRANCH | 11:41 |
dtantsur | then we can override $something on bugfix branches | 11:42 |
dtantsur | I will think about it, maybe we can use some zuul magic | 11:42 |
iurygregory | right, zuul can override some env vars | 11:43 |
iurygregory | and override the values | 11:43 |
iurygregory | it makes some sense | 11:43 |
dtantsur | yep. and we can ask zuul which branch is used for IPA, I guess | 11:43 |
* iurygregory is still waking up =) | 11:43 | |
iurygregory | yeah | 11:43 |
opendevreview | Dmitry Tantsur proposed openstack/metalsmith master: Provide a work around to fix the CI https://review.opendev.org/c/openstack/metalsmith/+/843112 | 12:02 |
opendevreview | Dmitry Tantsur proposed openstack/metalsmith master: Provide a work around to fix the CI https://review.opendev.org/c/openstack/metalsmith/+/843112 | 13:34 |
TheJulia | umm... why do I have this feeling like CI is detonating? | 13:51 |
TheJulia | At least, glancing at the channel chatter | 13:51 |
dtantsur | kinda. good morning TheJulia | 14:26 |
* dtantsur -> errands | 14:26 | |
opendevreview | Julia Kreger proposed openstack/ironic master: CI: Fix Multinode Grenade Testing https://review.opendev.org/c/openstack/ironic/+/840530 | 15:00 |
TheJulia | Well that is infuriating... I now get issues rooting back to the dss key length for ssh when I turned cleaning off. Hopefully this works and it is just some weird fluke. | 15:01 |
TheJulia | why is oslo log veresion conflicting now | 15:12 |
opendevreview | Julia Kreger proposed openstack/ironic-python-agent master: Update oslo.log requirement to 4.6.1 https://review.opendev.org/c/openstack/ironic-python-agent/+/843162 | 15:18 |
opendevreview | Verification of a change to openstack/ironic master failed: CI: Removing ironic job queue https://review.opendev.org/c/openstack/ironic/+/842597 | 16:38 |
TheJulia | iurygregory: speaking of ipa and python, even on master weirdness around oslo log. It feels like a new pip version dropped or soemthing | 20:45 |
TheJulia | iurygregory: speaking of ipa and python, even on master weirdness around oslo log. It feels like a new pip version dropped or something funky | 20:49 |
TheJulia | errrrr | 20:49 |
TheJulia | okay, time to step away from the computerbox | 20:49 |
iurygregory | TheJulia, I saw some weirdness in master also (when trying to merge the multipath patch) | 20:51 |
iurygregory | hummm or maybe we are using something with 3.6 and it's not compatible not sure.. | 20:51 |
TheJulia | I'm thinking we need a list of known issues | 20:51 |
TheJulia | oh | 20:51 |
TheJulia | you know what | 20:52 |
TheJulia | I bet it is incompatabilities in the ramdisk build | 20:52 |
TheJulia | I can look tomorrow, I need to take migraine meds | 20:52 |
iurygregory | makes sense, I remember it was a bit random the failure | 20:52 |
iurygregory | so maybe is just when we are using DIB | 20:52 |
iurygregory | in rax we switch to tinyipa the jobs automatically because they have nested virtualization I think | 20:53 |
TheJulia | Indeed | 20:55 |
TheJulia | So it could have been broken there for a few weeks or something | 20:56 |
iurygregory | https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/841562 we merged the change in ipa-b to build using py39... | 20:56 |
TheJulia | Hmm I bet we’re in the land of fun resolver issues the | 20:57 |
TheJulia | Then | 20:57 |
iurygregory | yeah | 20:57 |
TheJulia | Anyway, tomorrow at this point. Groceries are almost here | 20:57 |
opendevreview | Verification of a change to openstack/ironic stable/xena failed: Fix ironic-lib from source and branch detection in IPA builds https://review.opendev.org/c/openstack/ironic/+/839984 | 22:58 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!