| *** pmannidi is now known as pmannidi|remote | 01:30 | |
| opendevreview | Merged openstack/ironic master: Expand the IPMI documentation https://review.opendev.org/c/openstack/ironic/+/802019 | 01:44 |
|---|---|---|
| opendevreview | Merged openstack/ironic master: Document making bugfix releases and branches https://review.opendev.org/c/openstack/ironic/+/801903 | 01:44 |
| *** pmannidi|remote is now known as pmannidi | 01:54 | |
| *** pmannidi is now known as pmannidi|brb | 02:43 | |
| *** pmannidi|brb is now known as pmannidi | 04:22 | |
| *** rpittau|afk is now known as rpittau | 07:55 | |
| rpittau | good morning ironic! o/ | 07:55 |
| jssfr | good morning :) | 07:55 |
| janders | hey rpittau o/ | 08:15 |
| rpittau | hey janders :) | 08:15 |
| cenne | Good morning ironic! | 08:27 |
| *** pmannidi is now known as pmannidi|AFK | 09:59 | |
| dtantsur | good afternoon ironic | 11:05 |
| dtantsur | iurygregory: hey! do we have a date for the midcycle already? | 11:10 |
| iurygregory | dtantsur, only 5 people voted (maybe we can ask during the meeting for people to vote so we can decide) I will probably show-up in the upstream meeting | 11:35 |
| dtantsur | ack | 11:35 |
| dtantsur | cenne: have you solved your release notes problems? | 11:42 |
| cenne | it's uploading | 11:48 |
| cenne | got the mail just now. | 11:49 |
| cenne | :) | 11:49 |
| opendevreview | Verification of a change to openstack/ironic-specs failed: Clean up released features/items https://review.opendev.org/c/openstack/ironic-specs/+/801650 | 11:53 |
| opendevreview | cenne proposed openstack/python-ironicclient master: Adds support for 'boot_mode' and 'secure_boot' node resource fields https://review.opendev.org/c/openstack/python-ironicclient/+/802026 | 11:55 |
| cenne | dtansur: minor clarification. in regards to openstacksdk patch. is the 'is_' in is_secure_boot fine? or would just node.secure_boot be preferred. | 12:08 |
| dtantsur | cenne: I think the openstacksdk convention is to use "is_" | 12:09 |
| opendevreview | cenne proposed openstack/python-ironicclient master: Add support for 'boot_mode' and 'secure_boot' node resource fields https://review.opendev.org/c/openstack/python-ironicclient/+/802026 | 12:09 |
| cenne | dtantsur: cool, so i'll let it stay. | 12:11 |
| TheJulia | good morning | 13:15 |
| * TheJulia wipes sleep from eyes as sleep was... a challenge | 13:17 | |
| TheJulia | We're over 1.5 inches of rain for the last 24 hours | 13:21 |
| TheJulia | which... seems like a lot, but it is the most in..... over a year | 13:22 |
| rpittau | too much rain here, not enough rain in my burning homeland :/ | 13:23 |
| TheJulia | :( | 13:25 |
| rpittau | literally burning, there's an immense fire that started over 60 hours ago https://www.theguardian.com/world/2021/jul/26/firefighters-battle-wildfires-raging-across-south-west-sardinia | 13:26 |
| TheJulia | :( | 13:30 |
| TheJulia | Not good | 13:30 |
| dtantsur | morning TheJulia | 13:31 |
| dtantsur | ugh :( | 13:32 |
| TheJulia | brrraaaaiiinnnns | 14:06 |
| dtantsur | where??? | 14:08 |
| dtantsur | huh https://zuul.opendev.org/t/openstack/build/d88c26b996c84164867433c281327cec/log/sphinx-build-pdf.log | 14:16 |
| TheJulia | I just spent an hour replyign to the pile of email that appeared over the weekend | 14:17 |
| TheJulia | I'd <3 brains if they were to just appear and be full of noms | 14:17 |
| dtantsur | I can relate. I'm writing a blog post since Friday.. | 14:17 |
| cenne | dtantsur: Just checked. I also get an error building pdf locally. Investigating. | 14:23 |
| dtantsur | cenne: it was on your patch, yeah | 14:23 |
| cenne | I didn't make any other changes though. :( Strange. | 14:24 |
| opendevreview | Ruby Loo proposed openstack/ironic master: Minor updates to anaconda doc https://review.opendev.org/c/openstack/ironic/+/802314 | 14:29 |
| rpittau | cenne, dtantsur, FYI I was looking at that but I can't reproduce it locally | 14:33 |
| cenne | hmm. that's very odd. | 14:34 |
| dtantsur | I've issued a recheck command, let's see how it goes | 14:40 |
| TheJulia | who is running the meeting today? | 14:41 |
| dtantsur | that would be me | 14:41 |
| cenne | https://bugs.launchpad.net/openstack-i18n/+bug/1935742 > Looks like a known bug. | 14:43 |
| TheJulia | ok | 14:43 |
| JayF | fixed 7/22/2021 | 14:54 |
| JayF | means maybe we need to bump a requirement, perhaps, to make that build pass? | 14:55 |
| JayF | https://review.opendev.org/c/openstack/i18n/+/800322/2/bindep.txt#7 that was their fix | 14:56 |
| dtantsur | let's see how the recheck goes | 14:56 |
| JayF | we could also do the same? | 14:56 |
| dtantsur | the same patch passed very recently | 14:56 |
| JayF | > '! LaTeX Error: File `tgtermes.sty' not found.' | 14:57 |
| JayF | on another patch, on ironic | 14:57 |
| JayF | https://review.opendev.org/c/openstack/ironic/+/802314 | 14:57 |
| dtantsur | meh | 14:57 |
| dtantsur | yeah, anyone wants to do the bindep change? | 14:57 |
| rpittau | I can do it | 14:59 |
| dtantsur | thx! | 14:59 |
| dtantsur | #startmeeting ironic | 15:00 |
| opendevmeet | Meeting started Mon Jul 26 15:00:13 2021 UTC and is due to finish in 60 minutes. The chair is dtantsur. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:00 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:00 |
| opendevmeet | The meeting name has been set to 'ironic' | 15:00 |
| rpittau | o/ | 15:00 |
| dtantsur | #chair TheJulia | 15:00 |
| opendevmeet | Current chairs: TheJulia dtantsur | 15:00 |
| TheJulia | o/ | 15:00 |
| erbarr | o/ | 15:00 |
| stendulker | o/ | 15:00 |
| dtantsur | hello everyone! | 15:00 |
| dtantsur | welcome to the most ironic of all meetings | 15:00 |
| cenne | o/ | 15:00 |
| dtantsur | #link https://wiki.openstack.org/wiki/Meetings/Ironic is our agenda | 15:00 |
| rloo | o/ | 15:00 |
| TheJulia | wait!?! We added more ironic?!? | 15:00 |
| * TheJulia is confused with the levels of ironic | 15:00 | |
| dtantsur | there is never enough, is there? | 15:00 |
| dtantsur | #topic Announcements / Reminder | 15:01 |
| rpioso | \o | 15:01 |
| dtantsur | mmm, I wonder if #topic works any more :) | 15:01 |
| dtantsur | anyway | 15:01 |
| dtantsur | #info Spring 2 releases have been done! | 15:01 |
| dtantsur | thanks all for your contributions! | 15:01 |
| dtantsur | #info Please add ideas vote for the midcycle slot: https://etherpad.opendev.org/p/ironic-xena-midcycle | 15:01 |
| dtantsur | #link https://doodle.com/poll/mhh959u4s4rturxi doodle for midcycle | 15:02 |
| * dtantsur suspects some slots are already in the past | 15:02 | |
| dtantsur | #info Please add your ideas for the PTG: https://etherpad.opendev.org/p/ironic-yoga-ptg | 15:02 |
| dtantsur | As a reminder, it will be in October, 18-22 | 15:02 |
| dtantsur | anything else to announce or remind of? | 15:03 |
| dtantsur | #topic Review action items from previous meeting | 15:03 |
| TheJulia | Nothing from my end. | 15:04 |
| dtantsur | iurygregory has created a midcycle doodle, thank you! | 15:04 |
| dtantsur | #topic Review subteam status reports (capped at ten minutes) | 15:04 |
| dtantsur | #link https://etherpad.opendev.org/p/IronicWhiteBoard starting on line 66 | 15:04 |
| dtantsur | the number of bugs has reduced. if somebody has been triaging/fixing them - thank you! | 15:05 |
| dtantsur | JayF, zer0c00l, how has the story if the anaconda CI finished? I don't remember the last week well :) | 15:05 |
| JayF | AFAIK, there is no planned time for us to spend on additional anaconda ci | 15:06 |
| JayF | TheJulia said she'd give the CentOS people a heads up about the artifacts we'd need to avoid doing image-building work in the gate | 15:07 |
| TheJulia | dtantsur: triaging/cleaning up as of recent | 15:07 |
| dtantsur | nice re cleaning up! | 15:07 |
| TheJulia | I'm not quite sure I remebering committing to that | 15:07 |
| dtantsur | JayF: not sure about the others, I feel pretty uneasy about having the whole deploy interface not really covered | 15:07 |
| TheJulia | s/remembering/remember/ | 15:07 |
| TheJulia | We likely need to have real discussion with opendev folks though, since it is a narly build in CI for us to build them every time | 15:08 |
| JayF | dtantsur: I don't like the situation either; but we just don't have the time, and building CI went up an order of magnitude in complexity when we realized that centos does not publish any compatible image | 15:08 |
| dtantsur | JayF: what exactly do you lack? | 15:08 |
| TheJulia | A second stage image | 15:08 |
| JayF | It's documented in the storyboard story linked in the subteam report | 15:08 |
| dtantsur | what is it technically? a root fs? | 15:08 |
| JayF | yeah | 15:09 |
| TheJulia | *but* that can be generated from centos native hosts if memory serves, and someone has recently posted some centos jobs to a number of projects.... | 15:09 |
| JayF | building it is documented in our docs, but the tools needed (yum/yum-utils) are not in the Ubuntu in our gate which is the first problem :( | 15:09 |
| dtantsur | JayF: this ugly stuff cannot help you? https://opendev.org/openstack/metalsmith/src/branch/master/playbooks/integration/centos-image.yaml#L85-L125 | 15:09 |
| dtantsur | mm, yeah. we can have a bifrost job, it works on centos. | 15:09 |
| dtantsur | but note that centos8 doesn't have yum-utils either. | 15:10 |
| JayF | dtantsur: hm, I don't know. | 15:10 |
| JayF | It's all documented in that story, I haven't looked at it since filing that | 15:10 |
| dtantsur | JayF: that's what I did in metalsmith to get a kernel/ramdisk/root from a whole disk image | 15:10 |
| JayF | It looks closer than anything I've seen, but I just don't know for sure | 15:10 |
| TheJulia | Perhaps table this for now and schedule a specific time to get some contributors on a call to at least brainstorm and sort through it? | 15:11 |
| TheJulia | or maybe midcycle topic? | 15:11 |
| dtantsur | yeah, but let's not drop this on the floor. we have enough untested corners. | 15:12 |
| dtantsur | midcycle ++ | 15:12 |
| * TheJulia opens the etherpad to add it | 15:12 | |
| JayF | dtantsur: that's why I documented the crap outta the problem and put it on the etherpad :) | 15:12 |
| dtantsur | okay, adding | 15:14 |
| dtantsur | arne_wiebalck: have you heard anything from kaifeng about the node history? | 15:14 |
| TheJulia | arne_wiebalck is on vacation | 15:14 |
| dtantsur | TheJulia: have you chatted with janders re cleaning? | 15:14 |
| dtantsur | ah | 15:14 |
| TheJulia | s/vacation/holiday/ | 15:15 |
| TheJulia | dtantsur: I have not | 15:15 |
| TheJulia | my last week was slammed | 15:15 |
| dtantsur | I see. I'll try to remember (maybe he'll see the ping) | 15:15 |
| TheJulia | hopefully | 15:15 |
| dtantsur | \o/ re RBAC done | 15:16 |
| rloo | yay! | 15:16 |
| dtantsur | would it be too much to ask for Bifrost support? ;) | 15:16 |
| TheJulia | dtantsur: uhhhh | 15:16 |
| dtantsur | the answer "yes" is perfectly acceptable :) | 15:17 |
| TheJulia | dtantsur: lets midcycle that because that would require some policy enforcement internals | 15:17 |
| dtantsur | as an MVP we can only do it when keystone is enabled | 15:17 |
| TheJulia | I guess I feel the need to have a clarifying doc written for rbac | 15:17 |
| dtantsur | oh, at least fix the 2 conflicting docs | 15:17 |
| TheJulia | since presently the policy enforcement code is bypassed in noauth model | 15:17 |
| dtantsur | there was a patch for it, seems abandoned though | 15:18 |
| TheJulia | or non-keystone auth models | 15:18 |
| TheJulia | orly? | 15:18 |
| dtantsur | https://review.opendev.org/789082 | 15:19 |
| TheJulia | I guess that is a fair change to perform | 15:20 |
| TheJulia | \o/ ask refined | 15:20 |
| dtantsur | okay, is everyone done with the statuses? | 15:20 |
| dtantsur | we can continue the RBAC chat later | 15:20 |
| TheJulia | ++ | 15:21 |
| dtantsur | #topic Deciding on priorities for the coming week | 15:22 |
| dtantsur | I have 3 CI patches https://review.opendev.org/c/openstack/ironic-python-agent/+/801898 https://review.opendev.org/c/openstack/ironic-inspector/+/801873 https://review.opendev.org/c/openstack/ironic/+/801876 | 15:22 |
| opendevreview | Riccardo Pittau proposed openstack/python-ironicclient master: Fix for missing fonts in PDF jobs https://review.opendev.org/c/openstack/python-ironicclient/+/802324 | 15:22 |
| dtantsur | and https://review.opendev.org/c/openstack/ironic/+/800084 from cenne | 15:22 |
| dtantsur | any objections? | 15:23 |
| TheJulia | none | 15:23 |
| rpittau | nope | 15:23 |
| TheJulia | We've got some backports on to ussuri that would be good, but they are presently failing CI, it appears due to random timeouts :\ | 15:23 |
| opendevreview | Ruby Loo proposed openstack/ironic master: Minor updates to anaconda doc https://review.opendev.org/c/openstack/ironic/+/802314 | 15:23 |
| cenne | umm.. | 15:24 |
| * cenne raises hand | 15:24 | |
| dtantsur | should we remove https://review.opendev.org/c/openstack/ironic-specs/+/764801 for now? it has been without updates for months | 15:24 |
| dtantsur | cenne: we're listening :) | 15:24 |
| rpittau | sounds good | 15:24 |
| TheJulia | yeah, agree to remove | 15:24 |
| cenne | so, there's a few tests that are currently commented out that i was wondering if make sense to just remove | 15:25 |
| dtantsur | interesting, where is that? | 15:25 |
| TheJulia | and are there comments accompanying them? | 15:25 |
| cenne | ironic/tests/unit/conductor/test_manager.py Line 513, 604. | 15:26 |
| cenne | https://review.opendev.org/c/openstack/ironic/+/800084/11/ironic/tests/unit/conductor/test_manager.py | 15:27 |
| dtantsur | ah, you mean, in your patch, not in git master? | 15:27 |
| cenne | TheJulia: Sorry, I was going to ask today/this-week | 15:27 |
| cenne | Yes in the patch. | 15:27 |
| dtantsur | let's discuss it on the patch then? | 15:27 |
| cenne | okay. | 15:27 |
| TheJulia | ++ | 15:28 |
| dtantsur | anyone has anything else for the priorities? | 15:28 |
| TheJulia | not today | 15:29 |
| dtantsur | #topic OpenInfra Live | 15:30 |
| dtantsur | TheJulia: the mic is yours! | 15:30 |
| TheJulia | oh, so the tl;dr is some of us have started collaborating on a basic presentation on "what is ironic" | 15:30 |
| TheJulia | and the previously mentioned operator feedback loop session didn't really gain any traction as openinfra live. The consensus is we need the forums back. | 15:31 |
| TheJulia | That is it for OpenInfra Live. Any questions? Anyone want to collaborate on a high level what is ironic and maybe "why you should be using it!" | 15:32 |
| dtantsur | for which audience? | 15:32 |
| cenne | yay :) | 15:32 |
| TheJulia | for which part? | 15:32 |
| dtantsur | I'm not sure I understand what you propose if the proposal hasn't gained traction? | 15:32 |
| TheJulia | There were two proposals, one was for a feedback session and that didn't gain any traction | 15:32 |
| dtantsur | I actually wrote a few paragraphs of "why ironic" for my upcoming post, can share the draft with you | 15:33 |
| TheJulia | The ironic session for openinfra live is tentatively on August 12th | 15:33 |
| TheJulia | dtantsur: ++ | 15:33 |
| dtantsur | Aug 12th => no dtantsur | 15:33 |
| TheJulia | ack | 15:33 |
| dtantsur | I'll pm you once I render it | 15:33 |
| TheJulia | k | 15:33 |
| TheJulia | Safe to proceed to the next topic? | 15:33 |
| dtantsur | yep | 15:34 |
| TheJulia | #topic Cancelling review jams? | 15:34 |
| cenne | ~~When is it? (OpenInfra Live)~~. I am also working on a blog entry with similar title. | 15:34 |
| cenne | / August 12 ack. | 15:34 |
| TheJulia | So, JayF pointed out that I've been struggling to be present for Review Jams, and the tl;dr is I've had meetings take over my Tuesday calendar. | 15:34 |
| rpittau | that's unfortunately also my case :/ | 15:35 |
| JayF | I mainly noticed nobody at all was attending them | 15:35 |
| JayF | except me. | 15:35 |
| TheJulia | We've also struggled with attendance after the last major release, which makes me think they are good for end of cycle get things sorted/merged activities | 15:35 |
| dtantsur | Our team now also has a partly conflicting meeting | 15:35 |
| TheJulia | JayF: Well, truth be told, It helps to have someone present who is kind of hitting the drum to drive people to the meeting and topics to get listed | 15:36 |
| TheJulia | I just can't hit the drum on the boat to waterskiiing speed on Tuesdays :) | 15:36 |
| JayF | I'll also generally note, as a reviewer, we haven't had many outstanding ironic/ipa/ironiclib patches recently | 15:36 |
| TheJulia | (think, someone drumming and a bunch of rowers to reach water skiing speeds) | 15:36 |
| JayF | they tend to get in pretty quick | 15:36 |
| TheJulia | So I guess, any objections to cancelling them at least until the midcycle? | 15:37 |
| TheJulia | If we identify priorities there then we should consider resumption and appropriate scheduling then? | 15:37 |
| * dtantsur agrees to both | 15:37 | |
| JayF | ++ | 15:37 |
| TheJulia | Anyone objections? | 15:38 |
| rpittau | none from me | 15:38 |
| * TheJulia believes the motion carries | 15:38 | |
| TheJulia | So we're done with Discussion then | 15:38 |
| TheJulia | #topic Baremetal SIG | 15:39 |
| TheJulia | arne_wiebalck is out, and I don't believe there are any updates. Does anyone have anything sig related? | 15:39 |
| dtantsur | not me | 15:40 |
| dtantsur | #topic Who is going to run the next meeting? | 15:40 |
| TheJulia | Maybe a reminder, upcoming topics https://etherpad.opendev.org/p/bare-metal-sig <-- Around line 160 might interest people | 15:40 |
| dtantsur | a bit early, sorry :) | 15:40 |
| TheJulia | no worries | 15:40 |
| dtantsur | the question persists though: any volunteers? | 15:41 |
| TheJulia | I'll do it next week | 15:41 |
| rpittau | I can run the next one | 15:41 |
| dtantsur | we have a conflict, the PTL will break the tie! | 15:41 |
| rpittau | :D | 15:41 |
| TheJulia | lol | 15:41 |
| TheJulia | rpittau: if you really want to your welcome run the meeting :) | 15:41 |
| rpittau | eheh ok :) | 15:41 |
| dtantsur | The PTL's judgement has been announced! | 15:42 |
| TheJulia | lol | 15:42 |
| dtantsur | #topic Open discussion | 15:42 |
| dtantsur | The floor is open, does anyone has anything? | 15:42 |
| TheJulia | I feel like my glorious purpose is to go reply to yet another email | 15:42 |
| rpittau | I have one hopefully quick thing, the lower-constraints job is actually working as expected now https://review.opendev.org/c/openstack/ironic/+/801350 | 15:42 |
| dtantsur | s/now/for now/ :D | 15:43 |
| rpittau | lol | 15:43 |
| dtantsur | rpittau: you forgot the limit to master only | 15:43 |
| rpittau | I didn't mention that in the commit message? | 15:43 |
| dtantsur | rpittau: I mean something like https://opendev.org/openstack/ironic-python-agent-builder/commit/963b23e0ae703f230815b474f1a2bf32cb4e807f | 15:44 |
| dtantsur | so that we don't have yet another thing to remember when creating branches | 15:44 |
| rpittau | oh yeah, sure, I'll add that | 15:44 |
| dtantsur | I'm not 100% sure it works with templates.. but worth trying? | 15:44 |
| rpittau | yep, let's give that a try | 15:45 |
| dtantsur | thanks! | 15:45 |
| rpittau | np | 15:45 |
| dtantsur | anything else? | 15:46 |
| TheJulia | not from my point of view | 15:48 |
| dtantsur | thanks all! | 15:48 |
| dtantsur | #endmeeting | 15:49 |
| opendevmeet | Meeting ended Mon Jul 26 15:49:12 2021 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:49 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/ironic/2021/ironic.2021-07-26-15.00.html | 15:49 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/ironic/2021/ironic.2021-07-26-15.00.txt | 15:49 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/ironic/2021/ironic.2021-07-26-15.00.log.html | 15:49 |
| lmcgann | I guess i missed the open discussion part of the meeting, but I want to throw out there that we had our demo for the attestation interface last week and are putting together a video showcasing the workflow to share with you guys and the Keylime upstream team. It should be ready sometime this week. | 15:52 |
| TheJulia | lmcgann: awesome | 15:53 |
| opendevreview | Riccardo Pittau proposed openstack/ironic stable/wallaby: [DNM] test branches on template https://review.opendev.org/c/openstack/ironic/+/802329 | 15:59 |
| rpittau | just a brutal test ^ | 15:59 |
| dtantsur | hmm, zuul doesn't like it. maybe ask on #opendev? | 16:04 |
| rpittau | yeah | 16:05 |
| TheJulia | agenda list updated w/r/t review jams | 16:21 |
| JayF | I'll remove em from the whiteboard | 16:27 |
| TheJulia | JayF: thanks | 16:28 |
| TheJulia | Any reviews on https://review.opendev.org/c/openstack/ironic/+/801867 would be appropriated, we're trying to fix this down on train and need to stop on ussuri first of course. | 16:32 |
| TheJulia | err, appreciated | 16:32 |
| dtantsur | no way, I'm now afraid of my review being appropriated! | 16:32 |
| dtantsur | TheJulia: please make sure you also backport my documentation follow-up. Many distributions don't have snponly.ipxe | 16:33 |
| TheJulia | lol | 16:33 |
| TheJulia | oh right | 16:33 |
| dtantsur | also some good news: https://bugzilla.redhat.com/show_bug.cgi?id=1981799 is ON_QA | 16:34 |
| TheJulia | oh jeeze where was that | 16:34 |
| opendevreview | Julia Kreger proposed openstack/ironic stable/wallaby: Fix iPXE docs: snponly is not always available https://review.opendev.org/c/openstack/ironic/+/802344 | 16:36 |
| TheJulia | dtantsur: like ^ | 16:37 |
| TheJulia | ? | 16:37 |
| rpittau | good night! o/ | 16:37 |
| TheJulia | goodnight rpittau | 16:37 |
| *** rpittau is now known as rpittau|afk | 16:37 | |
| opendevreview | Julia Kreger proposed openstack/ironic stable/victoria: Fix iPXE docs: snponly is not always available https://review.opendev.org/c/openstack/ironic/+/802345 | 16:38 |
| dtantsur | yep | 16:39 |
| dtantsur | I wasn't aware the snponly change was backported | 16:39 |
| TheJulia | it wasn't | 16:39 |
| TheJulia | but the text is still valid | 16:39 |
| TheJulia | because the base text suggests you should override the setting | 16:40 |
| TheJulia | . o O (why!?!?) | 16:40 |
| TheJulia | the explicit bootloader patch is much older, I think it merged in victoria but it was a bugfix we never backported | 16:41 |
| TheJulia | and of course, now we need it on train :( | 16:41 |
| dtantsur | everyone on train! | 16:49 |
| * dtantsur should go eat something now | 16:50 | |
| TheJulia | choooo chooooooo | 17:00 |
| TheJulia | or chewwwww chewwwww | 17:00 |
| TheJulia | :) | 17:00 |
| dtantsur | lol, I like it! | 17:01 |
| opendevreview | Merged openstack/ironic stable/wallaby: Fix iPXE docs: snponly is not always available https://review.opendev.org/c/openstack/ironic/+/802344 | 17:12 |
| opendevreview | Merged openstack/ironic stable/victoria: Fix iPXE docs: snponly is not always available https://review.opendev.org/c/openstack/ironic/+/802345 | 17:23 |
| cenne | JayF: FTR doing a new full clone and copying the patch over (git format-patch, git am) fixed it (the releasenotes building issue I had). | 17:34 |
| JayF | \o/ | 17:35 |
| opendevreview | Bob Fournier proposed openstack/ironic master: For particular vendors set the boot device prior to intalling image https://review.opendev.org/c/openstack/ironic/+/802389 | 19:20 |
| opendevreview | Bob Fournier proposed openstack/ironic master: [WIP] For particular vendors set the boot device prior to intalling image https://review.opendev.org/c/openstack/ironic/+/802389 | 19:20 |
| bfournie | TheJulia: something like this ^ ? | 19:21 |
| opendevreview | Bob Fournier proposed openstack/ironic master: [WIP] For particular vendors set the boot device prior to installing image https://review.opendev.org/c/openstack/ironic/+/802389 | 19:32 |
| TheJulia | bfournie: basically yes | 20:12 |
| TheJulia | bfournie: The thought that comes to mind is that this is likely uefi specific behavior | 20:12 |
| TheJulia | but I kind of wonder if we should abandon trying to set to disk if we setup a bootloader | 20:13 |
| TheJulia | at least with uefi | 20:13 |
| bfournie | TheJulia: so with UEFI it we don't do the "set device" it will still boot using the bootloader | 20:16 |
| bfournie | ? | 20:16 |
| TheJulia | it should if we ran the install bootloader step | 20:21 |
| TheJulia | the caveat is if the bmc has been persistently overridden | 20:22 |
| TheJulia | I think we need a chart to map through that | 20:22 |
| bfournie | TheJulia: right, as part of the set boot device to disk we set it to persistent, previous it was one-time pxe, no? | 20:23 |
| TheJulia | well, we would need to make sure that it is absolutely only one time to pxe | 20:29 |
| TheJulia | I guess the fear I have is some vendors might take that one time as persistent | 20:30 |
| bfournie | TheJulia: oh I see, so we'd set it with the one-time flag but vendor would use it as persistent, and it may depend on *how* it was set, ipmi or redfish etc | 20:41 |
| TheJulia | exactly | 20:41 |
| bfournie | TheJulia: I guess we can keep it vendor specific for now and make it more generic after testing. i like removing the set device entirely for this vendor to test that first. I can easily rework the patch to do that | 20:45 |
| TheJulia | bfournie: that is exactly what I was thinking | 21:00 |
| TheJulia | iurygregory: dtantsur: worked on the slide deck for openinfra live. Added a few more slides, resorted a couple things, and also tried to get across "ironic is more than just deployment" | 21:00 |
| iurygregory | TheJulia, ack o/ I will look at it tomorrow =D | 21:01 |
| opendevreview | Steve Baker proposed openstack/ironic stable/train: Provide a path to set explicit ipxe bootloaders https://review.opendev.org/c/openstack/ironic/+/801669 | 21:30 |
| TheJulia | stevebaker: so I finally realized that the puppet manifest was only doing like 80% of it https://review.rdoproject.org/r/c/openstack/ironic-distgit/+/34691 | 22:18 |
| *** pmannidi is now known as pmannidi|AFK | 22:18 | |
| stevebaker | TheJulia: I'm relieved you're talking about tftp and not the ipxe template | 22:19 |
| TheJulia | :) | 22:20 |
| stevebaker | TheJulia: how does this work with containers and the tripleo tooling? Is it like other ironic services where dnsmasq is running in its own container, orchestrated by tripleo and managed by the host systemd? | 22:23 |
| TheJulia | No clue... I'm assuming someplace there is a container build toolkit that results from puppet manifests | 22:24 |
| TheJulia | Which might explain why what should have worked with them didn't work, but really there are just a ton of layers so hopefully that is tomorrow's question to peel back | 22:25 |
| stevebaker | TheJulia: ok, so the packaging needs to handle the pure packaged running as a systemd managed process case. The puppet needs to generate the config and be capable of starting a systemd managed process. But tripleo will only use the config generation bit of that. It looks like this file will need to replace the ironic_pxe_tftp container yaml with the dnsmasq version (and maybe remove some xinetd things too?) https://opendev. | 22:32 |
| stevebaker | org/openstack/tripleo-heat-templates/src/branch/master/deployment/ironic/ironic-pxe-container-puppet.yaml | 22:32 |
| stevebaker | this would be exercised in the featureset001 job | 22:33 |
| TheJulia | wow that is overly complicated | 22:33 |
| * TheJulia wonders how much is redundant | 22:33 | |
| stevebaker | it is yeah. Packaging doesn't care about containers, and container based service management is not standardised at the os/packaging level. So in the absense of kubernetes, tripleo has its own yaml -> ansible -> writing systemd units -> podman run | 22:36 |
| opendevreview | Merged openstack/ironic stable/ussuri: Provide a path to set explicit ipxe bootloaders https://review.opendev.org/c/openstack/ironic/+/801867 | 22:52 |
| *** ChanServ changes topic to "Bare Metal Provisioning | Status: http://bit.ly/ironic-whiteboard | Docs: http://docs.openstack.org/ironic/ | Bugs: https://storyboard.openstack.org/#!/project_group/75 | Contributors are generally present between 6 AM and 12 AM UTC, If we do not answer, please feel free to pose questions to openstack-discuss mailing list." | 22:56 | |
| *** pmannidi|AFK is now known as pmannidi | 23:36 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!