| *** 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 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!