| TheJulia | ugh, bifrost, why you fail on change to changeset with just a reno added :( | 00:33 |
|---|---|---|
| TheJulia | queue infinite sadness | 00:33 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Detect ilo6 and redirect to redfish https://review.opendev.org/c/openstack/ironic/+/888297 | 00:40 |
| iurygregory | yeah =( | 00:43 |
| TheJulia | iurygregory: How about recursive infinite sadness? | 00:51 |
| iurygregory | TheJulia, no! please no! | 00:51 |
| iurygregory | D: | 00:52 |
| TheJulia | but... but! :) | 00:52 |
| iurygregory | we want happy CI upstream | 00:52 |
| iurygregory | for infinite recursive sadness we have downstream upgrade jobs =) | 00:52 |
| opendevreview | Kyrylo Romanenko proposed openstack/python-ironicclient master: Tests for OSC baremetal node create command with options https://review.opendev.org/c/openstack/python-ironicclient/+/382352 | 00:53 |
| iurygregory | CI is happy, it gave +1 on my patch XD | 00:57 |
| TheJulia | \o/ | 01:02 |
| opendevreview | Merged openstack/ironic master: Fix bug in devstack-guide: readd server create https://review.opendev.org/c/openstack/ironic/+/906001 | 01:02 |
| opendevreview | Steve Baker proposed openstack/ironic master: [WIP] Replace cinderclient usage with openstacksdk https://review.opendev.org/c/openstack/ironic/+/900265 | 01:03 |
| opendevreview | Verification of a change to openstack/python-ironicclient master failed: add pyproject.toml to support pip 23.1 https://review.opendev.org/c/openstack/python-ironicclient/+/906077 | 01:27 |
| opendevreview | Merged openstack/sushy stable/2023.2: Handle exceptions after re-authentication https://review.opendev.org/c/openstack/sushy/+/904735 | 01:36 |
| opendevreview | Merged openstack/sushy stable/2023.1: Handle exceptions after re-authentication https://review.opendev.org/c/openstack/sushy/+/904736 | 01:36 |
| opendevreview | Merged openstack/sushy stable/zed: Handle exceptions after re-authentication https://review.opendev.org/c/openstack/sushy/+/904793 | 01:39 |
| opendevreview | Merged openstack/sushy stable/yoga: Handle exceptions after re-authentication https://review.opendev.org/c/openstack/sushy/+/904801 | 01:39 |
| opendevreview | Merged openstack/sushy stable/xena: Handle exceptions after re-authentication https://review.opendev.org/c/openstack/sushy/+/904737 | 01:39 |
| opendevreview | Merged openstack/sushy stable/wallaby: Handle exceptions after re-authentication https://review.opendev.org/c/openstack/sushy/+/904802 | 01:39 |
| opendevreview | Verification of a change to openstack/python-ironic-inspector-client master failed: add pyproject.toml to support pip 23.1 https://review.opendev.org/c/openstack/python-ironic-inspector-client/+/906079 | 01:50 |
| opendevreview | Adam McArthur proposed openstack/ironic-python-agent master: [codespell] Adding git-blame-ignore-revs to clear codespell changes https://review.opendev.org/c/openstack/ironic-python-agent/+/906599 | 01:52 |
| opendevreview | Merged openstack/bifrost master: fix keystone auth scope https://review.opendev.org/c/openstack/bifrost/+/906528 | 04:50 |
| rpittau | good morning ironic! o/ | 07:34 |
| dtantsur | TheJulia: thanks for handling the bifrost issue! | 07:57 |
| opendevreview | Mohammed Boukhalfa proposed openstack/sushy-tools master: Add fake_ipa inspection, lookup and heartbeater to fake system https://review.opendev.org/c/openstack/sushy-tools/+/875366 | 08:28 |
| rpittau | JayF: re: 890408, thanks! :) | 08:55 |
| rpittau | oh bifrost CI is back, good | 08:57 |
| dtantsur | should be, thanks to Julia | 09:16 |
| rpittau | now that I'm looking deeper at plantuml I'm ready to give up and just convert all the sequence diagrams to SVG or drop them :D | 09:39 |
| masghar | Is it missing features, or does it have a bunch of dependencies? | 09:45 |
| rpittau | both, plus complexity is way higher than seqdiag, not sure it's worth at all if we can generate svg externally and embed them | 09:46 |
| masghar | I see, well we could use seqdiag or just diagrams.net to generate an svg externally if you're me :) | 09:54 |
| rpittau | yep, I think I'm going to move all diagrams to images so we can at least remove the requirements for seqdiag and move forward with Pillow, then we'll see | 09:55 |
| rpittau | it's 9 diags, should be quick | 09:56 |
| masghar | I can make the diagrams in the meantime, if thats helpful | 10:12 |
| iurygregory | good morning Ironic | 11:06 |
| iurygregory | looking for another review in https://review.opendev.org/c/openstack/ironic/+/903379 =) | 11:09 |
| opendevreview | Merged openstack/bifrost master: Configure the new inspection implementation https://review.opendev.org/c/openstack/bifrost/+/896925 | 11:26 |
| dtantsur | yay \o/ | 11:29 |
| rpittau | masghar: thanks, we already have the diagrams, I just need to change the seqdiag entries to normal image :) | 11:30 |
| rpittau | dtantsur: cool! | 11:30 |
| masghar | rpittau: perfect, okay | 11:36 |
| opendevreview | Dmitry Tantsur proposed openstack/bifrost master: Wire in node auto-discovery https://review.opendev.org/c/openstack/bifrost/+/906635 | 11:42 |
| opendevreview | Verification of a change to openstack/bifrost master failed: Collect lshw output in json format https://review.opendev.org/c/openstack/bifrost/+/890408 | 12:03 |
| masghar | I am looking at this diagram: https://docs.openstack.org/ironic/latest/user/architecture.html#example-pxe-boot-and-direct-deploy-process and am wondering why the PXE environment for instance image is prepared after direct deploy is completed? Hasn't the instance image already been written to the disk? | 12:23 |
| iurygregory | JayF, regarding https://review.opendev.org/c/openstack/releases/+/906560 I think we are good, I can +1 if you want | 12:29 |
| rpittau | iurygregory, JayF, I'm also +1 about https://review.opendev.org/c/openstack/releases/+/906560 | 13:17 |
| dtantsur | masghar: sigh. This diagram is so ancient that it refers to booting final instances via network. A feature that I removed.. maybe more than a year ago already. | 13:26 |
| opendevreview | Merged openstack/sushy stable/2023.2: Handle a different related properties for missing TransferProtocolType https://review.opendev.org/c/openstack/sushy/+/905740 | 13:30 |
| opendevreview | Damien RANNOU proposed openstack/ironic-python-agent master: USB device discovery https://review.opendev.org/c/openstack/ironic-python-agent/+/902769 | 14:04 |
| TheJulia | good morning | 14:23 |
| masghar | dtantsur: no wonder! It confused me a bit. Maybe I can fix this up | 14:33 |
| masghar | Good morning TheJulia! | 14:33 |
| dtantsur | morning TheJulia | 14:33 |
| dtantsur | masghar: yeah, any references to booting instances (not ramdisks) from networking can be dropped | 14:34 |
| masghar | alright | 14:34 |
| TheJulia | dtantsur: by chance did you have a minute or three to ponder my reply on http boot interfaces for normal network boot? | 14:50 |
| opendevreview | Merged openstack/bifrost master: Collect lshw output in json format https://review.opendev.org/c/openstack/bifrost/+/890408 | 15:22 |
| JayF | iurygregory: dtantsur: re: yoga->unmaintained; we just should make sure if we need to make a release of yoga we do so first | 15:32 |
| iurygregory | JayF, well, it will have a release but with the yoga-eom tag | 15:43 |
| JayF | no, that's not a release, that's just a tag | 15:44 |
| JayF | just like when we'd tag yoga-em when something went to extended maintenance | 15:44 |
| JayF | or yoga-eol if it was end of life | 15:44 |
| JayF | unmaintained is like EM in that we don't make releases once we cross that threshold | 15:45 |
| JayF | so it's "final chance to release yoga Ironic" | 15:45 |
| iurygregory | JayF, got it, yeah I think it makes sense (just checked ironic and we added things it would be good to release) | 15:46 |
| opendevreview | Merged openstack/sushy master: Update supported python versions https://review.opendev.org/c/openstack/sushy/+/906218 | 16:08 |
| TheJulia | iurygregory: JayF: I've posted https://review.opendev.org/c/openstack/releases/+/906712 to get a new version of the tempest plugin out there. | 16:22 |
| iurygregory | TheJulia, looking now, just got back from lunch | 16:24 |
| dtantsur | TheJulia: I only read it briefly. IIRC it addressed my concerns, but I did not have time for a re-review. | 16:42 |
| TheJulia | okay, I realize it wasn't the answer you were originally hoping for when you were asking the question either, but I guess I conveyed enough context to get the underlying challenge across. In the grand scheme of things, adn this is a totally valid thing for metal3, is you could just encode in a HTTPClient payload response anyway for all that wonderful hardware that thinks PXE IS HTTPBoot :) | 16:44 |
| rpittau | good night! o/ | 16:47 |
| dtantsur | TheJulia: yeah, it probably just a flavor of ipxe for us | 16:51 |
| TheJulia | That whole challenge is kind of if at any point you need to explicitly tell a redfish bmc that it needs to only boot from HTTP, then that is definitely another boot_interface selection since the lower level mechanics with the bmc can't just take a default path at that point | 16:52 |
| opendevreview | Jay Faulkner proposed openstack/ironic-tempest-plugin master: WIP: Basic tests for sharding https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/906749 | 18:05 |
| iurygregory | TheJulia, if you have some time for review today can you check https://review.opendev.org/c/openstack/ironic/+/903379 ? already has a +2 | 19:22 |
| opendevreview | Merged openstack/ironic stable/2023.1: Don't create a hardlink to a symlink when handling file:// URLs https://review.opendev.org/c/openstack/ironic/+/906133 | 19:37 |
| TheJulia | iurygregory: question regarding https://review.opendev.org/c/openstack/ironic/+/903379/8/ironic/drivers/modules/redfish/firmware_utils.py#b79 | 19:38 |
| TheJulia | are those already constrained by redfish, or is it just what we're comfortable with? I guess I ask becuase we are aware of BMCs being able to update things like storage adapter firmware, network card firmware, etc | 19:39 |
| TheJulia | ahh, I see it was hard coded in the component processing previously | 19:51 |
| iurygregory | TheJulia, it's what we can support for now | 19:56 |
| iurygregory | To make nics and other components available it would require a lot more things, so we decided to keep simple and we can enhance things later | 19:57 |
| iurygregory | a lot more things you can read as (test FirmwareInventory/SoftwareInventory from redfish and try to update things in different hardware models ...) | 19:58 |
| TheJulia | fair, it looks like the update service api doesn't really have the primitives except for in softwareinventory, and I think that is a whole other schema | 20:01 |
| iurygregory | yeah, and a bit more complicated to match things.. | 20:02 |
| TheJulia | ... I sort of wonder if there is a way to... in a generic sense, extend and support that | 20:54 |
| TheJulia | I guess it woudl also help to see practical examples and I'm not sure there are any off hand | 21:05 |
| iurygregory | TheJulia, janders collected some data from Software/FirmwareInventory if I recall we have available downstream | 21:22 |
| opendevreview | Adam McArthur proposed openstack/ironic-inspector master: [codespell] Fixing Spelling Mistakes https://review.opendev.org/c/openstack/ironic-inspector/+/906756 | 21:22 |
| opendevreview | Adam McArthur proposed openstack/ironic-inspector master: [codespell] Adding Tox Target for Codespell https://review.opendev.org/c/openstack/ironic-inspector/+/906757 | 21:22 |
| opendevreview | Adam McArthur proposed openstack/ironic-inspector master: [codespell] Adding CI target for Tox Codespell https://review.opendev.org/c/openstack/ironic-inspector/+/906758 | 21:22 |
| opendevreview | Adam McArthur proposed openstack/bifrost master: [codespell] Fixing Spelling Mistakes https://review.opendev.org/c/openstack/bifrost/+/906759 | 21:32 |
| opendevreview | Adam McArthur proposed openstack/bifrost master: [codespell] Adding Tox Target for Codespell https://review.opendev.org/c/openstack/bifrost/+/906760 | 21:32 |
| opendevreview | Adam McArthur proposed openstack/bifrost master: [codespell] Adding CI target for Tox Codespell https://review.opendev.org/c/openstack/bifrost/+/906761 | 21:32 |
| adamcarthur5 | Apologies for the spam folks - should I start doing the Tox and CI commits in one? I think the spelling changes themselves should be separate though if we add a git-blame-ignore-revs down the line. | 21:35 |
| JayF | I'd be OK with reviewing them that way since we have consensus to add it non-voting to our jobs | 21:36 |
| TheJulia | Also, it is not spam ) | 21:36 |
| TheJulia | err :) | 21:36 |
| JayF | that is true as well, it's happiness in notification form lol | 21:37 |
| JayF | opendevreview is the dopamine bot /s | 21:37 |
| TheJulia | now, if only it could be some sort of pastry | 21:37 |
| JayF | see, I would've thought you'd go more pizza | 21:37 |
| JayF | we'll have to mix them: hot pockets! | 21:37 |
| TheJulia | Eh, I *do* need to go kneed the dough for tonight's pizza | 21:37 |
| JayF | you have a knead need? | 21:38 |
| TheJulia | well, the dough needs it's first one soon | 21:38 |
| JayF | Vanessa found some no-knead overnight neapolitan recipe which did a pretty good crust; not as good as when she went the long way but apparently it was very little prep and it just worked | 21:38 |
| TheJulia | I feel like need that recipe | 21:39 |
| JayF | I have proxied your request to Vanessa :D | 21:39 |
| JayF | TheJulia: https://www.kingarthurbaking.com/recipes/neapolitan-style-pizza-crust-recipe I'll note that king arthur flour has like, slightly different protein content than some flours, so if you're not using their brand you might wanna lookup a mix (I think it'd be a small amount of bread flour with all purpose) | 21:41 |
| TheJulia | oh yes, and they have modified versions and recommendations for different types | 21:41 |
| TheJulia | like... making a french baguette | 21:41 |
| JayF | Vanessa has a library of flours now lol | 21:42 |
| iurygregory | and now I'm hungry after reading about pizza | 21:43 |
| iurygregory | lol | 21:43 |
| opendevreview | Steve Baker proposed openstack/sushy-tools master: Handle nova instance having no image https://review.opendev.org/c/openstack/sushy-tools/+/906767 | 22:11 |
| opendevreview | Steve Baker proposed openstack/sushy-tools master: [WIP] add virtual-media-boot to openstack driver https://review.opendev.org/c/openstack/sushy-tools/+/906768 | 22:11 |
| stevebaker[m] | TheJulia: boop | 22:24 |
| opendevreview | Merged openstack/ironic master: RedfishFirmwareInterface - Unit Tests & More logs https://review.opendev.org/c/openstack/ironic/+/903379 | 22:25 |
| iurygregory | happy dance \o/ | 22:27 |
| TheJulia | o/ stevebaker[m] | 22:33 |
| TheJulia | stevebaker[m]: do you want an early review on your wip change or do you want me to hold off? | 22:33 |
| stevebaker[m] | Yeah feedback would be welcom at this point i think | 22:40 |
| TheJulia | "i think" ;) | 22:41 |
| TheJulia | not sure I get why the initial changes to staticdriver | 22:46 |
| TheJulia | left two comments, one might requiring looking at volume states to know for sure | 22:51 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!