| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2020-09-16 | |||
| 17:36:26 | lyarwood | sean-k-mooney: sorry just pushing something and I need to drop again, can we catch up about that backport stuff in the morning? | |
| 17:36:42 | lyarwood | s/morning/your\ morning/g ;) | |
| 17:36:47 | sean-k-mooney | then if i have an old souce and a new dest then i think we dont clean up the vifs | |
| 17:37:00 | sean-k-mooney | lyarwood: ya we can no rush | |
| 17:37:19 | sean-k-mooney | melwitt: im not sure how to handel that other then leave the extra call to unplug_vifs in the backport | |
| 17:37:47 | sean-k-mooney | melwitt: unplug_vifs shoudl be callable twice without any ill effects | |
| 17:38:36 | melwitt | yeah, well wouldn't you have to leave it on master (not only backports) for one cycle? to bridge the N-1 to N upgrade? | |
| 17:39:10 | sean-k-mooney | melwitt: well the proablem is i need to backport the cve issue to like queens eventually | |
| 17:39:43 | sean-k-mooney | what i think i would have to do is keep the call to unplug_vifs in cleanup but wrap it in a try excetp | |
| 17:39:59 | melwitt | I'm just saying re: leaving the extra call, I think you'd have to leave it on master too, not only leave it on the backports. because the upgrade issue exists for master as well | |
| 17:40:08 | sean-k-mooney | oh right | |
| 17:40:17 | sean-k-mooney | i woudl have to leave it on master for a cycle ya | |
| 17:40:46 | sean-k-mooney | ill put the patches up for peopel to see as 3 patches | |
| 17:41:10 | sean-k-mooney | 1 for the new call in resize. one for the cve fix which is alredy up for review and one for the extra call removal | |
| 17:41:28 | sean-k-mooney | and we can defer that last one for a cycle | |
| 17:42:04 | sean-k-mooney | and i guess we can see if peopel think of any other reason not to do it this way. | |
| 17:42:10 | melwitt | yeah, so AFAICT the backport concern here would be the complexity and impact to something that is supposed to be "stable" but it's needed for a CVE so I think that makes it backportable (again unless I'm missing something) | |
| 17:43:03 | sean-k-mooney | ya if we think its backportable and we are happy with it on master i guess i dont have to rush the backport either | |
| 17:43:17 | sean-k-mooney | e.g. we coudl wait a while and let it back | |
| 17:43:20 | sean-k-mooney | *bake | |
| 17:44:24 | melwitt | yeah I'm just concerned about there being a bug or edge case that crops up in the gate and then we have a fix to backport 5 times too | |
| 17:44:51 | melwitt | not the end of the world, we did it recently :P but the goal is to avoid that | |
| 17:45:10 | sean-k-mooney | :) ya im ok with that | |
| 17:45:26 | sean-k-mooney | this partcalar issue has been dragin on since 2018 or longer | |
| 17:45:27 | melwitt | let's see what other ppl think. I worry I'm missing something unbackportable about it | |
| 17:45:46 | sean-k-mooney | 2017-11-24 | |
| 17:46:03 | melwitt | yeah, I think that might be the same neutron security bug I emailed about downstream a long time ago | |
| 17:46:22 | sean-k-mooney | i started working on this when i was still at intel | |
| 17:46:31 | sean-k-mooney | we have fixed in in a number of edgecases | |
| 17:46:41 | sean-k-mooney | but there is still 1 that is unfixed | |
| 17:46:51 | sean-k-mooney | just trying to get the fine case closed | |
| 17:47:23 | melwitt | understood | |
| 17:47:52 | sean-k-mooney | https://review.opendev.org/#/c/602432/ fixes the final case but its failing or reverts because of the change in unplug behaivor wehn we get rid of the libvirt race | |
| 17:59:03 | sean-k-mooney | melwitt: oh one related question do we think this could be an RC1 candiate so that we could count victoria as the first release or would wallaby have to be the first release with X for the removal of the second call | |
| 17:59:18 | sean-k-mooney | i guess we can cross that bridge wehn we come to it | |
| 17:59:52 | sean-k-mooney | but im not really sure given how latent both issue are that it qualifies for RC1 | |
| 18:00:18 | sean-k-mooney | i would like it to but i would like many things to be true that are not :) | |
| 18:00:38 | melwitt | sean-k-mooney: yeah I think it'd be a hard sell for RC1 because of how latent | |
| 18:01:13 | sean-k-mooney | ya ok. when i leave the todo commont for remoinv it ill say after wallaby | |
| 18:01:35 | melwitt | but you could bring it up at the nova meeting tomorrow to get more input | |
| 18:02:28 | sean-k-mooney | ya. i asked the peroson that reported the other bug if they were ok with me updating there patch to just do the method move if so i should have a patch up before then if not ill submit a sperate patch for that i guess | |
| 18:02:51 | sean-k-mooney | in either case i can add it to the open disucssion secation at the end | |
| 18:03:06 | melwitt | but eh... looking at the release schedule RC1 is Sep 24, not sure we could get proper review done by then | |
| 18:03:26 | sean-k-mooney | ya its rather tight | |
| 18:03:40 | sean-k-mooney | ill just assume this will have to be backported to stable/victoria | |
| 18:03:50 | sean-k-mooney | its fine | |
| 18:04:02 | melwitt | yeah, realistically I think that will be the case | |
| 18:04:36 | sean-k-mooney | ya no worries. | |
| 18:05:17 | sean-k-mooney | ill leave it of the agenda so and just ask lee and other for review once the patches are ready | |
| 18:05:58 | sean-k-mooney | i havent looked at the rc-potential bugs list yet | |
| 18:06:05 | sean-k-mooney | do we have any/many | |
| 18:07:20 | sean-k-mooney | oh just one https://bugs.launchpad.net/nova/+bugs?field.tag=victoria-rc-potential | |
| 18:07:28 | sean-k-mooney | https://bugs.launchpad.net/nova/+bug/1882521 | |
| 18:07:28 | openstack | Launchpad bug 1882521 in OpenStack Compute (nova) "Failing device detachments on Focal" [High,Confirmed] - Assigned to Lee Yarwood (lyarwood) | |
| 18:08:08 | sean-k-mooney | and lee has raised a qemu bug for that https://bugs.launchpad.net/qemu/+bug/1894804 | |
| 18:08:08 | openstack | Launchpad bug 1894804 in QEMU "Second DEVICE_DELETED event missing during virtio-blk disk device detach" [Undecided,New] | |
| 18:08:27 | sean-k-mooney | so i guess that is not really an rc blocker | |
| 18:10:36 | sean-k-mooney | and danpb basically conrimed it as a qemu bug in https://bugs.launchpad.net/qemu/+bug/1894804/comments/3 | |
| 18:10:36 | openstack | Launchpad bug 1894804 in QEMU "Second DEVICE_DELETED event missing during virtio-blk disk device detach" [Undecided,New] | |
| 18:10:43 | melwitt | yeah AFAICT there isn't anything we can do to workaround that either | |
| 18:11:36 | sean-k-mooney | we might be able to mitigate it by using virtio-scsi instead of virtio-block | |
| 18:11:45 | sean-k-mooney | but ya not really | |
| 18:11:57 | sean-k-mooney | other then skip it in tempest for now | |
| 19:07:23 | openstackgerrit | Merged openstack/nova master: tests: Add regression test for bug 1894966 https://review.opendev.org/752210 | |
| 19:07:23 | openstack | bug 1894966 in OpenStack Compute (nova) "Create servergroup failed with unexpected error" [Low,In progress] https://launchpad.net/bugs/1894966 - Assigned to Stephen Finucane (stephenfinucane) | |
| 19:20:09 | openstackgerrit | Merged openstack/os-vif master: Update master for stable/victoria https://review.opendev.org/750926 | |
| 19:33:37 | openstackgerrit | Lee Yarwood proposed openstack/nova master: Add regression test for bug #1895696 https://review.opendev.org/752247 | |
| 19:33:37 | openstack | bug 1895696 in OpenStack Compute (nova) "unable to boot instance from encrypted volume created from a glance image of an encrypted volume" [Medium,In progress] https://launchpad.net/bugs/1895696 - Assigned to Lee Yarwood (lyarwood) | |
| 19:33:37 | openstackgerrit | Lee Yarwood proposed openstack/nova master: compute: Skip cinder_encryption_key_id check when booting from volume https://review.opendev.org/752090 | |
| 19:44:27 | openstackgerrit | Merged openstack/python-novaclient master: Remove install unnecessary packages https://review.opendev.org/751637 | |
| 20:39:45 | openstackgerrit | Merged openstack/os-vif master: Add Python3 wallaby unit tests https://review.opendev.org/750927 | |
| 21:49:22 | openstackgerrit | Merged openstack/nova master: Fix error that cannot overwrite policy rule for 'forced_host' https://review.opendev.org/751152 | |
| 22:43:08 | openstackgerrit | Merged openstack/nova stable/rocky: Removed the host FQDN from the exception message https://review.opendev.org/749612 | |
| 23:50:50 | brinzhang0 | gibi, gmann, stephenfin: change 500-->400 is ok for me too ^^ thanks | |
| #openstack-nova - 2020-09-17 | |||
| 05:44:37 | openstackgerrit | Brin Zhang proposed openstack/nova stable/ussuri: tests: Add regression test for bug 1894966 https://review.opendev.org/752371 | |
| 05:44:37 | openstack | bug 1894966 in OpenStack Compute (nova) "Create servergroup failed with unexpected error" [Low,In progress] https://launchpad.net/bugs/1894966 - Assigned to Stephen Finucane (stephenfinucane) | |
| 07:05:54 | openstackgerrit | Brin Zhang proposed openstack/nova master: WIP: nova supports password encrypted VNC https://review.opendev.org/622336 | |
| 07:16:38 | bauzas | good morning Nova | |
| 07:17:04 | bauzas | sean-k-mooney: sorry, was afk yesterday so wasn't able to answer your backportability question, just ping me when you're back up | |
| 07:19:31 | gibi | good morning | |
| 08:37:00 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Handle libvirt reporting incorrect 4k page quantities https://review.opendev.org/631038 | |
| 08:37:00 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Make overcommit check for pinned instance pagesize aware https://review.opendev.org/631053 | |
| 08:41:05 | lyarwood | stephenfin: https://review.opendev.org/#/q/topic:bug/1895696 - would you mind hitting this again | |
| 08:56:40 | lyarwood | gibi: ^ also if you wouldn't mind, it's a regression for an awkward slightly convoluted encrypted volume use case | |
| 08:57:11 | gibi | lyarwood: looking.. | |
| 08:58:45 | lyarwood | many thanks | |
| 09:17:36 | aarents | Hi Nova! | |
| 09:17:43 | aarents | lyarwood you where close to +2 on that https://review.opendev.org/#/c/743708/ few weeks ago, if you can have a look when possible thks! | |
| 09:21:04 | lyarwood | aarents: morning, looking | |
| 09:42:18 | lyarwood | aarents: sorry for the delay, LGTM still, can you follow up with a devstack change so we can see it running somewhere? | |
| 09:42:30 | lyarwood | aarents: I left an example in the change FWIW | |
| 09:59:47 | openstackgerrit | Merged openstack/nova stable/stein: Add checks for volume status when rebuilding https://review.opendev.org/748559 | |
| 10:11:41 | admin0 | hi all .. what do I need to do in the database when i rename a hypervisor | |
| 10:11:51 | admin0 | i changed the hostname and did a nova-compute restart | |
| 10:12:01 | admin0 | what do i need to do to associate running instances to the new name | |
| 10:12:49 | lyarwood | erm that isn't supported at all | |
| 10:13:02 | lyarwood | your placement allocations are going to be totally borked | |
| 10:13:40 | lyarwood | can you undo the hostname change, empty the host of any instances and then do the hostname change? | |
| 10:14:16 | lyarwood | you should be able to delete service in nova and resource provider in placement with the old hostname | |
| 10:14:25 | lyarwood | and migrate everything back into the host | |