opendevreview | Huy Mai proposed openstack/sushy-tools master: Add fake ironic python agent (fake_ipa) https://review.opendev.org/c/openstack/sushy-tools/+/875366 | 06:27 |
---|---|---|
opendevreview | Huy Mai proposed openstack/sushy-tools master: Add fake ironic python agent (fake_ipa) https://review.opendev.org/c/openstack/sushy-tools/+/875366 | 07:07 |
dtantsur | Back and declaring a complete scrollback bankruptcy :) | 10:54 |
janders | dtantsur LOL | 11:24 |
iurygregory | good morning Ironic | 12:49 |
TheJulia | Good morning | 12:50 |
dansmith | JayF: thanks for jumping on this.. could we get it approved? It's blocking the devstack performance patch https://review.opendev.org/c/openstack/ironic/+/918690 | 14:16 |
dansmith | (the OCaaS one we discussed last week) | 14:16 |
JayF | I can merge as a single core as it's a CI change, will land it | 14:16 |
dansmith | Oh I meant "who can we ask" but if you can do that, then great, thanks | 14:16 |
TheJulia | done | 14:16 |
JayF | Yeah, Ironic permits single core merging for CI/Documentation changes as long as the gate is happy | 14:17 |
dansmith | cool, thanks both | 14:17 |
JayF | helpful for unsticking things during less-populated hours | 14:17 |
dansmith | indeed | 14:17 |
JayF | rpittau: you have a moment to help me nail down https://etherpad.opendev.org/p/baremetal-sig-cern-june-5-2024 | 14:21 |
iurygregory | JayF, he is on PTO till May 27 | 14:22 |
JayF | do you have any idea how long his metal3 presentation was going to be | 14:22 |
JayF | because I have to have the agenda shipped before that | 14:22 |
JayF | and he had nothing in that ehterpad indicating | 14:23 |
JayF | I am literally going to be in Europe by the time he's back lol | 14:23 |
iurygregory | no idea =( dtantsur do you have an idea of how long would it be? | 14:23 |
dtantsur | absolutely no, sorry | 14:24 |
iurygregory | np! | 14:24 |
JayF | someone is going to pull a number outta nowhere then | 14:24 |
iurygregory | 45 | 14:24 |
JayF | you are going to make it be me? :| | 14:24 |
iurygregory | lol | 14:24 |
JayF | lets go 30, if he goes over that's better than giving someone with 20m planned a 45m slot lmao | 14:24 |
iurygregory | maybe 30 would do, | 14:24 |
iurygregory | was about to say 30 lol :D | 14:25 |
JayF | Are there any other Ironic/Metal3 cores going to be at CERN AFA-you-K? | 14:26 |
iurygregory | not sure, I know I won't be at sig meetup | 14:27 |
JayF | ack; I think the answer is "no" | 14:27 |
opendevreview | Julia Kreger proposed openstack/ironic master: docs: allow the state diagram to be larger https://review.opendev.org/c/openstack/ironic/+/919585 | 14:27 |
JayF | Just have another topic we need a prezi on, if we had a third person it'd be very nice that none of us would repeat | 14:27 |
JayF | (ironic deprecations / forward motion -- e.g. the 'stop using redfish' prezi, among others) | 14:28 |
TheJulia | JayF: that changeset above results in the diagram being ~20% bigger on my monitors | 14:29 |
TheJulia | In the rendered page, that is | 14:29 |
JayF | nice, I'll look once CI is done | 14:29 |
opendevreview | Verification of a change to openstack/ironic master failed: Remove SQLAlchemy tips jobs https://review.opendev.org/c/openstack/ironic/+/915792 | 14:49 |
opendevreview | Elod Illes proposed openstack/networking-generic-switch master: [CI] Remove not defined job from queue https://review.opendev.org/c/openstack/networking-generic-switch/+/919593 | 15:07 |
JayF | https://review.opendev.org/c/openstack/ironic/+/919353 is a pretty simple change that needs review too, although it's going to conflict with Julia's change | 15:08 |
opendevreview | Merged openstack/ironic master: [devstack] Upload images with --file instead of stdin https://review.opendev.org/c/openstack/ironic/+/918690 | 15:48 |
JayF | dansmith: ^ you're clear | 15:50 |
dansmith | JayF: tys | 15:50 |
TheJulia | JayF: I can rebase, that is fine | 16:25 |
TheJulia | A quick and easy metalsmith change to bring clarity to users :) | 17:08 |
TheJulia | https://review.opendev.org/c/openstack/metalsmith/+/918664 | 17:08 |
opendevreview | Merged openstack/ironic master: Remove SQLAlchemy tips jobs https://review.opendev.org/c/openstack/ironic/+/915792 | 17:22 |
*** dmellado5 is now known as dmellado | 17:23 | |
*** dmellado5 is now known as dmellado | 17:33 | |
*** dmellado8 is now known as dmellado | 17:45 | |
JayF | I was today years old when I realized we enabled auto tls by default when implementing it \o/ | 17:45 |
JayF | Good job Dmitry! | 17:46 |
JayF | One less hidden feature of Ironic to put in my slide deck \o/ | 17:46 |
opendevreview | Merged openstack/metalsmith master: Error with clarity when a bad upgrade was encountered https://review.opendev.org/c/openstack/metalsmith/+/918664 | 18:32 |
dtantsur | JayF: heh, yeah, it was the whole reason of having *auto* TLS | 18:48 |
JayF | actually... https://docs.google.com/presentation/d/1U_I0XC_A5RaZxUotqKGf02hWgMYuPyfSXhirxh9clcI/edit?usp=sharing | 18:48 |
JayF | ignoring the fact most of those slides are not populated properly | 18:48 |
JayF | is there any feature of Ironic I don't mention there that I should? | 18:49 |
dtantsur | I'll take a look tomorrow if you don't mind | 18:49 |
JayF | no rush, and really not just limited to you | 18:51 |
JayF | the idea is just, I want to give people a taste of all the cool stuff Ironic can do, just so they know the things to go google lol | 18:51 |
dtantsur | oh yeah, I do these from time to time | 18:52 |
JayF | and also to talk about the features that are "hidden" from view when using Ironic via metal3/nova | 18:52 |
dtantsur | things like BIOS settings, now firmware updates, servicing, various inspection aspects | 18:52 |
dtantsur | (literally just directed someone to find out WWN's using inspection data) | 18:52 |
JayF | I don't have bios settings or firmware updates yet, I had them under steps LOL | 18:53 |
JayF | I take those for granted too, apparently | 18:53 |
* JayF edits | 18:53 | |
dtantsur | don't forget turning secure boot on/off on fly | 18:53 |
* dtantsur will go get lunch and think more tomorrow | 18:53 | |
dtantsur | s/lunch/dinner/ of course | 18:54 |
JayF | every word you say is moving my progress bar in the wrong direction ;) | 18:54 |
JayF | hehehe | 18:54 |
dtantsur | :D | 18:54 |
TheJulia | I literally had someone ask when we will support secure boot yesterday, and had to point out we have for quite a long time | 19:02 |
JayF | the 'various inspection aspects', I am not sure even I know what detail people care about there | 19:04 |
JayF | but the answer should basically be "if you want it, and it's reasonable, we probably already inspect for it or would happily do it" | 19:04 |
opendevreview | Julia Kreger proposed openstack/ironic-tempest-plugin master: Handle invalid configuration for vif test execution https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/919544 | 20:16 |
opendevreview | Jay Faulkner proposed openstack/ironic master: Test devstack OSCaas against all Ironic jobs https://review.opendev.org/c/openstack/ironic/+/919620 | 20:35 |
JayF | I legit could go for some guidance around how folks tend to use inspector; would saying the flow is often Enroll->Validate->Inspect on node onboarding be correct? | 20:45 |
JayF | I know we also support discovery to some extent, but I think that's not frequently used now? | 20:45 |
TheJulia | Yeah, discovery is super weird, largely it is just a step of data collection with maybe some parameter setting mixed in for good measure | 20:48 |
JayF | I'm thinking of including a section of like ... "this exists but may not for long" and/or with some mild statement discouraging use | 20:51 |
JayF | discovery, chassis would be two things on that list for sure | 20:51 |
TheJulia | ++ | 20:59 |
JayF | I had an interesting feature pitched today -- what would you all think about the ability to require an out of band cleaning step to run before the overall automated cleaning process begins. Basically if you have a use case for "I need to do something out of band even before a ramdisk is booted" | 22:08 |
JayF | but since automated cleaning is dynamic in-band, we boot an agent ramdisk to discover those steps | 22:08 |
JayF | ooh yeah, partition images are going on that "there are dragons here" set of slides too lol | 22:11 |
opendevreview | Julia Kreger proposed openstack/ironic-tempest-plugin master: WIP: reboot the node in basic ops tests https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/918462 | 22:12 |
JayF | We really do need a review jam, we've got a lot of things slipping through it seems | 22:48 |
TheJulia | ++ | 22:48 |
JayF | I did a pass on ironic-week-prio | 22:55 |
TheJulia | I just took a look at a few, and a review jam would at least help me | 23:01 |
opendevreview | Merged openstack/ironic master: docs: Cleanup/revise Secure Boot docs https://review.opendev.org/c/openstack/ironic/+/915386 | 23:12 |
cid | What's a review jam? | 23:12 |
TheJulia | It is where a bunch of us get on a call and sift through open changes reviewing them as a group | 23:13 |
cid | Ok. Like on the weekly meeting | 23:13 |
TheJulia | sort of | 23:14 |
cid | Got it | 23:14 |
opendevreview | Verification of a change to openstack/ironic-tempest-plugin master failed: Additional test state fixes https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/918001 | 23:17 |
JayF | cid: TheJulia: you want the three of us to commit to a time to do a sync one tomorrow? Maybe early AM PDT? | 23:49 |
cid | Would definitely love to. | 23:50 |
TheJulia | I think that would be good. | 23:50 |
JayF | I am wide open, only meeting tomorrow on my calendar is with cid lol | 23:51 |
JayF | although I have 'finish the endless slide presentation' penciled into all open slots :D | 23:52 |
JayF | cid: TheJulia: 1500 UTC? | 23:53 |
JayF | aka 8am PDT | 23:53 |
TheJulia | That can work | 23:53 |
JayF | cid: 4pm your time, does that work for you, too? | 23:54 |
cid | It does | 23:54 |
JayF | adamcarthur5: ^ if you are available tomorrow, would be a good chance for you to join too | 23:55 |
JayF | Lets just meet up in IRC and whoever gets here first spins up a zoom/meet/whatever | 23:55 |
adamcarthur5 | Sorry Jay, I won't be around tomorrow :(( | 23:55 |
JayF | Not a problem, just wanted to invite you o/ | 23:56 |
TheJulia | iurygregory: would you happen to be around at 1500 utc tomorrow? | 23:56 |
JayF | yeah a third core, especially for the specs, would be a good idea | 23:56 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!