| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2025-03-14 | |||
| 14:28:28 | sean-k-mooney | although the seise of a whole impoved alot of things | |
| 14:30:48 | gibi | maybe in Flamingo :) | |
| 14:31:02 | gibi | less golang more python :) | |
| 14:33:39 | sean-k-mooney | :) | |
| 14:33:46 | gibi | I don't know what is the relationship of flamingos a pythons, who would eat who | |
| 14:34:24 | dansmith | pythons would eat flamingos no doubt | |
| 14:35:31 | gibi | cool then. (we have neither this side of the globe) | |
| 14:36:01 | gibi | and gopher (the golang mascot) would definitely loose againts python | |
| 16:03:02 | opendevreview | Balazs Gibizer proposed openstack/nova master: Reproduce bug/2098496 https://review.opendev.org/c/openstack/nova/+/941673 | |
| 16:03:03 | opendevreview | Balazs Gibizer proposed openstack/nova master: Ignore metadata tags in pci/stats _find_pool logic https://review.opendev.org/c/openstack/nova/+/944277 | |
| 16:09:56 | opendevreview | Balazs Gibizer proposed openstack/nova master: Ignore metadata tags in pci/stats _find_pool logic https://review.opendev.org/c/openstack/nova/+/944277 | |
| 17:12:54 | sean-k-mooney | werid i just got a destop notificaiton form a comment gibi made a few hours ago | |
| 17:13:28 | sean-k-mooney | dansmith: not that im disagreeign but dont we test migration with postgres in the tox jobs | |
| 17:13:41 | sean-k-mooney | i would not try and use it in production | |
| 17:14:01 | sean-k-mooney | but we do install it with bindep in the tox env | |
| 17:14:25 | sean-k-mooney | and the opertunistic database stuff should kick in but maybe that broke at some point | |
| 17:14:51 | sean-k-mooney | we defintlly dont have stong testin gthere as we only really apply the migration ot an empty db | |
| 17:18:37 | sean-k-mooney | ya nova.tests.unit.db.main.test_migrations.TestMigrationsWalkPostgreSQL.test_walk_versions is still running in tox | |
| 17:20:28 | dansmith | (back in a sec) | |
| 17:21:05 | sean-k-mooney | dansmith: its not urgent dont worry. teh prostgress test are still runing but obviously those migration tests dont test that much | |
| 17:21:35 | sean-k-mooney | pointing to https://governance.openstack.org/tc/resolutions/20170613-postgresql-status.html is still the best responce | |
| 17:23:10 | sean-k-mooney | i was just concerned that those opertuneistic tests broke at some point and we missed it but the pg and mysqlone are running in addtion to sqlite | |
| 17:26:37 | dansmith | sean-k-mooney: we do, but not full of real world data | |
| 17:27:05 | dansmith | so maybe I was less than accurate about 'we don't test it' but testing with an empty DB is not really testing IMHO | |
| 17:27:14 | dansmith | it makes sure all the schema SQL is right, but not much else | |
| 17:27:40 | sean-k-mooney | ya so i know we incorrectly set the coration type on at least one of the tables when movign to alembic | |
| 17:27:56 | sean-k-mooney | so that might cuase some issues only when there is data | |
| 17:28:07 | sean-k-mooney | or on some locales but not our default | |
| 17:28:40 | sean-k-mooney | i think neutron had a postgress job up until about a year ago | |
| 17:30:53 | dansmith | IMHO people that want to run postgres are more than welcome to test and fix the things | |
| 17:31:27 | sean-k-mooney | ya i dont think its unreasonable | |
| 17:32:05 | sean-k-mooney | i just know that it broke for them at some poitn and no one stepped up to fix it so they droped testing | |
| 17:32:47 | sean-k-mooney | oh wow | |
| 17:32:58 | sean-k-mooney | they actully dropped postgress code https://github.com/openstack/neutron/commit/16e4f0af89cc67b7a8901bfd357fd49b86de73c8 | |
| 17:33:28 | sean-k-mooney | i pointed out to stephen that we techinially still supprot it | |
| 17:33:45 | sean-k-mooney | we just dont test it | |
| 18:09:01 | sean-k-mooney | dansmith: im reluctantly +2 but +2 none the same on the one time use device spec | |
| 18:09:44 | dansmith | I will take a reluctant +2 over the alternative thanks :) | |
| 18:09:49 | sean-k-mooney | i lost that tab like 4 times today.... | |
| 18:11:03 | sean-k-mooney | dansmith: by the way you mentioned a crash when you used the qxl device to test this, you might have better luck with somethign like the random number generator or a virtio-blk device | |
| 18:11:32 | sean-k-mooney | or igb vfs if your qemu supports it | |
| 18:12:22 | sean-k-mooney | i have had better look with nic/block devices when palying with double passthough then graphics cards | |
| 18:12:33 | sean-k-mooney | not sure why but just my personal experince | |
| 18:14:17 | dansmith | yeah, no igb on this box's qemu but yeah I need to use a different device | |
| 18:14:34 | dansmith | honestly it reboots in seconds and I always need to restart nova services after each fail, so it hasn't been so bad :P | |
| 18:15:36 | sean-k-mooney | if you try e1000 instead i think that will work | |
| 18:15:57 | sean-k-mooney | it diefenly works with a doubel passthough of a phsyical e1000 nic but i think the virutal ones work too | |
| 19:13:04 | opendevreview | Dan Smith proposed openstack/nova master: Ignore metadata tags in pci/stats _find_pool logic https://review.opendev.org/c/openstack/nova/+/944277 | |
| #openstack-nova - 2025-03-16 | |||
| 02:23:46 | opendevreview | Takashi Natsume proposed openstack/nova master: Update contributor guide for 2025.2 Flamingo https://review.opendev.org/c/openstack/nova/+/944603 | |
| 13:36:37 | opendevreview | Takashi Kajinami proposed openstack/nova master: Fix description of [pci] alias https://review.opendev.org/c/openstack/nova/+/944611 | |
| 14:32:39 | IshanShanware[m] | sean-k-mooney:... (full message at <https://matrix.org/oftc/media/v1/media/download/AV6erZTaSkUc-1ptdYd_qKTJvrKZM-gcFd4g_bJVy6rXGULoskvXyHay2wXi9H0HKT8DjkWIuL9S_YoRXhpohZBCeV53gI0wAG1hdHJpeC5vcmcveVpPZUhSanJ3eFhCck52TXBrSVJFZklq>) | |
| 19:16:55 | Callum027 | sean-k-mooney: Looking at the agenda for the next meeting, kinda looks like it's still focusing on the current release and not planning for 2025.2. Should I add the image meta changes to the agenda for next week, or is this week fine? | |
| #openstack-nova - 2025-03-17 | |||
| 07:08:29 | r-taketn | hi. I have two entries for next PTG(About CCA, RDT/MPAM). Besides purchasing a ticket and submitting spec, what else should I prepare for the PTG? | |
| 08:03:56 | frickler | Uggla: could you have a quick look at https://review.opendev.org/c/openstack/releases/+/943801 pls? | |
| 09:19:58 | IshanShanware[m] | * sean-k-mooney:... (full message at <https://matrix.org/oftc/media/v1/media/download/Af0thDY-PzrHZLStDMYwU6GFaZIiMSA41JuprrmuPgJq6AY3dEeP8-2cHH70wc9rwy3S6QmtPw_bSYPSO95YLsdCeV64AjOgAG1hdHJpeC5vcmcvVUNuTVBJVllKcFlkQmR5bnhsc0V2Qllv>) | |
| 09:36:12 | frickler | Uggla: bauzas: also what's the status with rc1 for nova+placement? we're past the deadline now | |
| 09:36:43 | bauzas | frickler: as said in the nova rc1 patch, we're missing some patches before tagging | |
| 09:37:09 | bauzas | everything is tracked here https://etherpad.opendev.org/p/nova-epoxy-rc-potential | |
| 09:37:40 | bauzas | placement rc1 can be tagged tho, just double-checking the last dependencies | |
| 09:43:50 | frickler | bauzas: unless I miss something, the only remaining nova change is https://review.opendev.org/c/openstack/nova/+/943952 which is a pretty trivial but necessary change that keeps failing due to unrelated CI instability. let me know once you're ready to get this force-merged to unblock things :-) | |
| 09:44:29 | bauzas_ | frickler: not exactly, we're also missing a prelude patch that needs to be written as copy of the cycle highlights | |
| 09:45:02 | bauzas_ | that's a TODO on me this morning since we start to have a review quorum from the nova cores about the highlights | |
| 10:19:47 | opendevreview | Sylvain Bauza proposed openstack/nova master: Add Epoxy prelude section https://review.opendev.org/c/openstack/nova/+/944713 | |
| 11:29:10 | IshanShanware[m] | <IshanShanware[m]> "sean-k-mooney:..." <- bauzas: any thoughts here ? | |
| 12:14:46 | opendevreview | ribaudr proposed openstack/nova master: FUP improve comment accuracy and variable naming for tag removal https://review.opendev.org/c/openstack/nova/+/943124 | |
| 12:14:46 | opendevreview | ribaudr proposed openstack/nova master: FUP Remove unnecessary PCI check https://review.opendev.org/c/openstack/nova/+/944105 | |
| 12:14:47 | opendevreview | ribaudr proposed openstack/nova master: FUP improve and add integration tests for PCI SR-IOV servers https://review.opendev.org/c/openstack/nova/+/944106 | |
| 12:14:47 | opendevreview | ribaudr proposed openstack/nova master: FUP Add a warning to make non-explicit live migration request debugging easier https://review.opendev.org/c/openstack/nova/+/944133 | |
| 12:14:48 | opendevreview | ribaudr proposed openstack/nova master: FUP Update pci-passthrough and virtual-gpu documentation https://review.opendev.org/c/openstack/nova/+/944153 | |
| 13:45:54 | bauzas | cores, can we please give this a go ? https://review.opendev.org/c/openstack/nova/+/944713 | |
| 13:47:16 | dansmith | bauzas: I did three minutes ago | |
| 13:47:26 | bauzas | cool | |
| 13:49:28 | opendevreview | Sylvain Bauza proposed openstack/nova master: Add Epoxy prelude section https://review.opendev.org/c/openstack/nova/+/944713 | |
| 13:49:34 | bauzas | updated ^ | |
| 13:50:26 | dansmith | same | |
| 13:53:20 | dansmith | bauzas: wanna fix up that cycle highlights one too and I'll +2? | |
| 13:53:27 | bauzas | yup | |
| 13:54:16 | dansmith | bauzas: thanks for always being on top of these release things | |
| 13:54:32 | dansmith | uggla a de grandes chaussures à remplir | |
| 13:54:37 | bauzas | last time unless Uggla leaves me do that again in Flamingo :) | |
| 13:54:46 | bauzas | dansmith: lollylol | |
| 13:55:59 | bauzas | cycle highlights updated too | |
| 13:56:34 | dansmith | oh duh, I can't +2 that | |
| 13:56:41 | dansmith | sorry, but +1d :) | |
| 13:56:57 | opendevreview | Merged openstack/nova master: Update compute rpc alias for epoxy https://review.opendev.org/c/openstack/nova/+/943952 | |
| 13:57:53 | Uggla | @dansmith, you will learn French, that's a good start ! | |
| 13:58:21 | dansmith | I will not.. one french translation via google from me per cycle and that was the one for flamingo :) | |
| 13:59:28 | dansmith | the only one I really "know" is "voulez-vous coucher ... " you know the rest :) | |
| 13:59:31 | Uggla | @bauzas, I'll try to do it for flamingo, but I hope, you will be not far away to check it. ;) | |
| 14:00:49 | bauzas | Uggla: GMaps tells me a distance of 36 kms by road indeed | |
| 14:01:03 | Uggla | @dansmith, a useful one. ;) | |
| 14:01:58 | Uggla | bauzas, any chance you merge couple of FUP for vGPU before RC 1 ? | |
| 14:02:54 | dansmith | we're already late right? unless it's something critical I'd vote to stay the course, backport if it's important | |
| 14:05:21 | bauzas | Uggla: I'm a bit lost now, the bottom patches are on gate, but for the other 3 ones, there is a still -1 for the 3rd | |
| 14:05:54 | bauzas | Uggla: tbc, we can merge the two first as those are in the gate, but for the other ones, I don't think so unfortunately :( | |
| 14:07:45 | bauzas | if those two patches merge before we merge rc1, all good but I won't await them unfortunately | |
| 14:08:54 | Uggla | bauzas, the one for the doc is important. but it can be backported without pb. | |
| 14:09:05 | bauzas | yep, I know :( | |
| 14:10:09 | Uggla | I can put it down in the serie if it helps. | |
| 14:11:33 | Uggla | I have fixed the comments from @gibi and @sean-k-mooney, but it needs a review. Anyway let me know if I can help if not then we'll see and backport. | |