| arne_wiebalck | Good morning, Ironic! | 07:18 |
|---|---|---|
| janders | hey arne_wiebalck and Ironic o/ | 07:24 |
| janders | Happy New Year 2022! | 07:25 |
| arne_wiebalck | hey janders, good morning and happy new year! | 07:25 |
| rpittau | good morning ironic! o/ | 08:32 |
| arne_wiebalck | hey rpittau o/ | 09:05 |
| arne_wiebalck | rpittau: if you have a moment, could you please have another look at https://review.opendev.org/c/openstack/ironic-python-agent/+/821244 to see if we can move this one forward? | 09:06 |
| rpittau | hey arne_wiebalck :) | 09:07 |
| rpittau | sure thing, I'll have a look in a couple of minutes | 09:09 |
| arne_wiebalck | rpittau: thanks! | 09:09 |
| janders | hey rpittau o/ | 09:18 |
| rpittau | hey janders | 09:19 |
| rpittau | arne_wiebalck: it looks good to me, just a small thing, I think you don't need the new reqs in test-requirements.txt as you have already tooz in requirements.txt and kazoo is not imported directly | 10:11 |
| * dtantsur exits hibernation | 10:19 | |
| arne_wiebalck | rpittau: true, I can remove it | 10:20 |
| opendevreview | Arne Wiebalck proposed openstack/ironic-python-agent master: Burn-in: Dynamic network pairing https://review.opendev.org/c/openstack/ironic-python-agent/+/821244 | 10:23 |
| janders | hey dtantsur o/ | 10:26 |
| rpittau | arne_wiebalck: you left tooz :) | 10:30 |
| arne_wiebalck | sigh | 10:31 |
| arne_wiebalck | yes, sorry | 10:31 |
| opendevreview | Arne Wiebalck proposed openstack/ironic-python-agent master: Burn-in: Dynamic network pairing https://review.opendev.org/c/openstack/ironic-python-agent/+/821244 | 10:33 |
| iurygregory | good morning Ironic o/ | 11:11 |
| arne_wiebalck | thanks rpittau | 11:16 |
| janders | hey iurygregory o/ | 12:07 |
| iurygregory | janders, o/ | 12:07 |
| janders | See you tomorrow Ironic o/ have a great day everyone. | 12:58 |
| arne_wiebalck | bye janders o/ | 13:02 |
| iurygregory | bye janders o/ | 13:13 |
| dtantsur | rpittau: re your stream9 patch: deprecate stream8 at the same time? given python 3.6? | 14:08 |
| rpittau | mmmm in theory yes | 14:12 |
| rpittau | well in practice also :D | 14:12 |
| rpittau | I can add a note in the reno in the follow up | 14:12 |
| dtantsur | rpittau: oh, and you need to update the installation docs :) | 14:14 |
| rpittau | right! this follow up is becoming bigger and bigger :) | 14:15 |
| * dtantsur realizes he needs to change his testing environment to S9 | 14:15 | |
| * iurygregory should do the same | 14:17 | |
| TheJulia | Good morning | 15:02 |
| rpittau | good morning TheJulia :) | 15:02 |
| dtantsur | morning TheJulia | 15:05 |
| opendevreview | Merged openstack/bifrost master: Move /httpboot to /var/lib/ironic https://review.opendev.org/c/openstack/bifrost/+/822743 | 15:06 |
| rpittau | can I get one more review for https://review.opendev.org/c/openstack/bifrost/+/819058 please? :) | 15:08 |
| iurygregory | good morning TheJulia | 15:19 |
| TheJulia | rpittau: when will the node set be available ? | 15:35 |
| rpittau | TheJulia: technically it's already there, what's missing is the naming in the config | 15:36 |
| rpittau | the alias | 15:36 |
| TheJulia | rpittau: is there an ETA for that alias? | 15:37 |
| rpittau | TheJulia: actually let me double-check, it might be there already as it was planned for end of last year/beginning of new year | 15:42 |
| rpittau | yep it's there | 15:43 |
| TheJulia | rpittau: ahh, cool | 15:43 |
| rpittau | I can change that in the follow up as well | 15:43 |
| TheJulia | rpittau: the reason I was asking was because it would be awful to merge the change and have the use of cs9 (am I the only one whose brain things cs == counterstrike?) and it never get changed | 15:43 |
| TheJulia | for testing that is | 15:43 |
| TheJulia | rpittau: you know, I think that is fair to just in place edit and single core approve if it works. | 15:44 |
| rpittau | ok, let me change that now then | 15:44 |
| TheJulia | cool cool | 15:44 |
| rpittau | and yes, CS _is_ counterstrike | 15:45 |
| dtantsur | heh | 15:45 |
| * TheJulia wishes she didn't loose her sporks in the divorce so she could feel better about her old handle EvilSpork | 15:45 | |
| opendevreview | Riccardo Pittau proposed openstack/bifrost master: Run bifrost on CentOS Stream 9 https://review.opendev.org/c/openstack/bifrost/+/819058 | 15:46 |
| rpittau | done | 15:46 |
| TheJulia | Thanks! | 15:46 |
| opendevreview | Riccardo Pittau proposed openstack/bifrost master: Run bifrost on CentOS Stream 9 https://review.opendev.org/c/openstack/bifrost/+/819058 | 15:46 |
| rpittau | and typo fixed :D | 15:46 |
| TheJulia | \p/ | 15:46 |
| TheJulia | err | 15:46 |
| TheJulia | \o/ | 15:46 |
| TheJulia | any last reviews on https://github.com/OpenStackweb/ironic-website/pull/46 ? | 15:48 |
| TheJulia | we can't forget to occasionally post content there :( | 15:48 |
| rpittau | I'll have a quick look | 15:49 |
| TheJulia | we really ought to hold some sort of midcycle coupled with review jamming non-core contributor patches | 15:56 |
| dtantsur | ++ | 15:56 |
| iurygregory | ++ | 15:58 |
| rpittau | heh would be nice | 15:59 |
| TheJulia | would also be a good time to talk through some patch backstories | 16:00 |
| TheJulia | hopefully no relation to origins like GCC got | 16:00 |
| dtantsur | meeting? :) | 16:01 |
| * TheJulia wonders if she should start the meeting? | 16:01 | |
| iurygregory | I was about to ask :D | 16:01 |
| TheJulia | #startmeeting ironic | 16:01 |
| opendevmeet | Meeting started Mon Jan 10 16:01:40 2022 UTC and is due to finish in 60 minutes. The chair is TheJulia. Information about MeetBot at http://wiki.debian.org/MeetBot. | 16:01 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 16:01 |
| opendevmeet | The meeting name has been set to 'ironic' | 16:01 |
| TheJulia | #chair iurygregory | 16:01 |
| opendevmeet | Current chairs: TheJulia iurygregory | 16:01 |
| dtantsur | o/ | 16:01 |
| rpittau | o/ | 16:01 |
| erbarr | o/ | 16:01 |
| * TheJulia hands the microphone to iurygregory :) | 16:01 | |
| rpioso | o/ | 16:01 |
| iurygregory | o/ | 16:01 |
| ajya | o/ | 16:01 |
| iurygregory | Welcome back everyone! Happy New Year! | 16:02 |
| stendulker | o/ | 16:02 |
| rloo | o/ | 16:02 |
| iurygregory | #topic Announcements / Reminder | 16:03 |
| iurygregory | anyone has something to announce /remind us? | 16:03 |
| arne_wiebalck | o/ | 16:03 |
| * iurygregory does a quick check on openstack ML to see if there is something | 16:04 | |
| TheJulia | I think we've got 11 weeks until Yoga release | 16:04 |
| TheJulia | or maybe 12 | 16:05 |
| iurygregory | it's on Marc 11 so around 11 weeks | 16:06 |
| dtantsur | yeah, the first release freezes will be around mid-February | 16:06 |
| iurygregory | the second bugfix we will release on begin of Feb | 16:06 |
| iurygregory | #info check your emails for "2022 Individual Director Election and Bylaws Amendments" | 16:07 |
| iurygregory | if you are a contributor you probably received an email to vote =) | 16:07 |
| iurygregory | anything else? =) | 16:09 |
| TheJulia | I think that is it | 16:09 |
| iurygregory | ok, let's move on | 16:09 |
| iurygregory | #topic Review Action items from prior meeting | 16:10 |
| iurygregory | I've checked and we don't have any past action items | 16:10 |
| iurygregory | #topic Review subteam status reports | 16:10 |
| dtantsur | who cares about last year's action items? :) | 16:10 |
| iurygregory | only if we had something important XD | 16:11 |
| iurygregory | #link https://etherpad.opendev.org/p/IronicWhiteBoard | 16:11 |
| iurygregory | starting around L63 | 16:11 |
| TheJulia | we should really just dogpile on a troubleshooting/set of work one day | 16:13 |
| dtantsur | iurygregory: we haven't really talked about rootwrap/privsep for inspector | 16:15 |
| dtantsur | the iptables driver at least requires sudo | 16:15 |
| TheJulia | yeah | 16:16 |
| dtantsur | maybe we can make privsep optional when the dnsmasq driver is used? | 16:16 |
| TheJulia | I think steve baker may have looked at that recently... | 16:16 |
| TheJulia | I think that would be good but may also be fairly complex at the same time | 16:16 |
| dtantsur | yeah, this is what caused me to remember about inspector :) | 16:16 |
| TheJulia | oh good, at least my memory slightly works :) | 16:16 |
| dtantsur | not really, the dnsmasq driver does not use sudo by default, only when you provide restart/shutdown commands | 16:16 |
| iurygregory | dtantsur, good catch | 16:16 |
| dtantsur | I *think* | 16:17 |
| iurygregory | make it optional would be a good thing | 16:17 |
| TheJulia | dtantsur: I was more thinking from the entire process model standpoint but I don't have my head wrapped around it heavily | 16:17 |
| iurygregory | maybe I can sync with stevebaker[m] to see what he thinks (my timezone now permits this :D) | 16:19 |
| dtantsur | yeah... | 16:19 |
| dtantsur | ++ | 16:19 |
| * iurygregory iurygregory to talk with stevebaker[m] about privsep on inspector | 16:20 | |
| iurygregory | #action iurygregory to talk with stevebaker[m] about privsep on inspector | 16:20 |
| * iurygregory was using / instead of # <facepalm> | 16:20 | |
| iurygregory | we have updates on the topics, moving on | 16:21 |
| iurygregory | #topic Priorities for the coming week | 16:21 |
| iurygregory | #link https://review.opendev.org/q/hashtag:ironic-week-prio+status:open | 16:21 |
| dtantsur | I have some stuff to suggest! | 16:22 |
| iurygregory | any patches to add to our list? \o/ | 16:22 |
| iurygregory | dtantsur, go ahead :D | 16:22 |
| dtantsur | https://review.opendev.org/c/openstack/ironic/+/821219 enabled foreign keys for sqlite | 16:22 |
| dtantsur | https://review.opendev.org/c/openstack/ironic/+/820909 somewhat simplifies enabled_***_interfaces | 16:22 |
| rpittau | I added https://review.opendev.org/c/openstack/bifrost/+/819058 | 16:22 |
| ajya | can these be added - https://review.opendev.org/c/openstack/ironic/+/822781 https://review.opendev.org/c/openstack/ironic/+/823701/ ? | 16:23 |
| dtantsur | then we have bifrost combined ironic chain: https://review.opendev.org/c/openstack/bifrost/+/819649 https://review.opendev.org/c/openstack/bifrost/+/819640 https://review.opendev.org/c/openstack/bifrost/+/820390 https://review.opendev.org/c/openstack/bifrost/+/820479 | 16:23 |
| dtantsur | finally https://review.opendev.org/c/openstack/ironic/+/822329 allows web servers to disable our caching | 16:23 |
| dtantsur | any ideas/objections? | 16:24 |
| TheJulia | no objections here | 16:24 |
| TheJulia | I'm surprised we've not hit caching issues before | 16:24 |
| TheJulia | then again I bet folks tend to just use new file names or ID's | 16:24 |
| dtantsur | I don't think it's common for people to have images generated on fly | 16:25 |
| dtantsur | we may be the only ones :) | 16:25 |
| iurygregory | no objections from me also, ajya dtantsur feel free to add the hashtag =) | 16:25 |
| iurygregory | dtantsur, probably because we like to dangerous things :D | 16:26 |
| dtantsur | exactly :) | 16:26 |
| dtantsur | TheJulia: we also respect Last-Modified already | 16:26 |
| TheJulia | heh | 16:26 |
| TheJulia | dtantsur: true,, and that does address a huge chunk of it | 16:26 |
| dtantsur | but not Cache-Control: no-store | 16:27 |
| TheJulia | as long as it gets updated | 16:27 |
| ajya | iurygregory: thanks, done | 16:27 |
| TheJulia | although, I do remember we had someone who had a bogus last-modified from a server once | 16:27 |
| TheJulia | 4-ish years ago | 16:27 |
| dtantsur | sweeet | 16:27 |
| TheJulia | Anyway, onward? | 16:28 |
| dtantsur | yep | 16:28 |
| iurygregory | cool \o/ | 16:29 |
| iurygregory | #topic Baremetal SIG | 16:29 |
| iurygregory | #link https://etherpad.opendev.org/p/bare-metal-sig | 16:29 |
| iurygregory | arne_wiebalck, anything you would like to say? | 16:30 |
| arne_wiebalck | TheJulia: summarised the SIG's activities for the foundation's annual report. | 16:30 |
| arne_wiebalck | thanks TheJulia | 16:30 |
| TheJulia | Happy to help! | 16:30 |
| TheJulia | I'd love for us to somehow get some operators talking about size and scope | 16:30 |
| arne_wiebalck | For meetings, we have a tentative show-and-tell of an Ironic deployment scheduled for March. | 16:31 |
| TheJulia | granted, difficult | 16:31 |
| TheJulia | well | 16:31 |
| TheJulia | that could work too :) | 16:31 |
| arne_wiebalck | We have a free slot in February if someone feels like it :) | 16:31 |
| arne_wiebalck | There are some potential topics on the ethepad. | 16:31 |
| arne_wiebalck | *etherpad | 16:32 |
| arne_wiebalck | Unless someone comes forward, I will try to find a volunteer :-D | 16:32 |
| arne_wiebalck | Oh, and no meeting in January we said. | 16:33 |
| TheJulia | we should do a thing on scaling ironic | 16:33 |
| arne_wiebalck | That is it, I think. | 16:33 |
| TheJulia | NobodyCam: hint hint | 16:33 |
| NobodyCam | yawn, stretch Good Morning Ironic'ers | 16:34 |
| iurygregory | good morning NobodyCam :D | 16:34 |
| arne_wiebalck | hey NobodyCam o/ | 16:34 |
| NobodyCam | ++ on Scaling great topic | 16:34 |
| NobodyCam | Morning arne_wiebalck iurygregory, TheJulia :) o/ | 16:35 |
| TheJulia | It would be a good opportunity to talk about common pain points, ways to kind of route around them, and then kind of what does an ideal layout look like | 16:35 |
| TheJulia | And I think we could do a deck, if folks want to collaborate I can start it I guess and shoot for febuary ? | 16:35 |
| NobodyCam | yea | 16:36 |
| arne_wiebalck | sure | 16:36 |
| iurygregory | \o/ | 16:36 |
| iurygregory | great news :D | 16:36 |
| arne_wiebalck | we also have https://techblog.web.cern.ch/techblog/post/conductor-groups/ | 16:36 |
| arne_wiebalck | (scaling Ironic) which may give some ideas | 16:36 |
| NobodyCam | +++ | 16:36 |
| arne_wiebalck | deal then :), thanks! | 16:37 |
| arne_wiebalck | iurygregory: back to you I guess | 16:37 |
| iurygregory | arne_wiebalck, thank you! | 16:37 |
| iurygregory | #topic RFE review | 16:37 |
| iurygregory | anyone has an RFE that would like to discuss? | 16:37 |
| TheJulia | I haven't looked at RFE's or even bugs in storyboard recently due to everything going on | 16:39 |
| iurygregory | no worries, moving on :D | 16:39 |
| TheJulia | mainly bugs | 16:39 |
| iurygregory | #topic Open discussion | 16:39 |
| dtantsur | yeah, me neither | 16:39 |
| TheJulia | dtantsur: my most favorite one is what happens when the bmc disappears mid-authentication refresh | 16:40 |
| iurygregory | anything we should discuss? | 16:40 |
| dtantsur | TheJulia: sounds like a horror story | 16:40 |
| TheJulia | dtantsur: 4 bugs and counting wrapped into one patch :( | 16:41 |
| * TheJulia looks at the cats trying to remove her window blinds | 16:41 | |
| TheJulia | I've got nothing, besides we shoudl do a midcycle soon | 16:42 |
| rpittau | ++ | 16:43 |
| iurygregory | maybe we can start with a doodle? :D | 16:43 |
| iurygregory | who is interested in participating in the midcycle? | 16:43 |
| iurygregory | o/ | 16:43 |
| NobodyCam | I'd like to put an official happy new year in the record | 16:43 |
| TheJulia | iurygregory: o/ | 16:43 |
| * arne_wiebalck realises the mid cycle at CERN is almost 2 years ago already | 16:43 | |
| rpittau | arne_wiebalck: :/ | 16:44 |
| dtantsur | :( | 16:44 |
| iurygregory | ='( | 16:44 |
| dtantsur | c'mon, I'm not paying for the internet to get upset | 16:44 |
| TheJulia | arne_wiebalck: I was thinking of that this morning | 16:44 |
| arne_wiebalck | dtantsur: heh | 16:44 |
| iurygregory | dtantsur, you can blame arne_wiebalck this time | 16:44 |
| arne_wiebalck | once things are calm again, I guess we are happy to welcome you again :) | 16:44 |
| iurygregory | \o/ | 16:45 |
| rpittau | \o/ | 16:45 |
| * arne_wiebalck is trying to get dtantsur value for internet money | 16:45 | |
| dtantsur | I usually extract some suffering out of it | 16:45 |
| rpittau | I'm not sure we want to go down the internet-suffering path now! | 16:46 |
| iurygregory | rpittau++ | 16:46 |
| dtantsur | that's literally our profession :) | 16:46 |
| rpittau | :D | 16:46 |
| TheJulia | oh my | 16:47 |
| TheJulia | I suspect this meeting is Fin | 16:47 |
| iurygregory | and it's monday... so this make things worse | 16:47 |
| * iurygregory is trying to understand Fin... | 16:48 | |
| TheJulia | iurygregory: done | 16:48 |
| TheJulia | iurygregory: end, etc | 16:48 |
| dtantsur | I hope everyone who celebrates new year has enjoyed the holidays! | 16:48 |
| iurygregory | TheJulia, oh :D I was trying to find the word in english | 16:49 |
| iurygregory | so, who is going to send the email about the midcycle? | 16:49 |
| TheJulia | dtantsur: I need like another week off, can we somehow make the break longer? | 16:50 |
| dtantsur | +++ | 16:50 |
| dtantsur | I haven't had enough snow | 16:50 |
| TheJulia | snow++ | 16:50 |
| TheJulia | I might be able to go up the tramway, put some in a box, but I don't think it would make it to germany as snow :) | 16:51 |
| dtantsur | nope :) | 16:51 |
| dtantsur | at least so far no snow has made it to Düsseldorf without turning into a heavy rain | 16:51 |
| iurygregory | snow++ | 16:52 |
| TheJulia | I just had a depressing throught | 16:52 |
| TheJulia | about snow in general, and climate change | 16:52 |
| iurygregory | =( | 16:53 |
| dtantsur | there can be depressing thoughts there indeed | 16:53 |
| iurygregory | yeah =( | 16:54 |
| TheJulia | anyway, shall we end today's meeting so we can proceed with taking over the world | 16:54 |
| iurygregory | so, I think we are good with our meeting \o/ | 16:54 |
| iurygregory | yeah :D | 16:55 |
| iurygregory | I will run the next meeting | 16:55 |
| iurygregory | Thanks everyone! | 16:55 |
| rpittau | thanks iurygregory | 16:55 |
| TheJulia | cool, thanks everyone! | 16:55 |
| TheJulia | #endmeeting | 16:55 |
| opendevmeet | Meeting ended Mon Jan 10 16:55:26 2022 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:55 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/ironic/2022/ironic.2022-01-10-16.01.html | 16:55 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/ironic/2022/ironic.2022-01-10-16.01.txt | 16:55 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/ironic/2022/ironic.2022-01-10-16.01.log.html | 16:55 |
| * dtantsur is now eyeing places in Germany where you can get some snow experience without meeting too many people in the process | 16:56 | |
| rpittau | bye everyone, see you tomorrow o/ | 16:58 |
| TheJulia | dtantsur: as in moving? | 17:02 |
| dtantsur | TheJulia: well, in the near future visiting. moving is not off the tables as well, but highly depends on my wife's future career | 17:03 |
| TheJulia | Yeah | 17:03 |
| TheJulia | I honestly wish we lived up one of the local mountains | 17:04 |
| dtantsur | yeah | 17:04 |
| TheJulia | *but* getting up/down them can be a pain | 17:04 |
| dtantsur | I sometimes think we should move south, closer to the alps | 17:04 |
| dtantsur | Düsseldorf is nice, but the weather in winter is unbearable. and the landscape is flat as a table. | 17:05 |
| dtantsur | TheJulia: assuming you really need to get up/down :) | 17:06 |
| TheJulia | dtantsur: true... and the cold weather does cause my right knee to ache | 17:06 |
| dtantsur | :( | 17:07 |
| TheJulia | I remember you said you would be in Düsseldorf for about as long as you had said, but that was also before pandemic | 17:07 |
| dtantsur | yeah.. anyway, it's not up to me to decide at this point | 17:08 |
| TheJulia | yeah | 17:09 |
| dtantsur | I can work from home, but I'd highly prefer that my wife does not :D | 17:09 |
| TheJulia | :) | 17:09 |
| dtantsur | (she deals with antibiotic-resistant bacteria, among other things) | 17:09 |
| TheJulia | It can stress a relationship quite a bit, and even then changes can create new/additional stressors | 17:09 |
| TheJulia | and... yeah | 17:09 |
| opendevreview | Merged openstack/ironic master: Add dhcp options for each ip_version once https://review.opendev.org/c/openstack/ironic/+/823817 | 17:23 |
| opendevreview | Merged openstack/ironic master: Make bootloader installation failures fatal for whole disk images https://review.opendev.org/c/openstack/ironic/+/823723 | 17:23 |
| arne_wiebalck | hmm, ^^ didn't we rely on some flexibility during failures, e.g. for stream8 on top of s/w RAID? | 17:26 |
| opendevreview | Harald Jensås proposed openstack/ironic stable/xena: Add dhcp options for each ip_version once https://review.opendev.org/c/openstack/ironic/+/824012 | 17:26 |
| opendevreview | Harald Jensås proposed openstack/ironic stable/wallaby: Add dhcp options for each ip_version once https://review.opendev.org/c/openstack/ironic/+/824013 | 17:27 |
| dtantsur | arne_wiebalck: not sure. at least we don't seem to have left a common about it | 17:27 |
| dtantsur | hjensas_: bugfix/19.0 as well please | 17:28 |
| arne_wiebalck | dtantsur: I think s/w RAID on UEFI will break since it uses grub2-install which fails due to lack of secure boot support on stream8 | 17:28 |
| dtantsur | arne_wiebalck: I think we skip grub-install on UEFI, no? | 17:28 |
| opendevreview | Harald Jensås proposed openstack/ironic bugfix/19.0: Add dhcp options for each ip_version once https://review.opendev.org/c/openstack/ironic/+/824014 | 17:28 |
| arne_wiebalck | dtantsur: not on s/w RAID | 17:29 |
| dtantsur | cc mgoddard | 17:29 |
| arne_wiebalck | dtantsur: one more incentive to move to efibootmgr for s7w RAID | 17:29 |
| arne_wiebalck | *s/w RAID | 17:29 |
| hjensas_ | dtantsur: done, looks like the issue is in bugfix/18.x as well. Do we need to backport there as well? | 17:30 |
| dtantsur | is there a reason why we don't do it already? | 17:30 |
| arne_wiebalck | dtantsur: we carry this technical debt with us for a while now :-S | 17:30 |
| dtantsur | hjensas_: ideally, but the CI is pretty broken there now | 17:30 |
| arne_wiebalck | dtantsur: the "why" question was for me? | 17:31 |
| hjensas_ | dtantsur: ok, I'll propose the changes ... | 17:31 |
| dtantsur | arne_wiebalck: yep | 17:31 |
| opendevreview | Harald Jensås proposed openstack/ironic bugfix/18.1: Add dhcp options for each ip_version once https://review.opendev.org/c/openstack/ironic/+/824015 | 17:31 |
| arne_wiebalck | dtantsur: no reason, we just need to test and the code is quite convoluted | 17:31 |
| opendevreview | Harald Jensås proposed openstack/ironic bugfix/18.0: Add dhcp options for each ip_version once https://review.opendev.org/c/openstack/ironic/+/824016 | 17:31 |
| arne_wiebalck | dtantsur: I think stevebaker[m] had a look some months ago and saw no reason to not move | 17:31 |
| dtantsur | is it something either of you could take a look into? | 17:32 |
| dtantsur | I'd rather not rely on silencing errors forever :) | 17:32 |
| mgoddard | arne_wiebalck: oops, didn't mean to break SW RAID | 17:32 |
| arne_wiebalck | dtantsur: totally agree | 17:32 |
| mgoddard | could we go for a more specific condition for now? | 17:33 |
| arne_wiebalck | mgoddard: didn't assume this :-D | 17:33 |
| opendevreview | Merged openstack/ironic master: Use driver_internal_info methods for driver utils https://review.opendev.org/c/openstack/ironic/+/818505 | 17:33 |
| arne_wiebalck | mgoddard: and not sure you did, actually | 17:33 |
| mgoddard | e.g. wholedisk + UEFI + SW RAID | 17:33 |
| arne_wiebalck | right | 17:33 |
| arne_wiebalck | on stream8 | 17:33 |
| dtantsur | arne_wiebalck: would it be possible for you to test it once more? | 17:33 |
| arne_wiebalck | dtantsur: the just merged pacth? | 17:33 |
| dtantsur | yeah | 17:33 |
| arne_wiebalck | yes, will try some time later this week | 17:34 |
| * arne_wiebalck is upgrading Ironic tomorrow | 17:34 | |
| dtantsur | I'd expect https://opendev.org/openstack/ironic-python-agent/src/branch/master/ironic_python_agent/extensions/image.py#L791-L794 to always apply to whole disk images | 17:34 |
| dtantsur | partition images may be an issue, of course | 17:34 |
| dtantsur | anyway, need to go now, see you tomorrow | 17:36 |
| dtantsur | o/ | 17:36 |
| arne_wiebalck | o/ | 17:36 |
| arne_wiebalck | dtantsur: right, but I think we require the root fs UUID to be provided | 17:38 |
| opendevreview | Verification of a change to openstack/ironic master failed: Use driver_internal_info methods for redfish driver https://review.opendev.org/c/openstack/ironic/+/818508 | 17:41 |
| arne_wiebalck | bye everyone o/ | 18:12 |
| stevebaker[m] | good morning | 19:36 |
| TheJulia | good morning stevebaker[m] | 19:37 |
| stevebaker[m] | iurygregory, TheJulia : I spent yesterday getting my head around root/rootwrap/privsep for IPA/ironic/inspector. I think I know the shape of it all now | 19:38 |
| TheJulia | stevebaker[m]: \o/ | 19:42 |
| stevebaker[m] | as far as inspector goes, its just the dnsmasq restart thing. I think it can just be converted to privsep, and privsep will only launch a process when a dnsmasq stop/start is requested | 19:45 |
| stevebaker[m] | TheJulia: as for ironic, I'm going to convert this to use pycdlib, so no mount will be required to extract files from the iso https://opendev.org/openstack/ironic/src/branch/master/ironic/common/images.py#L603 | 19:47 |
| stevebaker[m] | TheJulia: but create_vfat_image will require privsep I think. Unless you can think of an alternative (some other mount-without-root mechanism, delete the function) https://opendev.org/openstack/ironic/src/branch/master/ironic/common/images.py#L84 | 19:50 |
| opendevreview | Verification of a change to openstack/ironic master failed: Use driver_internal_info methods for redfish driver https://review.opendev.org/c/openstack/ironic/+/818508 | 19:56 |
| TheJulia | I thought there was a python library out there to interact with fat filesystems directly | 20:08 |
| stevebaker[m] | TheJulia: yeah I found https://pypi.org/project/pyfatfs/ but the readme scared me off "It currently is in an early development state and expected to be unstable and contain bugs." Its not in openstack requirements, I haven't checked rpm packaging | 20:12 |
| TheJulia | Yeah, that would scare me off too | 20:20 |
| TheJulia | stevebaker[m]: I *wonder* if I was thinking of mtools | 20:21 |
| TheJulia | and my browser doesn't support ftp.... | 20:22 |
| TheJulia | what has the world come to | 20:22 |
| iurygregory | stevebaker[m], hey o/ | 20:28 |
| iurygregory | stevebaker[m], for ironic we are tracking things in https://storyboard.openstack.org/#!/story/2009704 | 20:29 |
| iurygregory | TheJulia, the idea is to convert to mtools | 20:30 |
| iurygregory | and the iso part we will be using pycdlib | 20:31 |
| iurygregory | so if I got right we only need privsep in the case dnsmasq? dtantsur mentioned that iptables would require sudo, so if all drivers in inspector will need privsep we won't make it optional | 20:34 |
| stevebaker[m] | iurygregory, dtantsur : I agree with all that, but please note I'm proposing to *add* a new privsep function to copy root-readable files out of /boot/efi/EFI/<redhat> https://review.opendev.org/c/openstack/ironic/+/821363 | 21:03 |
| iurygregory | stevebaker[m], any more context you can provide about it? no other way to do this without privsep? =) | 21:16 |
| stevebaker[m] | iurygregory: not that I can see. /boot/efi is mount root read only, and rpm packaging installs files as root read only. My proposal is that ironic carries this complexity so that all of the configuration tools can do less, and things just work | 21:19 |
| iurygregory | stevebaker[m], got it =) can you create a story for this when possible? | 21:20 |
| stevebaker[m] | iurygregory: the end result of that work is working grub network boot https://review.opendev.org/c/openstack/bifrost/+/807220 | 21:20 |
| stevebaker[m] | sure thing | 21:20 |
| stevebaker[m] | TheJulia: ok, mtools looks like the business, aside from terrible gnu docs | 21:49 |
| opendevreview | Verification of a change to openstack/ironic master failed: Ensure 'port' is up2date after binding:host_id https://review.opendev.org/c/openstack/ironic/+/823811 | 22:36 |
| opendevreview | Steve Baker proposed openstack/ironic master: Use mtools mcopy in create_vfat_image https://review.opendev.org/c/openstack/ironic/+/824075 | 22:37 |
| opendevreview | Steve Baker proposed openstack/ironic master: Use mtools mcopy in create_vfat_image https://review.opendev.org/c/openstack/ironic/+/824075 | 22:41 |
| janders | good morning Ironic o/ | 23:46 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!