| *** bhagyashris_ is now known as bhagyashris | 08:01 | |
| opendevreview | Tobias Urdin proposed openstack/nova master: Update announce self workaround opt description https://review.opendev.org/c/openstack/nova/+/826829 | 08:41 |
|---|---|---|
| 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:42 |
| 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:46 |
| 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:47 |
| 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:49 |
| opendevreview | Tobias Urdin proposed openstack/nova stable/train: libvirt: Add announce-self post live-migration workaround https://review.opendev.org/c/openstack/nova/+/825177 | 08:50 |
| *** bhagyashris_ is now known as bhagyashris | 08:53 | |
| 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 |
| bauzas | rosmaita: thanks for the heads up | 09:05 |
| bauzas | gmann: elodilles: I'm just preparing the stable-only changes | 09:05 |
| elodilles | bauzas: ack, thanks | 10:15 |
| elodilles | don't we use upper-constraints.txt for that job? as openstacksdk is pinned to 0.59.0 in there | 10:17 |
| elodilles | ohh, i see, the job has override-branch: https://opendev.org/openstack/openstacksdk/src/branch/master/.zuul.yaml#L40 | 10:27 |
| bauzas | elodilles: which is kinda normal (openstacksdk wants to pull always master when testing against projects' stable branches) | 10:35 |
| bauzas | honestly, I'm torn with the proposed direction | 10:36 |
| bauzas | I was about writing the patches when I thought about it | 10:36 |
| 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 |
| bauzas | and I wonder why we aren't just fixing the root caused | 10:36 |
| bauzas | frickler: correct, that's my opinion | 10:36 |
| bauzas | nova shouldn't pull the old sdk from xena | 10:36 |
| bauzas | rosmaita: ^ see I finally ended up with not providing the stable-only changes | 10:37 |
| 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:37 |
| 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:39 |
| * bauzas tries to find a bug report against this gate failure | 10:39 | |
| elodilles | yes, sounds reasonable to skip | 10:42 |
| bauzas | we should tho make the sdk job non-voting until this is fixed | 10:47 |
| * bauzas needs to disappear but I can do it | 10:49 | |
| 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:21 |
| opendevreview | Dmitrii Shcherbakov proposed openstack/nova master: [yoga] Add support for VNIC_TYPE_SMARTNIC https://review.opendev.org/c/openstack/nova/+/824835 | 11:34 |
| opendevreview | Dmitrii Shcherbakov proposed openstack/nova master: Bump os-traits to 2.7.0 https://review.opendev.org/c/openstack/nova/+/826675 | 11:34 |
| opendevreview | Dmitrii Shcherbakov proposed openstack/nova master: Filter computes without remote-managed ports early https://review.opendev.org/c/openstack/nova/+/812111 | 11:34 |
| 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:19 |
| 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:20 |
| ralonsoh | sean-k-mooney[m], (just a heads-up) https://review.opendev.org/c/openstack/neutron/+/826512 | 12:27 |
| ralonsoh | is that what Nova is expecting? do you need something more or different? let me know | 12:27 |
| opendevreview | Stephen Finucane proposed openstack/nova master: requirements: Remove os-xenapi https://review.opendev.org/c/openstack/nova/+/822747 | 12:29 |
| opendevreview | Stephen Finucane proposed openstack/nova master: Remove Python 2-specific imports https://review.opendev.org/c/openstack/nova/+/822748 | 12:29 |
| 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 |
| opendevreview | Stephen Finucane proposed openstack/nova master: Move optional build dependencies to 'extras' https://review.opendev.org/c/openstack/nova/+/822749 | 12:29 |
| opendevreview | Stephen Finucane proposed openstack/nova master: tox: Install extras https://review.opendev.org/c/openstack/nova/+/822750 | 12:29 |
| stephenfin | sean-k-mooney[m]: https://review.opendev.org/c/openstack/nova/+/822747/ | 12:29 |
| 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 |
| stephenfin | If you'd be so kind 0:) | 12:31 |
| gibi | sean-k-mooney[m]: done :) | 12:32 |
| gibi | stephenfin: done :) | 12:32 |
| stephenfin | \o/ ty! | 12:33 |
| *** dasm|off is now known as dasm | 13:30 | |
| *** dasm is now known as dasm|rover | 13:31 | |
| 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:38 |
| chateaulav | sean-k-mooney[m]: | 13:56 |
| tbarron | any nova folks want to join devstack-plugin-ceph meeting on use of cephadm? mtg now in #openstack-qa | 15:05 |
| sean-k-mooney | i could join but my main feed back is sure you totally should | 15:06 |
| sean-k-mooney | tbarron: as long as we only have to maintain one plugin for use itn the gate that is | 15:06 |
| sean-k-mooney | and can initally select which to use so we can transtion smothly | 15:07 |
| tbarron | sean-k-mooney: if you can briefly say that in the meeting it would be constructive | 15:07 |
| tbarron | rather than me reporting for nova | 15:07 |
| sean-k-mooney | ack i more or less said the same in the mail thread | 15:07 |
| frickler | bauzas: gibi: the fix for running sdk against xena has merged and I'm now working on further failures against ussuri | 15:18 |
| gibi | frickler: thanks! | 15:18 |
| bauzas | frickler: ++ | 15:18 |
| bauzas | thanks for having worked on it | 15:18 |
| sean-k-mooney | frickler: is sdk going to move to a non branched mode like tempest? | 16:00 |
| sean-k-mooney | frickler: i kind of stopped trackign the mail thread | 16:01 |
| frickler | sean-k-mooney: not sure about that, but afaict the idea has always been for the latest version to support older openstack, just like OSC does. it's just that there was no CI coverage for that for some time - like since newton was eoled | 16:35 |
| sean-k-mooney | frickler: ack well that is why i made refernce to tempest which is not branched and similarly support older release | 16:49 |
| frickler | possibly the branches will continue to be used in order to help distros, but I don't think they should be used for upstream testing by default like glance and cinder now did | 16:52 |
| sean-k-mooney | frickler: yep makes sense. proably going forward there is no need to create new branches but really thats up to the sdk team | 17:30 |
| opendevreview | Jonathan Race proposed openstack/nova master: Adds Pick guest CPU architecture based on host arch in libvirt driver support https://review.opendev.org/c/openstack/nova/+/822053 | 17:42 |
| opendevreview | Imran Hussain proposed openstack/nova master: [nova/libvirt] Support for checking and enabling SMM when needed https://review.opendev.org/c/openstack/nova/+/825496 | 17:45 |
| opendevreview | Jonathan Race proposed openstack/nova master: Adds Pick guest CPU architecture based on host arch in libvirt driver support https://review.opendev.org/c/openstack/nova/+/822053 | 17:57 |
| 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 | 18:16 |
| opendevreview | Dmitrii Shcherbakov proposed openstack/nova master: [yoga] Introduce remote_managed tag for PCI devices https://review.opendev.org/c/openstack/nova/+/824834 | 18:16 |
| opendevreview | Dmitrii Shcherbakov proposed openstack/nova master: [yoga] Add support for VNIC_TYPE_SMARTNIC https://review.opendev.org/c/openstack/nova/+/824835 | 18:16 |
| opendevreview | Dmitrii Shcherbakov proposed openstack/nova master: Bump os-traits to 2.7.0 https://review.opendev.org/c/openstack/nova/+/826675 | 18:16 |
| opendevreview | Dmitrii Shcherbakov proposed openstack/nova master: Filter computes without remote-managed ports early https://review.opendev.org/c/openstack/nova/+/812111 | 18:16 |
| opendevreview | Jonathan Race proposed openstack/nova master: Adds Pick guest CPU architecture based on host arch in libvirt driver support https://review.opendev.org/c/openstack/nova/+/822053 | 18:26 |
| opendevreview | Balazs Gibizer proposed openstack/placement master: Add microversion 1.39 to support any-trait queries https://review.opendev.org/c/openstack/placement/+/826719 | 18:28 |
| opendevreview | Jonathan Race proposed openstack/nova master: Adds Pick guest CPU architecture based on host arch in libvirt driver support https://review.opendev.org/c/openstack/nova/+/822053 | 18:31 |
| opendevreview | Stephen Finucane proposed openstack/nova master: Use unittest.mock instead of third party mock https://review.opendev.org/c/openstack/nova/+/714676 | 19:01 |
| opendevreview | sean mooney proposed openstack/nova master: [WIP] enable SMM mode when secure-boot is requested. https://review.opendev.org/c/openstack/nova/+/826931 | 19:06 |
| sean-k-mooney | stephenfin: kashyap ^ | 19:07 |
| chateaulav | sean-k-mooney: added initial ci content. first time working with nova ci, but i followed other examples. also rebased. | 19:09 |
| opendevreview | melanie witt proposed openstack/nova stable/xena: libvirt: Add announce-self post live-migration workaround https://review.opendev.org/c/openstack/nova/+/825064 | 19:10 |
| melwitt | sean-k-mooney: easy update to add the "domain will be considered tainted by libvirt" note to the config option help https://review.opendev.org/c/openstack/nova/+/826829 | 19:11 |
| gmann | bauzas: frickler I agree with the solution. that is how we test the things in branchless Tempest with feature flags. "if xyz present/cnofigured then tests it" | 19:11 |
| sean-k-mooney | melwitt: ack ill look at it now | 19:12 |
| melwitt | thx | 19:12 |
| sean-k-mooney | ok approved | 19:13 |
| sean-k-mooney | will we backport that seperatly or squash it in | 19:13 |
| melwitt | I squashed it in | 19:13 |
| sean-k-mooney | ack | 19:13 |
| sean-k-mooney | eandersson: spatel https://review.opendev.org/c/openstack/nova/+/826931 | 19:14 |
| sean-k-mooney | it will need test but we can likely just merge and updated version fo that and backport | 19:14 |
| sean-k-mooney | its not a nova bug but we can workaorund it in nova | 19:14 |
| spatel | Sweet!!! i can give it a try to this patch and see how it goes :) | 19:15 |
| sean-k-mooney | spatel: we will proably proceed with https://review.opendev.org/c/openstack/nova/+/825496 instead | 19:36 |
| spatel | that is also sounds good | 19:39 |
| spatel | as far as it get merge | 19:40 |
| opendevreview | Merged openstack/nova master: requirements: Remove os-xenapi https://review.opendev.org/c/openstack/nova/+/822747 | 21:06 |
| opendevreview | Merged openstack/nova master: Remove Python 2-specific imports https://review.opendev.org/c/openstack/nova/+/822748 | 21:06 |
| *** efried1 is now known as efried | 21:24 | |
| *** dasm|rover is now known as dasm|off | 21:47 | |
| opendevreview | melanie witt proposed openstack/nova master: Raise InstanceNotFound on fkey constraint fail saving info cache https://review.opendev.org/c/openstack/nova/+/826942 | 22:10 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!