| cardoe | I'm gonna start being more ruthless with ironic-week-prio... ya leave something with a -1 review with no update for 2 weeks... I'm removing it.. I'm telling folks to re-add it once they update it. | 00:18 |
|---|---|---|
| opendevreview | Merged openstack/sushy-tools master: Add get_processors() to AbstractSystemsDriver https://review.opendev.org/c/openstack/sushy-tools/+/965036 | 01:18 |
| opendevreview | Merged openstack/sushy-tools master: Expose System sub-resources based on driver support https://review.opendev.org/c/openstack/sushy-tools/+/965038 | 01:22 |
| opendevreview | Merged openstack/sushy-tools master: Add zombie cleanup for defunct SSH processes https://review.opendev.org/c/openstack/sushy-tools/+/963520 | 01:22 |
| opendevreview | Merged openstack/sushy-tools master: Drop redundant variable definition https://review.opendev.org/c/openstack/sushy-tools/+/953162 | 01:22 |
| opendevreview | Merged openstack/ironic stable/2025.1: Fix : AsRockRack Management via Redfish https://review.opendev.org/c/openstack/ironic/+/966779 | 01:39 |
| opendevreview | Merged openstack/ironic stable/2025.2: Add generic redfish compatible vendor name https://review.opendev.org/c/openstack/ironic/+/967498 | 01:39 |
| opendevreview | Merged openstack/ironic-tempest-plugin master: Remove useless cover target https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/962940 | 01:44 |
| opendevreview | Merged openstack/ironic master: Make sure [cors] allowed_origin accepts a list value https://review.opendev.org/c/openstack/ironic/+/966271 | 01:56 |
| opendevreview | Doug Goldstein proposed openstack/ironic master: chore: use state_path as the base path https://review.opendev.org/c/openstack/ironic/+/968092 | 02:11 |
| opendevreview | Merged openstack/networking-generic-switch stable/2025.1: Correctly set use_keys when key_file is set https://review.opendev.org/c/openstack/networking-generic-switch/+/950297 | 02:32 |
| opendevreview | Merged openstack/ironic master: Allow detach vif in available state https://review.opendev.org/c/openstack/ironic/+/948297 | 02:34 |
| cardoe | ^ now that's an interesting one given the issue I saw | 03:22 |
| cardoe | TheJulia: ^ | 03:22 |
| TheJulia | Oh, in your case it will still fail, but… you had a bad neutron state. Only so much we can really do | 03:24 |
| TheJulia | Anyway, back to it… | 03:25 |
| TheJulia | It https://usercontent.irccloud-cdn.com/file/N1J2cARe/1763781963.JPG | 03:26 |
| TheJulia | It does occur to me. While they are making our goal keeper earn his pay… that it’s likely a race worsened by the removal of eventlet. Since we were time slicing across green threads before and now it’s a whole shiny unwinding thread. | 03:28 |
| cardoe | Yep with my loaded down box. | 03:31 |
| cardoe | JayF: https://review.opendev.org/c/openstack/ironic-specs/+/501511 that's an interesting one given the runbooks stuff. We also discussed that previously. | 03:31 |
| TheJulia | The other aspect, and this one worries me. Is how quickly the db is running | 03:34 |
| TheJulia | Since the db is the lock holder and … shoot I want db metrics | 03:34 |
| TheJulia | Under heavy load, my experience has been MySQL is not the fastest on a writing a change | 03:35 |
| opendevreview | Merged openstack/ironic stable/2025.1: Memoize calls to bcrypt.checkpw https://review.opendev.org/c/openstack/ironic/+/958316 | 03:36 |
| opendevreview | Doug Goldstein proposed openstack/ironic stable/2025.2: Allow detach vif in available state https://review.opendev.org/c/openstack/ironic/+/968096 | 03:39 |
| opendevreview | Merged openstack/ironic stable/2025.1: No admin-set maintenance override on power sync https://review.opendev.org/c/openstack/ironic/+/963856 | 03:44 |
| opendevreview | Merged openstack/ironic-python-agent stable/2025.1: Hint at sector sizes when reporting an invalid written image https://review.opendev.org/c/openstack/ironic-python-agent/+/956662 | 03:50 |
| TheJulia | cardoe: I might toss a patch up, if I remember, to swap the code pattern to lock the row on the update. It might slow it down on that detach, but it should block long enough that we can be sure sure | 03:52 |
| TheJulia | Nvmd, thinking the wrong thing. It’s the node | 03:54 |
| opendevreview | Merged openstack/ironic stable/2025.2: Add upgrade check for iLO/iLO5 driver deprecation https://review.opendev.org/c/openstack/ironic/+/965158 | 04:06 |
| opendevreview | Merged openstack/ironic master: trivial: revise text from mutlinode job changes https://review.opendev.org/c/openstack/ironic/+/951766 | 04:36 |
| opendevreview | Merged openstack/ironic master: trivial: follow-up on ftp enable/disable flag https://review.opendev.org/c/openstack/ironic/+/942357 | 05:01 |
| opendevreview | Merged openstack/ironic master: Skip BMC detection in IPA for out-of-band management https://review.opendev.org/c/openstack/ironic/+/967166 | 07:07 |
| opendevreview | Merged openstack/ironic master: Fix spelling errors in unit test files https://review.opendev.org/c/openstack/ironic/+/967258 | 07:07 |
| opendevreview | nidhi proposed openstack/ironic master: Add Redfish LLDP data collection support to the Redfish inspection interface. https://review.opendev.org/c/openstack/ironic/+/967841 | 07:42 |
| opendevreview | Merged openstack/ironic master: Drop remaining inspector job https://review.opendev.org/c/openstack/ironic/+/967397 | 09:06 |
| opendevreview | Merged openstack/ironic master: Update devstack guides to raise RAM requirement https://review.opendev.org/c/openstack/ironic/+/967087 | 09:35 |
| opendevreview | Takashi Kajinami proposed openstack/ironic master: Remove inspector inspect interface https://review.opendev.org/c/openstack/ironic/+/967476 | 09:47 |
| opendevreview | Takashi Kajinami proposed openstack/ironic master: Remove inspector inspect interface https://review.opendev.org/c/openstack/ironic/+/967476 | 09:49 |
| opendevreview | Takashi Kajinami proposed openstack/ironic master: Drop reference to TripleO https://review.opendev.org/c/openstack/ironic/+/968108 | 09:49 |
| opendevreview | Merged openstack/ironic master: Drop reference to TripleO https://review.opendev.org/c/openstack/ironic/+/968108 | 15:01 |
| cardoe | I put Zulu through the paces overnight. | 15:57 |
| opendevreview | Merged openstack/ironic-python-agent master: Skip BMC detection when using out-of-band management https://review.opendev.org/c/openstack/ironic-python-agent/+/967167 | 16:02 |
| opendevreview | Merged openstack/ironic stable/2025.2: api: Allow bios values to be None https://review.opendev.org/c/openstack/ironic/+/963940 | 17:22 |
| opendevreview | Merged openstack/ironic master: Fix cleaning example in docs that sets the BMC clock to use correct arg https://review.opendev.org/c/openstack/ironic/+/963951 | 17:22 |
| opendevreview | Merged openstack/ironic master: trivial: fix benchmark data generation script https://review.opendev.org/c/openstack/ironic/+/955099 | 20:56 |
| opendevreview | Merged openstack/ironic-tempest-plugin master: Replace deprecated assertItemsEqual https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/953589 | 23:11 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!
