opendevreview | Verification of a change to openstack/ironic master failed: add snmp power action delay https://review.opendev.org/c/openstack/ironic/+/809902 | 00:07 |
---|---|---|
NobodyCam | if someone wanted to add a new mfg to the snmp driver. would that require a spec | 00:08 |
NobodyCam | we have a driver for `Server Technology` power PDU devices. happy to provide a patch for snmp. | 00:13 |
opendevreview | Verification of a change to openstack/ironic master failed: Follow up to Add support for verify steps https://review.opendev.org/c/openstack/ironic/+/812792 | 00:41 |
janders | ^ looks like CI is having a hard time | 00:43 |
opendevreview | Verification of a change to openstack/ironic master failed: Follow up to Add support for verify steps https://review.opendev.org/c/openstack/ironic/+/812792 | 01:03 |
opendevreview | Verification of a change to openstack/ironic master failed: add snmp power action delay https://review.opendev.org/c/openstack/ironic/+/809902 | 03:21 |
opendevreview | Merged openstack/ironic master: Follow up to Add support for verify steps https://review.opendev.org/c/openstack/ironic/+/812792 | 03:57 |
*** pmannidi is now known as pmannidi|AFK | 04:25 | |
*** pmannidi|AFK is now known as pmannidi | 05:39 | |
arne_wiebalck | Good morning, Ironic! | 06:04 |
janders | hey arne_wiebalck | 06:06 |
janders | Happy Friday | 06:06 |
arne_wiebalck | hey janders, good morning o/ | 06:07 |
opendevreview | Arne Wiebalck proposed openstack/ironic-python-agent master: Assert EFI part UUID is not None before editing fstab https://review.opendev.org/c/openstack/ironic-python-agent/+/813125 | 06:37 |
iurygregory | good morning arne_wiebalck janders and Ironic! Happy Friday! | 06:41 |
arne_wiebalck | hey iurygregory o/ | 06:42 |
janders | good morning iurygregory o/ | 06:58 |
rpittau | good morning ironic! o/ Happy Friday! | 07:16 |
iurygregory | morning rpittau o/ | 07:16 |
rpittau | hey iurygregory :) | 07:16 |
janders | good morning rpittau | 07:20 |
janders | Happy Friday | 07:20 |
rpittau | hey janders :) | 07:20 |
janders | (and thanks for the /approve :) | 07:20 |
rpittau | np! | 07:20 |
janders | quite a successful week work wise | 07:20 |
janders | thanks guys! | 07:20 |
rpittau | glad to hear that :) | 07:20 |
janders | if you have time, here's one more easy review for a Friday: https://review.opendev.org/c/openstack/ironic/+/812795 | 07:21 |
janders | (and CI finally passed after way-too-many rechecks) | 07:21 |
janders | hmm just what happened to my release note :D | 07:22 |
rpittau | CI quite unstable since a couple days | 07:22 |
janders | looks like a forgotten git add :/ | 07:23 |
janders | I will add the note together with any other fixing up that may be required to limit CI runs needed | 07:24 |
rpittau | it looks good to me, except for the release note missing | 07:25 |
opendevreview | Jacob Anders proposed openstack/ironic master: Do not append filename parameter to image URL when using local file https://review.opendev.org/c/openstack/ironic/+/812795 | 07:25 |
rpittau | janders: double added on the 4th line | 07:26 |
janders | rpittau good pickup | 07:27 |
opendevreview | Jacob Anders proposed openstack/ironic master: Do not append filename parameter to image URL when using local file https://review.opendev.org/c/openstack/ironic/+/812795 | 07:28 |
janders | thanks Riccardo | 07:35 |
rpittau | mmm standalone job is failing quite often recently | 07:39 |
janders | yes | 07:39 |
janders | looks like some storage related confusion | 07:40 |
rpittau | yeah | 07:40 |
janders | (I saw multiple mentions of a storage device missing) | 07:40 |
arne_wiebalck | rpittau is faster than arne_wiebalck's shadow this time :) | 08:04 |
rpittau | :) | 08:05 |
opendevreview | Verification of a change to openstack/ironic master failed: CI: Change CI ipxe file to snponly https://review.opendev.org/c/openstack/ironic/+/812167 | 08:15 |
opendevreview | Verification of a change to openstack/ironic master failed: Demote three warning messages https://review.opendev.org/c/openstack/ironic/+/812643 | 08:34 |
arne_wiebalck | I am seeing the EFI boot order changed after instantiation, in particular the EFI shell moving up before the disks. From what I see, this does not happen during the deployment (I logged the order during deploy until the node went down and Ironic should not touch the order anyway since this is s/w RAID and we do not call efibootmgr in this case), but during the first boot (?). This is on CentOS7. Anyone seen this or | 09:37 |
arne_wiebalck | an idea what this could be? | 09:37 |
arne_wiebalck | It looks like *something* is resetting the order to some default on the first boot. Changing the order back and rebooting is fine. | 09:37 |
iurygregory | we saw some weird things in some HW where after reboot they changed the boot order .-. | 09:39 |
arne_wiebalck | I see this on different hardware | 09:41 |
rpittau | from the ironic perspective I don't see what could have caused that | 09:50 |
arne_wiebalck | yeah ... I am also somewhat puzzled. Not that obvious to me how to observe when the change is happening (or by what). | 10:08 |
arne_wiebalck | anyway, I'll keep digging :) | 10:08 |
iurygregory | ironic-cores this is my idea for the schedule for the PTG https://paste.opendev.org/show/809872/ , we have 11 slots atm and 16 topics | 10:39 |
iurygregory | let me know what you think | 10:39 |
iurygregory | also non-cores are very welcome to provide feedback =) | 10:41 |
opendevreview | Merged openstack/ironic master: CI: Change CI ipxe file to snponly https://review.opendev.org/c/openstack/ironic/+/812167 | 11:10 |
opendevreview | Merged openstack/ironic master: Demote three warning messages https://review.opendev.org/c/openstack/ironic/+/812643 | 11:10 |
rpittau | at last we have support for python 3.10 in eventlet \o/ https://github.com/eventlet/eventlet/pull/715 | 12:15 |
*** sshnaidm is now known as sshnaidm|afk | 12:17 | |
rpittau | iurygregory: schedule looks ok to me, I was expecting the times to be earlier though | 12:17 |
iurygregory | rpittau, in any specific day or in all of them? .-. ( I think we are using the same UTC times from the Xena PTG basically) | 12:30 |
* iurygregory brb need to solve a few things | 12:53 | |
rpittau | iurygregory: all days, but that's ok, not a complain, just an observation :) | 12:54 |
rpittau | bye everyone, have a great weekend! o/ | 13:29 |
TheJulia | iurygregory: I'd plan on 4 topics a day, pair 2 less contentious with 2 more coneentious or uknown topics. Like Monday, I suspect the first two topics will be relatively quick. The last one may take a while, but souldn't consume the entire window imho. | 14:07 |
iurygregory | TheJulia, sure let me update | 14:14 |
iurygregory | TheJulia,https://paste.opendev.org/show/809885/ wdyt? I didn't plan 4 topics in each day (because it would fit in 4 days and we have 5) my idea would be start the first top of the next day in case we have free time, wdyt? | 15:05 |
TheJulia | 3rd/3rd party/ :) | 15:13 |
iurygregory | yeah I can update before sending the email =) | 15:13 |
iurygregory | and update in the etherpad | 15:13 |
TheJulia | wait, room kilo, did someone change our entries?!? | 15:14 |
iurygregory | TheJulia, thursday/friday I see Kilo in the ethercalc | 15:14 |
TheJulia | oh, someone was evil then | 15:14 |
iurygregory | https://ethercalc.openstack.org/8tum5yl1bx43 | 15:14 |
iurygregory | I see TC in the Juno room =P | 15:15 |
TheJulia | iurygregory: w/r/t wednesday, I'd honestly kind of like to also discuss options of passing through api opts to the conductor for smaller scale usages, but that may just fall out of the discussion of bulk operations as well | 15:15 |
iurygregory | TheJulia, makes sense to me, I will also add a section in the etherpad for "new topics" from the last hour and we can try to book more slots or fit in the ones we have | 15:17 |
TheJulia | ++ | 15:18 |
iurygregory | I will send and email with this information and the current schedule (we can review with the community on the upstream meeting on Monday - I will add a topic to the agenda) | 15:22 |
NobodyCam | Good Morning Ironic folks... and OFC.... | 15:28 |
NobodyCam | TGIF! | 15:28 |
TheJulia | good morning NobodyCam | 15:29 |
NobodyCam | Happy Friday TheJulia | 15:29 |
NobodyCam | I see the gate is finicky this morning | 15:30 |
TheJulia | I blame the rain | 15:31 |
TheJulia | I guess the rain is good, compact some of the loose dirt before the rig gets washed tomorrow | 15:31 |
* iurygregory facepalm, tks god I double checked the etherpad again - I missed two topics to add in the PTG schedule :D | 15:54 | |
TheJulia | iurygregory: these things happen | 15:56 |
TheJulia | generally, I take the entire text and chop it down | 15:56 |
TheJulia | well, a copy of | 15:56 |
iurygregory | yeah =) | 15:57 |
iurygregory | ok, it's 6pm friday \o/ time to go out for a beer :D | 16:00 |
iurygregory | and dinner... | 16:00 |
iurygregory | have a great weekend everyone o/ | 16:00 |
TheJulia | goodnight iurygregory | 16:02 |
opendevreview | Verification of a change to openstack/ironic master failed: add snmp power action delay https://review.opendev.org/c/openstack/ironic/+/809902 | 16:05 |
opendevreview | Verification of a change to openstack/ironic master failed: add snmp power action delay https://review.opendev.org/c/openstack/ironic/+/809902 | 16:38 |
opendevreview | Julia Kreger proposed openstack/ironic master: Retool devstack plugin to use pxe loaders configuration https://review.opendev.org/c/openstack/ironic/+/812188 | 17:25 |
opendevreview | Julia Kreger proposed openstack/ironic master: SRBAC - Prepare for additional services https://review.opendev.org/c/openstack/ironic/+/812411 | 17:26 |
TheJulia | NobodyCam: well, looks like your change is going to pass through the check queue, now the gate queue | 17:27 |
NobodyCam | :dance | 17:29 |
NobodyCam | woo hoo | 17:29 |
TheJulia | Any cores around to review https://review.opendev.org/c/openstack/ironic-inspector/+/812137 ? | 17:29 |
opendevreview | Verification of a change to openstack/ironic master failed: add snmp power action delay https://review.opendev.org/c/openstack/ironic/+/809902 | 17:40 |
opendevreview | Julia Kreger proposed openstack/ironic master: Remove debian packages file for devstack https://review.opendev.org/c/openstack/ironic/+/810314 | 17:40 |
opendevreview | Julia Kreger proposed openstack/metalsmith master: DNM: Test CI https://review.opendev.org/c/openstack/metalsmith/+/781549 | 17:58 |
TheJulia | hmm, worried uefi change broke metalsmith | 17:58 |
TheJulia | will know a little later | 17:58 |
opendevreview | Samuel Zuk proposed openstack/ironic-specs master: Power control intermediary spec proposal https://review.opendev.org/c/openstack/ironic-specs/+/808804 | 18:45 |
opendevreview | Merged openstack/ironic master: add snmp power action delay https://review.opendev.org/c/openstack/ironic/+/809902 | 19:32 |
NobodyCam | :dance | 19:34 |
NobodyCam | only 5 rechecks | 19:34 |
opendevreview | Merged openstack/ironic master: Do not append filename parameter to image URL when using local file https://review.opendev.org/c/openstack/ironic/+/812795 | 19:36 |
opendevreview | Verification of a change to openstack/ironic master failed: Update the list of supported database filters https://review.opendev.org/c/openstack/ironic/+/812979 | 19:36 |
TheJulia | hmmm | 19:56 |
TheJulia | hmm, yeah. Two metalsmith jobs are broken by the uefi default change it looks like... | 20:03 |
TheJulia | which I should have expected | 20:03 |
opendevreview | Julia Kreger proposed openstack/metalsmith master: CI: Fix jobs for UEFI default change https://review.opendev.org/c/openstack/metalsmith/+/813248 | 20:10 |
opendevreview | Julia Kreger proposed openstack/metalsmith master: CI: remove old centos7 job https://review.opendev.org/c/openstack/metalsmith/+/813249 | 20:10 |
opendevreview | Julia Kreger proposed openstack/ironic master: Test nova-compute fix https://review.opendev.org/c/openstack/ironic/+/813264 | 22:25 |
TheJulia | NobodyCam: ^^ | 22:25 |
* NobodyCam click link | 22:26 | |
TheJulia | zuul will be the real test | 22:27 |
NobodyCam | passing or not. gets my +1 alrady | 22:29 |
NobodyCam | *ready | 22:29 |
NobodyCam | ** all ready | 22:29 |
TheJulia | heh | 22:31 |
TheJulia | There, no breakfast excuses :) | 22:32 |
TheJulia | I have no idea if that will work against your version, fwiw | 22:32 |
TheJulia | but master branch seems pretty cut and dry, there are no other places it is called | 22:32 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!