| 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 |
|---|---|---|
| opendevreview | melanie witt proposed openstack/nova master: Follow up for unified limits https://review.opendev.org/c/openstack/nova/+/830918 | 03:46 |
| 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 | 08:40 |
| opendevreview | Andre Aranha proposed openstack/nova master: Change test fips job from centos8 to centos-9 https://review.opendev.org/c/openstack/nova/+/831917 | 09:50 |
| 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:01 |
| ralonsoh | I've added this patch to the Neutron drivers meeting today | 11:02 |
| ralonsoh | (to the meeting agenda) | 11:02 |
| gibi | ralonsoh: ack, I will join to the meeting | 11:02 |
| sean-k-mooney | gibi: my comments were just about showing that the profile mac was what was actully being used | 11:17 |
| sean-k-mooney | gibi so those are not needed if we are ok with that | 11:17 |
| gibi | OK, thanks | 11:17 |
| sean-k-mooney | i.e. the change would have demonstrated that the mac was not persisted to the port.mac | 11:18 |
| gibi | hm, then it still make sense to have that test case | 11:18 |
| gibi | to show that we are still not persisting | 11:18 |
| gibi | it in the port.mac_address | 11:18 |
| gibi | I will create a follow up then | 11:19 |
| gibi | and depending on the driver's meeting outcome I can squash it | 11:19 |
| *** bhagyashris|ruck is now known as bhagyashris | 11:36 | |
| sean-k-mooney | gibi: so that deprecation warning is actully already fixed in secretstorage | 12:18 |
| sean-k-mooney | but i had to manually install the latest version | 12:18 |
| sean-k-mooney | its fixed in 3.3.1 | 12:18 |
| sean-k-mooney | but for some reason 2.3.1 was insalled by devstack | 12:19 |
| sean-k-mooney | upperconstaits allwos 3.3.1 so not sure why this was not installed | 12:20 |
| gibi | interesting | 12:26 |
| gibi | I also see 2.3.1 in my devstack too | 12:28 |
| gibi | devstack stack.sh log does not explain where this is coming from | 12:32 |
| sean-k-mooney | it must be a downgrade form say keyring or something else | 12:36 |
| sean-k-mooney | actully it requires >=3.2 | 12:37 |
| sean-k-mooney | https://github.com/jaraco/keyring/blob/main/setup.cfg#L24 | 12:37 |
| sean-k-mooney | i have a really old verison fo keyring 18.0.1 | 12:38 |
| sean-k-mooney | which pulls in <3 | 12:38 |
| sean-k-mooney | https://github.com/jaraco/keyring/blob/18.0.1/setup.cfg#L35 | 12:38 |
| sean-k-mooney | so somehting is clamping the keyrign version which si then clamping the secretstorage version | 12:39 |
| sean-k-mooney | gibi python3-keyring is installed form the package manager | 12:50 |
| sean-k-mooney | so this might be form the base cloud image | 12:52 |
| sean-k-mooney | i dont have the issue on a centos9 vm so its likely not related to devstack | 12:53 |
| gibi | yeah I don't see that in the bindep either so probably it is the ubunut cloud image | 12:54 |
| bauzas | just in case, folks, I wrote something about the next PTG http://lists.openstack.org/pipermail/openstack-discuss/2022-March/027529.html | 13:53 |
| bauzas | I'd appreciate thoughts about the PTG schedule | 13:54 |
| gibi | looking | 13:56 |
| gibi | bauzas: nice proposals. I made my vote in the etherpad | 14:07 |
| bauzas | thanks | 14:07 |
| bauzas | gibi: about what we could discussing during the Asian time, I don't know yet | 14:08 |
| bauzas | but | 14:08 |
| bauzas | if we have enough quorum, we could discuss about something that US contributors don't need to want to discuss | 14:08 |
| gibi | yeah | 14:08 |
| gibi | and I also see that creating an asian friendly slot might encourage folks from that part of the world to propose topics | 14:09 |
| bauzas | yeah | 14:11 |
| *** dasm|off is now known as dasm | 14:20 | |
| *** iurygregory_ is now known as iurygregory | 14:28 | |
| opendevreview | Jan Hartkopf proposed openstack/nova-specs master: update userdata https://review.opendev.org/c/openstack/nova-specs/+/816542 | 15:29 |
| opendevreview | Dmitrii Shcherbakov proposed openstack/nova master: Fix migration with remote-managed ports & add FT https://review.opendev.org/c/openstack/nova/+/829974 | 15:42 |
| dmitriis | gibi: included your fix into my change, thanks again for finding the issue in the fixture! | 15:43 |
| gibi | dmitriis: no worries I also needed the fix | 15:43 |
| 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 |
| dmitriis | gibi: ah, right, it's in the resource tracker | 15:44 |
| 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:44 |
| 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:45 |
| gibi | dmitriis: I think the pattern is similar | 15:46 |
| gibi | let me know if you need help | 15:46 |
| dmitriis | gibi: ack, I'll do a follow-up change on top of the current one | 15:46 |
| sean-k-mooney | dmitriis: so im going to take a look at your patch shortly | 15:51 |
| sean-k-mooney | the fix your are talking about is for revert? | 15:52 |
| dmitriis | sean-k-mooney: yes, it's cold migration + revert | 15:52 |
| sean-k-mooney | ok so i mihgt need that too | 15:52 |
| dmitriis | sean-k-mooney: https://review.opendev.org/c/openstack/nova/+/829974/4..5 | 15:52 |
| sean-k-mooney | am i was hoping to get acces to 2 node system to do some vdpa move operation testing | 15:53 |
| sean-k-mooney | right now we are having issue with froman to get it to repovision with centos9 | 15:53 |
| sean-k-mooney | so im debating if i can use the test fixture to test vdpa move operations | 15:53 |
| 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:54 |
| sean-k-mooney | ah the fix is to do a deepcopy | 15:55 |
| 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:56 |
| 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 |
| gibi | sean-k-mooney: yeah I debated to add more deepcopy. I did not tried to reproduce the issue with the other fields | 15:57 |
| 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 |
| gibi | but I haven't digged down into it | 15:58 |
| sean-k-mooney | ya | 15:59 |
| 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:21 |
| 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:26 |
| 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:30 |
| opendevreview | sean mooney proposed openstack/nova stable/train: libvirt: Add announce-self post live-migration workaround https://review.opendev.org/c/openstack/nova/+/825177 | 16:32 |
| opendevreview | Takashi Natsume proposed openstack/placement master: Add zed spec directory https://review.opendev.org/c/openstack/placement/+/829532 | 17:03 |
| opendevreview | Takashi Natsume proposed openstack/os-vif master: Change minversion of tox to 3.18.0 https://review.opendev.org/c/openstack/os-vif/+/791970 | 17:04 |
| opendevreview | Takashi Natsume proposed openstack/os-resource-classes master: Change minversion of tox to 3.18.0 https://review.opendev.org/c/openstack/os-resource-classes/+/791974 | 17:04 |
| opendevreview | Takashi Natsume proposed openstack/placement master: Change minversion of tox to 3.18.0 https://review.opendev.org/c/openstack/placement/+/791971 | 17:04 |
| noonedeadpunk | hey there! I know it's friday evening, but I was wondering if you see https://review.opendev.org/c/openstack/nova/+/822749 as appropriate backport to stable branches? | 19:27 |
| noonedeadpunk | as powervm is really pita on older releases where you have py3.6 | 19:27 |
| noonedeadpunk | or well, should likely patch u-c instead to map pypowervm back to 1.1.24 where it was at least installing | 19:35 |
| elodilles | noonedeadpunk: req changes are not allowed to backport, but as I see it actually does not change them, just moves them to setup.cfg's [extra] section, so I think this is OK to backport | 20:41 |
| *** dasm is now known as dasm|off | 22:07 | |
| opendevreview | Mauricio Faria de Oliveira proposed openstack/nova stable/victoria: [stable-only] libvirt: UEFI: skip OVMF_CODE.secboot.fd on pc if possible https://review.opendev.org/c/openstack/nova/+/828979 | 23:00 |
| opendevreview | Mauricio Faria de Oliveira proposed openstack/nova stable/victoria: [stable-only] libvirt: UEFI: skip OVMF_CODE.secboot.fd on pc if possible https://review.opendev.org/c/openstack/nova/+/828979 | 23:01 |
| opendevreview | Mauricio Faria de Oliveira proposed openstack/nova stable/ussuri: [stable-only] libvirt: UEFI: skip OVMF_CODE.secboot.fd on pc if possible https://review.opendev.org/c/openstack/nova/+/828980 | 23:02 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!