| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-04-25 | |||
| 13:45:05 | bauzas | while I don't care of the CI changes as elodilles noted, I'm a bit torn whether we should include the propre gitreview target in the last build | |
| 13:45:17 | bauzas | "build" rather with quotes | |
| 13:45:44 | elodilles | from package point of view they are not included (pypi, tarball) so the release would just be unnecessary fuss for package consumers | |
| 13:46:18 | sean-k-mooney | elodilles: right but teh way the em tag is ment to work is it is ment ot allow you to restore the branch | |
| 13:46:31 | sean-k-mooney | so the em tag shoudl be on the last patch in the stable branch | |
| 13:46:54 | bauzas | then we should include the gitreview patch | |
| 13:47:22 | elodilles | sean-k-mooney: well, no, that's the *-eol you mean i think :) | |
| 13:47:33 | sean-k-mooney | oh sorry you are right | |
| 13:47:38 | sean-k-mooney | i was thinking of eol | |
| 13:47:42 | sean-k-mooney | for em we are not removing the branch | |
| 13:47:49 | sean-k-mooney | then ya its fine as is | |
| 13:47:55 | sean-k-mooney | no need to do another release | |
| 13:48:06 | sean-k-mooney | bauzas: do you agree? | |
| 13:48:56 | bauzas | hmmmm | |
| 13:49:18 | bauzas | I'm not *that* opiniated, I just want to be convinced | |
| 13:49:25 | bauzas | what's the purpose of the -em tag ? | |
| 13:49:46 | sean-k-mooney | really just to signal that no more releases will be made | |
| 13:52:40 | elodilles | yepp | |
| 13:54:48 | bauzas | that's my understanding | |
| 13:55:08 | bauzas | now, looking at https://review.opendev.org/q/project:openstack/python-novaclient+branch:stable/victoria | |
| 13:55:17 | bauzas | we only have 2 patches in the branch :) | |
| 13:55:26 | bauzas | one isn't required | |
| 13:55:38 | bauzas | the other one just updates the gitreview target | |
| 14:01:15 | elodilles | yepp, those are not part of the packages (even if we tag and release them) | |
| 14:07:34 | bauzas | elodilles: OK, then I'll change my vote | |
| 14:25:50 | opendevreview | Balazs Gibizer proposed openstack/nova master: Simulate bug 1969496 https://review.opendev.org/c/openstack/nova/+/838554 | |
| 14:25:51 | opendevreview | Balazs Gibizer proposed openstack/nova master: Allow claiming PCI PF if child VF is unavailable https://review.opendev.org/c/openstack/nova/+/838555 | |
| 14:26:54 | gibi | sean-k-mooney: finalized the fix ^^ | |
| 14:27:38 | opendevreview | Merged openstack/nova master: VMware: Split out VMwareAPISession https://review.opendev.org/c/openstack/nova/+/832156 | |
| 14:27:53 | sean-k-mooney | gibi: ack ill add that to my review list so for this week | |
| 14:28:01 | gibi | thanks! | |
| 14:28:47 | sean-k-mooney | ah nice you have all 3 parts. the repoduce, allowing it to claim and also fixing the inconsitent state | |
| 14:28:55 | gibi | yepp | |
| 14:36:50 | sean-k-mooney | gibi: is there a reason you did the simulation second in the series | |
| 14:37:19 | sean-k-mooney | i woudl have expected the startup patch to be last | |
| 14:37:33 | sean-k-mooney | so simulate, claim then start up | |
| 14:37:41 | sean-k-mooney | it works this way too just wondering | |
| 14:38:27 | gibi | the reason was that I started with teh cleanup path as it was easier to get to it. but no technical dependency so I can reorder if needed | |
| 14:39:51 | sean-k-mooney | no thats ok in principal once the start up check is merged you dont need the others | |
| 14:40:00 | sean-k-mooney | so that is why i was suggesting reordering | |
| 14:40:04 | sean-k-mooney | but i like having all 3 | |
| 14:40:27 | sean-k-mooney | (you would have to restart the agent to get this code hench the startup cleanup is enough) | |
| 14:42:02 | gibi | the current cleanup logic only works if the admin removes the VFs from the hypervisor first, then restart the agent. So if that removal (of the restart) is not possible then the claim-while-child-unavailable patch helps | |
| 14:42:14 | sean-k-mooney | ah you are right | |
| 14:42:35 | gibi | or I could create a wider cleanup logic that does not require the VF removal from the hypervisor | |
| 14:42:55 | gibi | just push the VF to available at startup | |
| 14:43:01 | gibi | regardles if it is removed or not | |
| 14:43:25 | gibi | and do this before the existing removed device cleanup logic | |
| 14:43:39 | gibi | we have full of options :) | |
| 14:45:35 | sean-k-mooney | im reviewing it now so unless others object lets keep this simple and go with what you have | |
| 14:45:48 | sean-k-mooney | we might want to do a larger cleanup later but this looks pretty backportable to me | |
| 14:46:33 | gibi | yepp it is pretty self contains now | |
| 14:46:50 | gibi | I will do the backports too | |
| 14:46:56 | gibi | once it lands on master | |
| 14:50:27 | opendevreview | Artom Lifshitz proposed openstack/nova master: Reproduce live migration rollback w/o multi port bindings error https://review.opendev.org/c/openstack/nova/+/839091 | |
| 14:50:28 | opendevreview | Artom Lifshitz proposed openstack/nova master: Fix LM rollback w/o multi port bindings extension https://review.opendev.org/c/openstack/nova/+/839227 | |
| 15:56:11 | opendevreview | Andre Aranha proposed openstack/nova master: Test setting the nova job to centos-9-stream https://review.opendev.org/c/openstack/nova/+/831844 | |
| 16:13:57 | sean-k-mooney | ah fips | |
| 16:14:02 | sean-k-mooney | i was like which job | |
| 16:30:41 | opendevreview | Dan Smith proposed openstack/nova master: DNM: Run against performance.json patch https://review.opendev.org/c/openstack/nova/+/838934 | |
| 17:45:11 | melwitt | gmann: this is a small patch involving deprecation of [api] config option that you might be interested in reviewing https://review.opendev.org/c/openstack/nova/+/836253 | |
| 18:05:45 | gmann | melwitt: thanks, +A. lgtm. | |
| 18:13:54 | melwitt | thanks gmann | |
| 18:32:40 | opendevreview | Jakub Libosvar proposed openstack/os-vif master: Check for hybrid plugging in OVS https://review.opendev.org/c/openstack/os-vif/+/839102 | |
| 19:39:55 | opendevreview | Merged openstack/nova master: Deprecate [api] use_forwarded_for https://review.opendev.org/c/openstack/nova/+/836253 | |
| 23:15:07 | opendevreview | Ghanshyam proposed openstack/nova master: Move centos stream testing to centos-9-stream https://review.opendev.org/c/openstack/nova/+/839275 | |
| #openstack-nova - 2022-04-26 | |||
| 01:00:32 | fzzf[m] | hi folks. I use nodepool connect openstack provider. in openstack, nova build instance failed. and nova list show instance is in BUILD state. and I can't delete it.... (full message at https://matrix.org/_matrix/media/r0/download/matrix.org/JmBToOhokPkttSFIalLvbSEy) | |
| 01:33:44 | opendevreview | Jorhson Deng proposed openstack/nova master: Reset the instance's task_state to None when the status is error https://review.opendev.org/c/openstack/nova/+/838991 | |
| 03:28:53 | fzzf[m] | * hi folks. I use nodepool connect openstack provider. in openstack, nova build instance failed. and nova list show instance is in BUILD state. and I can't delete it.... (full message at https://matrix.org/_matrix/media/r0/download/matrix.org/nhRIVjZthAjNvxgkNkdUczzV) | |
| 07:18:08 | Uggla | Good morning. o/ | |
| 07:37:56 | bauzas | Uggla: good morning | |
| 08:31:04 | gibi | o/ | |
| 08:56:04 | sean-k-mooney | gmann: dansmith care to readd your +w for moving the fips job to c9s https://review.opendev.org/c/openstack/nova/+/831844/7 | |
| 09:30:24 | fzzf[m] | hi folks. I use nodepool connect openstack provider. nova build instance failed. and nova list show instance is in BUILD state like follow. I delete it prompt No server with a name or ID. and in dashboard show have use these two instance, 8 vcpu, 16g memory, openstack is packstack train ,nova is 15.1.1... (full message at https://matrix.org/_matrix/media/r0/download/matrix.org/cXVvKGZbhOphiZZcinYshnCe) | |
| 10:30:31 | opendevreview | Balazs Gibizer proposed openstack/nova stable/train: Reproduce bug 1953359 https://review.opendev.org/c/openstack/nova/+/839353 | |
| 10:30:32 | opendevreview | Balazs Gibizer proposed openstack/nova stable/train: Extend the reproducer for 1953359 and 1952915 https://review.opendev.org/c/openstack/nova/+/839354 | |
| 10:30:33 | opendevreview | Balazs Gibizer proposed openstack/nova stable/train: [rt] Apply migration context for incoming migrations https://review.opendev.org/c/openstack/nova/+/839355 | |
| 11:03:07 | gibi | bauzas: I'm not when you clean up the nova meeting agenda but I now added a topic for Open Discussion for this week's meeting | |
| 11:03:14 | gibi | * I'm not sure | |
| 11:10:46 | sean-k-mooney | artom: by the way while you are fixing revert if you feel like updating https://review.opendev.org/c/openstack/nova/+/750217/2 i would not object. i should really get back to that but have not had the time | |
| 11:22:41 | opendevreview | Andrew Bonney proposed openstack/nova master: Fix segment-aware scheduling permissions error https://review.opendev.org/c/openstack/nova/+/839361 | |
| 11:57:36 | sean-k-mooney | woot! just tested cold migration with vdpa on real hardware with my WIP patch and it indeed worked as the func tests said it should | |
| 11:58:15 | sean-k-mooney | i need to test every thing else and then reivew the logs but this is a good sign | |
| 12:01:23 | sean-k-mooney | shelve and resize work too | |
| 12:09:39 | sean-k-mooney | evacuate works too and the device is correctly released if/when the source host is started after the evac | |
| 12:10:11 | sean-k-mooney | so that is all the move ops that should work exluding live-migration which qemu does not support | |
| 12:12:36 | sean-k-mooney | ok suspend is broken but that fine that is going to be split into another patch anyway and i am pretty sure i know why that is broken anyway. im using the generic pci device detach function and i shoudl be using the interface detach version since vdpa devices ues the interface not host dev elements like normal sriov vfs | |
| 12:14:14 | gibi | sean-k-mooney: nice! | |
| 12:16:14 | sean-k-mooney | im really happy i was able to more or less prove this out with the functional tests ahead of time | |
| 12:16:34 | sean-k-mooney | i kindof expected suspend to fail too which is why i wanted to test this on real hardware | |
| 12:25:11 | Uggla | sean-k-mooney, I guess you manage to install ceph on top of Centos Stream. Any hints to fix that: nothing provides "libpython3.6m.so.1.0()(64bit) needed by ceph-mgr-2:16.2.7-0.el8.x86_64" | |
| 12:32:30 | sean-k-mooney | Uggla: how are you isntalling ceph? with the devstack plugin or by hand? | |
| 12:33:31 | sean-k-mooney | by the way that looks like you are just missin python3-devel | |
| 12:34:02 | sean-k-mooney | also are you using centos stream 8 | |
| 12:34:07 | sean-k-mooney | that is not supported anymore | |
| 12:34:15 | Uggla | sean-k-mooney, devstack plugin (from manila local.conf example : https://docs.openstack.org/manila/latest/_downloads/e31f39ce83daac60c4348fde522cee1e/cephfs_local.conf) | |
| 12:34:18 | sean-k-mooney | you should be using centos stream 9 | |
| 12:34:35 | sean-k-mooney | openstack nolonger support python 3.6 | |
| 12:34:35 | Uggla | sean-k-mooney, it is a stream9 | |
| 12:34:54 | sean-k-mooney | libpython3.6 is python 3.6 | |