opendevreview | Merged openstack/ironic-lib stable/wallaby: Add platform:base-py[2|3] to bindep https://review.opendev.org/c/openstack/ironic-lib/+/830203 | 02:14 |
---|---|---|
arne_wiebalck | Good morning, Ironic! | 07:28 |
janders | morning arne_wiebalck (not sure if I can call it good given what's going on) | 07:35 |
arne_wiebalck | janders: yeah (good morning to you none the less :)) | 07:36 |
rpittau | good morning ironic! o/ | 08:30 |
arne_wiebalck | hey rpittau o/ | 08:58 |
rpittau | hey arne_wiebalck :) | 08:58 |
rpittau | interesting.... 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.txt | 09:03 |
opendevreview | Verification of a change to openstack/tenks master failed: CI: switch Ubuntu jobs to Focal 20.04 https://review.opendev.org/c/openstack/tenks/+/828509 | 09:31 |
dtantsur | terrible morning, ironic | 09:32 |
janders | dtantsur terrible morning :( | 09:33 |
opendevreview | Merged openstack/tenks master: CI: switch Ubuntu jobs to Focal 20.04 https://review.opendev.org/c/openstack/tenks/+/828509 | 10:05 |
opendevreview | Merged openstack/ironic master: Deprecate instance network boot https://review.opendev.org/c/openstack/ironic/+/828746 | 12:02 |
opendevreview | Verification of a change to openstack/ironic-inspector master failed: Support listening on a Unix socket https://review.opendev.org/c/openstack/ironic-inspector/+/827124 | 12:02 |
*** sshnaidm is now known as sshnaidm|off | 12:23 | |
TheJulia | dtantsur: what can we use to denote the start of a day when it is such a morning? | 12:40 |
TheJulia | if you say go back to bed, I'm all for this, after my first meeting today | 12:40 |
dtantsur | TheJulia: I'm basically out of words today, but going back to bed may be a wise choice | 12:43 |
* TheJulia offers a hug | 12:43 | |
TheJulia | That is how I was feeling last night | 12:44 |
dtantsur | a lot of hugs will be needed by many people the coming days | 12:44 |
dtantsur | I really hope pas-ha[m] and other folks are safe | 12:44 |
TheJulia | ditto | 12:45 |
TheJulia | I was thinking about them last night quite a bit | 12:45 |
janders | in general the term "morning" is popularly associated with light, but today consists of darkness, mostly | 12:48 |
dtantsur | yeah | 12:48 |
TheJulia | agreed | 12:48 |
iurygregory | Hello Ironic o/ | 12:49 |
janders | hey iurygregory o/ | 12:49 |
dtantsur | welcome to the new world iurygregory | 12:49 |
janders | I think we agree we liked the old one more | 12:49 |
iurygregory | dtantsur, yeah =( | 12:50 |
*** pmannidi is now known as pmannidi|Away | 12:50 | |
janders | git reset --hard for reality would come in handy | 12:50 |
dtantsur | remember 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 |
iurygregory | yeah | 12:50 |
iurygregory | CERN midcycle <3 | 12:51 |
TheJulia | dtantsur: I had a photo from my stop over in Paris on the way to Geneva come up on facebook yesterday | 12:51 |
TheJulia | it made me super sad :( | 12:51 |
iurygregory | awww =( | 12:51 |
janders | I wasn't there, but have similar feelings about Kubecon in San Diego. Weird stuff started happening with time-space after that... | 12:52 |
TheJulia | Kubecon San Diego was just... weird | 12:53 |
TheJulia | well, that entire week was weird | 12:53 |
TheJulia | but I had amazing fish tacos | 12:53 |
* TheJulia needs to go obtain more fish tacos | 12:53 | |
iurygregory | fish tacos? wow | 12:54 |
iurygregory | TheJulia, dtantsur I've updated the cycle-highlights for Yoga o/ | 12:54 |
arne_wiebalck | stevebaker[m]: it seems your latest commit to diskimage-builder breaks our IPA image building | 12:55 |
TheJulia | arne_wiebalck: ugh | 12:55 |
dtantsur | iurygregory: thanks! I'll get to it when I can. | 12:56 |
iurygregory | ack o/ | 12:56 |
arne_wiebalck | stevebaker[m]: I tried without https://github.com/openstack/diskimage-builder/commit/6a3b8d94821fe9f2398124415dcb1220920d4101 and it works again. | 12:56 |
iurygregory | I will be lurking around irc (since I'm on PTO today XD) | 12:56 |
TheJulia | arne_wiebalck: are you handing it a partition image?! | 12:56 |
arne_wiebalck | TheJulia: CentOS-Stream-GenericCloud-8-20220125.1.x86_64.qcow2 | 12:57 |
TheJulia | arne_wiebalck: do you hapen to get a link? | 12:58 |
arne_wiebalck | TheJulia: let me see ... | 12:58 |
arne_wiebalck | TheJulia: https://cloud.centos.org/centos/8-stream/x86_64/images/CentOS-Stream-GenericCloud-8-20220125.1.x86_64.qcow2 | 12:59 |
TheJulia | much appreciated, downloading | 12:59 |
arne_wiebalck | TheJulia: I am not specifying this image directly, I think dib uses this from the parameters we pass. | 13:01 |
arne_wiebalck | TheJulia: like DIB_RELEASE | 13:01 |
TheJulia | so my understanding was this was only used on whole disk images, I guess I'll know for sure in a few minutes | 13:01 |
arne_wiebalck | TheJulia: thanks! | 13:03 |
TheJulia | so... the image *is* a whole disk image with a partition | 13:09 |
TheJulia | arne_wiebalck: got any build logs or the command your using for latest dib? | 13:09 |
arne_wiebalck | TheJulia: yep, let me give you the relevant bit ... | 13:10 |
TheJulia | thanks | 13:10 |
arne_wiebalck | TheJulia: 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_wiebalck | TheJulia: line 71 is also interesting | 13:12 |
TheJulia | what version of lsblk do you have? | 13:12 |
TheJulia | and I guess that leads to what is this being run on os wise? | 13:12 |
janders | see you on Monday Ironic o/ I hope we can all get some rest | 13:12 |
TheJulia | goodnight janders | 13:12 |
arne_wiebalck | TheJulia: this is the pipeline runner, let me check | 13:13 |
arne_wiebalck | TheJulia: but none of the centos OSs I have checked know that option | 13:13 |
* TheJulia wonders how CI passed | 13:14 | |
arne_wiebalck | TheJulia: should be cs8 as well | 13:15 |
arne_wiebalck | TheJulia: the OS of the gitlab runner | 13:15 |
TheJulia | interesting, I don't have my p1 loopback partition locally | 13:22 |
arne_wiebalck | TheJulia: you think it is an issue with the container we use for the runner? | 13:26 |
TheJulia | maybe... | 13:28 |
TheJulia | dunno yet since github is not loading for me at the moment | 13:28 |
TheJulia | trying to find where this option got added | 13:28 |
rpittau | arne_wiebalck: centos stream 8 ? | 13:28 |
arne_wiebalck | rpittau: yes | 13:29 |
arne_wiebalck | TheJulia: rpittau: I will launch a container to look inside .. | 13:29 |
* TheJulia steps away to do the "look more presentable for a video call" dance | 13:29 | |
rpittau | arne_wiebalck: I don't have that up now, check the version of lsblk as suggested by TheJulia | 13:30 |
arne_wiebalck | rpittau: `lsblk from util-linux 2.32.1` | 13:30 |
arne_wiebalck | TheJulia: yes, the container has a number of loop back devices ... no idea, why | 13:31 |
opendevreview | Merged openstack/ironic master: Support listening on a Unix socket https://review.opendev.org/c/openstack/ironic/+/828508 | 13:31 |
opendevreview | Merged 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/+/829721 | 13:31 |
opendevreview | Merged 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/+/830356 | 13:31 |
rpittau | arne_wiebalck: PARTTYPENAME is present since 2.35 | 13:33 |
arne_wiebalck | TheJulia: wait, these are the device of the host :) | 13:33 |
arne_wiebalck | rpittau: aha | 13:33 |
arne_wiebalck | rpittau: not sure this is the problem, but for sure this does not help | 13:34 |
arne_wiebalck | rpittau: what is the version in stream 8 ? | 13:34 |
rpittau | 2.32.1 | 13:35 |
TheJulia | I wonder if stream9 was the base os for the build | 13:36 |
rpittau | that could explain the PARTTYPE | 13:36 |
rpittau | PARTYPENAME sorry | 13:37 |
rpittau | CS9 has 2.37.2 | 13:37 |
rpittau | yeah stevebaker[m] change is definitely not compatible with CS8 | 13:37 |
arne_wiebalck | yes, cs9 has 2.37.2 | 13:38 |
rpittau | yep :) | 13:39 |
arne_wiebalck | we clone dib for every IPA build from master | 13:39 |
TheJulia | okay, so... if someone wants to click the revert button | 13:39 |
TheJulia | I'll approve it | 13:39 |
rpittau | TheJulia: https://review.opendev.org/c/openstack/diskimage-builder/+/830717 | 13:43 |
TheJulia | much appreciated | 13:43 |
TheJulia | approved and workflowed | 13:43 |
arne_wiebalck | thanks rpittau and TheJulia ! | 13:43 |
rpittau | np :) | 13:44 |
opendevreview | Verification 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/+/829722 | 13:44 |
opendevreview | OpenStack Release Bot proposed openstack/ironic bugfix/20.0: Update .gitreview for bugfix/20.0 https://review.opendev.org/c/openstack/ironic/+/830837 | 15:08 |
dtantsur | going to an anti-war demo, see you tomorrow | 15:39 |
dtantsur | o/ | 15:39 |
TheJulia | o/ | 15:40 |
MahnoorAsghar | o/ | 15:43 |
TheJulia | being an adult is no fun | 16:00 |
opendevreview | Verification of a change to openstack/ironic-inspector master failed: Support listening on a Unix socket https://review.opendev.org/c/openstack/ironic-inspector/+/827124 | 16:20 |
TheJulia | hmm where did Iury disappear off to... | 16:37 |
arne_wiebalck | he is on pto today I think ... no? | 16:40 |
arne_wiebalck | (or was this yesterday?) | 16:41 |
* TheJulia shrugs | 16:41 | |
* arne_wiebalck maybe needs some pto, too | 16:41 | |
arne_wiebalck | it was today | 16:42 |
TheJulia | I definitely do | 16:45 |
TheJulia | I'll likley be away tomorrow. I've got one two hour meeting and then breakfast with bottomless memosas is tempting | 16:45 |
TheJulia | arne_wiebalck: w/r/t oslo policy see: https://bugs.launchpad.net/oslo.policy/+bug/1742569 | 17:03 |
TheJulia | tl;dr horizon | 17:03 |
rpittau | TheJulia: Iury's out today and tomorrow | 17:19 |
TheJulia | okay | 17:19 |
rpittau | see ya tomorrow o/ | 17:20 |
opendevreview | Verification 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/+/829722 | 17:34 |
arne_wiebalck | TheJulia: oh, ok | 17:44 |
* arne_wiebalck is trying to wrap his head around this | 17:46 | |
arne_wiebalck | TheJulia: there is still the issue of multiple aliases | 17:47 |
arne_wiebalck | TheJulia: which is the point where this breaks | 17:47 |
arne_wiebalck | TheJulia: or is this an Ironic special case? | 17:47 |
arne_wiebalck | bye everyone o/ | 18:03 |
opendevreview | Verification of a change to openstack/ironic-inspector master failed: Support listening on a Unix socket https://review.opendev.org/c/openstack/ironic-inspector/+/827124 | 19:25 |
opendevreview | Julia Kreger proposed openstack/sushy master: Fix session authentication issues https://review.opendev.org/c/openstack/sushy/+/821324 | 19:29 |
stevebaker[m] | good morning | 19:48 |
stevebaker[m] | I see that dib change created some ... issues | 19:49 |
arne_wiebalck | stevebaker[m]: :) | 20:00 |
arne_wiebalck | stevebaker[m]: it was pulled in while I was testing sth else ... quite confusing initially :-D | 20: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 logging | 20:02 |
arne_wiebalck | stevebaker[m]: in fact, I am not even sure the issue I saw is due to the lsblk version, i.e. if that is fatal | 20:02 |
arne_wiebalck | stevebaker[m]: right | 20:02 |
arne_wiebalck | stevebaker[m]: totally possible it is sth else in my image | 20: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 CS7 | 20:03 |
stevebaker[m] | actually maybe not CS7 as the host, thats real old | 20:03 |
arne_wiebalck | the host ? | 20:04 |
TheJulia | stevebaker[m]: yeah :( | 20:07 |
TheJulia | I wouldn't worry about Centos7 personally :) | 20:07 |
stevebaker[m] | actually that change broke tripleo centos-7 :-0 https://bugs.launchpad.net/tripleo/+bug/1962168 | 20:08 |
TheJulia | why is tripleo still building that?! | 20:22 |
TheJulia | well, RDO | 20:22 |
arne_wiebalck | do they have a stream 8 one as well? this should break as well | 20: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 partition | 20:58 |
stevebaker[m] | (and add back partition image support, for the hell of it) | 21:01 |
opendevreview | Steve Baker proposed openstack/ironic-python-agent-builder master: Make check-image-dib-centos9 voting https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/830909 | 22:31 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!