Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2020-09-28
09:27:09 openstackgerrit Stephen Finucane proposed openstack/nova master: Add reproducer for bug #1894095 https://review.opendev.org/749713
10:05:46 openstackgerrit Stephen Finucane proposed openstack/nova master: api: Add type hints for 'nova.compute.api' https://review.opendev.org/741623
10:05:47 openstackgerrit Stephen Finucane proposed openstack/nova master: tests: Remove unnecessary logic from resize, migrate tests https://review.opendev.org/743207
10:05:47 openstackgerrit Stephen Finucane proposed openstack/nova master: tests: Duplicate '_test_resize' to '_test_migrate' https://review.opendev.org/743206
10:05:48 openstackgerrit Stephen Finucane proposed openstack/nova master: api: Split 'nova.compute.API._resize' https://review.opendev.org/743208
10:05:48 openstackgerrit Stephen Finucane proposed openstack/nova master: api: Add 'nova.compute.API.migrate' https://review.opendev.org/741624
10:34:56 lyarwood is the zuul log UI broken for anyone else?
10:35:40 gibi lyarwood: a random result works for me https://zuul.opendev.org/t/openstack/build/ec16025bf0fe44bd827846cea466236d/log/job-output.txt
10:36:52 lyarwood weird, might be my connection
10:36:57 lyarwood it's just spinning for me
10:37:02 lyarwood thanks gibi
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

Earlier   Later