Monday, 2021-12-13

opendevreviewSteve Baker proposed openstack/ironic master: WIP use privsep to copy for loader_file_paths  https://review.opendev.org/c/openstack/ironic/+/82136300:11
*** pmannidi is now known as pmannidi|meet02:50
*** pmannidi|meet is now known as pmannidi|DM02:51
opendevreviewSteve Baker proposed openstack/ironic master: WIP use privsep to copy for loader_file_paths  https://review.opendev.org/c/openstack/ironic/+/82136303:05
*** pmannidi|DM is now known as pmannidi06:12
jingvar\o07:49
arne_wiebalckGood morning jingvar and Ironic!08:04
rpittaugood morning ironic! o/08:35
arne_wiebalckhey rpittau o/08:50
rpittauhey arne_wiebalck :)08:50
opendevreviewRiccardo Pittau proposed openstack/ironic master: Fix Mock objects as specs  https://review.opendev.org/c/openstack/ironic/+/82137008:55
dtantsurmorning ironic09:06
jandersgood morning jingvar arne_wiebalck rpittau dtantsur and Ironic o/09:08
arne_wiebalckhey janders o/09:11
rpittauhey janders :)09:12
opendevreviewDmitry Tantsur proposed openstack/ironic master: Adoption: do not validate boot interface when local booting  https://review.opendev.org/c/openstack/ironic/+/82153709:27
rpittaudtantsur: 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 behavior09:57
dtantsurrpittau: spec=object makes no sense10:13
dtantsurthe 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
dtantsurwe need to remove the double mocking, not please the auto-speccer10:14
dtantsurI guess (from the message), it's because tenacity is mocked somewhere?10:16
dtantsurin the nova case it's even simpler: we're literally mocking a mock10:18
dtantsurthe drivers are a bit more interesting, we may need a wrapper that checks if the mocked thing is already a mock10:20
rpittaualright, I'll have a deeper look, thanks for the hints10:38
opendevreviewDmitry Tantsur proposed openstack/ironic master: Document single process Ironic  https://review.opendev.org/c/openstack/ironic/+/82039611:52
janderssee you tomorrow Ironic o/11:57
dtantsurTheJulia: somehow, we still don't fail properly if the image does not have an EFI partition..13:12
jingvarI failed with missed linux-firmware in cloud images13:21
arne_wiebalckjingvar: image too big with linux-firmware ?13:43
TheJuliagood morning13:57
TheJuliadtantsur: 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 partition13:58
jingvararne_wiebalck: I'm not sure. But Rocky 8.5 cloudimage and probaly Ubuntu Focal - cut it14:06
opendevreviewAija Jauntēva proposed openstack/ironic master: Fix redfish update_firmware for newer Sushy  https://review.opendev.org/c/openstack/ironic/+/82157614:21
opendevreviewRiccardo Pittau proposed openstack/ironic master: [WIP] Fix Mock objects as specs  https://review.opendev.org/c/openstack/ironic/+/82137015:39
TheJuliaarne_wiebalck: hey, back in time, with software raid, What was the actual layout you guys were using for the images15:55
TheJuliaarne_wiebalck: I remember it had to be very particular to work15:55
rpittau#startmeeting ironic16:00
opendevmeetMeeting 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
opendevmeetUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.16:00
opendevmeetThe meeting name has been set to 'ironic'16:00
rlooo/16:00
arne_wiebalcko/16:00
rpiosoo/16:00
ajyao/16:00
* dtantsur is lurking16:00
rpittauHello everyone and welcome to the weekly Ironic meeting!16:00
rpittauOur agenda is on the wiki16:01
rpittau#link https://wiki.openstack.org/wiki/Meetings/Ironic#Agenda_for_next_meeting16:01
rpittau#topic Announcements / Reminders16:01
rpittau#info bugfix branches created for ironic, ironic-inspector, ironic-python-agent, and bifrost16:02
rpittauit 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
rloothx for doing that everyone!16:03
rpittauI'll be here until the 24th but I think lot of people will be out until next year first or second week16:04
TheJuliaSeems like I'm going to be out after the 21st or so16:05
rpittauanyone has anything else to remind/announce ?16:06
rpittaualright, moving on16:07
rpittau#topic Review Action items from prior meeting16:07
rpittauno action items from last time :)16:07
rpittau#topic Review subteam status reports16:07
rpittau#link https://etherpad.openstack.org/p/IronicWhiteBoard16:08
rpittauaround L6316:08
rpittaudtantsur: what's missing for the all-in-one ironic process ?16:13
dtantsurdocs at least16:13
dtantsurhttps://review.opendev.org/c/openstack/ironic/+/82039616:14
dtantsurand bifrost changes16:14
rpittauok16:14
dtantsurI can also offer a metal3 patch for review :D16:16
rpittau:)16:16
rpittauare we good with the updates?16:17
rpittauI'll take that as a yes!16:17
rpittau#topic Priorities for the coming week16:17
rpittau#link https://review.opendev.org/q/hashtag:ironic-week-prio+status:open16:18
rpittauit seems we have some space for reviews this week16:19
dtantsurI 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
dtantsurall 3 are relatively small16:19
arne_wiebalckdynamic network pairing for burn-in: https://review.opendev.org/c/openstack/ironic-python-agent/+/82124416:19
rpittaudtantsur: added16:20
arne_wiebalcknot super small, but maybe interesting to see if you can find holes :)16:20
rpittauarne_wiebalck: added that too16:20
arne_wiebalckrpittau: thanks16:20
rpittaulooks good, does anyone have anything else to add ?16:22
rpittaualright, moving on16:22
rpittauthere's nothing up for discussion this week, so jumping to....16:22
rpittau#topic Baremetal SIG16: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
rpittaucan't wait! :)16:23
arne_wiebalckfor the demo?16:23
arne_wiebalck;)16:23
rpittauyeah, very curious to see this metal3 thing in action :D16:23
arne_wiebalckheh16:23
dtantsurit won't be too long, I must warn - my environment keeps misbehaving16:23
dtantsurlol16:23
arne_wiebalckdtantsur: live demo?16:24
dtantsurI'm too lazy to record anything, so you'll have to bear with me :)16:24
arne_wiebalckha, with pleasure, I am sure it will be great16:24
dtantsuron the other hand, we can poke around together :)16:24
arne_wiebalcknice!16:24
arne_wiebalckI don't think there is anything else for the SIG16:25
arne_wiebalckwe may skip the jan meeting16:25
arne_wiebalcknot sure I find someone to present16:25
rpittausure16:25
arne_wiebalckunless there are volunteers ?16:25
TheJuliamy plate is overflowing at present :(16:25
arne_wiebalcksure, was more a joke, everyone should get some rest and I'll find someone for feb16:26
rpittausounds good16:27
rpittaulet's move on ?16:27
arne_wiebalckyes16:27
rpittauok!16:27
rpittauwe don't have RFEs to review, so jumping to16:27
rpittau#topic Open Discussion16:27
rpittauanything unplanned to discuss ?16:27
dtantsurrloo: re "are folks really using adoption"16:28
dtantsurYeah, 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
dtantsurSo, unlike in most cases, adoption is a routine process for us16:28
rloodtantsur: sweet, that's great!16:28
* dtantsur makes a note to talk about it tomorrow16:29
arne_wiebalcknot sure we have this, but maybe worth documenting how this works with nova16:29
arne_wiebalckadoption, I mean16:29
dtantsurlike "OMG terribly"? :D16:30
rloo[it doesn't play nice with nova right now]16:30
arne_wiebalckI have adopted a couple of thousand nodes into Nova and Ironic :)16:30
rloowe have too, at yahoo, but not via the adoption mechanism.16:31
dtantsurhave a blog post?16:31
arne_wiebalckhttps://techblog.web.cern.ch/techblog/post/ironic-nova-adoption/16:31
dtantsuryeah, I've heard about bright people doing 'openstack stack delete' on their tripleo, and then dancing around the fake drivers16:31
dtantsurto get their cloud back16:31
arne_wiebalckrloo: true, same here16:32
arne_wiebalckrloo: my point was to make this clear in our docs ... unless we have already16:33
arne_wiebalckrloo: so users do not get too excited :)16:33
dtantsurhttps://docs.openstack.org/ironic/latest/admin/adoption.html16:33
dtantsurnothing much abut nova16:33
arne_wiebalckdtantsur: right16:33
rloohmm. maybe we should add something at the top, wrt do-not-use-with-nova...16:34
arne_wiebalckdtantsur: I can suggest sth16:34
dtantsur++16:34
rloothx arne_wiebalck! :)16:34
rpittauthanks arne_wiebalck :)16:35
rpittauany other topic to discuss today ?16:35
rpittauseems we're good to proceed to the last topic16:36
rpittau#topic Who is going to run next year's first meeting?16:36
dtantsurI think we should appoint the PTL if he's back16:37
rpittausounds good :D16:37
rpittaucoll, thanks everyone, have a great week!16:38
rpittau#endmeeting16:38
opendevmeetMeeting ended Mon Dec 13 16:38:23 2021 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)16:38
opendevmeetMinutes:        https://meetings.opendev.org/meetings/ironic/2021/ironic.2021-12-13-16.00.html16:38
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/ironic/2021/ironic.2021-12-13-16.00.txt16:38
opendevmeetLog:            https://meetings.opendev.org/meetings/ironic/2021/ironic.2021-12-13-16.00.log.html16:38
arne_wiebalckthanks rpittau 16:39
*** amoralej is now known as amoralej|off16:44
rpittau good night! o/17:16
opendevreviewArne Wiebalck proposed openstack/ironic master: [doc] Adoption with Nova  https://review.opendev.org/c/openstack/ironic/+/82161118:28
arne_wiebalckrloo: dtantsur ^^18:28
rlooarne_wiebalck: +2, thx!18:36
arne_wiebalckrloo: thanks18:40
arne_wiebalckbye everyone, see you tomorrow o/18:40
rloobye arne_wiebalck! :)18:41
*** sshnaidm is now known as sshnaidm|afk19:06
dtantsurbye folks19:21
* TheJulia feels like a nap is really required19:33
jandersgood morning Ironic o/21:11
TheJuliao/ janders 22:26
jandershey TheJulia :)22:45
jandershow is it going?22:45
TheJuliait goes, listening to training I missed earlier today23:13

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