| opendevreview | Jay Faulkner proposed openstack/ironic master: API support for CRUD node.shard https://review.opendev.org/c/openstack/ironic/+/866235 | 00:41 |
|---|---|---|
| vanou | good morning ironic | 02:51 |
| arne_wiebalck | Good morning vanou and Ironic! | 07:31 |
| rpittau | good morning ironic! o/ | 08:24 |
| vanou | Hi arne_wiebalck rpittau | 08:58 |
| kubajj | Good morning vanou, rpittau, arne_wiebalck, and Ironic! | 09:17 |
| rpittau | JayF: re: virtualpdu, thanks! Did you find out anything? Otherwise I'm still up to send an email to the members of the core group | 09:50 |
| rpittau | when anyone has a moment, plese quick review https://review.opendev.org/c/openstack/metalsmith/+/869356 thanks! | 10:06 |
| opendevreview | Merged openstack/ironic master: [DOC] Add entry regarding cleaning+raid https://review.opendev.org/c/openstack/ironic/+/867674 | 10:28 |
| opendevreview | Merged openstack/networking-generic-switch master: Document ngs_trunk_ports and ngs_physical_networks https://review.opendev.org/c/openstack/networking-generic-switch/+/868313 | 10:33 |
| rpittau | I see there are 2 +2 for https://review.opendev.org/c/openstack/ironic/+/868521, anything else missing there? reviews appreciated :) | 10:34 |
| vanou | Hi kubajj | 10:40 |
| arne_wiebalck | hey rpittau kubajj o/ | 11:06 |
| rpittau | hey kubajj vanou arne_wiebalck :) | 11:10 |
| iurygregory | good morning Ironic o/ | 11:18 |
| samuelkunkel[m] | Hello Ironic | 11:32 |
| samuelkunkel[m] | I am facing a weird issue, maybe someone of you can give me a hint. I currently test redfish to replace the ipmitool interface for our newer server. | 11:33 |
| samuelkunkel[m] | Currently I dont have the ca present for those nodes so I try to ignore the cert validation. (setting baremetal node set --driver-info redfish_verify_ca=False) but it seems like the client automatically converts this to a string. | 11:33 |
| samuelkunkel[m] | Resulting in: "redfish_verify_ca": "False" | 11:33 |
| samuelkunkel[m] | and therefore failing with the error: [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: self signed certificate | 11:33 |
| samuelkunkel[m] | I will try it by just using the the api call directly, not wrapped by the openstack client... but maybe some of you saw that? | 11:33 |
| samuelkunkel[m] | * I will try it now by using the the api call directly instead of the openstackclient. but maybe some of you saw that? | 11:34 |
| dtantsur | samuelkunkel[m]: I have a feeling most of our code actually handles booleans as strings. Maybe not redfish_verify_ca. | 11:41 |
| samuelkunkel[m] | In this case the code fetches it, defaults to bool True & if its a string it validates it for the path. So verify_ca = driver_info.get('redfish_verify_ca', True) would return "False" which the next check if isinstance(verify_ca, str) would then be true as "False" is a string, correct? | 11:43 |
| dtantsur | we normally use something like strutils.boot_from_string | 11:43 |
| dtantsur | (I don't have the code in front of me) | 11:44 |
| samuelkunkel[m] | going to try setting it via the api directly | 11:55 |
| kubajj | dtantsur: could I get a re-review of https://review.opendev.org/c/openstack/ironic/+/866876 please? | 12:27 |
| dtantsur | kubajj: lgtm, congrats! | 12:38 |
| kubajj | dtantsur: thanks. Sorry JayF | 12:39 |
| dtantsur | the winner takes it aaaaaaallll | 12:39 |
| opendevreview | Merged openstack/metalsmith master: Remove not used old job https://review.opendev.org/c/openstack/metalsmith/+/869356 | 12:47 |
| opendevreview | Merged openstack/ironic master: API for node inventory https://review.opendev.org/c/openstack/ironic/+/866876 | 14:24 |
| opendevreview | Vanou Ishii proposed openstack/ironic master: [iRMC] identify BMC firmware version https://review.opendev.org/c/openstack/ironic/+/865074 | 14:37 |
| opendevreview | Vanou Ishii proposed openstack/ironic master: [iRMC] Handle IPMI incompatibility in iRMC S6 2.x https://review.opendev.org/c/openstack/ironic/+/865075 | 14:37 |
| vanou | TheJulia JayF: I update patch based on TheJulia's advice. If possible, please review these https://review.opendev.org/c/openstack/ironic/+/865074 https://review.opendev.org/c/openstack/ironic/+/865075 | 14:39 |
| JayF | good morning, we have a meeting nowish | 15:00 |
| JayF | #startmeeting | 15:00 |
| opendevmeet | JayF: Error: A meeting name is required, e.g., '#startmeeting Marketing Committee' | 15:00 |
| JayF | #startmeeting ironic | 15:00 |
| opendevmeet | Meeting started Mon Jan 16 15:00:53 2023 UTC and is due to finish in 60 minutes. The chair is JayF. 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 |
| iurygregory | o/ | 15:00 |
| matfechner | o/ | 15:00 |
| JayF | Good morning, who all is here? | 15:01 |
| dtantsur | o/ | 15:01 |
| JayF | I'll note today is a holiday in the US; Martin Luther King Jr day today | 15:01 |
| JayF | so I expect us to have less attendance than usual | 15:01 |
| JayF | #topic Announcements/Reminder | 15:02 |
| rpittau | o/ | 15:02 |
| JayF | Standing reminder to review patches tagged ironic-week-prio and to hashtag any patches ready for review with ironic-week-prio: https://tinyurl.com/ironic-weekly-prio-dash | 15:02 |
| JayF | #topic Review action items from previous meeting | 15:02 |
| JayF | iurygregory was going to create an etherpad for virtual ptg | 15:03 |
| JayF | iurygregory: if you've done that, share it here and I'll link the URL in the minutes | 15:04 |
| JayF | otherwise that's OK; we can tackle it this week | 15:04 |
| iurygregory | sharing now | 15:05 |
| iurygregory | https://etherpad.opendev.org/p/ironic-bobcat-ptg | 15:05 |
| JayF | #link https://etherpad.opendev.org/p/ironic-bobcat-ptg | 15:06 |
| JayF | I'll look over that sometime this week and help flesh any of it out | 15:06 |
| JayF | #topic Review ironic CI status | 15:06 |
| JayF | It's been a mess. Big thanks for rpittau for trying to stay on top of it | 15:06 |
| rpittau | "trying" is the right word! | 15:07 |
| JayF | I was just thinking; I'll try and spin wheels and often you'll try and gain traction | 15:07 |
| JayF | our CI is hugely complex :D | 15:07 |
| rpittau | :D | 15:07 |
| JayF | one good note about this; we're mostly done with zuul-config-errors | 15:07 |
| JayF | after the bugfix branch retirements last week | 15:07 |
| rpittau | JayF: related to CI, we should probably talk about virtualpdu | 15:08 |
| JayF | oh yeah, that's not on agenda though | 15:08 |
| JayF | I'll mark it for open discussion | 15:08 |
| rpittau | sure | 15:08 |
| JayF | One thing I'd like is if we cleaned up the last 3/4 zuul config errors this week | 15:08 |
| JayF | these are branches that never got the zuul queue: param removal fix | 15:09 |
| JayF | but are still new enough to be supported | 15:09 |
| JayF | I can't find the dang etherpads :( | 15:09 |
| JayF | ironic-inspector bugfix/10.12; ironic-inspector bugfix/10.7; IPE bugfix/2.1; IPE stable/train; python-ironicclient train | 15:11 |
| JayF | among others | 15:11 |
| JayF | I created a full list in an etherpad and promptly lost the URL to said etherpad | 15:12 |
| JayF | basically; go here and search "ironic" https://zuul.opendev.org/t/openstack/config-errors | 15:12 |
| JayF | if you wanna help with this | 15:12 |
| JayF | I'm not as sharp on fixing inspector and IPE so I really hope other folks are willing to take a look :D | 15:12 |
| JayF | #topic 2023.1 Work in Progress | 15:12 |
| rpittau | I'll have a look | 15:12 |
| JayF | I'd like, next week, to do a full review of this with folks... | 15:12 |
| JayF | we have a lot of stuff we said we wanted to do this cycle that I think has barely even started | 15:13 |
| JayF | it'll be a good input to the PTG planning to know what items are getting done | 15:13 |
| JayF | so I strongly encourage you that next week, if you're one of the champions for an item in the workstream for this cycle; please input a status update or be here at next meeting with something to put | 15:13 |
| JayF | just trying to see where we are before we go planning another six months | 15:14 |
| JayF | https://etherpad.opendev.org/p/IronicWorkstreams2023.1 here is the current status doc | 15:14 |
| JayF | and https://etherpad.opendev.org/p/IronicWorkstreams2023.1 | 15:14 |
| JayF | and https://specs.openstack.org/openstack/ironic-specs/priorities/2023-1-workitems.html is the spec we landed with the aspirational list | 15:14 |
| JayF | #link https://specs.openstack.org/openstack/ironic-specs/priorities/2023-1-workitems.html | 15:14 |
| JayF | #link https://etherpad.opendev.org/p/IronicWorkstreams2023.1 | 15:15 |
| JayF | Any comments before I move on? | 15:15 |
| JayF | taking the silence as an implicit no | 15:16 |
| JayF | next topic | 15:16 |
| JayF | #action Future of Bugfix releases punted to next meeting; needs to be on a day that's not a US holiday so we have more contributors here | 15:17 |
| JayF | #topic open discussion | 15:17 |
| JayF | rpittau: wanted to mention virtualpdu | 15:17 |
| rpittau | yep | 15:17 |
| rpittau | ironic CI depends on it but the development has stopped since a while | 15:18 |
| rpittau | now it doesn't work with python 3.10, meaning we can't move to jammy | 15:18 |
| rpittau | I put together some patches to fix that but we probably need to contact the cores | 15:18 |
| dtantsur | any luck emailing them? | 15:19 |
| rpittau | I haven't emailed them yet as I saw JayF mentioning he wanted to check on another side | 15:19 |
| JayF | I was volunteering to take it up with TC | 15:19 |
| JayF | if Cores were not responsive | 15:19 |
| rpittau | ok, so I'll email them first | 15:20 |
| JayF | and that this; if we use it in our CI; might make sense to be an openstack project under our governance if it's not currently in a good maintenance spot | 15:20 |
| rpittau | I'll do it right after the meeting | 15:20 |
| JayF | #action rpittau to contact cores of virtualpdu about his changes | 15:20 |
| JayF | rpittau: I suggest you also offer to be a core; or for ironic-core to be added | 15:20 |
| JayF | rpittau: just very awkward for our CI to depend on packages that are not super-actively maintained | 15:20 |
| rpittau | well it was working until focal, it went unnoticed until we tried to go to jammy | 15:21 |
| rpittau | but I agree, we should monitor dependencies more closely | 15:21 |
| JayF | I mean, who else is going to maintain a fake-PDU for CI usage but us? That's sorta our thing now LOL | 15:22 |
| rpittau | :) | 15:22 |
| JayF | If there's nothing else; I'm going to call the meeting :) | 15:23 |
| JayF | Alright, I think we're done. | 15:24 |
| JayF | #endmeeting | 15:24 |
| opendevmeet | Meeting ended Mon Jan 16 15:24:07 2023 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:24 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/ironic/2023/ironic.2023-01-16-15.00.html | 15:24 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/ironic/2023/ironic.2023-01-16-15.00.txt | 15:24 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/ironic/2023/ironic.2023-01-16-15.00.log.html | 15:24 |
| JayF | A heads up about me personally: I'm only in off-and-on today because of the holiday | 15:24 |
| dtantsur | JayF: I'd like to start working on the baremetal SIG meetup content, maybe some slides to guide us | 15:24 |
| JayF | So if you need me, ping me, but understand there will be high latency :D | 15:24 |
| dtantsur | I'll prototype something, will ping you once you're back | 15:24 |
| JayF | dtantsur: I think that's an excellent idea! | 15:24 |
| dtantsur | JayF: will you be up to have 2-5 minites of Ironic news | 15:24 |
| dtantsur | * minutes (ffs, I tried 4 times to type it and still typed wrong...) | 15:25 |
| JayF | dtantsur: if you do it on google slides or w/e, can you share to jay at gr-oss dot io | 15:25 |
| JayF | lol | 15:25 |
| dtantsur | I will | 15:25 |
| JayF | minutes :) | 15:25 |
| JayF | I am that way about some words, too | 15:25 |
| JayF | maintenance took me literally a decade of adult life to spell without stress | 15:25 |
| JayF | and english is my first language | 15:25 |
| JayF | dtantsur: I can riff for 2-5 minutes on anything. But in reality, I probably don't have that much "Ironic news" lol | 15:26 |
| dtantsur | :D | 15:31 |
| opendevreview | Riccardo Pittau proposed openstack/ironic-inspector bugfix/10.7: Fix CI https://review.opendev.org/c/openstack/ironic-inspector/+/870579 | 15:34 |
| opendevreview | Riccardo Pittau proposed openstack/ironic-inspector bugfix/10.12: Fix CI https://review.opendev.org/c/openstack/ironic-inspector/+/870581 | 15:35 |
| rpittau | JayF: should be all the zuul config errors list -> https://etherpad.opendev.org/p/zuul-config-errors | 15:39 |
| JayF | Nice! I love etherpad, but I wish there was a way to index all the ones you had created | 15:40 |
| JayF | Because I already aggregated those and lost the darn URL | 15:40 |
| rpittau | JayF, dtantsur, I sent the email to the virtualpdu cores, added both of you in CC | 15:56 |
| dtantsur | thnx! | 16:02 |
| dtantsur | arne_wiebalck: hey, do we have any baremetal SIG related artwork (to use in slides)? | 16:18 |
| arne_wiebalck | I don't think so. | 16:18 |
| dtantsur | k | 16:18 |
| dtantsur | I'm a bit hesitant to use pixie boots since we don't put the equality sign between ironic and baremetal ISG (or do we?) | 16:19 |
| arne_wiebalck | I can share the slide deck we had in Berlin for the forum session if that is helpful? | 16:19 |
| dtantsur | possibly, I don't remember how it looked :) | 16:19 |
| rpittau | good night! o/ | 16:29 |
| opendevreview | Jakub Jelinek proposed openstack/ironic master: Create [inventory] https://review.opendev.org/c/openstack/ironic/+/870470 | 16:36 |
| dtantsur | JayF: shared | 16:43 |
| kubajj | How do I rebase something without ruining the dependencies? I still don't understand gerrit. | 16:44 |
| dtantsur | kubajj: what exactly are you trying to do? | 16:46 |
| kubajj | So I had the API change and created the Create [inventory] (3 messages above this one) as a dependency to it yesterday. However, it wanted me to rebase it, so I did and now it says (not current) in the relation chain | 16:47 |
| dtantsur | kubajj: have you rebased it against git master? if yes, it's probably fine | 16:51 |
| dtantsur | where do you see "not current"? | 16:51 |
| kubajj | https://review.opendev.org/c/openstack/ironic/+/866876 in the relation chain | 16:52 |
| kubajj | dtantsur: I think I did | 16:52 |
| dtantsur | kubajj: right. So, the child of the API change is the older (1st) revision of your new patch. And the 2nd revision is no longer a direct child of the API change. | 16:52 |
| dtantsur | Hence the text. You don't need to worry in thsi case. | 16:52 |
| samuelkunkel[m] | TheJulia: I can reproduce the issue about redfish not being able to change the boot device on an idrac 6.10 on a DELL R750 (G15). Here are the logs https://paste.debian.net/plain/1267425. If you need anything more - let me know. | 16:54 |
| kubajj | dtantsur: which part of the logic should I move to inspect_utils (ref: https://review.opendev.org/c/openstack/ironic/+/866876/comments/22dfe9a1_6f6c34b5 )? | 17:02 |
| dtantsur | kubajj: the 'if store_data == ' block | 17:09 |
| dtantsur | (as well as storing the data) | 17:09 |
| kubajj | dtantsur: I see. Thanks | 17:10 |
| TheJulia | samuelkunkel[m]: thanks! The hash character is super suspect. Out of curiosity, is that in debug logging mode? If not, any chance of getting a debug logging output? Also, what firmware version did you upgrade from? | 17:15 |
| samuelkunkel[m] | You refer to debug logging mode for the conductor? | 17:16 |
| TheJulia | samuelkunkel[m]: yes please | 17:17 |
| samuelkunkel[m] | Should not be a big issue, need to check some other stuff but I can follow up later or tomorrow for the debug logging output. | 17:24 |
| samuelkunkel[m] | What firmware version are you refering to? | 17:25 |
| samuelkunkel[m] | for this case I updated the firmware for the idrac from 5.10 to 6.10 | 17:25 |
| samuelkunkel[m] | If this is what you are refering to. | 17:25 |
| kubajj | dtantsur: I assume I should also move the tests to test_inspect_utils.py, right? | 17:32 |
| kubajj | TheJulia: could I get a review of https://review.opendev.org/c/openstack/ironic/+/870470 please? It's just some wee refactoring. | 19:17 |
| kubajj | Also, what is this HPE Proliant? | 19:17 |
| stevebaker[m] | good morning | 19:44 |
| samuelkunkel[m] | TheJulia: https://paste.debian.net/plain/1267445 here you go :) | 20:10 |
| samuelkunkel[m] | s/1267445/1267446/ | 20:14 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!