| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-01-28 | |||
| 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] Include pf mac and vf num in port updates https://review.opendev.org/c/openstack/nova/+/824833 | |
| 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: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: Bump os-traits to 2.7.0 https://review.opendev.org/c/openstack/nova/+/826675 | |
| 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: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 | oslo_db.exception.DBDuplicateEntry: (pymysql.err.IntegrityError) (1062, "Duplic | |
| 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: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 | |
| 12:09:13 | sean-k-mooney | ya i have seen customer hit this with heat templates and i think the shift on stack installer hit this about 4 mounts ago | |
| 12:09:28 | sean-k-mooney | i previously chatted to mdbooth about it. | |
| 12:10:04 | sean-k-mooney | i know its recently come up again downstream. are you looking at that? | |
| 12:16:03 | stephenfin | yup, spotted downstream | |
| 12:17:50 | opendevreview | Balazs Gibizer proposed openstack/placement master: Add microversion 1.39 to support any-trait queries https://review.opendev.org/c/openstack/placement/+/826719 | |
| 12:19:05 | opendevreview | Balazs Gibizer proposed openstack/placement master: Add microversion 1.39 to support any-trait queries https://review.opendev.org/c/openstack/placement/+/826719 | |
| 12:23:50 | opendevreview | Balazs Gibizer proposed openstack/placement master: Add microversion 1.39 to support any-trait queries https://review.opendev.org/c/openstack/placement/+/826719 | |
| 13:01:59 | dmitriis | sean-k-mooney: can't cleanly apply some changes on top of https://review.opendev.org/c/openstack/nova/+/819494/ since I rebased mine to the tip. But I removed the conflicting modifications in bind-time/plug-time event related functions which are getting removed in that change. | |
| 13:02:44 | dmitriis | (wondering if it's OK to rebase Artom's PRs myself) | |
| 13:03:33 | sean-k-mooney | ack am you could artom will be online soon if you want to wait and ask but i dont really see any harm in that. i would move it after the os-traits patch | |
| 13:04:11 | sean-k-mooney | alternitvily i can review those now and see if we can merge it shortly | |
| 13:07:03 | dmitriis | sean-k-mooney: ack, let me rebase both of his changes and I can cherry-pick cleanly then and reorder the VNIC_SMARTNIC patch and the os-traits patch | |
| 13:07:53 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Reject duplicate port IDs in server create https://review.opendev.org/c/openstack/nova/+/827070 | |
| 13:08:05 | stephenfin | sean-k-mooney: ^ (gibi, you might interested also) | |
| 13:08:18 | sean-k-mooney | alreday have it open :) | |
| 13:08:33 | sean-k-mooney | i assume a 400 will be returned | |
| 13:08:36 | stephenfin | yup | |
| 13:08:51 | opendevreview | Dmitrii Shcherbakov proposed openstack/nova master: Add nova-ovs-hybrid-plug job https://review.opendev.org/c/openstack/nova/+/817303 | |
| 13:08:51 | stephenfin | I'm really hoping to avoid a microversion for that, since it's so obviously wrong as-is | |
| 13:09:00 | stephenfin | but I can't make that choice alone, of course :) | |
| 13:09:02 | sean-k-mooney | 400 is already a valid return code | |
| 13:09:07 | sean-k-mooney | so no microversion needed | |
| 13:09:16 | stephenfin | right, but it's currently returning a 201 | |
| 13:09:21 | stephenfin | since server create is async | |
| 13:09:34 | sean-k-mooney | its returning a 201 but the server end in error | |
| 13:09:38 | stephenfin | yeah | |
| 13:09:44 | sean-k-mooney | i think we have an excpetion for that | |
| 13:10:14 | stephenfin | so I'm arguing that since every other subsequent server op would fail, this is an acceptable change. We shouldn't break anyones automation | |
| 13:10:24 | sean-k-mooney | https://docs.openstack.org/nova/latest/contributor/microversions.html#when-do-i-need-a-new-microversion | |
| 13:10:37 | sean-k-mooney | so i would argue that thies is the first yes | |
| 13:10:46 | sean-k-mooney | did we silently fail to do what is asked | |