| *** mhen_ is now known as mhen | 01:45 | |
| opendevreview | Andrew Bonney proposed openstack/nova stable/2025.1: Don't reset port dns_name when shelving instances https://review.opendev.org/c/openstack/nova/+/956216 | 07:14 |
|---|---|---|
| sp-bmilanov | sean-k-mooney hi, maybe offtopic, but I just saw the "Assisted-By: Claude-Code" trailer at https://review.opendev.org/c/openstack/nova-specs/+/953940 ... I don't know if you've discussed it somewhere, but can you share a bit about your experience writing the spec and how you used Claude-Code? What were you most pleased with / most frustrated with? Do you have any advice if anyone else wants to use an LLM for writing spe | 07:22 |
| sp-bmilanov | nts? Not looking to use something at the moment personally, but I am curious about the experience of others (even more so in the context of OpenStack) | 07:22 |
| opendevreview | Ian Watson proposed openstack/nova master: Add migration conditional for resource tracker https://review.opendev.org/c/openstack/nova/+/955690 | 10:24 |
| opendevreview | Stephen Finucane proposed openstack/nova master: Fix doc comment on manager role change https://review.opendev.org/c/openstack/nova/+/956189 | 11:03 |
| gboutry | Hello Nova, I'm looking for arm64 live migration capability, and it looks like it's currently not support: https://docs.openstack.org/nova/latest/user/support-matrix.html#operation_live_migrate | 12:10 |
| gboutry | Can someone confirm this docs is up to date and this is unsupported? | 12:10 |
| opendevreview | Andrew Bonney proposed openstack/nova stable/2025.1: Don't reset port dns_name when shelving instances https://review.opendev.org/c/openstack/nova/+/956216 | 12:18 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Simplify servers views (1/3) https://review.opendev.org/c/openstack/nova/+/956231 | 12:22 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Simplify servers views (2/3) https://review.opendev.org/c/openstack/nova/+/956232 | 12:22 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Simplify servers views (3/3) https://review.opendev.org/c/openstack/nova/+/956233 | 12:22 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for servers APIs (1/6) https://review.opendev.org/c/openstack/nova/+/956234 | 12:23 |
| stephenfin | incoming | 13:09 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Address issues with images APIs https://review.opendev.org/c/openstack/nova/+/956102 | 13:09 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Separate volume, snapshot and volume attachments https://review.opendev.org/c/openstack/nova/+/952347 | 13:09 |
| opendevreview | Stephen Finucane proposed openstack/nova master: tests: Use valid UUIDs for cinder resources https://review.opendev.org/c/openstack/nova/+/952935 | 13:09 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Only apply "soft" additionalProperties validation to requests https://review.opendev.org/c/openstack/nova/+/952936 | 13:09 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for volumes APIs https://review.opendev.org/c/openstack/nova/+/952348 | 13:09 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for snapshots APIs https://review.opendev.org/c/openstack/nova/+/952349 | 13:09 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for volume attachments APIs https://review.opendev.org/c/openstack/nova/+/952350 | 13:09 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for floating IP APIs https://review.opendev.org/c/openstack/nova/+/952972 | 13:09 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for security group APIs https://review.opendev.org/c/openstack/nova/+/952973 | 13:09 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for services APIs https://review.opendev.org/c/openstack/nova/+/953196 | 13:09 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for server usage audit log APIs https://review.opendev.org/c/openstack/nova/+/953209 | 13:09 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for tenant network APIs https://review.opendev.org/c/openstack/nova/+/956088 | 13:09 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for simple tenant usage APIs https://review.opendev.org/c/openstack/nova/+/956096 | 13:09 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for limits API https://review.opendev.org/c/openstack/nova/+/956139 | 13:09 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for quota class sets API https://review.opendev.org/c/openstack/nova/+/956140 | 13:09 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for quota sets API https://review.opendev.org/c/openstack/nova/+/956141 | 13:09 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for migrations API https://review.opendev.org/c/openstack/nova/+/956142 | 13:09 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for server migrations API https://review.opendev.org/c/openstack/nova/+/956143 | 13:09 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for server tags API https://review.opendev.org/c/openstack/nova/+/956144 | 13:09 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for server metadata APIs https://review.opendev.org/c/openstack/nova/+/956145 | 13:09 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Simplify servers views (1/3) https://review.opendev.org/c/openstack/nova/+/956231 | 13:09 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Simplify servers views (2/3) https://review.opendev.org/c/openstack/nova/+/956232 | 13:09 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Simplify servers views (3/3) https://review.opendev.org/c/openstack/nova/+/956233 | 13:09 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for servers APIs (1/6) https://review.opendev.org/c/openstack/nova/+/956234 | 13:09 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for servers APIs (2/6) https://review.opendev.org/c/openstack/nova/+/956236 | 13:09 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for servers APIs (3/6) https://review.opendev.org/c/openstack/nova/+/956237 | 13:10 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for servers APIs (4/6) https://review.opendev.org/c/openstack/nova/+/956238 | 13:10 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for servers APIs (5/6) https://review.opendev.org/c/openstack/nova/+/956239 | 13:10 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for servers APIs (6/6) https://review.opendev.org/c/openstack/nova/+/956240 | 13:10 |
| opendevreview | Stephen Finucane proposed openstack/nova master: tests: Invert validation check https://review.opendev.org/c/openstack/nova/+/956241 | 13:10 |
| stephenfin | gmaan: sean-k-mooney: that is _everything_ | 13:10 |
| sean-k-mooney | :) | 13:10 |
| sean-k-mooney | gboutry: i bleive it works, im not aware of anythign the prevents it form workign for libvirt with kvm at least | 13:11 |
| sean-k-mooney | gboutry: the main issue is we do not have a devstack job that runs on aarch64 to test it let alone a mutli node one | 13:12 |
| sean-k-mooney | we have had tirdpary cis validate that feature in the past for other archatechtures | 13:13 |
| gboutry | Thank you very much sean-k-mooney | 13:22 |
| opendevreview | melanie witt proposed openstack/nova master: WIP libvirt: Get info with abs path, rebase with rel path https://review.opendev.org/c/openstack/nova/+/955039 | 16:19 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for migrations API https://review.opendev.org/c/openstack/nova/+/956142 | 16:32 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for server migrations API https://review.opendev.org/c/openstack/nova/+/956143 | 16:32 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for server tags API https://review.opendev.org/c/openstack/nova/+/956144 | 16:32 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for server metadata APIs https://review.opendev.org/c/openstack/nova/+/956145 | 16:32 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Simplify servers views (1/3) https://review.opendev.org/c/openstack/nova/+/956231 | 16:32 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Simplify servers views (2/3) https://review.opendev.org/c/openstack/nova/+/956232 | 16:32 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Simplify servers views (3/3) https://review.opendev.org/c/openstack/nova/+/956233 | 16:32 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for servers APIs (1/6) https://review.opendev.org/c/openstack/nova/+/956234 | 16:32 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for servers APIs (2/6) https://review.opendev.org/c/openstack/nova/+/956236 | 16:32 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for servers APIs (3/6) https://review.opendev.org/c/openstack/nova/+/956237 | 16:32 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for servers APIs (4/6) https://review.opendev.org/c/openstack/nova/+/956238 | 16:32 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for servers APIs (5/6) https://review.opendev.org/c/openstack/nova/+/956239 | 16:32 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for servers APIs (6/6) https://review.opendev.org/c/openstack/nova/+/956240 | 16:32 |
| opendevreview | Stephen Finucane proposed openstack/nova master: tests: Invert validation check https://review.opendev.org/c/openstack/nova/+/956241 | 16:32 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for server shares APIs https://review.opendev.org/c/openstack/nova/+/956266 | 16:32 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Allow to start unit test without eventlet https://review.opendev.org/c/openstack/nova/+/953436 | 16:33 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Run unit test with threading mode https://review.opendev.org/c/openstack/nova/+/953475 | 16:33 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: [test]RPC using threading or eventlet selectively https://review.opendev.org/c/openstack/nova/+/953815 | 16:33 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: [CI]Make nova-tox-py312-threading voting https://review.opendev.org/c/openstack/nova/+/955791 | 16:33 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: [vncproxy]Handle ssl.wrap_socket removal in py312 https://review.opendev.org/c/openstack/nova/+/955915 | 16:33 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Make RBD Tpool usage conditional https://review.opendev.org/c/openstack/nova/+/956089 | 16:34 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Make libvirt Tpool proxying conditional https://review.opendev.org/c/openstack/nova/+/956090 | 16:34 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Fix ProviderTree copying with threading Lock https://review.opendev.org/c/openstack/nova/+/956091 | 16:34 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: [test]Further categorization of disabled unit tests https://review.opendev.org/c/openstack/nova/+/956092 | 16:34 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Warn on long task wait time for executor https://review.opendev.org/c/openstack/nova/+/952666 | 16:34 |
| gmaan | stephenfin: sure? I was happy on tuesday that i finished the series :P | 16:38 |
| stephenfin | I mean, I can find more things 🙃 But no, done. I wasn't submitting them all at once because...107 (!) patches is a lot of patches | 16:39 |
| gmaan | I know and understand. | 16:50 |
| sean-k-mooney | gmaan: speaking of patchs i left some comment on your docs fix yesterday | 17:14 |
| sean-k-mooney | oh stephen adressed them | 17:14 |
| gmaan | sean-k-mooney: yeah, I updated those and further stephenfin refined those more | 17:14 |
| sean-k-mooney | ok ill review that now and we can hopefully land that soonish | 17:15 |
| gmaan | thanks | 17:16 |
| opendevreview | melanie witt proposed openstack/nova master: DNM test volume snapshot NFS https://review.opendev.org/c/openstack/nova/+/956282 | 17:51 |
| opendevreview | melanie witt proposed openstack/nova master: DNM test volume snapshot NFS https://review.opendev.org/c/openstack/nova/+/956282 | 17:54 |
| melwitt | sean-k-mooney: you are too fast 😂 | 17:54 |
| sean-k-mooney | hehe it was just good timing | 17:55 |
| melwitt | I actually did it right on my WIP patch but now I want to test the baseline and see if the existing volume snapshot scenario test repro's the bug or if I need to add something to the tempest test | 17:55 |
| sean-k-mooney | this is for the removal of the last snapshot or a diffent nfs snapthot bug | 17:56 |
| sean-k-mooney | sorry its for https://review.opendev.org/c/openstack/nova/+/955039 | 17:57 |
| sean-k-mooney | the absulte path issue | 17:57 |
| sean-k-mooney | melwitt: did we confirm if devstack-plugin-nfs-tempest-full is doing cinder nfs nova nfs or both | 17:58 |
| melwitt | sean-k-mooney: oh bleh I forgot about that | 17:59 |
| sean-k-mooney | i see only one compute so i assuem cidner nfs but not nova on nfs | 17:59 |
| melwitt | yeah I expect you're right | 17:59 |
| sean-k-mooney | it could be both but it would not test live migraiton | 17:59 |
| melwitt | thanks for mentioning it. my brain is just 😑 | 18:00 |
| sean-k-mooney | ya so its cinder nfs and that will verify the issue your tryign to fix | 18:01 |
| sean-k-mooney | https://zuul.opendev.org/t/openstack/build/ae12c161604b49e49e05b8ccef739785/log/controller/logs/etc/cinder/cinder_conf.txt | 18:01 |
| sean-k-mooney | so the nova-ovs-hybrid plug job currently test well ml2/ovs with hybrid plug for move ops but also spice on debian 12 | 18:03 |
| sean-k-mooney | i want to rename that to nova-alt | 18:03 |
| sean-k-mooney | so like nova-next but no implciation that the alternitive config it is testing are intended to become new defaults | 18:03 |
| sean-k-mooney | i think we could slot nfs testing into that too | 18:04 |
| melwitt | ah ok | 18:05 |
| melwitt | I'd at least name it nova-alt-config because nova-alt sounds too vague to me | 18:07 |
| sean-k-mooney | but want if i want a dark mode emo job :) | 19:10 |
| sean-k-mooney | but ya good idea i just want to name it somehting better then it is currenlty if we extend the focus of the job | 19:11 |
| opendevreview | Merged openstack/nova master: Fix doc comment on manager role change https://review.opendev.org/c/openstack/nova/+/956189 | 19:12 |
| melwitt | yep agreed | 19:16 |
| melwitt | nova-goth | 19:16 |
| melwitt | the dark practice of using nfs | 19:17 |
| melwitt | ok so the existing tempest test doesn't trigger the bug so I'll try adding something to it | 19:18 |
| sean-k-mooney | oh so you wont see it in single node | 19:24 |
| sean-k-mooney | so the relitive path bug happens because thing are mounted at diffent locations based on cidner volume vs nova presecitve right | 19:25 |
| sean-k-mooney | so i think to see it you will need a 2 node jobs wherne you run nova compute on a serepte ndoe form cidner | 19:26 |
| melwitt | yes | 19:26 |
| sean-k-mooney | ya so you will need to tweak the job to do that by usign the multi node node set and disbale n-cpu on the contoller | 19:26 |
| melwitt | I wonder if the lack of a second snapshot is related | 19:27 |
| sean-k-mooney | yes perhaps try that first | 19:27 |
| melwitt | ++ | 19:27 |
| sean-k-mooney | im not sure if it iss but that was what war reported right multiple snaphost and it was the last oen that failed to be removed | 19:27 |
| melwitt | right. the repro steps also say to create 2 or more snapshots | 19:28 |
| sean-k-mooney | im not entirly sure why that woudl matter but it woudl likely explain why it was missed | 19:28 |
| sean-k-mooney | beyond the fact we dont test nfs i mean :) | 19:29 |
| melwitt | heh | 19:29 |
| opendevreview | melanie witt proposed openstack/nova master: DNM test volume snapshot NFS https://review.opendev.org/c/openstack/nova/+/956282 | 20:02 |
| mikal | sean-k-mooney: https://review.opendev.org/c/openstack/nova/+/950643 is ready for a quick re-review whenever you have a minute. | 20:46 |
| opendevreview | melanie witt proposed openstack/nova master: WIP libvirt: Get info with abs path, rebase with rel path https://review.opendev.org/c/openstack/nova/+/955039 | 21:05 |
| melwitt | ok cool the use of two snapshots did repro the bug without needing to go to multinode | 21:32 |
| sean-k-mooney | mikal: well i just got back from a dog walk and put food in the oven so sure i have 20 mins | 21:36 |
| sean-k-mooney | mikal: looks good to me | 21:39 |
| sean-k-mooney | melwitt: i wonder why it needs two, do you know? | 21:40 |
| melwitt | sean-k-mooney: something to do with nova rebasing the image. there's something where when you delete a volume snapshot while the instance is stopped, nova will end up rebasing the image which corrupts the metadata and then attempt to delete the last snapshot fails on the cinder side | 21:47 |
| melwitt | so the repro is, boot from volume, create two volume snapshots, stop instance, delete the newest volume snapshot (works), delete the oldest volume snapshot (fails) | 21:49 |
| melwitt | this is what I uploaded to do ^ https://review.opendev.org/c/openstack/tempest/+/956302/1/tempest/scenario/test_volume_boot_pattern.py | 21:49 |
| sean-k-mooney | melwitt: so nova is not ment to allow that | 21:54 |
| sean-k-mooney | when the instnace is stopped nova returns a 400 to cinder | 21:55 |
| sean-k-mooney | when it calls the assisted voluem snapshot delete api | 21:55 |
| sean-k-mooney | ya that tesss looks resonable | 21:56 |
| gmaan | sean-k-mooney: that is only case (nova reject based on instance state )when merge_target_file is None https://github.com/openstack/nova/blob/master/nova/compute/api.py#L5926C34-L5926C51 | 21:57 |
| gmaan | NOT None | 21:57 |
| gmaan | when i looked into this bug, i also got confused with when nova reject based on instance state and when not | 21:57 |
| sean-k-mooney | hum | 21:58 |
| sean-k-mooney | well that is proably too narrorw | 21:58 |
| gmaan | yeah | 21:58 |
| sean-k-mooney | i mean we could make it work offline | 21:58 |
| sean-k-mooney | but for right now there are obviously other cases where it can fail | 21:58 |
| sean-k-mooney | https://bugs.launchpad.net/nova/+bug/1919487 | 21:59 |
| sean-k-mooney | that was the bug lee was tryign to mitigate | 21:59 |
| sean-k-mooney | ok so thinking about this | 22:00 |
| sean-k-mooney | if (delete_info.get('merge_target_file') is not None and | 22:00 |
| sean-k-mooney | instance.vm_state != vm_states.ACTIVE | 22:00 |
| sean-k-mooney | ): | 22:01 |
| melwitt | not supposed to allow it? I did not know that | 22:01 |
| sean-k-mooney | so if there is a merge_target_file | 22:01 |
| sean-k-mooney | and the vm is stopped | 22:01 |
| melwitt | coincidence I guess that it works fine with relative paths | 22:01 |
| sean-k-mooney | then we raise | 22:01 |
| melwitt | well, I guess let me wait for my test first. maybe it won't pass | 22:02 |
| sean-k-mooney | melwitt: well i was thinkign ti was not inteded | 22:02 |
| sean-k-mooney | becuase the logs i was looking at had a 400 saying it was not supported | 22:02 |
| melwitt | oh, huh. | 22:02 |
| sean-k-mooney | and if we added it then it was not inteded | 22:02 |
| sean-k-mooney | so i think that was hitting this check | 22:02 |
| sean-k-mooney | there are too bugs downstream for the issue your looking at fixing | 22:04 |
| sean-k-mooney | this is in the cinder log of the other one | 22:05 |
| sean-k-mooney | Call to Nova delete snapshot failed: novaclient.exceptions.BadRequest: Instance 9b578630-6c54-4481-8c49-6e0dd5d7e1fa in vm_state stopped. Cannot volume_snapshot_delete while the instance is in this state. (HTTP 400) | 22:05 |
| melwitt | oh 🫤 | 22:05 |
| sean-k-mooney | in the othe rbug they had 3 snapshot | 22:07 |
| sean-k-mooney | the first one deleted fine | 22:07 |
| melwitt | it just looked like such an easy simple thing to put it back to using relative path for the qemu-img rebase | 22:08 |
| sean-k-mooney | the scond one the invalid backing file | 22:09 |
| sean-k-mooney | the third once got that error above | 22:10 |
| sean-k-mooney | melwitt: well it is | 22:10 |
| sean-k-mooney | we shoud fix that first | 22:10 |
| sean-k-mooney | when you have 3 snapshot deliting the most recnet breaked deletin the next oldest becuase of the backing file | 22:11 |
| melwitt | I had no idea the depths of this issue 🙂 | 22:11 |
| melwitt | yeah. that seems to be what happens with the 2 snapshots | 22:11 |
| sean-k-mooney | melwitt: do you remember the last time you fix an nfs snapshot issue.... | 22:11 |
| melwitt | lol | 22:11 |
| melwitt | twas a journey | 22:12 |
| sean-k-mooney | so i dont know if the backing file issue is having sidefect | 22:12 |
| sean-k-mooney | i.e. if that is fixed i dont knwo if the other isssue will happen | 22:13 |
| melwitt | oh, I see what you mean | 22:13 |
| melwitt | hells yeah the test passes | 22:13 |
| sean-k-mooney | oh with your chage | 22:14 |
| sean-k-mooney | cool | 22:14 |
| sean-k-mooney | can you try 3 snapshot? | 22:14 |
| melwitt | yeah. I thought it would but I wanted to have proof | 22:14 |
| melwitt | sean-k-mooney: yeah I can try that. in what order do you want to delete? newest then middle then oldest or? | 22:14 |
| sean-k-mooney | yep that what they did | 22:16 |
| melwitt | k | 22:16 |
| sean-k-mooney | although i dont know if they waited. in the down stream bug they are suing the cli and issue the 3 delete back to back then list the snapshot | 22:16 |
| *** tosky_ is now known as tosky | 22:50 | |
| melwitt | sean-k-mooney: for your tomorrow - test just passed with 3 snapshots also | 23:30 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!