| *** bauzas0 is now known as bauzas | 01:03 | |
| opendevreview | Merged openstack/nova stable/2025.1: Cache [pci]alias parsing https://review.opendev.org/c/openstack/nova/+/952457 | 02:30 |
|---|---|---|
| opendevreview | ChungWon Lee proposed openstack/nova master: Add regression test for bug #2134375 https://review.opendev.org/c/openstack/nova/+/984900 | 06:03 |
| opendevreview | ChungWon Lee proposed openstack/nova master: Fix port remain while error on update port with multi network https://review.opendev.org/c/openstack/nova/+/983672 | 08:25 |
| opendevreview | Kamil Sambor proposed openstack/nova master: Deprecate eventlet concurrency mode https://review.opendev.org/c/openstack/nova/+/987331 | 08:59 |
| opendevreview | ChungWon Lee proposed openstack/nova master: Fix port remain while error on update port with multi network https://review.opendev.org/c/openstack/nova/+/983672 | 10:03 |
| opendevreview | Stephen Finucane proposed openstack/os-vif master: Deprecate version module https://review.opendev.org/c/openstack/os-vif/+/987636 | 11:53 |
| opendevreview | Stephen Finucane proposed openstack/osc-placement master: Deprecate version attribute https://review.opendev.org/c/openstack/osc-placement/+/987669 | 14:43 |
| opendevreview | Stephen Finucane proposed openstack/os-traits master: Deprecate version attribute https://review.opendev.org/c/openstack/os-traits/+/987671 | 14:45 |
| opendevreview | Dan Smith proposed openstack/nova master: Fix resize to unpermitted flavors https://review.opendev.org/c/openstack/nova/+/987704 | 15:15 |
| dansmith | Uggla: this ^ is a public security bug fix.. we decided to do this outside embargo because it's not super impactful.. still would be good to try to prioritize review, merge, and backport I think | 15:17 |
| Uggla | dansmith, yep, I set it in the list, I will warn in the next upstream meeting. Btw, I think we can also ask to have it in a sprint. | 15:20 |
| dansmith | I'd rather not mix our downstream team organization with things like this.. | 15:25 |
| dansmith | bauzas: FYI ^ about the flavor fix, maybe you can hit that quicker since you have context | 15:25 |
| bauzas | dansmith: already reviewing it :) | 15:29 |
| dansmith | bauzas: nice thanks :D | 15:29 |
| bauzas | +2 | 15:31 |
| bauzas | 'd | 15:31 |
| dansmith | excellent thanks.. | 15:32 |
| dansmith | maybe gmaan or melwitt could have a looksee | 15:33 |
| gibi | dansmith: do we want to add a release notes to that bugfix? Other that I'm +2 | 15:43 |
| dansmith | gibi: I dunno, I don't really think we need renos for every bug fix, especially when it's not changing important behavior. The bug is referenced and it's very detailed, and while it is a security thing, it's not very impactful. | 15:44 |
| dansmith | if you think this needs one for some reason I can surely add it | 15:45 |
| gibi | dansmith: I have no specific reason for a reno so I approved the patch as is | 15:47 |
| gibi | I left a follow up request to consider | 15:47 |
| elodilles | melwitt: unfortunately i haven't found any fix for the grenade-skip-level-always job (unmaintained/2024.1 -> stable/2025.1) yet :/ it's on my TODO (but with lower priority). in nova, it was set non-voting some time ago already. in placement i also added the same workaround to unblock the gate (i'd appreciate a review on the patch o:)): https://review.opendev.org/c/openstack/placement/+/980761 | 16:02 |
| opendevreview | Dan Smith proposed openstack/nova master: Refactor get_flavor_by_flavor_id() to require context https://review.opendev.org/c/openstack/nova/+/987723 | 16:31 |
| dansmith | gibi: ^ straight from the devil's mouth to your eyeballs. Minimal checking so far, but I'll go over it in more detail in gerrit | 16:31 |
| gibi | dansmith: looks OK to me | 16:54 |
| dansmith | gibi: ack, thanks for catching my typo, I'll rev both quickly | 16:56 |
| opendevreview | Dan Smith proposed openstack/nova master: Refactor get_flavor_by_flavor_id() to require context https://review.opendev.org/c/openstack/nova/+/987723 | 16:59 |
| gibi | dansmith: thanks. +2 | 17:05 |
| dansmith | o7 | 17:05 |
| sean-k-mooney | dansmith: i +1'd no need to pull it form the gate or anythin but it woudl be good to add a release note for this i think | 17:47 |
| sean-k-mooney | the fix looks good overall so no objection to merging it as is | 17:48 |
| sean-k-mooney | gibi: dansmith by the way i have seen the updates on the pci spec. i have not had time to review them and likely wont till next week but | 17:57 |
| sean-k-mooney | i do see the pain point in the configuration options | 17:57 |
| opendevreview | Dan Smith proposed openstack/nova-specs master: Add unpin-az spec https://review.opendev.org/c/openstack/nova-specs/+/986539 | 18:38 |
| sean-k-mooney | dansmith: https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/986818 does not fix the image pull failure in the ceph jobs but it shoudl help some fo the other transiant failure. if you keep seeing podman failure perhaps we shoudl condier retrying that step as well | 20:32 |
| sean-k-mooney | dansmith: im assuming the 503 was just a tempry failure so hopefully yoru recheck goes through | 20:33 |
| sean-k-mooney | it failed on https://github.com/openstack/devstack-plugin-ceph/blob/master/devstack/lib/cephadm#L373-L374 | 20:36 |
| sean-k-mooney | and we are only usign that to determin which verion of cephadm to install https://github.com/openstack/devstack-plugin-ceph/blob/master/devstack/lib/cephadm#L166-L185 | 20:40 |
| sean-k-mooney | so i feel like we could improve that although it may just fail later | 20:40 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!