| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2020-09-18 | |||
| 08:45:19 | bauzas | those two concerns aren't interrelated | |
| 08:45:44 | bauzas | (AFAICT) | |
| 08:46:00 | lyarwood | right, we did however say that we would bump the min versions in V | |
| 08:46:08 | lyarwood | and that we would also move the next versions in V | |
| 08:46:46 | lyarwood | gibi: correct we don't need to change the min to move to focal | |
| 08:46:49 | bauzas | bumping minimums is somehow understandable | |
| 08:47:03 | bauzas | more than us moving to Focal, tbh | |
| 08:47:11 | openstackgerrit | Liang Fang proposed openstack/nova master: [WIP] rbd patch for volume local cache https://review.opendev.org/726762 | |
| 08:47:28 | bauzas | anyway, I want us to stop focusing on distros, and rather be discussing on QEMU versions | |
| 08:47:28 | stephenfin | I personally class the bumping of minimums more important and less risky than the focal move | |
| 08:47:32 | lyarwood | tbh I'd personally drop the entire move to focal at this point | |
| 08:47:34 | stephenfin | oh, jinx :) | |
| 08:47:44 | lyarwood | \o/ | |
| 08:48:03 | gibi | droping Focal needs to be agreed with the TC | |
| 08:48:12 | bauzas | stephenfin: yeah I'm kinda leading to this conclusion | |
| 08:48:24 | lyarwood | I'm amazed that we continued with it after M2 tbh | |
| 08:48:31 | lyarwood | move early or don't move at all | |
| 08:48:36 | stephenfin | apologies if this was discussed yesterday, but couldn't we have a single non-voting tempest job on Focal? | |
| 08:48:50 | bauzas | gibi: I don't understand why the OpenStack community makes an hard sell on a OS thing | |
| 08:48:59 | stephenfin | to satisfy that request | |
| 08:49:11 | bauzas | I thought we only considered dependencies and not OSes | |
| 08:49:13 | lyarwood | yeah but I get the feeling that gmann's changes are to the base jobs | |
| 08:49:15 | bauzas | to be frank | |
| 08:49:23 | lyarwood | we could overwrite the node labels in our tree however | |
| 08:49:25 | gibi | bauzas: me neither but I think to make the communication clear we have to be on the same side with the TC | |
| 08:49:37 | bauzas | okay, let's be pragmatic | |
| 08:50:08 | bauzas | the problem is with a specific QEMU version that races on volume attach/detach, right? | |
| 08:50:13 | stephenfin | ah, you mean the parent 'openstack-*' and 'legacy-*' jobs? | |
| 08:50:25 | bauzas | and past this version, QEMU isn't fixed yet | |
| 08:50:30 | bauzas | am i correct? | |
| 08:50:54 | lyarwood | stephenfin: yeah | |
| 08:51:04 | bauzas | so, we can legitamately bump the minimums without being impacted by the QEMU issue | |
| 08:51:11 | bauzas | legitimatly | |
| 08:51:14 | lyarwood | bauzas: that's the issue with focal and the ussuri UCA yes | |
| 08:51:41 | lyarwood | bauzas: but we can only bump the mins *if* we use the train UCA, ussuri UCA or move to focal | |
| 08:51:42 | bauzas | lyarwood: because the ussuri UCA ships a qemu version that's impacted, right? | |
| 08:51:55 | lyarwood | bauzas: correct it's the same versions as shipped in focal | |
| 08:52:08 | lyarwood | bauzas: with the train UCA being a little older and not impacted by the bug | |
| 08:52:11 | bauzas | ok, so the problem is with UCA, not the distro | |
| 08:52:26 | lyarwood | bauzas: well both, it's in the core distro from focal | |
| 08:52:32 | bauzas | ah I see | |
| 08:52:39 | bauzas | gotcha | |
| 08:52:41 | bauzas | so | |
| 08:52:49 | bauzas | I'm all documenting this | |
| 08:53:07 | gibi | buming minimum means we change the libvirt version on stable/ussuri too | |
| 08:53:13 | gibi | is it OK? | |
| 08:53:18 | bauzas | gibi: why ? | |
| 08:53:22 | gibi | did we do that before in other stable branches? | |
| 08:53:30 | lyarwood | gibi: yeah we've done that in the past | |
| 08:53:42 | stephenfin | I guess we'll change the version tested if we backport, yes. We won't change the versions allowed though | |
| 08:53:43 | lyarwood | bauzas: grenade | |
| 08:53:45 | gibi | lyarwood: so it is sort of a normal thing at a min bump? | |
| 08:53:48 | brinzhang_ | gibi: IMO, in V release is enough | |
| 08:54:03 | lyarwood | gibi: iirc the last time we did this was due to QEMU/libvirt bugs | |
| 08:54:16 | bauzas | lyarwood: what? | |
| 08:54:24 | lyarwood | gibi: so we enabled the UCA to workaround these and had to do it on an older stable release for grenade as well | |
| 08:54:31 | bauzas | lyarwood: I thought grenade was upgrading on the target | |
| 08:54:42 | brinzhang_ | Cyborg done this today https://review.opendev.org/#/q/project:openstack/cyborg+branch:master+topic:migrate-to-focal | |
| 08:54:43 | lyarwood | bauzas: it does but it doesn't rerun devstack | |
| 08:54:54 | brinzhang_ | bump to the min version | |
| 08:55:06 | lyarwood | bauzas: so we'd need to use the train UCA on stable/ussuri to ensure n-cpu from master would work | |
| 08:55:23 | lyarwood | stephenfin: correct, it's just the tested version in stable/ussuri being changed | |
| 08:55:42 | lyarwood | and again I don't think anyone is using the min versions of libvirt and QEMU at the moment | |
| 08:56:06 | lyarwood | if you're using Ubuntu you're using the UCAs | |
| 08:56:20 | lyarwood | if you're using el distro you;re already way past these versions on stable/ussuri | |
| 08:56:56 | bauzas | ok, either way | |
| 08:57:04 | bauzas | if we merge the bumps | |
| 08:57:28 | bauzas | are we sure that the new minimums are satisfied by the train UCA ? | |
| 08:57:40 | bauzas | because | |
| 08:57:50 | bauzas | we could just don't care about train UCA | |
| 08:57:59 | lyarwood | bauzas: yes see https://review.opendev.org/#/c/746981/ | |
| 08:58:12 | bauzas | but if we would like to tackle the other issue, we would need to deploy a Bionic with Train UCA | |
| 08:58:23 | lyarwood | bauzas: specifically https://zuul.opendev.org/t/openstack/build/318b52b1aaf6473787405c9ed1ca0bc0/log/controller/logs/dpkg-l.txt | |
| 08:58:30 | bauzas | perfect, then I'm clear | |
| 08:58:37 | lyarwood | bauzas: that change Depends-On the devstack change pulling in the train UCA | |
| 08:58:50 | lyarwood | bauzas: all of the non grenade jobs are working | |
| 08:58:59 | gibi | OK, so it is not unprecedentet to change the tested libvirt version on a stable branch | |
| 08:59:00 | bauzas | okay, so I personnally feel that bumping minimums are reasonably safe | |
| 08:59:02 | gibi | so lets do it then | |
| 08:59:09 | bauzas | +1 | |
| 08:59:16 | gibi | bump the minimum and turn on train UCA in stable/ussuri | |
| 08:59:40 | bauzas | and let's do what we approved yesterday for the Focal thing | |
| 08:59:51 | bauzas | the gmann's work on jobs | |
| 09:00:08 | openstackgerrit | Takashi Natsume proposed openstack/nova master: Remove six.moves https://review.opendev.org/727224 | |
| 09:00:21 | bauzas | lyarwood: sean-k-mooney: that being said, we need to document the gap | |
| 09:00:47 | openstack | Launchpad bug 1882521 in OpenStack Compute (nova) "Failing device detachments on Focal" [High,Confirmed] - Assigned to Lee Yarwood (lyarwood) | |
| 09:00:47 | bauzas | I guess we can use https://bugs.launchpad.net/nova/+bug/1882521 as a base start | |
| 09:01:11 | openstackgerrit | Takashi Natsume proposed openstack/nova master: Remove six.iteritems/itervalues/iterkeys https://review.opendev.org/727757 | |
| 09:01:12 | gibi | bauzas: could you drop the -2 from https://review.opendev.org/#/c/749707 then? | |
| 09:01:22 | bauzas | gibi: I was on it | |
| 09:01:25 | openstackgerrit | Takashi Natsume proposed openstack/nova master: Remove six.byte2int/int2byte https://review.opendev.org/727777 | |
| 09:01:27 | gibi | coolio | |
| 09:01:28 | lyarwood | ubuntu-- https://zuul.opendev.org/t/openstack/build/fc4fd2d6fcd14618899060c4e84edca0/log/job-output.txt#6579 one small thing to fix here for the evacuation tests also | |
| 09:01:44 | openstackgerrit | Takashi Natsume proposed openstack/nova master: Remove six.binary_type/integer_types/string_types https://review.opendev.org/728094 | |
| 09:01:52 | lyarwood | bauzas: and you mean outside of https://review.opendev.org/#/c/746981/6/releasenotes/notes/victoria-libvirt-version-bump-e1a09b3a72ee56a4.yaml | |
| 09:01:58 | openstackgerrit | Takashi Natsume proposed openstack/nova master: Remove six.text_type (1/2) https://review.opendev.org/728109 | |
| 09:02:24 | openstackgerrit | Takashi Natsume proposed openstack/nova master: Remove six.text_type (2/2) https://review.opendev.org/728117 | |
| 09:02:59 | openstackgerrit | Takashi Natsume proposed openstack/nova master: doc: correct the link to user/flavor.rst https://review.opendev.org/740176 | |
| 09:03:09 | bauzas | lyarwood: no, it's a matter of communicating the caveat we found with Focal | |
| 09:03:16 | openstackgerrit | Takashi Natsume proposed openstack/nova master: doc: Fix rendering in the accelerator support doc https://review.opendev.org/751574 | |