| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2020-09-17 | |||
| 16:31:58 | sean-k-mooney | so it wont work on a paused guest | |
| 16:32:20 | sean-k-mooney | im not sure about virtio-scisi but i suspect that wont work either | |
| 16:32:29 | stephenfin | bauzas: we're questioning whether the live parameter makes sense here https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L2247 | |
| 16:32:50 | stephenfin | or if we should be outright rejecting a request to attach/detach a device from a paused guest | |
| 16:32:54 | sean-k-mooney | stephenfin: i think we shoudl be passing both live and persisted in almsot every case | |
| 16:33:18 | sean-k-mooney | stephenfin: i dont know of any reason we would not want to update both | |
| 16:33:23 | sean-k-mooney | always | |
| 16:33:53 | sean-k-mooney | well i guess if the vm was off | |
| 16:34:03 | sean-k-mooney | but in htat case we dont need to update the domain | |
| 16:34:10 | sean-k-mooney | we will just recreated it when we start it | |
| 16:36:28 | stephenfin | okay, let me see if I can untangle this /o\ | |
| 16:43:48 | sean-k-mooney | this could explain why we are getting out of sync | |
| 16:44:08 | sean-k-mooney | there is at least one bug where the live and perstient domains get out of sync | |
| 16:44:21 | sean-k-mooney | if we are attaching with persistent and detaching with live that would do it | |
| 16:48:00 | lyarwood | sean-k-mooney: https://review.opendev.org/#/c/747123/ FYI | |
| 16:48:05 | lyarwood | sean-k-mooney: moved to the train UCA | |
| 16:57:34 | sean-k-mooney | cool if you need more testing i can try that but ill just see what the ci says first | |
| 17:00:40 | openstackgerrit | sean mooney proposed openstack/nova master: Fix unplugging VIF when migrate/resize VM https://review.opendev.org/751642 | |
| 17:00:40 | openstackgerrit | sean mooney proposed openstack/nova master: libvirt: delegate ovs plug to os-vif https://review.opendev.org/602432 | |
| 17:01:11 | sean-k-mooney | stephenfin: gibi i have not fixed the functest failure but hopefully its simpler to figure out now that i have pushed the patches | |
| 17:01:29 | sean-k-mooney | stephenfin: ill keep debuging it locally and see if i can figure it out | |
| 17:29:23 | openstackgerrit | Lee Yarwood proposed openstack/nova master: WIP zuul: Introduce nova-multinode-evacuate https://review.opendev.org/744883 | |
| 19:00:21 | openstackgerrit | Merged openstack/nova master: api: Set min, maxItems for server_group.policies field https://review.opendev.org/752211 | |
| 19:00:31 | openstackgerrit | Merged openstack/nova master: Add regression test for bug #1895696 https://review.opendev.org/752247 | |
| 19:00:31 | openstack | bug 1895696 in OpenStack Compute (nova) "unable to boot instance from encrypted volume created from a glance image of an encrypted volume" [High,In progress] https://launchpad.net/bugs/1895696 - Assigned to Lee Yarwood (lyarwood) | |
| 19:59:44 | openstackgerrit | Merged openstack/nova master: compute: Skip cinder_encryption_key_id check when booting from volume https://review.opendev.org/752090 | |
| 20:02:22 | openstackgerrit | Lee Yarwood proposed openstack/nova master: zuul: Introduce nova-multinode-evacuate https://review.opendev.org/744883 | |
| 20:02:23 | openstackgerrit | Lee Yarwood proposed openstack/nova master: zuul: Replace nova-live-migration with zuulv3 jobs https://review.opendev.org/752557 | |
| 20:04:54 | openstackgerrit | Lee Yarwood proposed openstack/nova master: zuul: Introduce nova-evacuate https://review.opendev.org/744883 | |
| 20:04:54 | openstackgerrit | Lee Yarwood proposed openstack/nova master: zuul: Replace nova-live-migration with zuulv3 jobs https://review.opendev.org/752557 | |
| 20:24:37 | openstackgerrit | Lee Yarwood proposed openstack/nova master: fakelibvirt: Use versionutils to set min versions found in the driver https://review.opendev.org/749707 | |
| 20:24:38 | openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: Bump MIN_{LIBVIRT,QEMU}_VERSION and NEXT_MIN_{LIBVIRT,QEMU}_VERSION https://review.opendev.org/746981 | |
| 20:24:39 | openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: Remove MIN_LIBVIRT_FILE_BACKED_DISCARD_VERSION https://review.opendev.org/746982 | |
| 20:24:39 | openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: Remove MIN_{LIBVIRT,QEMU}_NATIVE_TLS_VERSION https://review.opendev.org/746983 | |
| 20:24:39 | openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: Remove MIN_LIBVIRT_BETTER_SIGKILL_HANDLING https://review.opendev.org/746984 | |
| 20:24:40 | openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: Remove MIN_LIBVIRT_VIDEO_MODEL_VERSIONS https://review.opendev.org/746985 | |
| 20:24:40 | openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: Remove MIN_{LIBVIRT,QEMU}_PMEM_SUPPORT https://review.opendev.org/746986 | |
| 20:28:15 | openstackgerrit | Lee Yarwood proposed openstack/nova stable/queens: libvirt: Provide VIR_MIGRATE_PARAM_PERSIST_XML during live migration https://review.opendev.org/747976 | |
| 20:56:16 | openstackgerrit | Merged openstack/nova stable/ussuri: Move confirm resize under semaphore https://review.opendev.org/751353 | |
| 22:55:45 | gregwork | is it possible to adjust the weight of vcpu qupotas in nova | |
| 22:56:21 | gregwork | like lets say i have a flavor that has the extra spec of cpushares 256 and is part of a cpu pinset of 4 cores | |
| 22:56:45 | gregwork | this would be a very limited amount of cpu time an instance could occupy during contention | |
| 23:35:30 | openstackgerrit | Merged openstack/nova stable/train: libvirt:driver:Disallow AIO=native when 'O_DIRECT' is not available https://review.opendev.org/747610 | |
| #openstack-nova - 2020-09-18 | |||
| 00:00:29 | openstackgerrit | Jeremy Stanley proposed openstack/nova master: DNM: Testing PDF builds https://review.opendev.org/752585 | |
| 00:34:04 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: DNM: Testing PDF builds https://review.opendev.org/752585 | |
| 07:51:22 | bauzas | good Friday Nova | |
| 08:04:25 | lyarwood | Morning \o | |
| 08:05:18 | lyarwood | sean-k-mooney / bauzas / stephenfin / gibi ; so some good news, the libvirt version bump https://review.opendev.org/#/c/746981/ works with the train UCA https://review.opendev.org/#/c/747123/ | |
| 08:05:31 | lyarwood | the only issue remaining appears to be with how the grenade jobs are deployed | |
| 08:06:04 | lyarwood | oh they are, weird | |
| 08:09:50 | lyarwood | ah right I need to backport this to stable/ussuri | |
| 08:10:18 | lyarwood | fun chicken and egg problem here | |
| 08:11:15 | kashyap | lyarwood: Mornin, I still haven't gotten to the Focal thing; was beset with a worser storm downstream, if you know what I mean :D | |
| 08:12:54 | lyarwood | np | |
| 08:14:21 | nightmare_unreal | Hello is there a glance dedicated channel ? | |
| 08:14:50 | nightmare_unreal | nvm found it | |
| 08:19:09 | gibi | lyarwood: what exactly need to be backported to ussir? | |
| 08:19:13 | gibi | ussuri | |
| 08:21:16 | lyarwood | gibi: https://review.opendev.org/#/c/752641/ I believe, do devstack uses the train UCA when deplying the stable/ussuri env during a grenade run | |
| 08:21:37 | lyarwood | that or I could hack enabling the UCA into grenade on master somehow | |
| 08:22:04 | lyarwood | so* devstack sorry | |
| 08:22:28 | gibi | hm so in ussuri the train UCA has newer libvirt than the default packages? | |
| 08:24:00 | lyarwood | yes correct | |
| 08:24:39 | lyarwood | the train UCA has the new MIN_{LIBVIRT,QEMU}_VERSIONs we are looking to set in the current bump on master | |
| 08:24:59 | lyarwood | as well | |
| 08:26:24 | gibi | this means that we now change what libvirt version we use for testing the upgrade from ussuri | |
| 08:36:22 | lyarwood | gibi: we would also change the version of libvirt installed when testing stable/ussuri as a whole | |
| 08:36:39 | lyarwood | gibi: but tbh I don't think anyone is actually using the default versions installed in bionic | |
| 08:38:59 | bauzas | sorry I was focusing on some downstream bugs | |
| 08:39:03 | bauzas | a tl:dr: please ? | |
| 08:40:47 | lyarwood | bauzas: sean-k-mooney suggested that we use the Train UCA in Victoria so we can land the libvirt version bump *without* hitting the QEMU issue found in focal and the Ussuri UCA. | |
| 08:41:17 | bauzas | lyarwood: for all jobs ? | |
| 08:41:17 | lyarwood | bauzas: This mostly works but we need to also use the Train UCA on stable/ussuri to allow grenade jobs on master to pass | |
| 08:41:26 | lyarwood | bauzas: yes for all jobs | |
| 08:41:35 | bauzas | lyarwood: so it would be a Focal + train UCA for all jobs, right? | |
| 08:41:50 | lyarwood | bauzas: no just bionic | |
| 08:41:58 | lyarwood | bauzas: this doesn't change the focal situation | |
| 08:42:10 | lyarwood | bauzas: it just means we can land the libvirt version bump ahead of the focal change | |
| 08:42:33 | lyarwood | bauzas: so we don't end up landing the version bump after rc | |
| 08:42:35 | bauzas | lyarwood: I raised a concern yesterday about us bumping minimums so close to RC1 | |
| 08:42:50 | bauzas | but I heard people wanting us to bump | |
| 08:43:01 | lyarwood | bauzas: we don't have a choice if the TC are pushing us to move to focal | |
| 08:43:20 | bauzas | well, TC aren't operators | |
| 08:43:25 | bauzas | some of them are | |
| 08:43:48 | bauzas | but I don't want us to hold all our gate so close to RC1 because we found some bugs unexpectedly | |
| 08:44:10 | bauzas | telling "we had to do it because $TC" is a terrible explanation | |
| 08:44:13 | lyarwood | I also hate making changes like this so close to RC tbh | |
| 08:44:25 | lyarwood | but if we don't support bionic | |
| 08:44:33 | bauzas | okay, let's be pragmatic and discuss about the change surface to be as minimal as we can | |
| 08:44:48 | gibi | do we need to change the min libvirt version to be able to switch to Focal? | |
| 08:44:58 | bauzas | there are two different concerns here | |
| 08:45:04 | bauzas | 1/ people want to bump versions | |
| 08:45:10 | bauzas | 2/ TC wants us to move to Focal | |
| 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 | |