| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-03-03 | |||
| 15:16:32 | sean-k-mooney | the job is non voting right | |
| 15:17:09 | sean-k-mooney | i think we could proceed with this failure unless it reliably fails every time in the fips job? | |
| 15:18:52 | ade_lee_ | sean-k-mooney, fair enough -- maybe what we do then is change the job to be for centos-9 - and then merge it | |
| 15:19:22 | sean-k-mooney | basically what i woudl hope is when we fix this normally it would be fixed for fips | |
| 15:26:28 | ade_lee_ | sean-k-mooney, ack - I'll update to centos 9 - and then ping for reviews. do we have any sense of when it will be fixed ? its showing up in glance reviews, cinder etc .. | |
| 15:26:56 | ade_lee_ | sean-k-mooney, a BZ will be super helpful so I can track things | |
| 15:30:09 | sean-k-mooney | so we kind fo do have one for qemu and there are a few cix issues | |
| 15:30:29 | sean-k-mooney | we dont have a singel one for nova for example because its not really a nova issue | |
| 15:30:45 | sean-k-mooney | we think that some of the recent bugfixes in qemu and libvirt will help | |
| 15:33:26 | opendevreview | ribaudr proposed openstack/python-novaclient master: Microversion 2.91: Support specifying destination host to unshelve https://review.opendev.org/c/openstack/python-novaclient/+/831651 | |
| 16:19:07 | tobias-urdin | sean-k-mooney: no hurry, let me know when you have a second :) | |
| 16:19:44 | sean-k-mooney | tobias-urdin: sorry im in a meeting downstream which is why i did not respond | |
| 16:34:21 | opendevreview | Merged openstack/python-novaclient stable/yoga: Update .gitreview for stable/yoga https://review.opendev.org/c/openstack/python-novaclient/+/831704 | |
| 16:34:24 | opendevreview | Merged openstack/python-novaclient stable/yoga: Update TOX_CONSTRAINTS_FILE for stable/yoga https://review.opendev.org/c/openstack/python-novaclient/+/831705 | |
| 17:28:04 | Uggla | sean-k-mooney, fyi now the unshelve to host is complete with client part as well. | |
| 17:56:30 | opendevreview | Merged openstack/osc-placement stable/yoga: Update .gitreview for stable/yoga https://review.opendev.org/c/openstack/osc-placement/+/831695 | |
| 17:57:28 | opendevreview | Merged openstack/osc-placement stable/yoga: Update TOX_CONSTRAINTS_FILE for stable/yoga https://review.opendev.org/c/openstack/osc-placement/+/831698 | |
| 18:12:17 | sean-k-mooney | Uggla: ack ill try and review what you have proably monday | |
| 18:33:57 | Uggla | sean-k-mooney, no hurries I think. | |
| 18:51:08 | sean-k-mooney | are all patches in https://review.opendev.org/q/topic:bp%252Funshelve-to-host | |
| 18:51:37 | sean-k-mooney | if so ill add that to my review-list bookmark folder for monday | |
| 20:13:55 | opendevreview | Ade Lee proposed openstack/nova master: Test setting the nova job to centos-9-stream https://review.opendev.org/c/openstack/nova/+/831844 | |
| #openstack-nova - 2022-03-04 | |||
| 03:46:01 | opendevreview | melanie witt proposed openstack/nova master: Follow up for unified limits https://review.opendev.org/c/openstack/nova/+/830918 | |
| 03:46:01 | opendevreview | melanie witt proposed openstack/nova master: Enable unified limits in the nova-next job https://review.opendev.org/c/openstack/nova/+/789963 | |
| 08:40:04 | Uggla | sean-k-mooney, yep all patches (specs, api, novaclient, openstackclient) are available under this topic : https://review.opendev.org/q/topic:bp%252Funshelve-to-host | |
| 09:50:49 | opendevreview | Andre Aranha proposed openstack/nova master: Change test fips job from centos8 to centos-9 https://review.opendev.org/c/openstack/nova/+/831917 | |
| 11:01:40 | gibi | sean-k-mooney: hi! ralonsoh left +2 on the neutron PF MAC patch https://review.opendev.org/c/openstack/neutron/+/829247/2 So it seem neutron is OK not to persist the MAC in the port.mac_address just in the binding profile. You had comments about the test case in that patch. Do you still feel that is needed? | |
| 11:02:20 | ralonsoh | I've added this patch to the Neutron drivers meeting today | |
| 11:02:32 | ralonsoh | (to the meeting agenda) | |
| 11:02:56 | gibi | ralonsoh: ack, I will join to the meeting | |
| 11:17:30 | sean-k-mooney | gibi: my comments were just about showing that the profile mac was what was actully being used | |
| 11:17:43 | sean-k-mooney | gibi so those are not needed if we are ok with that | |
| 11:17:51 | gibi | OK, thanks | |
| 11:18:17 | sean-k-mooney | i.e. the change would have demonstrated that the mac was not persisted to the port.mac | |
| 11:18:35 | gibi | hm, then it still make sense to have that test case | |
| 11:18:47 | gibi | to show that we are still not persisting | |
| 11:18:54 | gibi | it in the port.mac_address | |
| 11:19:05 | gibi | I will create a follow up then | |
| 11:19:21 | gibi | and depending on the driver's meeting outcome I can squash it | |
| 12:18:33 | sean-k-mooney | gibi: so that deprecation warning is actully already fixed in secretstorage | |
| 12:18:46 | sean-k-mooney | but i had to manually install the latest version | |
| 12:18:51 | sean-k-mooney | its fixed in 3.3.1 | |
| 12:19:02 | sean-k-mooney | but for some reason 2.3.1 was insalled by devstack | |
| 12:20:05 | sean-k-mooney | upperconstaits allwos 3.3.1 so not sure why this was not installed | |
| 12:26:34 | gibi | interesting | |
| 12:28:32 | gibi | I also see 2.3.1 in my devstack too | |
| 12:32:02 | gibi | devstack stack.sh log does not explain where this is coming from | |
| 12:36:39 | sean-k-mooney | it must be a downgrade form say keyring or something else | |
| 12:37:35 | sean-k-mooney | actully it requires >=3.2 | |
| 12:37:37 | sean-k-mooney | https://github.com/jaraco/keyring/blob/main/setup.cfg#L24 | |
| 12:38:25 | sean-k-mooney | i have a really old verison fo keyring 18.0.1 | |
| 12:38:45 | sean-k-mooney | which pulls in <3 | |
| 12:38:47 | sean-k-mooney | https://github.com/jaraco/keyring/blob/18.0.1/setup.cfg#L35 | |
| 12:39:07 | sean-k-mooney | so somehting is clamping the keyrign version which si then clamping the secretstorage version | |
| 12:50:54 | sean-k-mooney | gibi python3-keyring is installed form the package manager | |
| 12:52:04 | sean-k-mooney | so this might be form the base cloud image | |
| 12:53:02 | sean-k-mooney | i dont have the issue on a centos9 vm so its likely not related to devstack | |
| 12:54:47 | gibi | yeah I don't see that in the bindep either so probably it is the ubunut cloud image | |
| 13:53:56 | bauzas | just in case, folks, I wrote something about the next PTG http://lists.openstack.org/pipermail/openstack-discuss/2022-March/027529.html | |
| 13:54:08 | bauzas | I'd appreciate thoughts about the PTG schedule | |
| 13:56:55 | gibi | looking | |
| 14:07:03 | gibi | bauzas: nice proposals. I made my vote in the etherpad | |
| 14:07:11 | bauzas | thanks | |
| 14:08:05 | bauzas | gibi: about what we could discussing during the Asian time, I don't know yet | |
| 14:08:10 | bauzas | but | |
| 14:08:46 | bauzas | if we have enough quorum, we could discuss about something that US contributors don't need to want to discuss | |
| 14:08:55 | gibi | yeah | |
| 14:09:32 | gibi | and I also see that creating an asian friendly slot might encourage folks from that part of the world to propose topics | |
| 14:11:41 | bauzas | yeah | |
| 15:29:40 | opendevreview | Jan Hartkopf proposed openstack/nova-specs master: update userdata https://review.opendev.org/c/openstack/nova-specs/+/816542 | |
| 15:42:04 | opendevreview | Dmitrii Shcherbakov proposed openstack/nova master: Fix migration with remote-managed ports & add FT https://review.opendev.org/c/openstack/nova/+/829974 | |
| 15:43:35 | dmitriis | gibi: included your fix into my change, thanks again for finding the issue in the fixture! | |
| 15:43:54 | gibi | dmitriis: no worries I also needed the fix | |
| 15:44:11 | gibi | dmitriis: the fun thing is that it is not the fixture that is faulty it is the nova code handling the migration context | |
| 15:44:28 | dmitriis | gibi: ah, right, it's in the resource tracker | |
| 15:44:46 | gibi | dmitriis: probably it does not cause fault in a real deployment just in the test env, but the logic in the code is wrong | |
| 15:45:43 | dmitriis | gibi: I see. I'll look into storing pf_mac_address and vf_num in extra_info. That's a bit different from the case you were working on | |
| 15:46:02 | gibi | dmitriis: I think the pattern is similar | |
| 15:46:10 | gibi | let me know if you need help | |
| 15:46:56 | dmitriis | gibi: ack, I'll do a follow-up change on top of the current one | |
| 15:51:52 | sean-k-mooney | dmitriis: so im going to take a look at your patch shortly | |
| 15:52:05 | sean-k-mooney | the fix your are talking about is for revert? | |
| 15:52:16 | dmitriis | sean-k-mooney: yes, it's cold migration + revert | |
| 15:52:37 | sean-k-mooney | ok so i mihgt need that too | |
| 15:52:51 | dmitriis | sean-k-mooney: https://review.opendev.org/c/openstack/nova/+/829974/4..5 | |
| 15:53:03 | sean-k-mooney | am i was hoping to get acces to 2 node system to do some vdpa move operation testing | |
| 15:53:25 | sean-k-mooney | right now we are having issue with froman to get it to repovision with centos9 | |
| 15:53:46 | sean-k-mooney | so im debating if i can use the test fixture to test vdpa move operations | |
| 15:54:30 | sean-k-mooney | i think everything or mostly everything is in place to do that but i like to test things with real hardware too | |
| 15:55:20 | sean-k-mooney | ah the fix is to do a deepcopy | |
| 15:56:17 | sean-k-mooney | ya im not sure that a real deployment would hit the issue since we likely wont have the same instnace object | |
| 15:57:10 | sean-k-mooney | that kind of rasies the question should we be deepcopying the instance.numa_topotlogy or any of the other data | |
| 15:57:53 | gibi | sean-k-mooney: yeah I debated to add more deepcopy. I did not tried to reproduce the issue with the other fields | |
| 15:58:34 | gibi | it is only a problem for the pci_device as the tracker _appends_ the newly claimed devices so maybe it is not an issue to the other fields | |
| 15:58:53 | gibi | but I haven't digged down into it | |
| 15:59:20 | sean-k-mooney | ya | |
| 16:21:14 | opendevreview | sean mooney proposed openstack/nova stable/wallaby: libvirt: Add announce-self post live-migration workaround https://review.opendev.org/c/openstack/nova/+/825178 | |
| 16:26:55 | opendevreview | sean mooney proposed openstack/nova stable/victoria: libvirt: Add announce-self post live-migration workaround https://review.opendev.org/c/openstack/nova/+/825175 | |
| 16:30:05 | opendevreview | sean mooney proposed openstack/nova stable/ussuri: libvirt: Add announce-self post live-migration workaround https://review.opendev.org/c/openstack/nova/+/825176 | |
| 16:32:00 | opendevreview | sean mooney proposed openstack/nova stable/train: libvirt: Add announce-self post live-migration workaround https://review.opendev.org/c/openstack/nova/+/825177 | |