| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-03-03 | |||
| 15:02:16 | sean-k-mooney | what do the ObjectTempUrlTest test do | |
| 15:02:25 | sean-k-mooney | are they swift related im not familar with them | |
| 15:03:38 | sean-k-mooney | GET https://149.202.163.165:8080/v1/AUTH_ab7063290b7341eeb77f5198d9e09903/tempest-TestContainer-735377846/tempest-TestObject-1768297629 | |
| 15:03:47 | sean-k-mooney | that looks like possible swift to me | |
| 15:05:34 | sean-k-mooney | ade_lee_: in anycase the rescue failure does not look fips related | |
| 15:07:03 | tobias-urdin | sean-k-mooney: any input on above? | |
| 15:09:03 | ade_lee_ | sean-k-mooney, ack - I didn't think it was, but unfortunately , it is blocking the fips patches. Do we have any workarounds/possible fixes ? I'll ask the swift folks about the swift issues. | |
| 15:10:10 | ade_lee_ | sean-k-mooney, is there a BZ /launchpad to track these libvirt/qemu issues? | |
| 15:10:37 | gibi | sean-k-mooney: yeah, the base case retry is something we can remove when we switch to qemu 6.2 as a minimum | |
| 15:10:49 | gibi | (or something around 6.2 Im not sure) | |
| 15:15:26 | sean-k-mooney | we proably could make it conditonal on the version before we raise our minium | |
| 15:15:36 | gibi | yeah, good point | |
| 15:15:57 | sean-k-mooney | tobias-urdin: oh i missed your follow ups | |
| 15:16:02 | sean-k-mooney | ill read back one sec | |
| 15:16:18 | sean-k-mooney | ade_lee_: why is it blocking? | |
| 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: Enable unified limits in the nova-next job https://review.opendev.org/c/openstack/nova/+/789963 | |
| 03:46:01 | opendevreview | melanie witt proposed openstack/nova master: Follow up for unified limits https://review.opendev.org/c/openstack/nova/+/830918 | |
| 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 | |