Thursday, 2022-02-24

opendevreviewMerged openstack/ironic-lib stable/wallaby: Add platform:base-py[2|3] to bindep  https://review.opendev.org/c/openstack/ironic-lib/+/83020302:14
arne_wiebalckGood morning, Ironic!07:28
jandersmorning arne_wiebalck (not sure if I can call it good given what's going on)07:35
arne_wiebalckjanders: yeah (good morning to you none the less :))07:36
rpittaugood morning ironic! o/08:30
arne_wiebalckhey rpittau o/08:58
rpittauhey arne_wiebalck :)08:58
rpittauinteresting.... I didn't change the try-except-finally logic in images.py but it seems it's skipping the finally clause in one test -> https://1db75cf0b12d832dc11e-0da77774ba1e870a646f19d85e727a87.ssl.cf5.rackcdn.com/819121/13/check/openstack-tox-py39/75272ed/job-output.txt09:03
opendevreviewVerification of a change to openstack/tenks master failed: CI: switch Ubuntu jobs to Focal 20.04  https://review.opendev.org/c/openstack/tenks/+/82850909:31
dtantsurterrible morning, ironic09:32
jandersdtantsur terrible morning :(09:33
opendevreviewMerged openstack/tenks master: CI: switch Ubuntu jobs to Focal 20.04  https://review.opendev.org/c/openstack/tenks/+/82850910:05
opendevreviewMerged openstack/ironic master: Deprecate instance network boot  https://review.opendev.org/c/openstack/ironic/+/82874612:02
opendevreviewVerification of a change to openstack/ironic-inspector master failed: Support listening on a Unix socket  https://review.opendev.org/c/openstack/ironic-inspector/+/82712412:02
*** sshnaidm is now known as sshnaidm|off12:23
TheJuliadtantsur: what can we use to denote the start of a day when it is such a morning?12:40
TheJuliaif you say go back to bed, I'm all for this, after my first meeting today12:40
dtantsurTheJulia: I'm basically out of words today, but going back to bed may be a wise choice12:43
* TheJulia offers a hug12:43
TheJuliaThat is how I was feeling last night12:44
dtantsura lot of hugs will be needed by many people the coming days12:44
dtantsurI really hope pas-ha[m] and other folks are safe12:44
TheJuliaditto12:45
TheJuliaI was thinking about them last night quite a bit12:45
jandersin general the term "morning" is popularly associated with light, but today consists of darkness, mostly12:48
dtantsuryeah12:48
TheJuliaagreed12:48
iurygregoryHello Ironic o/12:49
jandershey iurygregory o/12:49
dtantsurwelcome to the new world iurygregory 12:49
jandersI think we agree we liked the old one more12:49
iurygregorydtantsur, yeah =(12:50
*** pmannidi is now known as pmannidi|Away12:50
jandersgit reset --hard for reality would come in handy12:50
dtantsurremember some of us went to Geneva for a meetup in early 2020? who could know it was the last good time for years to come?12:50
iurygregoryyeah12:50
iurygregoryCERN midcycle <312:51
TheJuliadtantsur: I had a photo from my stop over in Paris on the way to Geneva come up on facebook yesterday12:51
TheJuliait made me super sad :(12:51
iurygregoryawww =(12:51
jandersI wasn't there, but have similar feelings about Kubecon in San Diego. Weird stuff started happening with time-space after that...12:52
TheJuliaKubecon San Diego was just... weird12:53
TheJuliawell, that entire week was weird12:53
TheJuliabut I had amazing fish tacos12:53
* TheJulia needs to go obtain more fish tacos12:53
iurygregoryfish tacos? wow12:54
iurygregoryTheJulia, dtantsur I've updated the cycle-highlights for Yoga o/12:54
arne_wiebalckstevebaker[m]: it seems your latest commit to diskimage-builder breaks our IPA image building12:55
TheJuliaarne_wiebalck: ugh12:55
dtantsuriurygregory: thanks! I'll get to it when I can.12:56
iurygregoryack o/12:56
arne_wiebalckstevebaker[m]: I tried without https://github.com/openstack/diskimage-builder/commit/6a3b8d94821fe9f2398124415dcb1220920d4101 and it works again.12:56
iurygregoryI will be lurking around irc (since I'm on PTO today XD)12:56
TheJuliaarne_wiebalck: are you handing it a partition image?!12:56
arne_wiebalckTheJulia: CentOS-Stream-GenericCloud-8-20220125.1.x86_64.qcow212:57
TheJuliaarne_wiebalck: do you hapen to get a link?12:58
arne_wiebalckTheJulia: let me see ...12:58
arne_wiebalckTheJulia: https://cloud.centos.org/centos/8-stream/x86_64/images/CentOS-Stream-GenericCloud-8-20220125.1.x86_64.qcow212:59
TheJuliamuch appreciated, downloading12:59
arne_wiebalckTheJulia: I am not specifying this image directly, I think dib uses this from the parameters we pass.13:01
arne_wiebalckTheJulia: like DIB_RELEASE13:01
TheJuliaso my understanding was this was only used on whole disk images, I guess I'll know for sure in a few minutes13:01
arne_wiebalckTheJulia: thanks!13:03
TheJuliaso... the image *is* a whole disk image with a partition13:09
TheJuliaarne_wiebalck: got any build logs or the command your using for latest dib?13:09
arne_wiebalckTheJulia: yep, let me give you the relevant bit ...13:10
TheJuliathanks13:10
arne_wiebalckTheJulia: this is from the download to where the mount fails (which is how I found it): https://paste.opendev.org/show/bbzklVIhyuxQDbSILI1k/13:11
arne_wiebalckTheJulia: line 71 is also interesting13:12
TheJuliawhat version of lsblk do you have?13:12
TheJuliaand I guess that leads to what is this being run on os wise?13:12
janderssee you on Monday Ironic o/ I hope we can all get some rest13:12
TheJuliagoodnight janders 13:12
arne_wiebalckTheJulia: this is the pipeline runner, let me check13:13
arne_wiebalckTheJulia: but none of the centos OSs I have checked know that option13:13
* TheJulia wonders how CI passed13:14
arne_wiebalckTheJulia: should be cs8 as well13:15
arne_wiebalckTheJulia: the OS of the gitlab runner13:15
TheJuliainteresting, I don't have my p1 loopback partition locally13:22
arne_wiebalckTheJulia: you think it is an issue with the container we use for the runner?13:26
TheJuliamaybe...13:28
TheJuliadunno yet since github is not loading for me at the moment13:28
TheJuliatrying to find where this option got added13:28
rpittauarne_wiebalck: centos stream 8 ?13:28
arne_wiebalckrpittau: yes13:29
arne_wiebalckTheJulia: rpittau: I will launch a container to look inside ..13:29
* TheJulia steps away to do the "look more presentable for a video call" dance13:29
rpittauarne_wiebalck: I don't have that up now, check the version of lsblk as suggested by TheJulia 13:30
arne_wiebalckrpittau: `lsblk from util-linux 2.32.1`13:30
arne_wiebalckTheJulia: yes, the container has a number of loop back devices ... no idea, why13:31
opendevreviewMerged openstack/ironic master: Support listening on a Unix socket  https://review.opendev.org/c/openstack/ironic/+/82850813:31
opendevreviewMerged openstack/ironic-python-agent stable/xena: Stop defaulting the standalone job to netboot; use the correct boot mode  https://review.opendev.org/c/openstack/ironic-python-agent/+/82972113:31
opendevreviewMerged openstack/ironic-python-agent bugfix/8.4: Stop defaulting the standalone job to netboot; use the correct boot mode  https://review.opendev.org/c/openstack/ironic-python-agent/+/83035613:31
rpittauarne_wiebalck: PARTTYPENAME is present since 2.3513:33
arne_wiebalckTheJulia: wait, these are the device of the host :)13:33
arne_wiebalckrpittau: aha13:33
arne_wiebalckrpittau: not sure this is the problem, but for sure this does not help13:34
arne_wiebalckrpittau: what is the version in stream 8 ?13:34
rpittau2.32.113:35
TheJuliaI wonder if stream9 was the base os for the build13:36
rpittauthat could explain the PARTTYPE13:36
rpittauPARTYPENAME sorry13:37
rpittauCS9 has 2.37.213:37
rpittauyeah stevebaker[m] change is definitely not compatible with CS813:37
arne_wiebalckyes, cs9 has 2.37.213:38
rpittauyep :)13:39
arne_wiebalckwe clone dib for every IPA build from master13:39
TheJuliaokay, so... if someone wants to click the revert button13:39
TheJuliaI'll approve it13:39
rpittauTheJulia: https://review.opendev.org/c/openstack/diskimage-builder/+/83071713:43
TheJuliamuch appreciated13:43
TheJuliaapproved and workflowed13:43
arne_wiebalckthanks rpittau and TheJulia !13:43
rpittaunp :)13:44
opendevreviewVerification of a change to openstack/ironic-python-agent bugfix/8.3 failed: Stop defaulting the standalone job to netboot; use the correct boot mode  https://review.opendev.org/c/openstack/ironic-python-agent/+/82972213:44
opendevreviewOpenStack Release Bot proposed openstack/ironic bugfix/20.0: Update .gitreview for bugfix/20.0  https://review.opendev.org/c/openstack/ironic/+/83083715:08
dtantsurgoing to an anti-war demo, see you tomorrow15:39
dtantsuro/15:39
TheJuliao/15:40
MahnoorAsgharo/15:43
TheJuliabeing an adult is no fun16:00
opendevreviewVerification of a change to openstack/ironic-inspector master failed: Support listening on a Unix socket  https://review.opendev.org/c/openstack/ironic-inspector/+/82712416:20
TheJuliahmm where did Iury disappear off to...16:37
arne_wiebalckhe is on pto today I think ... no?16:40
arne_wiebalck(or was this yesterday?)16:41
* TheJulia shrugs16:41
* arne_wiebalck maybe needs some pto, too16:41
arne_wiebalckit was today16:42
TheJuliaI definitely do16:45
TheJuliaI'll likley be away tomorrow. I've got one two hour meeting and then breakfast with bottomless memosas is tempting16:45
TheJuliaarne_wiebalck: w/r/t oslo policy see: https://bugs.launchpad.net/oslo.policy/+bug/174256917:03
TheJuliatl;dr horizon17:03
rpittauTheJulia: Iury's out today and tomorrow17:19
TheJuliaokay17:19
rpittausee ya tomorrow o/17:20
opendevreviewVerification of a change to openstack/ironic-python-agent bugfix/8.3 failed: Stop defaulting the standalone job to netboot; use the correct boot mode  https://review.opendev.org/c/openstack/ironic-python-agent/+/82972217:34
arne_wiebalckTheJulia: oh, ok17:44
* arne_wiebalck is trying to wrap his head around this17:46
arne_wiebalckTheJulia: there is still the issue of multiple aliases17:47
arne_wiebalckTheJulia: which is the point where this breaks17:47
arne_wiebalckTheJulia: or is this an Ironic special case?17:47
arne_wiebalckbye everyone o/18:03
opendevreviewVerification of a change to openstack/ironic-inspector master failed: Support listening on a Unix socket  https://review.opendev.org/c/openstack/ironic-inspector/+/82712419:25
opendevreviewJulia Kreger proposed openstack/sushy master: Fix session authentication issues  https://review.opendev.org/c/openstack/sushy/+/82132419:29
stevebaker[m]good morning19:48
stevebaker[m]I see that dib change created some ... issues19:49
arne_wiebalckstevebaker[m]: :)20:00
arne_wiebalckstevebaker[m]: it was pulled in while I was testing sth else ... quite confusing initially :-D20:00
stevebaker[m]I don't think PARTTYPENAME needs to be read at all, all images with EFI partitions appear to have the correct GUID, and the --output-all was just for debug logging20:02
arne_wiebalckstevebaker[m]: in fact, I am not even sure the issue I saw is due to the lsblk version, i.e. if that is fatal20:02
arne_wiebalckstevebaker[m]: right20:02
arne_wiebalckstevebaker[m]: totally possible it is sth else in my image20:03
stevebaker[m]I don't think the image version is the issue, its the host. I'll make sure it works on CS8 and CS720:03
stevebaker[m]actually maybe not CS7 as the host, thats real old20:03
arne_wiebalckthe host ?20:04
TheJuliastevebaker[m]: yeah :(20:07
TheJuliaI wouldn't worry about Centos7 personally :)20:07
stevebaker[m]actually that change broke tripleo centos-7 :-0 https://bugs.launchpad.net/tripleo/+bug/196216820:08
TheJuliawhy is tripleo still building that?!20:22
TheJuliawell, RDO20:22
arne_wiebalck do they have a stream 8 one as well? this should break as well20:22
stevebaker[m]arne_wiebalck: I'm not sure PARTTYPENAME is your issue. FSTYPE="" so it gets ignored. I need to replicate that. I probably need to put some logic back which says if there is only one device, then assume it is the root partition20:58
stevebaker[m](and add back partition image support, for the hell of it)21:01
opendevreviewSteve Baker proposed openstack/ironic-python-agent-builder master: Make check-image-dib-centos9 voting  https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/83090922:31

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