Tuesday, 2024-05-14

opendevreviewHuy Mai proposed openstack/sushy-tools master: Add fake ironic python agent (fake_ipa)  https://review.opendev.org/c/openstack/sushy-tools/+/87536606:27
opendevreviewHuy Mai proposed openstack/sushy-tools master: Add fake ironic python agent (fake_ipa)  https://review.opendev.org/c/openstack/sushy-tools/+/87536607:07
dtantsurBack and declaring a complete scrollback bankruptcy :)10:54
jandersdtantsur LOL11:24
iurygregorygood morning Ironic 12:49
TheJuliaGood morning12:50
dansmithJayF: thanks for jumping on this.. could we get it approved? It's blocking the devstack performance patch https://review.opendev.org/c/openstack/ironic/+/91869014:16
dansmith(the OCaaS one we discussed last week)14:16
JayFI can merge as a single core as it's a CI change, will land it14:16
dansmithOh I meant "who can we ask" but if you can do that, then great, thanks14:16
TheJuliadone14:16
JayFYeah, Ironic permits single core merging for CI/Documentation changes as long as the gate is happy14:17
dansmithcool, thanks both14:17
JayFhelpful for unsticking things during less-populated hours14:17
dansmithindeed14:17
JayFrpittau: you have a moment to help me nail down https://etherpad.opendev.org/p/baremetal-sig-cern-june-5-202414:21
iurygregoryJayF, he is on PTO till May 27 14:22
JayFdo you have any idea how long his metal3 presentation was going to be14:22
JayFbecause I have to have the agenda shipped before that14:22
JayFand he had nothing in that ehterpad indicating14:23
JayFI am literally going to be in Europe by the time he's back lol14:23
iurygregoryno idea =( dtantsur do you have an idea of how long would it be?14:23
dtantsurabsolutely no, sorry14:24
iurygregorynp!14:24
JayFsomeone is going to pull a number outta nowhere then14:24
iurygregory4514:24
JayFyou are going to make it be me? :| 14:24
iurygregorylol14:24
JayFlets go 30, if he goes over that's better than giving someone with 20m planned a 45m slot lmao14:24
iurygregorymaybe 30 would do, 14:24
iurygregorywas about to say 30 lol :D14:25
JayFAre there any other Ironic/Metal3 cores going to be at CERN AFA-you-K?14:26
iurygregorynot sure, I know I won't be at sig meetup14:27
JayFack; I think the answer is "no"14:27
opendevreviewJulia Kreger proposed openstack/ironic master: docs: allow the state diagram to be larger  https://review.opendev.org/c/openstack/ironic/+/91958514:27
JayFJust have another topic we need a prezi on, if we had a third person it'd be very nice that none of us would repeat14:27
JayF(ironic deprecations / forward motion -- e.g. the 'stop using redfish' prezi, among others)14:28
TheJuliaJayF: that changeset above results in the diagram being ~20% bigger on my monitors14:29
TheJuliaIn the rendered page, that is14:29
JayFnice, I'll look once CI is done14:29
opendevreviewVerification of a change to openstack/ironic master failed: Remove SQLAlchemy tips jobs  https://review.opendev.org/c/openstack/ironic/+/91579214:49
opendevreviewElod Illes proposed openstack/networking-generic-switch master: [CI] Remove not defined job from queue  https://review.opendev.org/c/openstack/networking-generic-switch/+/91959315:07
JayFhttps://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 change15:08
opendevreviewMerged openstack/ironic master: [devstack] Upload images with --file instead of stdin  https://review.opendev.org/c/openstack/ironic/+/91869015:48
JayFdansmith: ^ you're clear15:50
dansmithJayF: tys15:50
TheJuliaJayF: I can rebase, that is fine16:25
TheJuliaA quick and easy metalsmith change to bring clarity to users :)17:08
TheJuliahttps://review.opendev.org/c/openstack/metalsmith/+/91866417:08
opendevreviewMerged openstack/ironic master: Remove SQLAlchemy tips jobs  https://review.opendev.org/c/openstack/ironic/+/91579217:22
*** dmellado5 is now known as dmellado17:23
*** dmellado5 is now known as dmellado17:33
*** dmellado8 is now known as dmellado17:45
JayFI was today years old when I realized we enabled auto tls by default when implementing it \o/17:45
JayFGood job Dmitry!17:46
JayFOne less hidden feature of Ironic to put in my slide deck \o/17:46
opendevreviewMerged openstack/metalsmith master: Error with clarity when a bad upgrade was encountered  https://review.opendev.org/c/openstack/metalsmith/+/91866418:32
dtantsurJayF: heh, yeah, it was the whole reason of having *auto* TLS18:48
JayFactually... https://docs.google.com/presentation/d/1U_I0XC_A5RaZxUotqKGf02hWgMYuPyfSXhirxh9clcI/edit?usp=sharing18:48
JayFignoring the fact most of those slides are not populated properly18:48
JayFis there any feature of Ironic I don't mention there that I should?18:49
dtantsurI'll take a look tomorrow if you don't mind18:49
JayFno rush, and really not just limited to you18:51
JayFthe 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 lol18:51
dtantsuroh yeah, I do these from time to time18:52
JayFand also to talk about the features that are "hidden" from view when using Ironic via metal3/nova18:52
dtantsurthings like BIOS settings, now firmware updates, servicing, various inspection aspects18:52
dtantsur(literally just directed someone to find out WWN's using inspection data)18:52
JayFI don't have bios settings or firmware updates yet, I had them under steps  LOL18:53
JayFI take those for granted too, apparently18:53
* JayF edits18:53
dtantsurdon't forget turning secure boot on/off on fly18:53
* dtantsur will go get lunch and think more tomorrow18:53
dtantsurs/lunch/dinner/ of course18:54
JayFevery word you say is moving my progress bar in the wrong direction ;) 18:54
JayFhehehe18:54
dtantsur:D18:54
TheJuliaI literally had someone ask when we will support secure boot yesterday, and had to point out we have for quite a long time19:02
JayFthe 'various inspection aspects', I am not sure even I know what detail people care about there19:04
JayFbut 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
opendevreviewJulia Kreger proposed openstack/ironic-tempest-plugin master: Handle invalid configuration for vif test execution  https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/91954420:16
opendevreviewJay Faulkner proposed openstack/ironic master: Test devstack OSCaas against all Ironic jobs  https://review.opendev.org/c/openstack/ironic/+/91962020:35
JayFI 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
JayFI know we also support discovery to some extent, but I think that's not frequently used now?20:45
TheJuliaYeah, discovery is super weird, largely it is just a step of data collection with maybe some parameter setting mixed in for good measure20:48
JayFI'm thinking of including a section of like ... "this exists but may not for long" and/or with some mild statement discouraging use20:51
JayFdiscovery, chassis would be two things on that list for sure20:51
TheJulia++20:59
JayFI 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
JayFbut since automated cleaning is dynamic in-band, we boot an agent ramdisk to discover those steps22:08
JayFooh yeah, partition images are going on that "there are dragons here" set of slides too lol22:11
opendevreviewJulia Kreger proposed openstack/ironic-tempest-plugin master: WIP: reboot the node in basic ops tests  https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/91846222:12
JayFWe really do need a review jam, we've got a lot of things slipping through it seems22:48
TheJulia++22:48
JayFI did a pass on ironic-week-prio22:55
TheJuliaI just took a look at a few, and a review jam would at least help me23:01
opendevreviewMerged openstack/ironic master: docs: Cleanup/revise Secure Boot docs  https://review.opendev.org/c/openstack/ironic/+/91538623:12
cidWhat's a review jam?23:12
TheJuliaIt is where a bunch of us get on a call and sift through open changes reviewing them as a group23:13
cidOk. Like on the weekly meeting 23:13
TheJuliasort of23:14
cidGot it23:14
opendevreviewVerification of a change to openstack/ironic-tempest-plugin master failed: Additional test state fixes  https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/91800123:17
JayFcid: TheJulia: you want the three of us to commit to a time to do a sync one tomorrow? Maybe early AM PDT?23:49
cidWould definitely love to.23:50
TheJuliaI think that would be good.23:50
JayFI am wide open, only meeting tomorrow on my calendar is with cid lol23:51
JayFalthough I have 'finish the endless slide presentation' penciled into all open slots :D 23:52
JayFcid: TheJulia: 1500 UTC?23:53
JayFaka 8am PDT23:53
TheJuliaThat can work23:53
JayFcid: 4pm your time, does that work for you, too?23:54
cidIt does23:54
JayFadamcarthur5: ^ if you are available tomorrow, would be a good chance for you to join too23:55
JayFLets just meet up in IRC and whoever gets here first spins up a zoom/meet/whatever23:55
adamcarthur5Sorry Jay, I won't be around tomorrow :((23:55
JayFNot a problem, just wanted to invite you o/23:56
TheJuliaiurygregory: would you happen to be around at 1500 utc tomorrow?23:56
JayFyeah a third core, especially for the specs, would be a good idea23:56

Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!