| rpittau | good morning ironic! o/ | 07:30 |
|---|---|---|
| kubajj | good morning rpittau, and ironic! o/ | 07:53 |
| rpittau | hey kubajj :) | 08:06 |
| opendevreview | Riccardo Pittau proposed openstack/ironic master: [docs] Add procedure to EoL and delete old bugfix branch https://review.opendev.org/c/openstack/ironic/+/926648 | 08:13 |
| rpittau | FYI final sushy and ironic-lib releases for 2024.2 are in the oven :) | 08:31 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic-tempest-plugin master: A simple standalone test for in-band inspection https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/925981 | 08:36 |
| opendevreview | Graeme Moss proposed openstack/networking-generic-switch master: Add Supermicro switches to allow for supported write config https://review.opendev.org/c/openstack/networking-generic-switch/+/926886 | 09:28 |
| opendevreview | Graeme Moss proposed openstack/networking-generic-switch stable/2024.1: Add Supermicro switches to allow for supported write config https://review.opendev.org/c/openstack/networking-generic-switch/+/926887 | 09:30 |
| opendevreview | Merged openstack/python-ironicclient master: support passing disable_ramdisk for clean and service state https://review.opendev.org/c/openstack/python-ironicclient/+/924895 | 09:34 |
| opendevreview | Graeme Moss proposed openstack/networking-generic-switch master: Add Supermicro switches to allow for supported write config https://review.opendev.org/c/openstack/networking-generic-switch/+/926886 | 09:59 |
| opendevreview | Robert Hoffmann proposed openstack/ironic-inspector master: Fix: ignore resolve error while looking up node https://review.opendev.org/c/openstack/ironic-inspector/+/925326 | 10:03 |
| opendevreview | Graeme Moss proposed openstack/networking-generic-switch master: Add Supermicro switches to allow for supported write config https://review.opendev.org/c/openstack/networking-generic-switch/+/926886 | 11:15 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: Better handle missing inspection_network https://review.opendev.org/c/openstack/ironic/+/926899 | 11:16 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: Enable the inspection test in the redfish standalone job https://review.opendev.org/c/openstack/ironic/+/926647 | 11:18 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: DNM: trying without inspection_network https://review.opendev.org/c/openstack/ironic/+/926902 | 11:19 |
| pas-ha[m] | hi folks, would you care to tag a new release of python-ironicclient? the current latest version 5.7.0 is missing a critical patch for py3.12 compat, and py312 is currently default on latest ubuntu lts and in brew on mac... https://review.opendev.org/c/openstack/python-ironicclient/+/914523 | 12:52 |
| rpittau | pas-ha[m]: the client releases are coming soon, we're at the end of the dev cycle | 12:53 |
| pas-ha[m] | cool, thanks | 12:53 |
| rpittau | pas-ha[m]: precisely next week, just checked the release schedule | 12:55 |
| opendevreview | Graeme Moss proposed openstack/networking-generic-switch master: Add Supermicro switches to allow for supported write config https://review.opendev.org/c/openstack/networking-generic-switch/+/926886 | 13:00 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic-tempest-plugin master: A simple standalone test for in-band inspection https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/925981 | 13:29 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: Better handle missing inspection_network https://review.opendev.org/c/openstack/ironic/+/926899 | 13:32 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: Enable the inspection test in the redfish standalone job https://review.opendev.org/c/openstack/ironic/+/926647 | 13:33 |
| opendevreview | cid proposed openstack/ironic master: docs-audit-2024: Labeling references https://review.opendev.org/c/openstack/ironic/+/925691 | 14:03 |
| cardoe | Is there a "Replaces" or some such syntax for gerrit to supersede another change? | 14:50 |
| iurygregory | rpittau, have you finished the cycle highlights ? | 14:50 |
| rpittau | iurygregory: not yet! I plan to put the patch up first thing next week | 14:51 |
| iurygregory | rpittau, ack! | 14:51 |
| iurygregory | tks | 14:51 |
| JayF | cardoe: use the change ID from the change you want to supplant | 15:00 |
| JayF | That's how geritt tracks changes | 15:01 |
| JayF | iurygregory: I have someone downstream who was hoping to get an example of using the firmware interface. Do you think you'll have a doc drafted up this week? | 15:01 |
| JayF | (on ilo with redfish driver) | 15:01 |
| iurygregory | JayF, it's on my list for next week, but I will push a draft up today | 15:02 |
| iurygregory | o/ | 15:02 |
| iurygregory | I'm planning on updating https://docs.openstack.org/ironic/latest/admin/firmware-updates.html | 15:02 |
| iurygregory | to include the information about the firmware interface | 15:03 |
| JayF | awesome, thanks | 15:24 |
| rpittau | bye everyone, see you on monday! o/ | 15:27 |
| iurygregory | rpittau, bye o/ | 15:27 |
| JayF | \o | 15:32 |
| iurygregory | JayF, the images you used in the Ironic Toolbox were generated by AI right? do you mind tell me what you used ? :D | 15:34 |
| JayF | Microsoft Designer is what I use for most of that stuff at the moment. Not generally my preferred AI overall but it's free and at hand. | 15:34 |
| JayF | I will warn you; you will get some small % backlash for using obviously-AI-generated images | 15:35 |
| iurygregory | I can imagine that.. | 15:36 |
| iurygregory | tks! | 15:36 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic-tempest-plugin master: A simple standalone test for in-band inspection https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/925981 | 15:50 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: Enable the inspection test in the redfish standalone job https://review.opendev.org/c/openstack/ironic/+/926647 | 15:50 |
| dtantsur | I"m properly annoyed at how many iterations this ^^ is taking.. | 15:55 |
| TheJulia | iterations are always many | 15:56 |
| opendevreview | Jay Faulkner proposed openstack/ironic master: Clarify how to configure for non-master branch https://review.opendev.org/c/openstack/ironic/+/926939 | 17:32 |
| opendevreview | James Denton proposed openstack/ironic master: Update network_data.json to include dns nameservers https://review.opendev.org/c/openstack/ironic/+/926685 | 18:23 |
| TheJulia | jamesdenton: just thinking/wondering, what when there are multiple servers... | 18:31 |
| jamesdenton | multiple servers works, i was simply following protocol for tests | 18:34 |
| TheJulia | okay | 18:35 |
| jamesdenton | but i can definitely update the test to include 2 dns servers instead, if you'd rather | 18:36 |
| TheJulia | I just kind of wonder if it makes sense to change one of the tests so it does more than one, given it is a for loop | 18:36 |
| TheJulia | but *shrugs* | 18:36 |
| jamesdenton | sure, let me spin that | 18:36 |
| opendevreview | James Denton proposed openstack/ironic master: Update network_data.json to include dns nameservers https://review.opendev.org/c/openstack/ironic/+/926685 | 18:51 |
| opendevreview | James Denton proposed openstack/ironic master: Update network_data.json to include dns nameservers https://review.opendev.org/c/openstack/ironic/+/926685 | 18:52 |
| opendevreview | Merged openstack/ironic stable/2023.1: [CI][stable only] fix zuul config https://review.opendev.org/c/openstack/ironic/+/926638 | 19:16 |
| JayF | jamesdenton: https://review.opendev.org/c/openstack/ironic/+/926685/4 sorry for asking for more churn :( but on the bright side it means we'll backport it? | 19:27 |
| JayF | I shoulda caught the missing relnote on ps1 when I reviewed | 19:27 |
| jamesdenton | release note... no biggie. | 19:35 |
| JayF | I'll quickly core review if you can update, thank you :) | 19:37 |
| *** melwitt is now known as jgwentworth | 19:38 | |
| *** jgwentworth is now known as melwitt | 19:38 | |
| opendevreview | James Denton proposed openstack/ironic master: Update network_data.json to include dns nameservers https://review.opendev.org/c/openstack/ironic/+/926685 | 19:43 |
| jamesdenton | \o\ /o/ \o\ /o/ | 19:44 |
| TheJulia | Woot! | 19:44 |
| iurygregory | while writing the docs for firmware updates (via firmware interface) to add to https://docs.openstack.org/ironic/latest/admin/firmware-updates.html , I've noticed we "Only Redfish driver supports firmware updates currently." , but iLO was the first driver to introduce the idea of a clean step (2 to be precise) that would allow updating firmware... | 20:36 |
| JayF | yeah, but it doesn't use firmware interface | 20:39 |
| JayF | and that driver is on the way to death so... | 20:40 |
| JayF | although I was thinking of proposing an agent firmware update interface plugin | 20:40 |
| cardoe | jamesdenton: I don't like your Python style. -1 ;) | 20:45 |
| cardoe | JayF: I meant like https://review.opendev.org/c/openstack/ironic/+/926227 implements what https://review.opendev.org/c/openstack/ironic/+/902107 was a different way and a different scope | 20:46 |
| JayF | You're probably used to seeing openstack python code from 5-7 years ago like most deployers /s | 20:47 |
| JayF | cardoe: oh, author would abandon the undesired change and link to the new one | 20:47 |
| cardoe | okay. Just wanted to make sure I was doing the right thing. | 20:47 |
| jamesdenton | cardoe I learned Python from Stack Overflow, so.. | 20:47 |
| JayF | I got the good python tutorial when I was at Rackspace :) | 20:48 |
| JayF | Glyph (of twisted fame) on a whiteboard is a great way to learn python, A+++ do recommend | 20:48 |
| JayF | lol | 20:48 |
| cardoe | jamesdenton: I'm just trolling cause we finally got ya back on IRC. | 20:48 |
| jamesdenton | I am also Guest972 in an IRC bouncer in a browser tab that i cannot find | 20:49 |
| cardoe | I mean I close tabs when my box OOMs so I can't judge. | 20:49 |
| jamesdenton | i have 72GB in this iMac and don't have that problem, usually. Though Brave/Chrome really like to test the limit | 20:50 |
| jamesdenton | @JayF i am not sure i have the aptitude for that. | 20:51 |
| cardoe | The person that invented browser tabs figured there would only be a handful. The first day they gave it to someone with ADHD I can only imagine their reaction was "What have I done!?!" | 20:51 |
| JayF | honestly it was a pretty good deal, I taught him a devop or two, he taught me about how inheretance works | 20:51 |
| jamesdenton | browser tabs == bookmarks | 20:51 |
| JayF | I'm pretty sure when I left Rackspace | 20:52 |
| JayF | there was still a bouncer alive from someone who had been off the team for like, 2-3 years | 20:52 |
| jamesdenton | no doubt :D | 20:52 |
| JayF | in one of our OFTC-but-unofficial chat channels | 20:52 |
| cardoe | I was pretty sure when they shut down the OpenStack part of the Castle that the cloud would blink off. Cause there was some box under someone's old desk keeping something alive. | 20:53 |
| jamesdenton | that was the day the spanning tree loop was broken and the internet got faster | 20:54 |
| JayF | I worked at the Island of Misfit Toys | 20:55 |
| JayF | and before you ask Blacksburg or SF, both :) | 20:55 |
| jamesdenton | opposite ends | 20:55 |
| JayF | There has to be an incredibly small number of folks who worked there for like, 9 years, and managed to avoid Texas lol | 20:55 |
| JayF | (living there, I mean) | 20:56 |
| jamesdenton | "the lucky few" | 20:56 |
| JayF | you said it, not me | 20:57 |
| JayF | I will note my wife, who married me when I worked for Rackspace (literally in the racker rally room in BCB), made me swear to never ask her to move to tx :D | 20:57 |
| jamesdenton | in hindsight, not a bad move | 20:57 |
| JayF | we're pretty well planted in Washington these days | 21:00 |
| JayF | highly recommended :D | 21:00 |
| jamesdenton | i've only been to Seattle - seemed nice enough | 21:01 |
| opendevreview | Iury Gregory Melo Ferreira proposed openstack/ironic master: Firmware Update via Firmware Interface Docs https://review.opendev.org/c/openstack/ironic/+/926961 | 21:02 |
| iurygregory | JayF, new interfaces are always welcome :D | 21:02 |
| iurygregory | JayF, the initial draft ^ | 21:02 |
| iurygregory | I probably need to reorganize a bit, since we can now do firmware update via servicing (so it's not "manual cleaning") | 21:03 |
| JayF | I'm sorta gathering ideas for next cycle, kea dhcp + in-band firmware interface implementation are up there | 21:03 |
| JayF | iurygregory: if you want a bonus point, you can use a runbook in one of the examples :D | 21:05 |
| iurygregory | JayF, can you add a comment in the patch so I don't forget? I will only touch this next week :D | 21:06 |
| iurygregory | tomorrow is day off for me | 21:06 |
| iurygregory | good night everyone o/ | 21:08 |
| JayF | \o | 21:09 |
| JayF | enjoy the time off! | 21:09 |
| cardoe | What ya doing with kea dhcp? | 21:09 |
| cardoe | JayF: | 21:09 |
| JayF | jamesdenton: Seattle is fine, but like, I grew up in NC, and Washington definitely has some places that feel a bit unpretentious while still being in driving distance of all the nice amenities ... I wouldn't think that's rep of the whole state | 21:09 |
| cardoe | Real bonus points if ya make firmware update not require the ramdisk ;) | 21:10 |
| JayF | cardoe: https://bugs.launchpad.net/ironic/+bug/2026757 | 21:10 |
| JayF | cardoe: tl;dr: dnsmasq is crashy in CI, maybe in production but timing makes it irrelevant, and crashy is bad | 21:10 |
| JayF | upstream dnsmasq doesn't have an idea how to fix it and appear to have stopped trying | 21:11 |
| cardoe | I have this crazy idea of a k8s-ified DHCP | 21:12 |
| JayF | I agree with an adjective in the middle of that sentence lol | 21:12 |
| cardoe | Memory management in C is hard is basically the summary of that bug. | 21:15 |
| JayF | and they refactored it to try to fix a bug and somehow made it worse | 21:17 |
| JayF | but TBH, this experience is more like, I feel awkward using software in production that has a known-memory-safety issue that they gave up on | 21:17 |
| JayF | makes me ask the question: what about if they had a CVE that only impacted Ironic-like use cases? | 21:18 |
| TheJulia | i don't think they expect users to constantly change records and send hup signals | 21:20 |
| cardoe | Well DHCP as a protocol isn't hard. The hard part for most of the implementations is the database. | 21:20 |
| cardoe | Nobody wants to write out an entire config for every single device. So we've got these shorten forms of "if this then that record gets included" | 21:23 |
| TheJulia | well, if you believe the folks I had on a call earlier, everything always get the same exact config | 21:23 |
| TheJulia | .... not in reality | 21:23 |
| JayF | I view it in terms of bottlenecks: even if our use case is weird, right now our throughput bottleneck for a region is effectively "how fast dnsmasq can update" | 21:24 |
| JayF | so like "we don't expect X" is a bad answer in any event, clearly we need it and are using it | 21:25 |
| JayF | this is why I wanna do kea support | 21:25 |
| TheJulia | well, then maybe it is time to try and do something | 21:29 |
| TheJulia | *something* | 21:29 |
| opendevreview | cid proposed openstack/ironic-python-agent master: Check for existence of an IPMI device https://review.opendev.org/c/openstack/ironic-python-agent/+/926973 | 23:49 |
| cid | o/ | 23:57 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!