opendevreview | Steve Baker proposed openstack/ironic master: WIP use privsep to copy for loader_file_paths https://review.opendev.org/c/openstack/ironic/+/821363 | 00:11 |
---|---|---|
*** pmannidi is now known as pmannidi|meet | 02:50 | |
*** pmannidi|meet is now known as pmannidi|DM | 02:51 | |
opendevreview | Steve Baker proposed openstack/ironic master: WIP use privsep to copy for loader_file_paths https://review.opendev.org/c/openstack/ironic/+/821363 | 03:05 |
*** pmannidi|DM is now known as pmannidi | 06:12 | |
jingvar | \o | 07:49 |
arne_wiebalck | Good morning jingvar and Ironic! | 08:04 |
rpittau | good morning ironic! o/ | 08:35 |
arne_wiebalck | hey rpittau o/ | 08:50 |
rpittau | hey arne_wiebalck :) | 08:50 |
opendevreview | Riccardo Pittau proposed openstack/ironic master: Fix Mock objects as specs https://review.opendev.org/c/openstack/ironic/+/821370 | 08:55 |
dtantsur | morning ironic | 09:06 |
janders | good morning jingvar arne_wiebalck rpittau dtantsur and Ironic o/ | 09:08 |
arne_wiebalck | hey janders o/ | 09:11 |
rpittau | hey janders :) | 09:12 |
opendevreview | Dmitry Tantsur proposed openstack/ironic master: Adoption: do not validate boot interface when local booting https://review.opendev.org/c/openstack/ironic/+/821537 | 09:27 |
rpittau | dtantsur: I'm not sure I understand your comment on https://review.opendev.org/c/openstack/ironic/+/821370, with spec=object we should disallow the double-mock behavior | 09:57 |
dtantsur | rpittau: spec=object makes no sense | 10:13 |
dtantsur | the error Corey pasted says it all: Cannot autospec attr 'call' from target <ironic.common.glance_service.image_service.GlanceImageService object at 0x7f3899ba0b50> as it has already been mocked out. | 10:14 |
dtantsur | we need to remove the double mocking, not please the auto-speccer | 10:14 |
dtantsur | I guess (from the message), it's because tenacity is mocked somewhere? | 10:16 |
dtantsur | in the nova case it's even simpler: we're literally mocking a mock | 10:18 |
dtantsur | the drivers are a bit more interesting, we may need a wrapper that checks if the mocked thing is already a mock | 10:20 |
rpittau | alright, I'll have a deeper look, thanks for the hints | 10:38 |
opendevreview | Dmitry Tantsur proposed openstack/ironic master: Document single process Ironic https://review.opendev.org/c/openstack/ironic/+/820396 | 11:52 |
janders | see you tomorrow Ironic o/ | 11:57 |
dtantsur | TheJulia: somehow, we still don't fail properly if the image does not have an EFI partition.. | 13:12 |
jingvar | I failed with missed linux-firmware in cloud images | 13:21 |
arne_wiebalck | jingvar: image too big with linux-firmware ? | 13:43 |
TheJulia | good morning | 13:57 |
TheJulia | dtantsur: we have code to extract it from the folder from partition images, but it sounds like the image in that case is entirely invalid if it is a wholedisk image missing an efi partition | 13:58 |
jingvar | arne_wiebalck: I'm not sure. But Rocky 8.5 cloudimage and probaly Ubuntu Focal - cut it | 14:06 |
opendevreview | Aija Jauntēva proposed openstack/ironic master: Fix redfish update_firmware for newer Sushy https://review.opendev.org/c/openstack/ironic/+/821576 | 14:21 |
opendevreview | Riccardo Pittau proposed openstack/ironic master: [WIP] Fix Mock objects as specs https://review.opendev.org/c/openstack/ironic/+/821370 | 15:39 |
TheJulia | arne_wiebalck: hey, back in time, with software raid, What was the actual layout you guys were using for the images | 15:55 |
TheJulia | arne_wiebalck: I remember it had to be very particular to work | 15:55 |
rpittau | #startmeeting ironic | 16:00 |
opendevmeet | Meeting started Mon Dec 13 16:00:17 2021 UTC and is due to finish in 60 minutes. The chair is rpittau. Information about MeetBot at http://wiki.debian.org/MeetBot. | 16:00 |
opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 16:00 |
opendevmeet | The meeting name has been set to 'ironic' | 16:00 |
rloo | o/ | 16:00 |
arne_wiebalck | o/ | 16:00 |
rpioso | o/ | 16:00 |
ajya | o/ | 16:00 |
* dtantsur is lurking | 16:00 | |
rpittau | Hello everyone and welcome to the weekly Ironic meeting! | 16:00 |
rpittau | Our agenda is on the wiki | 16:01 |
rpittau | #link https://wiki.openstack.org/wiki/Meetings/Ironic#Agenda_for_next_meeting | 16:01 |
rpittau | #topic Announcements / Reminders | 16:01 |
rpittau | #info bugfix branches created for ironic, ironic-inspector, ironic-python-agent, and bifrost | 16:02 |
rpittau | it seems we've released all the bugfix branches last week, great :) | 16:02 |
rpittau | #info this is the last meeting of 2021, see you on Monday January 10th 2022 ! | 16:03 |
rloo | thx for doing that everyone! | 16:03 |
rpittau | I'll be here until the 24th but I think lot of people will be out until next year first or second week | 16:04 |
TheJulia | Seems like I'm going to be out after the 21st or so | 16:05 |
rpittau | anyone has anything else to remind/announce ? | 16:06 |
rpittau | alright, moving on | 16:07 |
rpittau | #topic Review Action items from prior meeting | 16:07 |
rpittau | no action items from last time :) | 16:07 |
rpittau | #topic Review subteam status reports | 16:07 |
rpittau | #link https://etherpad.openstack.org/p/IronicWhiteBoard | 16:08 |
rpittau | around L63 | 16:08 |
rpittau | dtantsur: what's missing for the all-in-one ironic process ? | 16:13 |
dtantsur | docs at least | 16:13 |
dtantsur | https://review.opendev.org/c/openstack/ironic/+/820396 | 16:14 |
dtantsur | and bifrost changes | 16:14 |
rpittau | ok | 16:14 |
dtantsur | I can also offer a metal3 patch for review :D | 16:16 |
rpittau | :) | 16:16 |
rpittau | are we good with the updates? | 16:17 |
rpittau | I'll take that as a yes! | 16:17 |
rpittau | #topic Priorities for the coming week | 16:17 |
rpittau | #link https://review.opendev.org/q/hashtag:ironic-week-prio+status:open | 16:18 |
rpittau | it seems we have some space for reviews this week | 16:19 |
dtantsur | I would like to add https://review.opendev.org/c/openstack/ironic/+/821537 https://review.opendev.org/c/openstack/ironic/+/820904 and https://review.opendev.org/c/openstack/ironic/+/821010/ | 16:19 |
dtantsur | all 3 are relatively small | 16:19 |
arne_wiebalck | dynamic network pairing for burn-in: https://review.opendev.org/c/openstack/ironic-python-agent/+/821244 | 16:19 |
rpittau | dtantsur: added | 16:20 |
arne_wiebalck | not super small, but maybe interesting to see if you can find holes :) | 16:20 |
rpittau | arne_wiebalck: added that too | 16:20 |
arne_wiebalck | rpittau: thanks | 16:20 |
rpittau | looks good, does anyone have anything else to add ? | 16:22 |
rpittau | alright, moving on | 16:22 |
rpittau | there's nothing up for discussion this week, so jumping to.... | 16:22 |
rpittau | #topic Baremetal SIG | 16:23 |
rpittau | #info Tomorrow Tuesday Dec 14, 2021 at 2 PM UTC -> Dmitry Tantsur on "Overview of Metal3 (with demo!)" | 16:23 |
dtantsur | :) | 16:23 |
rpittau | can't wait! :) | 16:23 |
arne_wiebalck | for the demo? | 16:23 |
arne_wiebalck | ;) | 16:23 |
rpittau | yeah, very curious to see this metal3 thing in action :D | 16:23 |
arne_wiebalck | heh | 16:23 |
dtantsur | it won't be too long, I must warn - my environment keeps misbehaving | 16:23 |
dtantsur | lol | 16:23 |
arne_wiebalck | dtantsur: live demo? | 16:24 |
dtantsur | I'm too lazy to record anything, so you'll have to bear with me :) | 16:24 |
arne_wiebalck | ha, with pleasure, I am sure it will be great | 16:24 |
dtantsur | on the other hand, we can poke around together :) | 16:24 |
arne_wiebalck | nice! | 16:24 |
arne_wiebalck | I don't think there is anything else for the SIG | 16:25 |
arne_wiebalck | we may skip the jan meeting | 16:25 |
arne_wiebalck | not sure I find someone to present | 16:25 |
rpittau | sure | 16:25 |
arne_wiebalck | unless there are volunteers ? | 16:25 |
TheJulia | my plate is overflowing at present :( | 16:25 |
arne_wiebalck | sure, was more a joke, everyone should get some rest and I'll find someone for feb | 16:26 |
rpittau | sounds good | 16:27 |
rpittau | let's move on ? | 16:27 |
arne_wiebalck | yes | 16:27 |
rpittau | ok! | 16:27 |
rpittau | we don't have RFEs to review, so jumping to | 16:27 |
rpittau | #topic Open Discussion | 16:27 |
rpittau | anything unplanned to discuss ? | 16:27 |
dtantsur | rloo: re "are folks really using adoption" | 16:28 |
dtantsur | Yeah, we're using it very actively in Metal3. Not only to adopt externally provisioned nodes, but also to migrate nodes between Ironic's (e.g. when pivoting from the bootstrap cluster to the final one) and to recover the database if the Ironic pod gets deleted. | 16:28 |
dtantsur | So, unlike in most cases, adoption is a routine process for us | 16:28 |
rloo | dtantsur: sweet, that's great! | 16:28 |
* dtantsur makes a note to talk about it tomorrow | 16:29 | |
arne_wiebalck | not sure we have this, but maybe worth documenting how this works with nova | 16:29 |
arne_wiebalck | adoption, I mean | 16:29 |
dtantsur | like "OMG terribly"? :D | 16:30 |
rloo | [it doesn't play nice with nova right now] | 16:30 |
arne_wiebalck | I have adopted a couple of thousand nodes into Nova and Ironic :) | 16:30 |
rloo | we have too, at yahoo, but not via the adoption mechanism. | 16:31 |
dtantsur | have a blog post? | 16:31 |
arne_wiebalck | https://techblog.web.cern.ch/techblog/post/ironic-nova-adoption/ | 16:31 |
dtantsur | yeah, I've heard about bright people doing 'openstack stack delete' on their tripleo, and then dancing around the fake drivers | 16:31 |
dtantsur | to get their cloud back | 16:31 |
arne_wiebalck | rloo: true, same here | 16:32 |
arne_wiebalck | rloo: my point was to make this clear in our docs ... unless we have already | 16:33 |
arne_wiebalck | rloo: so users do not get too excited :) | 16:33 |
dtantsur | https://docs.openstack.org/ironic/latest/admin/adoption.html | 16:33 |
dtantsur | nothing much abut nova | 16:33 |
arne_wiebalck | dtantsur: right | 16:33 |
rloo | hmm. maybe we should add something at the top, wrt do-not-use-with-nova... | 16:34 |
arne_wiebalck | dtantsur: I can suggest sth | 16:34 |
dtantsur | ++ | 16:34 |
rloo | thx arne_wiebalck! :) | 16:34 |
rpittau | thanks arne_wiebalck :) | 16:35 |
rpittau | any other topic to discuss today ? | 16:35 |
rpittau | seems we're good to proceed to the last topic | 16:36 |
rpittau | #topic Who is going to run next year's first meeting? | 16:36 |
dtantsur | I think we should appoint the PTL if he's back | 16:37 |
rpittau | sounds good :D | 16:37 |
rpittau | coll, thanks everyone, have a great week! | 16:38 |
rpittau | #endmeeting | 16:38 |
opendevmeet | Meeting ended Mon Dec 13 16:38:23 2021 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:38 |
opendevmeet | Minutes: https://meetings.opendev.org/meetings/ironic/2021/ironic.2021-12-13-16.00.html | 16:38 |
opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/ironic/2021/ironic.2021-12-13-16.00.txt | 16:38 |
opendevmeet | Log: https://meetings.opendev.org/meetings/ironic/2021/ironic.2021-12-13-16.00.log.html | 16:38 |
arne_wiebalck | thanks rpittau | 16:39 |
*** amoralej is now known as amoralej|off | 16:44 | |
rpittau | good night! o/ | 17:16 |
opendevreview | Arne Wiebalck proposed openstack/ironic master: [doc] Adoption with Nova https://review.opendev.org/c/openstack/ironic/+/821611 | 18:28 |
arne_wiebalck | rloo: dtantsur ^^ | 18:28 |
rloo | arne_wiebalck: +2, thx! | 18:36 |
arne_wiebalck | rloo: thanks | 18:40 |
arne_wiebalck | bye everyone, see you tomorrow o/ | 18:40 |
rloo | bye arne_wiebalck! :) | 18:41 |
*** sshnaidm is now known as sshnaidm|afk | 19:06 | |
dtantsur | bye folks | 19:21 |
* TheJulia feels like a nap is really required | 19:33 | |
janders | good morning Ironic o/ | 21:11 |
TheJulia | o/ janders | 22:26 |
janders | hey TheJulia :) | 22:45 |
janders | how is it going? | 22:45 |
TheJulia | it goes, listening to training I missed earlier today | 23:13 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!