| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2024-09-13 | |||
| 14:05:38 | artom | We would need to do something like soft-reboot it after the live migration? | |
| 14:05:41 | sean-k-mooney | https://review.opendev.org/c/openstack/nova/+/913842 | |
| 14:06:11 | sean-k-mooney | artom: it should have caused the instance to crash and caused the connectivy test to fail when we ssh in | |
| 14:06:25 | gibi | artom: in the volume booted case I think the instance will remain ACTIVE. For the shared storage case I don't know what happen when the local disk is deleted under it | |
| 14:06:35 | sean-k-mooney | we also have one broken the back and forth live migration | |
| 14:06:55 | artom | Ah, yeah, that would probably expose this | |
| 14:07:50 | artom | OK, I think I've saturated - I've literally had a feverish night tossing and turning because of some mystery illness one of the kids (presumably the baby) brought home. I'll do go dumb paperwork stuff :P | |
| 14:08:14 | bauzas | sean-k-mooney: does the ceph job check shared storage with cephfs ? | |
| 14:08:21 | sean-k-mooney | bauzas: no | |
| 14:08:24 | bauzas | or is it just rbd ? | |
| 14:08:37 | bauzas | okay, then we wouldn't see the bug | |
| 14:08:38 | sean-k-mooney | its images_type=rbd | |
| 14:08:41 | gibi | artom: take care | |
| 14:08:43 | bauzas | I see | |
| 14:08:48 | sean-k-mooney | right i said i need to create a new job for this | |
| 14:08:56 | sean-k-mooney | i tought we had an nfs job already | |
| 14:09:04 | bauzas | artom: joys of being parent of kids going back at school | |
| 14:09:05 | sean-k-mooney | we have - devstack-plugin-nfs-tempest-full: | |
| 14:09:19 | sean-k-mooney | in the experimatal line but not in check | |
| 14:09:51 | sean-k-mooney | bauzas: basically we have 2 chioices i can continue enabling numa testing in the ceph job | |
| 14:09:59 | bauzas | so you'd add numa live-migration tests in there ? | |
| 14:10:07 | sean-k-mooney | or i could pivort and create a shared storage migration job and put them in that | |
| 14:10:13 | bauzas | I'm cool but we're just adding more tests to the CI | |
| 14:10:23 | sean-k-mooney | we dont have a job that would catch this currently | |
| 14:10:32 | sean-k-mooney | we do not have any testing with nova on shared storage | |
| 14:11:34 | sean-k-mooney | bauzas: so im saying i can create one and also put the numa stuff in it, or i can proceed with putting the numa stuff in teh ceph jobs and we can create a seperate nova-shared-stroage-migration job | |
| 14:12:22 | sean-k-mooney | bauzas: do you have a prefernce? | |
| 14:12:37 | bauzas | kill shared storage with fire ? | |
| 14:12:44 | sean-k-mooney | :) | |
| 14:13:18 | bauzas | you can have a shared image backend, why would you mind using NFS which would create you don't know ? | |
| 14:13:41 | sean-k-mooney | well i cant argue agaisnt that but we coudl certenly make it less teribad in the future if we included this as part of the image backend refactor | |
| 14:13:53 | bauzas | another good case, yes | |
| 14:14:08 | sean-k-mooney | like if we had image_typs=nfs that would simplfy things a lot | |
| 14:15:21 | sean-k-mooney | bauzas: lets pause this dicussion for now and we can loop back to the tempest coverage after rc1 | |
| 14:17:03 | sean-k-mooney | bauzas: https://github.com/openstack/devstack-plugin-nfs currently just supprot seting up cinder on nfs so to test this i woudl eiter need to extend that or just put it in the nova devstack plugin which is honestly simpler | |
| 14:18:34 | sean-k-mooney | when we have a little more time ill try and figure out which job to extend to give us the most test coverage without needing extra josb | |
| 14:21:13 | sean-k-mooney | im thinkg of using nova-ovs-hybrid-plug to test shared storage and numa and then just leave the ceph job as is | |
| 14:25:33 | sean-k-mooney | summerised that here https://review.opendev.org/c/openstack/nova/+/913842/comments/ee7e862a_c1fd1b8c | |
| 14:26:25 | sean-k-mooney | frickler: is there a dnm that testing your openstack client patch | |
| 14:27:02 | sean-k-mooney | frickler: if not i need to rebase one of my patches to adress a nit anyway so i might add a depends on to it | |
| 14:56:47 | gibi | bauzas: which way you want to land the shared storage fix: a) I add the UT to supamatt's commit directly or b) I push the UT separately and then I can +2 the commit with the fix still | |
| 14:58:56 | bauzas | can we squash both ? | |
| 14:59:13 | bauzas | you can +2 the patch, I'm fine | |
| 14:59:25 | bauzas | I'll explain in my comment why it's fine | |
| 14:59:40 | bauzas | (and I want to stop by the top of the hour :) ) | |
| 15:01:28 | gibi | OK I will push the squash in 5 mins | |
| 15:08:12 | opendevreview | OpenStack Release Bot proposed openstack/placement stable/2024.2: Update .gitreview for stable/2024.2 https://review.opendev.org/c/openstack/placement/+/929290 | |
| 15:08:13 | opendevreview | OpenStack Release Bot proposed openstack/placement stable/2024.2: Update TOX_CONSTRAINTS_FILE for stable/2024.2 https://review.opendev.org/c/openstack/placement/+/929291 | |
| 15:08:15 | opendevreview | OpenStack Release Bot proposed openstack/placement master: Update master for stable/2024.2 https://review.opendev.org/c/openstack/placement/+/929292 | |
| 15:09:35 | opendevreview | Balazs Gibizer proposed openstack/nova master: Fix regression with live migration on shared storage https://review.opendev.org/c/openstack/nova/+/928970 | |
| 15:09:40 | gibi | bauzas: ^^ | |
| 15:09:45 | gibi | sorry it was 8 minutes | |
| 15:09:56 | gibi | but I can offer an extended commit message in return :) | |
| 15:12:19 | gibi | cc artom: ^^ | |
| 15:16:49 | gibi | and I need to drop now | |
| 15:16:59 | opendevreview | sean mooney proposed openstack/nova master: allow upgrade of pre-victoria InstanceNUMACells https://review.opendev.org/c/openstack/nova/+/929187 | |
| 15:22:13 | bauzas | sean-k-mooney: can you please review https://review.opendev.org/c/openstack/nova/+/928970 ? | |
| 15:22:52 | frickler | sean-k-mooney: no, I only tested locally, feel free to add a dep to verify | |
| 15:24:27 | sean-k-mooney | frickler:i have +2 the client patch after verifyign the filed in the sdk and novaclient | |
| 15:24:45 | sean-k-mooney | frickler: i have also added it as a depedn on in https://review.opendev.org/c/openstack/nova/+/929187 | |
| 15:24:55 | sean-k-mooney | altough i dont knwo fi the jobs are set up to pull in osc form souce | |
| 15:25:10 | sean-k-mooney | bauzas: sure ill take a look now | |
| 15:25:27 | bauzas | thanks, now I need to go off | |
| 15:25:33 | sean-k-mooney | frickler: i think at this point we just need someone form the sdk team to +w the clinet patch | |
| 15:26:13 | sean-k-mooney | bauzas: o/ enjoy your weekend | |
| 15:26:29 | bauzas | thanks | |
| 15:30:40 | sean-k-mooney | +2w, the approch looks good. i woudl have prefer a functioanl regression test but the coverage is enough to proceed with. ill look at tempest ci coverage for this next week if i have time | |
| 20:02:01 | opendevreview | Merged openstack/nova master: Add Dalmatian prelude section https://review.opendev.org/c/openstack/nova/+/928995 | |
| #openstack-nova - 2024-09-14 | |||
| 20:31:42 | opendevreview | sean mooney proposed openstack/nova master: [WIP] provide a nix flake for simple dev envs https://review.opendev.org/c/openstack/nova/+/929354 | |
| 20:31:42 | opendevreview | sean mooney proposed openstack/nova master: [tox] add LOCALE_ARCHIVE to passenv https://review.opendev.org/c/openstack/nova/+/929353 | |
| 20:34:26 | opendevreview | sean mooney proposed openstack/nova master: [WIP] provide a nix flake for simple dev envs https://review.opendev.org/c/openstack/nova/+/929354 | |
| 20:34:26 | opendevreview | sean mooney proposed openstack/nova master: [tox] add LOCALE_ARCHIVE to passenv https://review.opendev.org/c/openstack/nova/+/929353 | |
| 22:37:12 | opendevreview | sean mooney proposed openstack/nova master: [WIP] provide a nix flake for simple dev envs https://review.opendev.org/c/openstack/nova/+/929354 | |
| 22:38:30 | opendevreview | sean mooney proposed openstack/nova master: [WIP] provide a nix flake for simple dev envs https://review.opendev.org/c/openstack/nova/+/929354 | |
| #openstack-nova - 2024-09-15 | |||
| 12:27:13 | opendevreview | sean mooney proposed openstack/nova master: [WIP] provide a nix flake for simple dev envs https://review.opendev.org/c/openstack/nova/+/929354 | |
| 12:27:14 | opendevreview | sean mooney proposed openstack/nova master: [tox] enable piwheels to speed up tox https://review.opendev.org/c/openstack/nova/+/929366 | |
| #openstack-nova - 2024-09-16 | |||
| 06:18:54 | opendevreview | Takashi Kajinami proposed openstack/nova master: Drop SQLALCHEMY_WARN_20 https://review.opendev.org/c/openstack/nova/+/929395 | |
| 06:22:56 | opendevreview | Takashi Kajinami proposed openstack/placement master: Drop SQLALCHEMY_WARN_20 https://review.opendev.org/c/openstack/placement/+/929397 | |
| 06:25:18 | opendevreview | Takashi Kajinami proposed openstack/nova master: Drop SQLALCHEMY_WARN_20 https://review.opendev.org/c/openstack/nova/+/929395 | |
| 06:25:41 | opendevreview | Takashi Kajinami proposed openstack/placement master: Drop SQLALCHEMY_WARN_20 https://review.opendev.org/c/openstack/placement/+/929397 | |
| 06:40:02 | auniyal | Zuul UI updated https://zuul.openstack.org/status, looks pretty good, ++ to who ever is responsible | |
| 07:31:56 | gibi | yeah nice UI rewamp | |
| 10:04:13 | gibi | bauzas: do you aware that we have a blocked gate still even though the openstackclient evacuate fix is merged in https://review.opendev.org/c/openstack/python-openstackclient/+/929236 ? | |
| 10:04:23 | gibi | do we need a new client release and a upper constraint bump? | |
| 10:06:03 | gibi | elodilles: maybe you have more context on this ^^ | |
| 11:53:43 | bauzas | gibi: yup, I was aware of the problem, hence my question in the #openstack-release channel | |
| 11:54:18 | sean-k-mooney | bauzas: if your talking about th osc issue | |
| 11:54:32 | sean-k-mooney | stephenfin just porposed a 7.1.1 release to adress it | |
| 11:54:41 | bauzas | gibi: we need https://review.opendev.org/c/openstack/python-openstackclient/+/929402 to be merged and then creating a new release | |
| 11:54:54 | bauzas | sean-k-mooney: okay | |
| 11:55:22 | sean-k-mooney | https://review.opendev.org/c/openstack/releases/+/929454 | |
| 11:56:14 | gibi | hm 7.1.1 is released from master but we have a backport https://review.opendev.org/c/openstack/python-openstackclient/+/929402 merged to 2024.2 | |
| 11:56:22 | gibi | so I'm a bit confused which one to consume | |
| 11:56:51 | gibi | hm sorry 7.1.1 seems to point to the stable branch | |
| 11:57:05 | sean-k-mooney | we willl need to pull in 7.1.1 into 2024.2's uc | |
| 11:57:10 | sean-k-mooney | after the release | |
| 11:57:17 | gibi | yeah that is the next step after the release then | |
| 11:57:33 | sean-k-mooney | i just told stephen about the job failure and he is fixign the typo | |
| 11:57:57 | sean-k-mooney | ah already done | |
| 11:59:04 | sean-k-mooney | i could make depends on work but i think its will take about the same amount of time to get resulst that way as waitign for the release | |