| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-01-27 | |||
| 18:23:35 | rosmaita | bauzas: i posted these to deal with the openstacksdk-functional-devstack failures in the stable branches: https://review.opendev.org/q/topic:fix-sdk-func-job ... want me to do a set for nova? | |
| 18:25:32 | ganso | EugenMayer: that is still not working, but it is not working for other reason (I am to blame), just a min | |
| 18:25:56 | ganso | EugenMayer: this: https://review.opendev.org/c/openstack/nova/+/800114 | |
| 18:27:39 | ganso | EugenMayer: apparently this might work CONF.workarounds.disable_group_policy_check_upcall | |
| 18:28:27 | EugenMayer | interesting | |
| 18:29:37 | sean-k-mooney | ganso: you can use the old microversion but we really dont like supproting forced live migrations | |
| 18:29:47 | sean-k-mooney | ganso: if yo just want to specify the host you can now do that | |
| 18:29:53 | sean-k-mooney | but it wont bypass the schduler | |
| 18:30:50 | sean-k-mooney | evacuate does not use live migration by the way | |
| 18:31:04 | ganso | sean-k-mooney: yea but apparently it is not a matter of bypassing the scheduler anymore, I think, but I'm confused to how this worked in this past... my patch broke the functionality, but I can't see exactly how the forced host could also bypass the anti-affinity check before | |
| 18:31:35 | ganso | sean-k-mooney: oh it doesn't? I won't run pre_live_migration checks if it is a live-evacuation? | |
| 18:31:38 | EugenMayer | sean-k-mooney i recall you helped me unstack an instance which had an endless/stack task (image backup) running. I tried to reset state but i still cannot restart the instance or check the logs/console. Are there any leftovers? | |
| 18:31:43 | sean-k-mooney | ganso: it wont bypass the late affintiy check on the compute | |
| 18:32:01 | sean-k-mooney | you can only bypass that by disabling it on the host temporaly | |
| 18:33:14 | sean-k-mooney | ganso: live evacuation are not a thing at the api level its a nova client thing | |
| 18:33:31 | sean-k-mooney | so nova host-evacuate-live | |
| 18:33:46 | sean-k-mooney | is just a clisnt side for loop that does live migrations | |
| 18:34:07 | ganso | sean-k-mooney: oh, therefore it will use live-migration code at the backend | |
| 18:34:40 | sean-k-mooney | ganso: the workaroudn you can do for new is select a singel host, temperally disable the upcall then migrate teh host to the host useing a force live migratio nwith the old microversion | |
| 18:34:45 | sean-k-mooney | then renable the upcall | |
| 18:34:47 | sean-k-mooney | ganso: yes | |
| 18:35:22 | ganso | sean-k-mooney: thanks! | |
| 18:36:02 | sean-k-mooney | lol im glad you coudl read that. i am slightly distracted and that had more typos then i usually have | |
| 18:38:24 | ganso | sean-k-mooney: lol np it isn't hard to spellcheck while reading xD | |
| 19:05:37 | elodilles | sean-k-mooney: sorry for the late answer. so, no, i didn't know about any pip issue in centos job. though my local devstack started to fail with the very same point (pip bootstrap; ubuntu focal host), so it's weird :S | |
| 19:06:24 | sean-k-mooney | elodilles: pip upsteream now require py3.7+ | |
| 19:06:30 | sean-k-mooney | since 3.6 is eol | |
| 19:06:54 | sean-k-mooney | my local hack which i have not push is to skip installing pip form pypi if pip is already installed | |
| 19:07:04 | elodilles | it's interesting, because py3.8 is on the host | |
| 19:07:21 | sean-k-mooney | is that you default python | |
| 19:07:30 | elodilles | sean-k-mooney: i did exactly the same as a temporary fix | |
| 19:07:45 | elodilles | yepp, 3.8 is the default | |
| 19:07:49 | sean-k-mooney | ya so we used to install pip form pypi in the past becasue the default was old | |
| 19:08:03 | sean-k-mooney | but im thinking maybe we shoudl rever to the package manager version | |
| 19:08:14 | sean-k-mooney | or at least have an option too | |
| 19:08:43 | elodilles | good question | |
| 19:21:28 | elodilles | i need to leave now... but my feeling is that we will see the issue in other jobs as well in the coming days, so some kind of general fix might be needed | |
| 21:25:53 | opendevreview | Dmitrii Shcherbakov proposed openstack/nova master: [yoga] Add PCI VPD Capability Handling https://review.opendev.org/c/openstack/nova/+/808199 | |
| 21:25:53 | opendevreview | Dmitrii Shcherbakov proposed openstack/nova master: [yoga] Include pf mac and vf num in port updates https://review.opendev.org/c/openstack/nova/+/824833 | |
| 21:25:54 | opendevreview | Dmitrii Shcherbakov proposed openstack/nova master: [yoga] Introduce remote_managed tag for PCI devices https://review.opendev.org/c/openstack/nova/+/824834 | |
| 21:25:54 | opendevreview | Dmitrii Shcherbakov proposed openstack/nova master: [yoga] Add support for VNIC_TYPE_SMARTNIC https://review.opendev.org/c/openstack/nova/+/824835 | |
| 21:25:55 | opendevreview | Dmitrii Shcherbakov proposed openstack/nova master: Bump os-traits to 2.7.0 https://review.opendev.org/c/openstack/nova/+/826675 | |
| 21:25:55 | opendevreview | Dmitrii Shcherbakov proposed openstack/nova master: Filter computes without remote-managed ports early https://review.opendev.org/c/openstack/nova/+/812111 | |
| #openstack-nova - 2022-01-28 | |||
| 08:41:32 | opendevreview | Tobias Urdin proposed openstack/nova master: Update announce self workaround opt description https://review.opendev.org/c/openstack/nova/+/826829 | |
| 08:42:56 | opendevreview | Tobias Urdin proposed openstack/nova stable/xena: libvirt: Add announce-self post live-migration workaround https://review.opendev.org/c/openstack/nova/+/825064 | |
| 08:46:08 | opendevreview | Tobias Urdin proposed openstack/nova stable/wallaby: libvirt: Add announce-self post live-migration workaround https://review.opendev.org/c/openstack/nova/+/825178 | |
| 08:47:27 | opendevreview | Tobias Urdin proposed openstack/nova stable/victoria: libvirt: Add announce-self post live-migration workaround https://review.opendev.org/c/openstack/nova/+/825175 | |
| 08:49:45 | opendevreview | Tobias Urdin proposed openstack/nova stable/ussuri: libvirt: Add announce-self post live-migration workaround https://review.opendev.org/c/openstack/nova/+/825176 | |
| 08:50:45 | opendevreview | Tobias Urdin proposed openstack/nova stable/train: libvirt: Add announce-self post live-migration workaround https://review.opendev.org/c/openstack/nova/+/825177 | |
| 09:05:04 | bauzas | gmann: elodilles: damn shit, we're got hit by http://lists.openstack.org/pipermail/openstack-discuss/2022-January/026905.html with https://review.opendev.org/c/openstack/nova/+/825930 | |
| 09:05:14 | bauzas | rosmaita: thanks for the heads up | |
| 09:05:43 | bauzas | gmann: elodilles: I'm just preparing the stable-only changes | |
| 10:15:15 | elodilles | bauzas: ack, thanks | |
| 10:17:47 | elodilles | don't we use upper-constraints.txt for that job? as openstacksdk is pinned to 0.59.0 in there | |
| 10:27:34 | elodilles | ohh, i see, the job has override-branch: https://opendev.org/openstack/openstacksdk/src/branch/master/.zuul.yaml#L40 | |
| 10:35:51 | bauzas | elodilles: which is kinda normal (openstacksdk wants to pull always master when testing against projects' stable branches) | |
| 10:36:06 | bauzas | honestly, I'm torn with the proposed direction | |
| 10:36:19 | bauzas | I was about writing the patches when I thought about it | |
| 10:36:32 | frickler | sdk should check for the qos extension being present and skip those tests if not, that's a bug in sdk IMO | |
| 10:36:33 | bauzas | and I wonder why we aren't just fixing the root caused | |
| 10:36:44 | bauzas | frickler: correct, that's my opinion | |
| 10:36:52 | bauzas | nova shouldn't pull the old sdk from xena | |
| 10:37:19 | bauzas | rosmaita: ^ see I finally ended up with not providing the stable-only changes | |
| 10:37:57 | bauzas | gibi: sean-k-mooney[m]: gmann: elodilles: your thoughts would be appreciated but I agree with frickler, we should just ask sdk to avoid those checks | |
| 10:39:17 | gibi | frickler, bauzas: I agree. The test should be skipped if the extension is not present. This is how tempest also skips similar tests when run against older neutron | |
| 10:42:31 | elodilles | yes, sounds reasonable to skip | |
| 10:47:06 | bauzas | we should tho make the sdk job non-voting until this is fixed | |
| 11:21:28 | frickler | https://review.opendev.org/c/openstack/openstacksdk/+/826873 should be the fix, not sure if a depends-on job from xena will properly test it, I'll try so set something up locally | |
| 11:34:03 | opendevreview | Dmitrii Shcherbakov proposed openstack/nova master: [yoga] Add support for VNIC_TYPE_SMARTNIC https://review.opendev.org/c/openstack/nova/+/824835 | |
| 11:34:04 | opendevreview | Dmitrii Shcherbakov proposed openstack/nova master: Bump os-traits to 2.7.0 https://review.opendev.org/c/openstack/nova/+/826675 | |
| 11:34:05 | opendevreview | Dmitrii Shcherbakov proposed openstack/nova master: Filter computes without remote-managed ports early https://review.opendev.org/c/openstack/nova/+/812111 | |
| 12:19:03 | stephenfin | bauzas: gibi: sean-k-mooney[m]: melwitt: (others) I think it's time we deprecated the PowerVM driver and (ideally) marked its dependencies as optional. https://review.opendev.org/c/openstack/nova/+/819366 https://review.opendev.org/c/openstack/nova/+/822749 | |
| 12:20:00 | stephenfin | bauzas: gibi: sean-k-mooney[m]: melwitt: (others) Context is https://github.com/powervm/pypowervm/pull/17 The main library behind that driver looks dead, unlike the ZVM one https://review.opendev.org/c/openstack/nova/+/819365 | |
| 12:27:11 | ralonsoh | sean-k-mooney[m], (just a heads-up) https://review.opendev.org/c/openstack/neutron/+/826512 | |
| 12:27:34 | ralonsoh | is that what Nova is expecting? do you need something more or different? let me know | |
| 12:29:27 | opendevreview | Stephen Finucane proposed openstack/nova master: requirements: Remove os-xenapi https://review.opendev.org/c/openstack/nova/+/822747 | |
| 12:29:27 | opendevreview | Stephen Finucane proposed openstack/nova master: Remove Python 2-specific imports https://review.opendev.org/c/openstack/nova/+/822748 | |
| 12:29:28 | opendevreview | Stephen Finucane proposed openstack/nova master: requirements: Bump minimum testtools to 2.5.0 https://review.opendev.org/c/openstack/nova/+/822820 | |
| 12:29:28 | opendevreview | Stephen Finucane proposed openstack/nova master: Move optional build dependencies to 'extras' https://review.opendev.org/c/openstack/nova/+/822749 | |
| 12:29:29 | opendevreview | Stephen Finucane proposed openstack/nova master: tox: Install extras https://review.opendev.org/c/openstack/nova/+/822750 | |
| 12:29:43 | stephenfin | sean-k-mooney[m]: https://review.opendev.org/c/openstack/nova/+/822747/ | |
| 12:31:15 | stephenfin | gibi: Thanks for the review. Note that there are three trivial patches ahead of that 'extras' change https://review.opendev.org/c/openstack/nova/+/822747/ https://review.opendev.org/c/openstack/nova/+/822748/ https://review.opendev.org/c/openstack/nova/+/822820/ | |
| 12:31:24 | stephenfin | If you'd be so kind 0:) | |
| 12:32:48 | gibi | sean-k-mooney[m]: done :) | |
| 12:32:57 | gibi | stephenfin: done :) | |
| 12:33:03 | stephenfin | \o/ ty! | |
| 13:38:08 | chateaulav | sean-k-mooney: to ensure im not wasting time in the wrong direction. im working on the nova ci job for emulation, and referencing other jobs as examples. to make sure im going the right direction, i can look at having a post-run playbook to execute the required commands for testing similar to the live migration job, is that correct. | |
| 13:56:57 | chateaulav | sean-k-mooney[m]: | |
| 15:05:40 | tbarron | any nova folks want to join devstack-plugin-ceph meeting on use of cephadm? mtg now in #openstack-qa | |
| 15:06:07 | sean-k-mooney | i could join but my main feed back is sure you totally should | |
| 15:06:36 | sean-k-mooney | tbarron: as long as we only have to maintain one plugin for use itn the gate that is | |
| 15:07:02 | sean-k-mooney | and can initally select which to use so we can transtion smothly | |
| 15:07:10 | tbarron | sean-k-mooney: if you can briefly say that in the meeting it would be constructive | |
| 15:07:18 | tbarron | rather than me reporting for nova | |
| 15:07:33 | sean-k-mooney | ack i more or less said the same in the mail thread | |
| 15:18:18 | frickler | bauzas: gibi: the fix for running sdk against xena has merged and I'm now working on further failures against ussuri | |
| 15:18:34 | gibi | frickler: thanks! | |
| 15:18:38 | bauzas | frickler: ++ | |
| 15:18:43 | bauzas | thanks for having worked on it | |
| 16:00:50 | sean-k-mooney | frickler: is sdk going to move to a non branched mode like tempest? | |