| opendevreview | Ghanshyam Maan proposed openstack/placement master: Update policy scope enforcement testing https://review.opendev.org/c/openstack/placement/+/987568 | 01:06 |
|---|---|---|
| opendevreview | Merged openstack/nova master: Add functional tests for compute service graceful shutdown https://review.opendev.org/c/openstack/nova/+/991743 | 01:22 |
| opendevreview | Merged openstack/nova master: Restore 'fork' start method for daemon mode on Python 3.14 https://review.opendev.org/c/openstack/nova/+/987815 | 03:44 |
| opendevreview | Shalini Srivastava proposed openstack/nova master: Filter invalid kwargs in volume_api.create to prevent TypeError https://review.opendev.org/c/openstack/nova/+/993306 | 04:49 |
| opendevreview | Ghanshyam Maan proposed openstack/placement master: Update policy scope enforcement testing https://review.opendev.org/c/openstack/placement/+/987568 | 04:55 |
| priteau | Hello, I have a vGPU question: say I have two device addresses configured for mdev, they get their own resource provider created in placement. Then I remove one of the devices because I don't want vgpu on it anymore. It appears that the resource provider doesn't get deleted from placement. Should I delete the rp manually? | 09:14 |
| priteau | I searched the code and found only two calls to delete_resource_provider outside of test code. They appear to be linked to deletion of a node or service, but not to an update of the mdev configuration | 09:16 |
| opendevreview | Shalini Srivastava proposed openstack/nova master: Filter invalid kwargs in volume_api.create to prevent TypeError https://review.opendev.org/c/openstack/nova/+/993306 | 10:29 |
| LarsErikP | sean-k-mooney: hi! Any chance you could look at this again? <angel emoji> https://bugs.launchpad.net/nova/+bug/2062127 would be very nice to get a fix for this.. | 10:36 |
| opendevreview | Kamil Sambor proposed openstack/nova master: Add concurrency_backend config option for Nova services https://review.opendev.org/c/openstack/nova/+/993760 | 10:36 |
| sean-k-mooney | LarsErikP: looks like its in merge conflict | 11:10 |
| sean-k-mooney | so im not actully working on that Rajat was workong on it at some point | 11:11 |
| sean-k-mooney | i added a fucntioal reporcuder for them becauses they didnt have time but i was not planing to take over that work in general | 11:12 |
| LarsErikP | ok. I understand. so.. how do we resolve the merge conflict? | 11:12 |
| sean-k-mooney | it will need to be rebased | 11:13 |
| LarsErikP | ok. So basically, Rajat might have to pick it up again? | 11:13 |
| sean-k-mooney | and https://review.opendev.org/c/openstack/nova/+/916409/4/nova/compute/manager.py#3763 needs to be fixed | 11:13 |
| sean-k-mooney | well or anyone else with time | 11:14 |
| sean-k-mooney | the thing is i can proably rebase this and fix the bug quickly but then i cant +2 it | 11:14 |
| sean-k-mooney | and i also wont have time to keep chekcign up on this | 11:15 |
| LarsErikP | I understand. I'm sorry I can't really help fixing it, since I'm not a developer :P I'm just really keen on getting it fixed | 11:16 |
| sean-k-mooney | on sec ill do it quickly but best thign to do woudl be then add it to open dicuss in the next nova meetign and see if you can find reviews | 11:17 |
| LarsErikP | Discussing it in the meeting sounds like it would help, imo =) Thanks | 11:19 |
| sean-k-mooney | benifits of ai... is while i might not normally have time to do this right now i have time to tell it to od it. | 11:24 |
| LarsErikP | haha! It's good for something, for sure! | 11:26 |
| opendevreview | sean mooney proposed openstack/nova master: Add regression test for bug 2062127 https://review.opendev.org/c/openstack/nova/+/976654 | 11:49 |
| opendevreview | sean mooney proposed openstack/nova master: Fix: Rebuild volume backed server with snapshot image https://review.opendev.org/c/openstack/nova/+/916409 | 11:49 |
| sean-k-mooney | LarsErikP: i think ^ should be good | 11:49 |
| sean-k-mooney | LarsErikP: https://github.com/openstack/nova/commit/29e1dc8b435e3bda14ba7660a2ff9500a1674c16 was the conflcit | 11:51 |
| sean-k-mooney | LarsErikP: so pretty trivial over all | 11:51 |
| sean-k-mooney | just renameing deadline to timeout since its the time in second to wait not a datetime in the future to compare agaisnt | 11:52 |
| LarsErikP | aha. ok | 11:53 |
| LarsErikP | cool! Thanks for helping out, and bring us a little closer to fixing this =) | 11:54 |
| opendevreview | Masanori Ueno proposed openstack/nova master: Add regression test for bug #2153425 https://review.opendev.org/c/openstack/nova/+/988777 | 14:47 |
| opendevreview | Masanori Ueno proposed openstack/nova master: live-migration: Ensure limits are set before _call_livem_checks_on_host https://review.opendev.org/c/openstack/nova/+/990212 | 14:52 |
| isaacvicente[m] | hi guys, could someone take a quick look at https://review.opendev.org/c/openstack/nova/+/990815? | 16:53 |
| opendevreview | sean mooney proposed openstack/nova master: Add regression test for bug 2062127 https://review.opendev.org/c/openstack/nova/+/976654 | 16:56 |
| opendevreview | sean mooney proposed openstack/nova master: Fix: Rebuild volume backed server with snapshot image https://review.opendev.org/c/openstack/nova/+/916409 | 16:56 |
| sean-k-mooney | LarsErikP: might help if i do that rename in the repoducer patch too :) | 16:56 |
| *** haleyb is now known as haleyb|out | 22:49 | |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!