| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2020-09-28 | |||
| 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 | openstackgerrit | Merged openstack/nova master: Add reproducer for bug #1894095 https://review.opendev.org/749713 | |
| 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: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: Migrate nova-grenade-multinode job to zuulv3 native https://review.opendev.org/742056 | |
| 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: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 | |
| 17:24:15 | lyarwood | gmann: if there's already a job in cinder then we can just use that, I couldn't see one in the plugin itself | |
| 17:24:28 | lyarwood | tosky: how so? stable/victoria is already cut | |
| 17:24:41 | lyarwood | tosky: are we really changing things in CI after RC? | |
| 17:24:43 | tosky | lyarwood: devstack-gate is branchless | |
| 17:24:48 | lyarwood | ah crap | |
| 17:25:00 | tosky | and it needs explicit support of each new release | |
| 17:25:01 | gmann | lyarwood: let me try those but as separate patch. i am +A on your patch and removed -W from mine | |
| 17:25:13 | tosky | that was the point of the legacy removal :) | |
| 17:25:28 | lyarwood | okay well it looks like we are backporting all of this to stable/victoria after all | |
| 17:25:59 | gmann | tosky: lyarwood i do not think openstack is 100% on zuulv3 and we can remove d-g support for victoria | |
| 17:26:09 | tosky | gmann: we are almost there | |
| 17:26:09 | dansmith | gmann: sorry just catching up | |
| 17:26:17 | tosky | and right now nova risk being the blocker | |
| 17:26:28 | tosky | check my work in progress etherpad | |
| 17:26:29 | dansmith | gmann: I think that losing the live migration on ceph coverage for really any amount of time, is problematic | |
| 17:26:35 | gmann | i expect d-g to be removed in W or X. | |
| 17:27:08 | gmann | tosky: yeah but we can remove dog support if it is 100% done and no dependency at all. | |
| 17:27:19 | tosky | isn't the ceph job a matter of including the devstack-plugin-ceph plugin and delegating the configuration to it? | |
| 17:27:21 | gmann | tosky: and do not forget the 3rd party CI/CD. | |
| 17:27:38 | tosky | gmann: 3rd party CI/CD have been told about that | |
| 17:27:45 | lyarwood | tosky: no, we need a multinode / multi compute job for these tests | |
| 17:27:57 | lyarwood | tosky: the standard job is just single host | |
| 17:28:28 | lyarwood | cinder-plugin-ceph-tempest-mn-aa looks like it's multinode but I don't think it's deploying two computes | |
| 17:28:52 | tosky | lyarwood: I mean, if you get the standard job, derive a the new job from it, change the nodeset and add devstack-plugin-ceph, what else will be missing? | |
| 17:29:05 | dansmith | the post-config change right? | |
| 17:29:19 | lyarwood | I'm pretty sure the plugin doesn't import the ceph keyring | |
| 17:29:27 | lyarwood | the old LM job would scp things around iirc | |
| 17:29:33 | lyarwood | we could hack something up quickly for that | |
| 17:29:44 | lyarwood | I just wanted to write a role for it etc | |
| 17:29:47 | lyarwood | into the plugin itself | |
| 17:30:21 | tosky | just as a general picture regarding the zuul v3 migration: right now, there are just: | |