| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-01-28 | |||
| 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: Remove Python 2-specific imports https://review.opendev.org/c/openstack/nova/+/822748 | |
| 12:29:27 | opendevreview | Stephen Finucane proposed openstack/nova master: requirements: Remove os-xenapi https://review.opendev.org/c/openstack/nova/+/822747 | |
| 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: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: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? | |
| 16:01:08 | sean-k-mooney | frickler: i kind of stopped trackign the mail thread | |
| 16:35:32 | 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:49:33 | sean-k-mooney | frickler: ack well that is why i made refernce to tempest which is not branched and similarly support older release | |
| 16:52:45 | 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 | |
| 17:30: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:42:29 | 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:45:07 | 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:57:10 | 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:16:15 | 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:15 | 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: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:17 | opendevreview | Dmitrii Shcherbakov proposed openstack/nova master: Filter computes without remote-managed ports early https://review.opendev.org/c/openstack/nova/+/812111 | |
| 18:16:17 | opendevreview | Dmitrii Shcherbakov proposed openstack/nova master: Bump os-traits to 2.7.0 https://review.opendev.org/c/openstack/nova/+/826675 | |
| 18:26:31 | 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:28:52 | 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:31:53 | 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 | |
| 19:01:49 | opendevreview | Stephen Finucane proposed openstack/nova master: Use unittest.mock instead of third party mock https://review.opendev.org/c/openstack/nova/+/714676 | |
| 19:06:58 | 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:07:19 | sean-k-mooney | stephenfin: kashyap ^ | |
| 19:09:39 | chateaulav | sean-k-mooney: added initial ci content. first time working with nova ci, but i followed other examples. also rebased. | |
| 19:10:02 | 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:11:19 | 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:30 | 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:12:28 | sean-k-mooney | melwitt: ack ill look at it now | |
| 19:12:37 | melwitt | thx | |
| 19:13:05 | sean-k-mooney | ok approved | |
| 19:13:17 | sean-k-mooney | will we backport that seperatly or squash it in | |
| 19:13:24 | melwitt | I squashed it in | |
| 19:13:28 | sean-k-mooney | ack | |
| 19:14:05 | sean-k-mooney | eandersson: spatel https://review.opendev.org/c/openstack/nova/+/826931 | |
| 19:14:43 | sean-k-mooney | it will need test but we can likely just merge and updated version fo that and backport | |
| 19:14:55 | sean-k-mooney | its not a nova bug but we can workaorund it in nova | |
| 19:15:43 | spatel | Sweet!!! i can give it a try to this patch and see how it goes :) | |
| 19:36:27 | sean-k-mooney | spatel: we will proably proceed with https://review.opendev.org/c/openstack/nova/+/825496 instead | |
| 19:39:54 | spatel | that is also sounds good | |
| 19:40:04 | spatel | as far as it get merge | |
| 21:06:25 | opendevreview | Merged openstack/nova master: requirements: Remove os-xenapi https://review.opendev.org/c/openstack/nova/+/822747 | |
| 21:06:32 | opendevreview | Merged openstack/nova master: Remove Python 2-specific imports https://review.opendev.org/c/openstack/nova/+/822748 | |
| 22:10:19 | opendevreview | melanie witt proposed openstack/nova master: Raise InstanceNotFound on fkey constraint fail saving info cache https://review.opendev.org/c/openstack/nova/+/826942 | |
| #openstack-nova - 2022-01-29 | |||
| 08:11:35 | opendevreview | Merged openstack/nova stable/xena: Update centos 8 py36 functional job nodeset to centos stream 8 https://review.opendev.org/c/openstack/nova/+/825930 | |
| 18:41:02 | 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 | |
| #openstack-nova - 2022-01-31 | |||
| 08:54:52 | opendevreview | Pierre Libeau proposed openstack/nova master: Move file system freeze after end of mirroring https://review.opendev.org/c/openstack/nova/+/803713 | |
| 09:21:05 | kashyap | sean-k-mooney: Will look the SMM thing | |
| 09:55:00 | plibeau3 | bauzas: hello when you have time to read https://review.opendev.org/c/openstack/nova/+/803713/ QEMU community give some input | |
| 09:59:27 | bauzas | plibeau3: ack, I can take a look on it this afternoon | |
| 09:59:58 | stephenfin | sean-k-mooney: Can you look at https://review.opendev.org/c/openstack/nova/+/822749/ today? | |
| 10:34:25 | 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 | |
| 11:10:25 | 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 | |
| 11:51:43 | gibi | Uggla: everybody could nominate Z names, but only the TC votes (this will change in A release) | |
| 11:52:15 | Uggla | gibi, oh ok thx gibi. | |
| 11:59:24 | sean-k-mooney | stephenfin: yep ill look now | |
| 12:01:24 | sean-k-mooney | stephenfin: the only feedback i had is do we want to have the extras installed in the ci but you might be doing that in the next patch | |
| 12:01:50 | sean-k-mooney | which you are | |
| 12:01:58 | sean-k-mooney | so no im happy with both patches | |
| 12:02:13 | stephenfin | openstack server create --flavor m1.small --image fedora-iso --port e3caceae-2c74-4114-9f3f-262a37fc1971 --port e3caceae-2c74-4114-9f3f-262a37fc1971 test-server --wait | |
| 12:02:27 | stephenfin | ate entry 'fa:16:3e:62:88:38/e3caceae-2c74-4114-9f3f-262a37fc1971-0' for key 'virtual_interfaces.uniq_virtual_interfaces0address0deleted'") | |
| 12:02:27 | stephenfin | oslo_db.exception.DBDuplicateEntry: (pymysql.err.IntegrityError) (1062, "Duplic | |
| 12:04:48 | sean-k-mooney | stephenfin: yep | |
| 12:04:57 | sean-k-mooney | repating the port fails there | |
| 12:05:04 | sean-k-mooney | this is a known failure mode | |
| 12:05:09 | sean-k-mooney | we could block that in the api | |
| 12:05:15 | stephenfin | We probably should | |
| 12:05:46 | stephenfin | There were a few bugs related to this when I Googled but none had resolutions in them | |