| opendevreview | melanie witt proposed openstack/nova master: DNM test ceph ipv6 https://review.opendev.org/c/openstack/nova/+/984623 | 00:42 |
|---|---|---|
| opendevreview | Taketani Ryo proposed openstack/nova master: doc: clarify SEV/SEV-ES behavior with locked_memory https://review.opendev.org/c/openstack/nova/+/971300 | 00:53 |
| opendevreview | Ghanshyam Maan proposed openstack/nova master: Add functional test for cleanup_host cancelling queued live migrations https://review.opendev.org/c/openstack/nova/+/986477 | 02:18 |
| opendevreview | Ghanshyam Maan proposed openstack/nova master: DNM: Test removal of enforce_scope https://review.opendev.org/c/openstack/nova/+/986479 | 02:44 |
| gibi | melwitt: I will take a look | 04:33 |
| gibi | melwitt: the only other option I see on the code level is to change the default of wait_for_state_change | 04:35 |
| gibi | maybe just change it in the coverage job via an env var passed? | 04:35 |
| gibi | we have OS_TEST_TIMEOUT as an example for a similar thing. We can have another var like OS_SLOW_ENV_WAIT_FACTOR that defaults to 1 and we can set it to 2 in the coverage env. Then this var can be multiplied in any sleep or retry count we see impacted by the slowness in the coverage env | 04:41 |
| *** ykarel_ is now known as ykarel | 07:44 | |
| opendevreview | Lajos Katona proposed openstack/nova master: Use SDK for Neutron subnets https://review.opendev.org/c/openstack/nova/+/962190 | 08:20 |
| opendevreview | Lajos Katona proposed openstack/nova master: Use SDK for Neutron extensions https://review.opendev.org/c/openstack/nova/+/962270 | 08:20 |
| opendevreview | Lajos Katona proposed openstack/nova master: Use SDK for Neutron floating IPs https://review.opendev.org/c/openstack/nova/+/962604 | 08:20 |
| *** sambork_ is now known as sambork | 09:50 | |
| lajoskatona | Uggla, bauzas: Hi, if you have few minutes I have a small doc update patch for mdev_class: https://review.opendev.org/c/openstack/nova/+/983089 (just a follow-up for bp/generic-mdevs https://review.opendev.org/q/topic:%22bp/generic-mdevs%22) | 10:14 |
| opendevreview | Kamil Sambor proposed openstack/nova master: Change nova-alt-configurations job https://review.opendev.org/c/openstack/nova/+/983179 | 12:46 |
| opendevreview | Merged openstack/nova master: Doc: add short doc section for mdev_class https://review.opendev.org/c/openstack/nova/+/983089 | 13:28 |
| opendevreview | Florian proposed openstack/nova master: Add check for PCIe devices attach limit for volume and ports https://review.opendev.org/c/openstack/nova/+/955584 | 13:39 |
| opendevreview | Dan Smith proposed openstack/nova-specs master: Add unpin-az spec https://review.opendev.org/c/openstack/nova-specs/+/986539 | 13:59 |
| opendevreview | Dan Smith proposed openstack/nova-specs master: Add unpin-az spec https://review.opendev.org/c/openstack/nova-specs/+/986539 | 14:27 |
| opendevreview | Balazs Gibizer proposed openstack/nova-specs master: add pci passthrough groups spec for 2026.2 https://review.opendev.org/c/openstack/nova-specs/+/975635 | 14:31 |
| melwitt | gibi: yeah I kinda thought about a wait multiplier too | 15:12 |
| opendevreview | melanie witt proposed openstack/nova master: DNM test ceph ipv6 https://review.opendev.org/c/openstack/nova/+/984623 | 15:20 |
| opendevreview | melanie witt proposed openstack/nova master: DNM test ceph ipv6 https://review.opendev.org/c/openstack/nova/+/984623 | 16:43 |
| melwitt | gmaan: another unit test stabilization for CI testenv:cover https://review.opendev.org/c/openstack/nova/+/986468 not sure if this will likely be the last "one-off" but if it's a more widespread problem, there are a couple of review comments in there with idea | 16:55 |
| gmaan | melwitt: lgtm, thanks | 17:33 |
| melwitt | thanks gmaan | 17:49 |
| melwitt | sean-k-mooney: was finally able to prove that ipv6 brackets in ceph mon addrs passed to libvirt without stripping the brackets, would fail https://review.opendev.org/c/openstack/nova/+/984623/18 | 17:51 |
| melwitt | I will likely need your help to figure out where and how the proper place to propose all of this ipv6 stuff haha .... for my test I crammed most of it into a pre-run script as you could see | 17:52 |
| sean-k-mooney | melwitt: oh cool | 17:54 |
| sean-k-mooney | i mean that works. it shoudl get added into the devstack roels | 17:55 |
| sean-k-mooney | but we can figure that out now that we have a working baseline | 17:55 |
| melwitt | yeah that was my thinking. hack it for the POC but then propose them in the proper places if it got to working | 17:56 |
| sean-k-mooney | as long as we are calling it a hack im ok with it if we call it a tempory solution it will out live cochroaches | 17:57 |
| melwitt | claude thought some of it should go under openstack/zuul-jobs so maybe that's true, maybe it isn't haha | 17:57 |
| sean-k-mooney | tempory colution that work never get fixed | 17:57 |
| sean-k-mooney | melwitt: yes some of it should | 17:57 |
| sean-k-mooney | we do the multi node bridge configuraiton in zuul/zuul-jobs | 17:57 |
| sean-k-mooney | https://opendev.org/zuul/zuul-jobs/src/branch/master/roles/multi-node-bridge | 17:58 |
| melwitt | yeah. I just didn't want to make the depends-on web really big while I was trial-and-erroring on it | 17:58 |
| melwitt | yes exactly what it was saying | 17:58 |
| sean-k-mooney | so we shudl proably make that proply do ipv6 as well | 17:58 |
| melwitt | yes exactly | 17:58 |
| sean-k-mooney | https://opendev.org/zuul/zuul-jobs/src/branch/master/roles/multi-node-bridge/tasks/switch.yaml | 17:58 |
| sean-k-mooney | same for the host file https://opendev.org/zuul/zuul-jobs/src/branch/master/roles/multi-node-hosts-file/tasks/main.yaml | 17:59 |
| melwitt | I cloned all of the relevant repos and claude knew where stuff should go, which was really cool considering I would have zero idea. I know you told me first but I also had claude look at all of it locally and it made all of the connections between them | 18:00 |
| sean-k-mooney | so if you want to see a beautful hack that i have not updated in a long time | 18:00 |
| melwitt | sure | 18:00 |
| sean-k-mooney | https://github.com/SeanMooney/ard/tree/master/molecule | 18:00 |
| sean-k-mooney | so i dont know if you knwo what molecule is | 18:01 |
| sean-k-mooney | btu its how you test ansible | 18:01 |
| sean-k-mooney | https://github.com/SeanMooney/ard/blob/master/molecule/default/molecule.yml | 18:01 |
| sean-k-mooney | i defiend a moluel role using vagrnat/libvtr/qemu | 18:01 |
| JayF | https://review.opendev.org/c/openstack/nova/+/983753 this is a high quality change to the Ironic virt driver which would have a significant impact on Ironic users, it's also simpler and easy to understand than most ironic virt driver changes. Your attention would be appreciated :D | 18:01 |
| sean-k-mooney | that create a contoler and comptue and then runs the devstack ansible playbooks form the ci jobs on them | 18:01 |
| melwitt | no I didn't know what molecule is. I had heard you say it before but I never knew what you were talking about haha | 18:02 |
| sean-k-mooney | so you can do molecule up adn it will use the ci roles to deploy a 2 node devstack | 18:02 |
| melwitt | ahh that's pretty cool | 18:02 |
| sean-k-mooney | melwitt: i used to ocationlly use it to test jobs changes locally | 18:03 |
| sean-k-mooney | JayF: ay yes that woudl be good to do and yes its pretty simple | 18:03 |
| sean-k-mooney | melwitt: i should see if it still work, but you could point it at phsyicla system too and just run it | 18:04 |
| JayF | you also got a 3-in-1 ironic cores review on it (me/cli.f/ci.d in a GR-OSS team review) | 18:04 |
| JayF | although admittedly some % of that time was spent explaining resechedules in nova :D | 18:04 |
| sean-k-mooney | JayF: technially they nolonger exist | 18:04 |
| sean-k-mooney | we replaced it with alternitive hosts | 18:04 |
| sean-k-mooney | so we can aovid going back to the schduler | 18:05 |
| JayF | is there a meaningful difference from an Ironic standpoint? | 18:05 |
| sean-k-mooney | no | 18:05 |
| sean-k-mooney | the diffent is es dont goback to the chsduelr if ther first host fails | 18:05 |
| JayF | Nice, so you just get to avoid the double dip in the scheduler | 18:05 |
| sean-k-mooney | we callulate alterntive in advnace and just select the next one | 18:05 |
| sean-k-mooney | yep | 18:05 |
| JayF | if those alternatives are scheduled to in the meantime | 18:05 |
| sean-k-mooney | the allcoation fails super early adn we go to the next one | 18:06 |
| JayF | ack, so it is a meaningful change for Ironic users :) | 18:06 |
| sean-k-mooney | basiclly the first thing we do is see if the canidate allcation is still valid and if not we go to the next | 18:06 |
| JayF | there's nearly zero chance that after a timed out deployment (hours) those allocs would still be valid | 18:06 |
| sean-k-mooney | well perhasp but this also changed in like pike or rocky | 18:07 |
| sean-k-mooney | it was part fo the cell_v2 work | 18:07 |
| JayF | Nice to know, albeit WAY too late, about the behavior though. Could be surprising for an Ironic user. | 18:07 |
| * JayF files it away in his nova shelf | 18:07 | |
| JayF | (I'm not saying you all had some responsibility to tell us; mainly annoyed I've had a flawed mental model for bordering-on a decade) | 18:08 |
| JayF | cid: clif: Worth reading the backlog if you wanna fill in the gaps around how retrying actually works now ^^^ | 18:09 |
| dansmith | yeah I think at this point it has worked that way for longer than it worked the other way :) | 18:15 |
| JayF | dansmith: I still find myself un-learning things that I "learned" from the old, hacked to @%$! cloud at my first OpenStack employer :) | 18:16 |
| JayF | this is added to the list, a good 12 years later :( | 18:16 |
| opendevreview | Merged openstack/nova master: Increase wait retries for resize reschedule with qos port test https://review.opendev.org/c/openstack/nova/+/986468 | 19:05 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!