| opendevreview | Merged openstack/ironic-python-agent stable/2026.1: Fix: Remove legacy agent_token fallback code https://review.opendev.org/c/openstack/ironic-python-agent/+/997615 | 00:22 |
|---|---|---|
| opendevreview | Steve Baker proposed openstack/ironic master: Switch ironic-tempest-standalone-advanced to autodetect deploy https://review.opendev.org/c/openstack/ironic/+/997666 | 01:29 |
| opendevreview | Riccardo Pittau proposed openstack/bifrost master: Use cirros 0.6.3 as default cirros version https://review.opendev.org/c/openstack/bifrost/+/932175 | 07:54 |
| opendevreview | Merged openstack/ironic-python-agent stable/2025.1: Fix: Remove legacy agent_token fallback code https://review.opendev.org/c/openstack/ironic-python-agent/+/997624 | 10:26 |
| opendevreview | Riccardo Pittau proposed openstack/bifrost master: Use cirros 0.6.3 as default cirros version https://review.opendev.org/c/openstack/bifrost/+/932175 | 13:19 |
| TheJulia | brraaaains | 13:41 |
| dtantsur | WHERE??? | 14:01 |
| TheJulia | I has NONE | 14:04 |
| dtantsur | Damn, I thought you intersected a braincell... I've been catching one the whole morning. | 14:05 |
| TheJulia | I am not an orange. | 14:06 |
| TheJulia | I'm more void ;) | 14:06 |
| dtantsur | :D | 14:07 |
| dtantsur | In our pair, my wife is void and I'm orange | 14:08 |
| TheJulia | heh | 14:17 |
| TheJulia | speaking of void and oranges, they defended the bed from the corgi this morning | 14:28 |
| TheJulia | cardoe: would you be available for a specs review jam session next week ? | 14:29 |
| dtantsur | That's the spirit! How die the overlord endure the rebellion? | 14:29 |
| cardoe | TheJulia: yes. Can I take the call from the lazy river? | 14:29 |
| TheJulia | he went up front in the RV because the void bops him on the head.... a lot. | 14:30 |
| dtantsur | Work-from-tent is always approved of and encouraged | 14:30 |
| TheJulia | cardoe: .... I mean... I guess so?! | 14:30 |
| cardoe | Well then yes. | 14:30 |
| TheJulia | I mean, I'm a prime work form RV'er. | 14:30 |
| TheJulia | Anyway, what day and approximately what time would be good for you ? | 14:30 |
| TheJulia | taking the corgi for his walk before my next meeting | 14:31 |
| cardoe | usually 8am to 9:30am central are terrible for me | 14:31 |
| cardoe | Let's call it 10am | 14:32 |
| cardoe | So 10am and later any day of the week should be good. | 14:32 |
| cardoe | Except Monday | 14:32 |
| * cardoe glares at Outlook for refreshing oddly. | 14:33 | |
| dtantsur | Should be 5pm for me, pretty doable (I can normally do 6pm too). Only Thursday already fully booked. | 14:36 |
| cardoe | If ya want I'll get up earlier. | 14:37 |
| dtantsur | Not a problem for me, I work later hours. Will be an issue if we want e.g. rpittau there. | 14:38 |
| * dtantsur works smth like 11-7 or even later sometimes | 14:38 | |
| cardoe | I usually work 8-5 local time during the summer | 14:39 |
| dtantsur | Timezones, right? A lot of meetings start at 5pm for me (including the Ironic one) | 14:42 |
| cardoe | Yep | 14:43 |
| TheJulia | can we translate it all to UTC to collalesce on a time ? | 14:49 |
| * TheJulia doesn't know the timezones in people's heads | 14:49 | |
| dtantsur | In the summer, I normally work until 1700 UTC. My RH calendar is up-to-date if you want to check it. | 14:50 |
| TheJulia | so if we did 10am central, that would be 8 am for me which *should* work... I think. | 14:51 |
| dtantsur | TheJulia: it's the Tuesday's security slot, right? Without Jay, we may just as well take it. | 14:52 |
| TheJulia | so technically, I'm already triple booked for that slot, but let me see if I can get rid of one of them | 14:53 |
| dtantsur | I can do 1600 UTC on Tue or Wed too. If you ask nicely, I can even do Friday :) | 14:54 |
| opendevreview | Merged openstack/networking-generic-switch master: Try Two: Split NGS admin docs https://review.opendev.org/c/openstack/networking-generic-switch/+/997461 | 15:00 |
| dtantsur | \o/ | 15:01 |
| opendevreview | Merged openstack/virtualbmc master: Migrate setup configuration to pyproject.toml https://review.opendev.org/c/openstack/virtualbmc/+/997579 | 15:22 |
| TheJulia | meeting invite sent, I got my other meeting I moved | 15:49 |
| TheJulia | or at least, promised to be move due to the other person having a conflict | 15:49 |
| opendevreview | Takashi Kajinami proposed openstack/python-ironicclient master: Add openstack-python3-next-jobs job https://review.opendev.org/c/openstack/python-ironicclient/+/997754 | 15:53 |
| opendevreview | Takashi Kajinami proposed openstack/python-ironicclient master: Remove Python 3.10 support https://review.opendev.org/c/openstack/python-ironicclient/+/997755 | 15:53 |
| opendevreview | Takashi Kajinami proposed openstack/ironic master: Remove Python 3.10 support https://review.opendev.org/c/openstack/ironic/+/997756 | 15:55 |
| opendevreview | Merged openstack/virtualbmc master: Update supported python versions https://review.opendev.org/c/openstack/virtualbmc/+/997580 | 16:40 |
| TheJulia | Any demand/interest for risc-v out there? | 16:40 |
| opendevreview | Takashi Kajinami proposed openstack/sushy master: Add openstack-python3-next-jobs https://review.opendev.org/c/openstack/sushy/+/997769 | 16:44 |
| opendevreview | Takashi Kajinami proposed openstack/sushy master: Remove Python 3.10 support https://review.opendev.org/c/openstack/sushy/+/997770 | 16:44 |
| opendevreview | Verification of a change to openstack/tenks master failed: Add openstack.cloud collection to requirements https://review.opendev.org/c/openstack/tenks/+/993060 | 17:02 |
| opendevreview | Julia Kreger proposed openstack/ironic-specs master: Libssh based switch configuration drivers? https://review.opendev.org/c/openstack/ironic-specs/+/997653 | 17:39 |
| JayF | TheJulia: in a "that's cool" sense, sure. I don't see it getting much traction in larger orgs (yet?) | 17:45 |
| opendevreview | Jay Faulkner proposed openstack/ironic-python-agent master: Allow limiting what config can be set in mDNS https://review.opendev.org/c/openstack/ironic-python-agent/+/997638 | 17:48 |
| opendevreview | Julia Kreger proposed openstack/networking-generic-switch master: Add Juniper Junos multicast BUM replication support https://review.opendev.org/c/openstack/networking-generic-switch/+/981307 | 19:21 |
| rm_work[m] | hey do you guys know/remember why Ironic RPC addresses conductors by embedding the hostname into the topic on the client side (get_topic_for -> ironic.conductor_manager.<host>) instead of passing it as server= on prepare()? the conductor binds with `Target(server=self.host)`, but the client never sets server, even though it even parses the host back out of the topic in _prepare_call. is that just historical, or does something | 19:52 |
| rm_work[m] | depend on it being in the topic string? | 19:52 |
| rm_work[m] | I ran into this in the http-rpc work and worked around it, but i'm at the point where i'm going through and trying to clean up and validate all my assumptions and this still stands out to me as an odd one-off | 19:52 |
| TheJulia | so the jsonrpc stuffs was largely for just single api and single conductor. The messaging model is fanout mdoel because the client doesn't lock to a specific conductor, they are addressed based upon the destination, and each conductor is bound to their specific topic for the api to address to it | 19:57 |
| TheJulia | (and conductor itself) | 19:58 |
| opendevreview | Julia Kreger proposed openstack/networking-generic-switch master: Add opt-in MTU management for switch ports https://review.opendev.org/c/openstack/networking-generic-switch/+/984832 | 20:00 |
| opendevreview | Julia Kreger proposed openstack/networking-generic-switch master: Add opt-in port carrier bounce on bind https://review.opendev.org/c/openstack/networking-generic-switch/+/984833 | 20:00 |
| opendevreview | Julia Kreger proposed openstack/networking-generic-switch master: Add opt-in STP edge port and BPDU guard support https://review.opendev.org/c/openstack/networking-generic-switch/+/984847 | 20:00 |
| rm_work[m] | hmm ok, I think I'm going to make a kind of garbage CR just so I can visualize and explain the issue and you can tell me why i'm dumb maybe? XD | 20:05 |
| rm_work[m] | with AI these days I find it easier/quicker just to do that with code than be vague trying to badly explain what i'm trying to figure out lol | 20:08 |
| opendevreview | Mathieu Parent proposed openstack/ironic-python-agent master: Allow auto_tls_allowed_clock_skew via kernel cmdline or vmedia https://review.opendev.org/c/openstack/ironic-python-agent/+/997788 | 20:20 |
| TheJulia | fair, I guess :) | 20:30 |
| TheJulia | I REALLY think we should default out to a larger clock skew window | 20:34 |
| opendevreview | Adam Harwell proposed openstack/ironic master: Address conductors by Target.server, not host-in-topic https://review.opendev.org/c/openstack/ironic/+/997789 | 20:39 |
| rm_work[m] | TheJulia: ^^ | 20:39 |
| rm_work[m] | AI seems to think that wouldn't immediately break everything but like, I do not trust that assessment XD | 20:39 |
| TheJulia | so, that is not that different | 20:41 |
| TheJulia | fwiw | 20:41 |
| rm_work[m] | not that different, as in... it would be fine? | 20:42 |
| TheJulia | it might be | 20:42 |
| rm_work[m] | I am spinning up a job right now (regardless of what happens with this) that tests Ironic with my HTTP-RPC stuff, so I could just Depends-On this CR and see what happens | 20:43 |
| rm_work[m] | it removes a very fragile bit where we have to dot-walk the topic to get the host on the oslo.messaging side | 20:43 |
| rm_work[m] | and just... makes Ironic match literally every other service | 20:43 |
| TheJulia | yeah, I'm curious how CI takes it | 20:44 |
| rm_work[m] | the other oddity I'm working around is that ironic includes the service name in the topic (ironic.conductor_manager) | 20:45 |
| TheJulia | That would be a super breaking change | 20:45 |
| rm_work[m] | but it's... fine, mostly just cosmetic I think | 20:45 |
| TheJulia | yeah | 20:45 |
| rm_work[m] | yeah I figured that part out :D | 20:45 |
| rm_work[m] | I have just had to special-case it | 20:46 |
| rm_work[m] | HTTP-RPC is super duper cool BTW | 20:48 |
| rm_work[m] | RMQ can go fly a kite | 20:48 |
| TheJulia | heh, yeah | 20:49 |
| rm_work[m] | we completely removed RMQ from our prod clouds | 20:49 |
| rm_work[m] | it is glorious | 20:49 |
| rm_work[m] | went from an incident a week that was RMQ related to ... nothing | 20:49 |
| TheJulia | nice | 20:57 |
| rm_work[m] | i kinda want to go scream from rooftops that everyone needs to immediately go Rabbit hunting and purge it from the ecosystem :D | 21:00 |
| rm_work[m] | but I think a lot of folks likely just don't have the time or interest in trying something experimental T_T | 21:00 |
| TheJulia | yeah, and I think it sort of gets miss-used due to the nature | 21:03 |
| TheJulia | and its harder to tease all that apart | 21:03 |
| rm_work[m] | kafka works brilliantly for notification transport, and http-rpc just drops in to replace rabbit, very easy, but I am sure there are edge cases | 21:04 |
| rm_work[m] | though it has worked with every service we run and that is quite a few non-standard | 21:04 |
| opendevreview | Julia Kreger proposed openstack/networking-generic-switch master: Add ngs_ssh_allowed_algorithms for SSH cipher control https://review.opendev.org/c/openstack/networking-generic-switch/+/991225 | 21:05 |
| TheJulia | yeah, I've only been through a few project's rpc code and its a bit of a blur to me | 21:06 |
| TheJulia | I just know I had a broken deployment once when we lost the message bus contents | 21:06 |
| TheJulia | but that was also a super duper long time ago | 21:06 |
| rm_work[m] | if you ever get a wild hair and just want to see what a better system looks like, hit me up :P | 21:07 |
| rm_work[m] | I have lots of devstack jobs that spin it up you could look at too | 21:07 |
| rm_work[m] | though realistically I realize that'll never happen because ya'll are so busy lol | 21:07 |
| opendevreview | Merged openstack/ironic-python-agent stable/2025.2: Fix: Remove legacy agent_token fallback code https://review.opendev.org/c/openstack/ironic-python-agent/+/997449 | 21:11 |
| TheJulia | always busy, unfortunately | 22:44 |
| rm_work | https://review.opendev.org/c/openstack/oslo.messaging/+/997783?tab=change-view-tab-header-zuul-results-summary | 23:10 |
| rm_work | So that job tests ironic with my original workaround | 23:11 |
| rm_work | I’m going to set it to use the CR I posted earlier and we’ll see | 23:11 |
| rm_work | If it passes, do you think you’d consider approving? I mean not without possibly some cleanup and thorough review, but I mean conceptually no hard blockers? | 23:13 |
| TheJulia | eh, quite likely, but I would need to sit down and take a good look at it just to make sure. | 23:50 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!