| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2020-09-28 | |||
| 10:39:14 | lyarwood | ah cool libvirtd now auto restarts on focal >_< | |
| 10:43:46 | openstackgerrit | Lee Yarwood proposed openstack/nova master: nova-evacuate: Disable libvirtd systemd service during negative tests https://review.opendev.org/754684 | |
| 10:48:02 | gibi | lyarwood: do we need this on stable/victoria too ^^ ? | |
| 10:48:18 | lyarwood | gibi: yeah I'll create a bug now | |
| 10:48:54 | gibi | thanks | |
| 10:49:45 | lyarwood | gibi: actually it didn't make it | |
| 10:49:55 | lyarwood | gibi: so we will need a different stable-only fix there | |
| 10:50:06 | lyarwood | gibi: as I assume the live migration job is broken | |
| 10:51:05 | gibi | ahh I see | |
| 10:55:23 | openstackgerrit | Lee Yarwood proposed openstack/nova stable/victoria: [stable-only] nova-live-migration: Disable libvirtd during negative tests https://review.opendev.org/754688 | |
| 11:54:14 | openstackgerrit | Lee Yarwood proposed openstack/nova master: compute: Lock by instance.uuid lock during swap_volume https://review.opendev.org/754695 | |
| 12:03:06 | openstackgerrit | Merged openstack/nova master: libvirt: Use local variable instead of CONF https://review.opendev.org/749319 | |
| 12:48:05 | openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: Remove MIN_{LIBVIRT,QEMU}_NATIVE_TLS_VERSION https://review.opendev.org/746983 | |
| 12:48:06 | openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: Remove MIN_LIBVIRT_VIDEO_MODEL_VERSIONS https://review.opendev.org/746985 | |
| 12:48:06 | openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: Remove MIN_LIBVIRT_BETTER_SIGKILL_HANDLING https://review.opendev.org/746984 | |
| 12:48:07 | openstackgerrit | Lee Yarwood proposed openstack/nova master: WIP libvirt: Bump MIN_{LIBVIRT,QEMU}_VERSION and NEXT_MIN_{LIBVIRT,QEMU}_VERSION https://review.opendev.org/754700 | |
| 12:48:07 | openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: Remove MIN_{LIBVIRT,QEMU}_PMEM_SUPPORT https://review.opendev.org/746986 | |
| 12:48:08 | openstackgerrit | Lee Yarwood proposed openstack/nova master: WIP libvirt: Remove MIN_LIBVIRT_VIR_ERR_DEVICE_MISSING https://review.opendev.org/754702 | |
| 12:48:08 | openstackgerrit | Lee Yarwood proposed openstack/nova master: WIP libvirt: Remove MIN_{LIBVIRT,QEMU}_BLOCKDEV https://review.opendev.org/754701 | |
| 12:48:09 | openstackgerrit | Lee Yarwood proposed openstack/nova master: WIP libvirt: Remove MIN_LIBVIRT_S390X_CPU_COMPARE https://review.opendev.org/754704 | |
| 12:48:09 | openstackgerrit | Lee Yarwood proposed openstack/nova master: WIP libvirt: Remove MIN_LIBVIRT_VTPM https://review.opendev.org/754703 | |
| 12:59:25 | openstackgerrit | Lee Yarwood proposed openstack/nova master: nova-evacuate: Disable libvirtd service and sockets during negative tests https://review.opendev.org/754684 | |
| 13:11:49 | openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: Support extending locally attached rbd volumes https://review.opendev.org/752069 | |
| 14:00:27 | openstackgerrit | Takashi Natsume proposed openstack/nova master: Fix a hacking test https://review.opendev.org/754713 | |
| 14:02:20 | openstackgerrit | Takashi Natsume proposed openstack/nova master: Add a hacking rule for assert_has_calls https://review.opendev.org/754714 | |
| 14:09:01 | openstackgerrit | Takashi Natsume proposed openstack/nova master: Add a hacking rule for assert_has_calls https://review.opendev.org/754714 | |
| 15:21:17 | sean-k-mooney | corvus: do you plan to work on https://review.opendev.org/#/c/700083/5 more in the future, im off today and tomorrow so i wont get around to workign on my alpine element for a couple of days but your containerfile element could be a shortcut to creating a replacemnt for cirros based on somethign well maintinted like alpine | |
| 15:27:09 | sean-k-mooney | oh wrong window :) | |
| 15:27:30 | corvus | sean-k-mooney: it's not currently in my top-10 list of things to do, so please do feel free to run with it. i do still find it interesting and would love to see it working. | |
| 15:29:36 | sean-k-mooney | ya its not super high in my todo list too but this is interesting enought that i might work on it on and off. we dont often hit issue with cirros but having a light alternitive would be nice when we do. | |
| 15:30:18 | sean-k-mooney | corvus: we ocationally see kernel crashses in the gate with the cirros images but its rare enough that its not normally a problem | |
| 16:51:14 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Set instance host and drop migration under lock https://review.opendev.org/754815 | |
| 16:54:38 | lyarwood | gibi / stephenfin ; https://review.opendev.org/#/c/746983/ any issues if I +W these and the patches on top? | |
| 17:11:17 | gmann | lyarwood: gibi dansmith nova grenade multinode zuulv3 version working fine now and ready to review- https://review.opendev.org/#/c/742056/ | |
| 17:26:55 | openstack | bug 1894095 in OpenStack Compute (nova) "Running periodic task during live migration results in incorrect usage" [Medium,Confirmed] https://launchpad.net/bugs/1894095 - Assigned to Stephen Finucane (stephenfinucane) | |
| 17:26:55 | openstackgerrit | Merged openstack/nova master: Add reproducer for bug #1894095 https://review.opendev.org/749713 | |
| 17:38:35 | dansmith | gmann: am I reading it right, that this drops the "after the first test, reconfigure for ceph and run the live migration tests again" behavior? | |
| 17:58:39 | gmann | dansmith: yeah, ceph configure things will be done later, same way for live migration job - https://review.opendev.org/#/c/752557/6/.zuul.yaml@72 | |
| 17:59:42 | dansmith | gmann: okay just kinda sucks to lose that coverage in the meantime | |
| 17:59:49 | dansmith | what's holding that up? just time to do it? | |
| 18:06:23 | gmann | dansmith: i think it's not so urgent to finish so time is ok, let me try PS2 again where it cover the complete run_tests.sh | |
| 18:35:11 | NobodyCam | Good Morning Nova folks, Off the wall question, Is it possible to change the mgmt IP of a deployed hypervisor? | |
| #openstack-nova - 2020-09-29 | |||
| 03:18:45 | openstackgerrit | Federico Ressi proposed openstack/nova master: Requires default python3-devel package on CentOS 8 https://review.opendev.org/752073 | |
| 07:20:47 | gibi | lyarwood: hi! I think the original reviewers are happy to plug the +W back quickly | |
| 07:22:02 | gibi | lyarwood: I'm trying to think about a case when it is not safe to self +W after a rebase but so far I found nothing | |
| 08:01:32 | lyarwood | gibi: ack thanks, yeah I couldn't think of one but just wanted to make sure I wasn't breaking an unwritten rule :) | |
| 08:02:05 | lyarwood | gibi: https://review.opendev.org/#/c/754684/ should be correct now, I missed that the service now has .socket services associated with it | |
| 08:06:31 | kashyap | lyarwood: Morning; I see all bits for F32 infra support has merged | |
| 08:06:48 | kashyap | lyarwood: Is it worth doing a "recheck" on the 'q35' change to see the "refreshed" result? | |
| 08:10:05 | lyarwood | it already depended on the changes anyway | |
| 08:17:50 | kashyap | Oh, right | |
| 09:51:25 | lyarwood | stephenfin: https://review.opendev.org/#/c/754684/ - would you mind taking a look at this, should fix nova-evacuate now it's on focal | |
| 10:02:15 | openstackgerrit | Lee Yarwood proposed openstack/nova master: zuul: Replace nova-live-migration with zuulv3 jobs https://review.opendev.org/752557 | |
| 10:05:11 | openstackgerrit | Lee Yarwood proposed openstack/nova master: zuul: Replace nova-live-migration with zuulv3 jobs https://review.opendev.org/752557 | |
| 10:05:11 | openstackgerrit | Lee Yarwood proposed openstack/nova master: Migrate nova-grenade-multinode job to zuulv3 native https://review.opendev.org/742056 | |
| 10:05:43 | lyarwood | gmann: ^ rebased my change on yours so I don't break the grenade jobs | |
| 10:13:06 | openstackgerrit | wu.shiming proposed openstack/python-novaclient master: Remove the unused coding style modules https://review.opendev.org/754961 | |
| 10:54:09 | hemanth_n | Hi nova folks, appreciate any final reviews and closure on https://review.opendev.org/#/c/749175/ thanks | |
| 11:33:33 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Set instance host and drop migration under lock https://review.opendev.org/754815 | |
| 12:08:06 | gibi | o/ | |
| 12:49:17 | openstackgerrit | Merged openstack/nova master: libvirt: Remove MIN_{LIBVIRT,QEMU}_NATIVE_TLS_VERSION https://review.opendev.org/746983 | |
| 15:09:24 | CeeMac | hi | |
| 15:09:52 | CeeMac | is there a way to track progress of a volume migration beyond the 'os-vol-mig-status-attr:migstat' attribute? | |
| 15:10:13 | CeeMac | i've had a volume in 'migrating' status for over an hour now and I have a horrible feeling its stuck somehow | |
| 15:10:22 | CeeMac | sorry wrong channel! | |
| 15:19:47 | openstackgerrit | Merged openstack/nova master: libvirt: Remove MIN_LIBVIRT_BETTER_SIGKILL_HANDLING https://review.opendev.org/746984 | |
| 15:27:41 | gmann | lyarwood: thanks. dansmith and I chatted about it yesterday. let's cover the ceph storage also to avoid coverage. | |
| 15:43:55 | tosky | gmann: but the existing grenade job didn't cover ceph | |
| 15:44:05 | tosky | gmann: talking about https://review.opendev.org/#/c/742056/ | |
| 15:44:16 | gmann | tosky: yeah, there is no grenade job for ceph | |
| 15:45:12 | tosky | I would say than there is no reason for the W-1 | |
| 15:45:15 | tosky | it could be added later | |
| 15:48:20 | gmann | lyarwood: we can go with three options: 1. keep the run_test.sh script like PS2 (https://review.opendev.org/#/c/742056/2) and later convert that script to ansible roles, 2. keep ceph as legacy job(or zuulv3 with run_test.sh script) and rest convert to zuulv3, 3. keep legacy job and convert it completely along with ceph storage. which one you prefer ? | |
| 15:48:43 | gmann | tosky: it lose the coverage of ceph storage which is tested with legacy job | |
| 15:49:52 | tosky | gmann: shouldn't https://review.opendev.org/752557 get W-1 then, instead of https://review.opendev.org/#/c/742056/ ? | |
| 15:50:26 | gmann | tosky: both. | |
| 15:51:15 | gmann | current legacy job (grenade and live migration) run the tests on ceph too | |
| 15:51:42 | gmann | with runtime setup of ceph on multi nodes | |
| 15:54:15 | tosky | oh | |
| 15:54:29 | tosky | without devstack-plugin-ceph then? | |
| 15:54:33 | tosky | I guess it's a thing from the past | |
| 16:05:39 | gmann | no, by using devstack-plugin-ceph with cloning and running the setup from there. | |
| 16:06:03 | gmann | but we can use devstack-plugin-ceph with separate ceph job as added as TODO in those patches | |
| 16:36:13 | openstackgerrit | Merged openstack/python-novaclient master: Remove the unused coding style modules https://review.opendev.org/754961 | |
| 17:14:56 | lyarwood | gmann / tosky ; tbh I think we can leave ceph as a TODO for the W cycle | |
| 17:15:23 | lyarwood | gmann / tosky ; for both grenade and LM as it's going to take lots of work in the plugin to get multinode jobs working | |
| 17:15:48 | lyarwood | gmann / tosky ; we previously hacked ceph support into the legacy jobs, I'd like to do it correctly this time | |
| 17:15:58 | gmann | lyarwood: and keep running ceph as part of legacy job? | |
| 17:16:05 | gmann | for victoria | |
| 17:16:43 | lyarwood | gmann: well I wasn't going to backport things to stable/victoria now unless we had to for this tbh | |
| 17:17:18 | lyarwood | gmann: if we are then we can backport the eventual multinode ceph stuff as well | |
| 17:17:41 | lyarwood | gmann: but I really can't think of a ceph specific bug caught by either job in the last few cycles FWIW | |
| 17:18:08 | gmann | lyarwood: ohk, as victoria branch is cut now so anyways these patches are going in W. | |
| 17:20:54 | gmann | dansmith: are you ok for ceph as TODO in W cycle ? Victoria gate will be no change and keep ceph testing in legacy job same way. | |
| 17:22:35 | lyarwood | to be clear, it's the live migration, grenade and evacuation tests on ceph that are going to be a TODO early in this cycle | |
| 17:22:47 | gmann | yeah. | |
| 17:22:55 | lyarwood | we just need a multinode job in the plugin itself and everything else is trivial tbh | |
| 17:23:11 | tosky | but legacy is going to break | |
| 17:23:19 | gmann | lyarwood: cinder has multinode ceph job but not grenade | |
| 17:23:25 | tosky | as devstack-gate won't support victoria | |