| *** dmellado030 is now known as dmellado03 | 02:00 | |
| opendevreview | Jacob Anders proposed openstack/ironic master: Add states.SERVICING and SERVICEWAIT to _FASTTRACK_HEARTBEAT_ALLOWED https://review.opendev.org/c/openstack/ironic/+/915389 | 04:04 |
|---|---|---|
| opendevreview | Jacob Anders proposed openstack/sushy-oem-idrac master: [WIP] Wait for BIOS configuration job to complete https://review.opendev.org/c/openstack/sushy-oem-idrac/+/915092 | 04:54 |
| rpittau | good morning ironic! o/ | 07:12 |
| rpittau | stevebaker[m]: great finding on the centos image, I had a suspect as the job started failing after the latest update, but didn't think about that change specifically! | 07:13 |
| stevebaker[m] | Looks like there is a different failure on the legacy bios job, i think we should disable the partition image test if we want that job to be green soon | 07:15 |
| rpittau | stevebaker[m]: the legacy job is failing because of a known issue with tinycore, we've disabled it in ironic already | 07:18 |
| rpittau | 'm very tempted to just get rid of it | 07:18 |
| stevebaker[m] | Ah ok, i will not worry about that then :) | 07:19 |
| opendevreview | Riccardo Pittau proposed openstack/metalsmith master: Mark legacy job as non-voting for the time being https://review.opendev.org/c/openstack/metalsmith/+/916040 | 07:22 |
| rpittau | just moving it out of the way for now ^ | 07:26 |
| opendevreview | Merged openstack/ironic stable/2024.1: Remove deprecation warning by setting schema https://review.opendev.org/c/openstack/ironic/+/915953 | 09:15 |
| dtantsur | iurygregory, diablo_rojo, I'm happy to co-mentor and can come up with any number of tasks with any complexity in in nearly any area of ironic :) | 09:26 |
| rpittau | dtantsur, iurygregory, diablo_rojo, likewise, I will be available to co-mentor and help in general :) | 09:28 |
| dtantsur | JayF, cid: ideally, we need a release note and new unit tests for https://review.opendev.org/c/openstack/ironic/+/913793 because it adds a whole new format of microversion headers | 10:21 |
| masghar | diablo_rojo, iurygregory, dtantsur, rpittau: generally interested in being a co-mentor, would be happy to help however I can :) | 10:27 |
| dtantsur | The next step is to collect project ideas | 10:28 |
| opendevreview | Merged openstack/ironic master: Add states.SERVICING and SERVICEWAIT to _FASTTRACK_HEARTBEAT_ALLOWED https://review.opendev.org/c/openstack/ironic/+/915389 | 10:34 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic stable/2024.1: Add states.SERVICING and SERVICEWAIT to _FASTTRACK_HEARTBEAT_ALLOWED https://review.opendev.org/c/openstack/ironic/+/916085 | 10:36 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic stable/2023.2: Add states.SERVICING and SERVICEWAIT to _FASTTRACK_HEARTBEAT_ALLOWED https://review.opendev.org/c/openstack/ironic/+/916086 | 10:36 |
| opendevreview | Jacob Anders proposed openstack/ironic master: [WIP] Correct logging call in _heartbeat_service_wait method https://review.opendev.org/c/openstack/ironic/+/915394 | 11:51 |
| *** srelf__ is now known as ContinuitySR | 12:05 | |
| opendevreview | Jacob Anders proposed openstack/sushy-oem-idrac master: [WIP] Wait for BIOS configuration job to complete https://review.opendev.org/c/openstack/sushy-oem-idrac/+/915092 | 13:06 |
| TheJulia | good morning | 13:08 |
| iurygregory | dtantsur, rpittau masghar I will create a doc so we can add ideas o/ | 13:26 |
| iurygregory | good morning Ironic o/ | 13:26 |
| * iurygregory totally forgot about irc till now lol | 13:27 | |
| TheJulia | More coffee time?! | 13:34 |
| iurygregory | TheJulia, yes! | 13:42 |
| rpittau | one moka won't be enough at this point | 13:48 |
| opendevreview | Jacob Anders proposed openstack/ironic master: [WIP] Correct logging call in _heartbeat_service_wait method https://review.opendev.org/c/openstack/ironic/+/915394 | 14:03 |
| TheJulia | Jacob is up really late | 14:04 |
| TheJulia | Is anyone able to load http://rst.ninjs.org/ | 14:09 |
| JayF | No | 14:11 |
| iurygregory | nope | 14:17 |
| rpittau | unless it's a complete white page it does not load for me | 14:17 |
| dtantsur | neither here | 14:33 |
| TheJulia | pity | 14:40 |
| dtantsur | Don't worry, nothing works today | 14:46 |
| TheJulia | oh, well | 15:25 |
| cid | This is one of the changes we discussed on the ironic arm ci pairing session the other day. Currently have one core review: | 16:09 |
| cid | https://review.opendev.org/c/openstack/ironic/+/915934?usp=search | 16:09 |
| rpittau | thanks cid, I've approved it | 16:10 |
| cid | that was fast :D | 16:11 |
| rpittau | :) | 16:16 |
| rpittau | good night! o/ | 16:16 |
| TheJulia | goodnight rpittau | 16:16 |
| JayF | .o(should we install python-ironicclient in devstack so `baremetal $thing` works in addition to `openstack baremetal $thing`) | 16:19 |
| TheJulia | ... I thought we already did | 16:21 |
| JayF | not in the primary OS, at least | 16:21 |
| TheJulia | && the OSC command line stuffs sets any appropriate flags | 16:21 |
| TheJulia | but... hmm | 16:21 |
| JayF | I am very confused about why `openstack token issue` now gives me a multiline token | 16:25 |
| JayF | and how to put that into a curl request | 16:25 |
| * JayF trying to help cid debug his .json api change | 16:25 | |
| * TheJulia feels very confused | 16:32 | |
| JayF | TheJulia: we have a fernet token in keystone with a newline | 16:40 |
| JayF | TheJulia: and I'm down a 15 minute rabbithole trying to figure how to get it to work lol | 16:40 |
| TheJulia | We're pulling a token to call placement right? | 16:46 |
| TheJulia | $(openstack token issue) should hold the newline, and then pass the ouput in ${VAR_NAME} to prevent expansion?! | 16:47 |
| dtantsur | JayF: to answer your earlier question, python-ironicclient is required for both `baremetal $thing` and `openstack baremetal $thing` | 16:48 |
| JayF | TheJulia: I think it was picnic/pebkac/id10t | 16:48 |
| dtantsur | JayF: have you seen my comment about microversion-parse and a release note? | 16:49 |
| JayF | not yet | 16:50 |
| dtantsur | cid: around? | 16:50 |
| dtantsur | cid: https://review.opendev.org/c/openstack/ironic/+/913793 went in without a release note and unit tests. It would be fine if microversion-parse was *just* a replacement, but it actually adds a pretty important feature. | 16:51 |
| dtantsur | with your change, Ironic now should support microversion headers in the form of OpenStack-API-Version: baremetal 1.87 | 16:52 |
| dtantsur | it would be great to test this new behavior and provide a release note for it. | 16:52 |
| diablo_rojo | iurygregory: I am back :) Did you find a friend? | 16:53 |
| dtantsur | maybe the real openstack was the friends we've found along the way | 16:53 |
| diablo_rojo | dtantsur: thats my philosophy | 16:54 |
| diablo_rojo | I have found many a friend in openstack | 16:54 |
| dtantsur | true | 16:54 |
| dtantsur | diablo_rojo: to your actual question, he found rpittau, masghar and myself, a small mentoring community even | 16:55 |
| dtantsur | we just need to come up with a project (or projects?) | 16:55 |
| diablo_rojo | Correct. | 16:55 |
| diablo_rojo | I can kick off an email thread with the instructor and you can say hi/update there when you have something - ideally in the next week or so? | 16:56 |
| dtantsur | things have been a bit crazy downstream, but we can probably make it next week | 16:56 |
| dtantsur | (I'll let iurygregory coordinate the effort, I guess, since I already coordinate too many things) | 16:57 |
| diablo_rojo | dtantsur: such delegation :D Well done. | 16:57 |
| dtantsur | :) | 16:57 |
| diablo_rojo | I myself struggle with that at times. | 16:57 |
| diablo_rojo | I don't think I have masghar 's email addy - can you dm it to me dtantsur ? | 16:58 |
| dtantsur | sure | 16:58 |
| diablo_rojo | Thread has been kicked off :) | 17:02 |
| dtantsur | great, thank you! | 17:03 |
| opendevreview | Julia Kreger proposed openstack/ironic-specs master: WIP: Intermediate networking service https://review.opendev.org/c/openstack/ironic-specs/+/916126 | 17:05 |
| TheJulia | Well, that might be crazy | 17:05 |
| * iurygregory back from lunch | 17:06 | |
| dtantsur | crazy++ | 17:06 |
| dtantsur | TheJulia: file name appreciated! | 17:06 |
| TheJulia | Quickly, order the self-hugging jackets! | 17:06 |
| iurygregory | diablo_rojo, I just saw your email o/ and I think dtantsur already provide you some answers =D | 17:07 |
| diablo_rojo | iurygregory: you are correct | 17:07 |
| TheJulia | I... kind of want to name the entire thing asylumn, which sort of works when you think about the "over the wall use modeling" with network management | 17:07 |
| diablo_rojo | Sounds like you got the village to sign up with you | 17:07 |
| iurygregory | :D | 17:07 |
| iurygregory | it happens | 17:07 |
| diablo_rojo | I am still naming you the ringleader iurygregory | 17:07 |
| diablo_rojo | Lol | 17:08 |
| iurygregory | \o/ | 17:08 |
| dtantsur | are we a fellowship of the ringleader then? | 17:08 |
| diablo_rojo | Fellowship of Pixie Boots maybe. | 17:08 |
| iurygregory | this makes more sense lol | 17:08 |
| TheJulia | well, then we likely have multiple ring leaders in the Fellowship of Pixie Boots | 17:08 |
| iurygregory | ++ | 17:09 |
| diablo_rojo | TheJulia: I do not doubt that one bit. The real question is if the are 3 of you or 7... or heavens forbid 9? | 17:10 |
| dtantsur | TheJulia: after this conversation, you can call your new project ironic-village. Because the ironic village is where networking happens! | 17:10 |
| diablo_rojo | I like it. | 17:11 |
| TheJulia | dtantsur: but do we get ironic themed jackets in the ironic village? | 17:11 |
| dtantsur | how can we not to? | 17:11 |
| dtantsur | TheJulia: the spec does not look outrageously wrong from a quick look (except for formatting - sorry!) | 17:12 |
| TheJulia | oh, no worries, I am fixing the whitespace now | 17:13 |
| TheJulia | formatting wise, I need to find a local rst editor since I've always used the web render | 17:13 |
| TheJulia | and now it is no more it seems | 17:13 |
| dtantsur | I'll hopefully give it a careful read in the coming days. I may start asking questions about integration points and upgrades. | 17:13 |
| TheJulia | Definitely, I think a openstack-discuss post will occur tomorrow | 17:14 |
| opendevreview | Julia Kreger proposed openstack/ironic-specs master: WIP: Intermediate networking service https://review.opendev.org/c/openstack/ironic-specs/+/916126 | 17:21 |
| TheJulia | Okay, that should take care of formatting stuffs mostly | 17:21 |
| opendevreview | Julia Kreger proposed openstack/ironic master: DNM: Break tempest on purpose to figure out what tests might need to be addressed https://review.opendev.org/c/openstack/ironic/+/916128 | 17:28 |
| opendevreview | Merged openstack/ironic master: Load machine type from command-line https://review.opendev.org/c/openstack/ironic/+/915934 | 17:31 |
| JayF | diablo_rojo: Do we have any general "learn how to openstack" resources? Trying to find something to help round out cid's knowledge | 17:32 |
| cid | dtantsur: ahh, totally forgot about that , I will push those change to upstream ASAP. | 17:51 |
| dtantsur | thanks! | 17:51 |
| *** dmellado033 is now known as dmellado03 | 17:54 | |
| TheJulia | oooh ahh my new switch is arriving today | 18:04 |
| JayF | TheJulia: not today, but I think I will need someone with crayons and small words to help talk me through some of the points in the networking spec; left high level comments including one which might demonstrate my WTF-ery directly | 18:44 |
| TheJulia | heh, okay | 18:45 |
| JayF | I'm just really, really, really flummoxed over how any MVP could be done without touching rest api | 18:46 |
| TheJulia | we *already* have the vif attach/detach logic | 18:48 |
| TheJulia | so if we just loosen the regex, which we long agreed to do and never did... | 18:48 |
| TheJulia | there is our mechanism | 18:48 |
| JayF | so basically the "MVP" as described there is what I already thought worked, where you just slip a vlan id in the right place and magic happens | 18:54 |
| JayF | I just all the sudden had a weird feeling that Ironic *being the ML2 plugin* for Neutron might be an interesting way to think about this medium-term | 18:55 |
| JayF | to make the neutron/non-neutron paths more similar | 18:55 |
| TheJulia | I wasn't thinking ironic be that, but that we could teach networking-baremtal or somethign else to bridge the call across | 18:57 |
| TheJulia | so there is that continuity | 18:57 |
| TheJulia | Mentally, that sort of results in a best of all possible cases outcome even though it is little more weird to reach | 18:58 |
| TheJulia | and doesn't cause us to block off possible paths | 18:58 |
| TheJulia | and mens that should someone have a proprietary plugin, it might just work too | 18:59 |
| JayF | Ironic, meaning, a project under ironic | 19:05 |
| JayF | not ir-api/cond/etc | 19:05 |
| opendevreview | Julia Kreger proposed openstack/ironic master: DNM: Break tempest on purpose to figure out what tests might need to be addressed https://review.opendev.org/c/openstack/ironic/+/916128 | 19:05 |
| TheJulia | huh? | 19:05 |
| JayF | I meant ngs is what I was thinking | 19:06 |
| TheJulia | ahh yeah | 19:07 |
| opendevreview | Julia Kreger proposed openstack/ironic master: docs: document stance on partition image use https://review.opendev.org/c/openstack/ironic/+/915388 | 19:25 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Docs: Remove outdated RBAC content https://review.opendev.org/c/openstack/ironic/+/915385 | 19:27 |
| opendevreview | Julia Kreger proposed openstack/ironic master: docs: Cleanup/revise Secure Boot docs https://review.opendev.org/c/openstack/ironic/+/915386 | 19:34 |
| opendevreview | Julia Kreger proposed openstack/ironic master: docs: update ilo docs regarding status -> use redfish https://review.opendev.org/c/openstack/ironic/+/915387 | 19:35 |
| opendevreview | Julia Kreger proposed openstack/ironic-tempest-plugin master: CI: Increment stable jobs for 2024.1/drop Zed https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/914931 | 19:52 |
| TheJulia | Hey folks, I know tempest plugins stuffs are not the most popular items to check, but some quick reviews on https://review.opendev.org/q/project:openstack/ironic-tempest-plugin+status:open+hashtag:ironic-week-prio items would be appreicated | 20:32 |
| iurygregory | will add to my list to look at it | 20:33 |
| iurygregory | fighting downstream CI today... | 20:33 |
| opendevreview | Steve Baker proposed openstack/metalsmith master: Use bootloader element instead of grub2 https://review.opendev.org/c/openstack/metalsmith/+/916018 | 20:57 |
| TheJulia | Thanks, I know that feeling too | 21:05 |
| TheJulia | stevebaker[m]: might need to disable the legacy job with https://review.opendev.org/c/openstack/metalsmith/+/916018 as well | 21:06 |
| stevebaker[m] | Yeah, I'm still trying to get the uefi job to go green, the depends-on didn't seem to apply but it has landed now | 21:07 |
| TheJulia | yeah, the jobs afaik only install latest | 21:27 |
| opendevreview | Steve Baker proposed openstack/metalsmith master: Use bootloader element instead of grub2 https://review.opendev.org/c/openstack/metalsmith/+/916018 | 22:46 |
| opendevreview | Julia Kreger proposed openstack/ironic master: DNM: Break tempest on purpose to figure out what tests might need to be addressed https://review.opendev.org/c/openstack/ironic/+/916128 | 23:02 |
| stevebaker[m] | ugh, the playbook installs dib from pip, so it'll need a release | 23:12 |
| opendevreview | Steve Baker proposed openstack/metalsmith master: Use bootloader element instead of grub2 https://review.opendev.org/c/openstack/metalsmith/+/916018 | 23:21 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!