Posted	Nick	Remark
#openstack-nova - 2019-10-05
19:39:12	openstackgerrit	Stephen Finucane proposed openstack/nova master: WIP https://review.opendev.org/686818
19:39:12	openstackgerrit	Stephen Finucane proposed openstack/nova master: docs: Blast most references to nova-network https://review.opendev.org/686817
#openstack-nova - 2019-10-06
12:21:27	openstackgerrit	Son Pham proposed openstack/nova-specs master: Up load new blueprint snapshot-vms-with-state-of-processes https://review.opendev.org/686934
14:13:32	openstackgerrit	Lenny Verkhovsky proposed openstack/os-vif master: WIP: Testing CI https://review.opendev.org/686937
18:24:11	openstackgerrit	Son Pham proposed openstack/nova-specs master: add snapshot blueprint https://review.opendev.org/686948
18:51:41	openstackgerrit	do3meli proposed openstack/nova master: Allow VMs to use unaddressed ports https://review.opendev.org/533249
#openstack-nova - 2019-10-07
01:36:30	openstackgerrit	Takashi NATSUME proposed openstack/nova master: api-ref: Fix security groups parameters https://review.opendev.org/686517
02:11:41	SonPham	Hi. what must i do after review my nova-spec to https://review.opendev.org/?
03:25:08	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Update contributor guide for Ussuri https://review.opendev.org/685630
07:22:14	bauzas	good morning Nova
08:23:01	gibi	bauzas: good morning
08:34:15	bauzas	gibi: good morning :)
09:27:19	stephenfin	gibi: Morning. Could I get you to look at https://review.opendev.org/#/c/684341/ and https://review.opendev.org/#/c/684342/ , since you've been looking at the rest
09:27:22	stephenfin	Please
09:27:24	stephenfin	:)
10:20:32	openstackgerrit	Stephen Finucane proposed openstack/nova master: VMware:fix get VM root device conditional verify https://review.opendev.org/454441
10:21:45	openstackgerrit	Stephen Finucane proposed openstack/nova master: VMware:fix get VM root device conditional verify https://review.opendev.org/454441
10:24:06	openstackgerrit	Stephen Finucane proposed openstack/nova master: Check enable_new_services option in service.py https://review.opendev.org/540962
10:26:42	openstackgerrit	Stephen Finucane proposed openstack/nova master: resolve confusing state when rebuilding https://review.opendev.org/535665
10:30:10	gibi	stephenfin: will try to look at today
10:37:04	openstackgerrit	Rico Lin proposed openstack/nova master: [Gate fix] Avoid use cell_uuid before assignment https://review.opendev.org/686996
10:50:49	openstackgerrit	Stephen Finucane proposed openstack/nova master: Improve error log when snapshot fails https://review.opendev.org/356020
10:58:06	openstackgerrit	Stephen Finucane proposed openstack/nova master: Handle unset 'connection_info' https://review.opendev.org/686997
10:58:36	openstackgerrit	Stephen Finucane proposed openstack/nova master: libvirt: don't call sync_guest_time if qga is not enabled https://review.opendev.org/524836
10:59:16	openstackgerrit	Stephen Finucane proposed openstack/nova master: Clear instance root_device_name in rebuild operation https://review.opendev.org/464991
11:00:44	openstackgerrit	Stephen Finucane proposed openstack/nova master: Use Placement API to check resource usage https://review.opendev.org/528953
12:11:48	openstackgerrit	sean mooney proposed openstack/nova master: support pci numa affinity policies in flavor and image https://review.opendev.org/674072
12:37:19	openstackgerrit	Stephen Finucane proposed openstack/nova master: Pass user context detaching volume on exceptions https://review.opendev.org/560115
12:48:23	openstackgerrit	Stephen Finucane proposed openstack/nova master: Remove RBD swap disk if resizing to a flavor without swap https://review.opendev.org/534711
13:05:17	openstackgerrit	Stephen Finucane proposed openstack/nova master: Convert floating IP tests to ddt https://review.opendev.org/471360
13:17:43	mdbooth	mriedem: If you've got a minute, I wonder if https://review.opendev.org/#/c/683922/1 is sufficiently baked yet.
13:18:32	mriedem	we haven't even released train
13:18:35	mriedem	so no i don't think so
13:18:39	mriedem	i'm talking months
13:19:09	mdbooth	Ack, I'll make a note in the review.
13:19:24	efried	mdbooth: at a glance, it looks like you're going to need a train backport too if you intend to land it in stein and earlier.
13:19:45	mdbooth	efried: It's in master
13:20:09	efried	mdbooth: master is ussuri rn
13:20:35	mdbooth	Ah, yes. I think we landed it just after the branch.
13:21:05	mriedem	no it'll be in train
13:22:09	mdbooth	Ok, thanks
13:23:14	mdbooth	efried: Yep, double checked and it's in train
13:23:18	efried	ah, yup
13:24:08	efried	aspiers: ?
13:40:53	sean-k-mooney	efried: just to clarify when i update my spec with the core liaison section do i qualify as a liaison or should i find a core to add.
13:42:10	openstackgerrit	Balazs Gibizer proposed openstack/nova-specs master: Support move operations with qos ports - Ussuri https://review.opendev.org/686779
13:42:12	efried	sean-k-mooney: I think you qualify. That will be a thing reviewers of your spec would be able to debate.
13:42:56	sean-k-mooney	ok ill rebase my spec on your patch and set it to me for now
13:44:08	openstackgerrit	Stephen Finucane proposed openstack/nova master: Avoid raise InstanceNotRunning exception https://review.opendev.org/541152
13:45:05	stephenfin	kashyap: Think you could look into this to see if it makes sense? https://review.opendev.org/#/c/589769/
13:59:11	openstack	bug 1709118 in OpenStack Compute (nova) "_ContextAuthPlugin needs a refresh" [Low,Confirmed] https://launchpad.net/bugs/1709118
13:59:11	openstackgerrit	Stephen Finucane proposed openstack/nova master: Remove workaround for bug #1709118 https://review.opendev.org/524316
14:00:03	openstackgerrit	Silvan Kaiser proposed openstack/nova stable/queens: Fixes multi-registry config in Quobyte driver https://review.opendev.org/687058
14:01:18	openstackgerrit	Matt Riedemann proposed openstack/nova master: Add boot from volume functional test with a huge request https://review.opendev.org/686734
14:01:19	openstackgerrit	Matt Riedemann proposed openstack/nova master: Cache image GETs for multi-create/multi-BDM requests https://review.opendev.org/686835
14:04:28	mriedem	lyarwood: i think https://review.opendev.org/#/c/679449/ is probably about the only thing worth getting into the next stable/rocky release if you want to push that through and then i'll queue up the release
14:05:36	lyarwood	mriedem: ack I'll take a look now
14:09:11	tellesnobrega	mriedem, hi, quick question nova reset-state command. I'm wondering if the command delete all ghost vms on the db? In case we do a backup of the DB and restore that later and the vms are gone, how can we clean up the DB
14:10:47	mriedem	it does not
14:10:58	mriedem	all reset-state does is change the vm_state on the instance and set task_state to None
14:11:11	mriedem	it has no interaction with the compute service
14:11:34	mriedem	to clean up the db you use the nova-manage db archive_deleted_rows command
14:11:48	tellesnobrega	mriedem, awesome
14:11:50	mriedem	with the --purge option if you really want to delete the data and not just move it to shadow tables
14:11:51	tellesnobrega	thanks, I will take a look
14:12:01	mriedem	info is here https://docs.openstack.org/nova/latest/cli/nova-manage.html
14:12:10	tellesnobrega	mriedem, thanks
14:14:33	openstackgerrit	Stephen Finucane proposed openstack/nova master: Prevent using blank volume as boot device https://review.opendev.org/171984
14:25:52	gibi	stephenfin: I have one question in https://review.opendev.org/#/c/686795/2/nova/tests/fixtures.py@1255
14:27:26	gshippey	Instance wont migrate to a newly provisioned compute host, this is the error in the nova-compute log: During detach_volume, instance disappeared.: InstanceNotFound: Instance 2b5cdc09-e451-4e1d-9044-4d85ffd29b51 could not be found. Has anyone seen anything similar before?
14:28:02	stephenfin	looking
14:28:08	openstackgerrit	Stephen Finucane proposed openstack/nova master: Remove fstype param from ploop init https://review.opendev.org/444970
14:29:02	stephenfin	gibi: Replied
14:30:04	openstackgerrit	Eric Fried proposed openstack/nova master: Allow versioned discovery unauthenticated https://review.opendev.org/685181
14:30:08	efried	alex_xu: ^
14:30:14	efried	thanks for the review.
14:30:28	mriedem	stephenfin: kind of hard to care about any ploop/vz patches when their ci hasn't worked in probably over a year
14:31:25	stephenfin	That one was fairly trivial so it shouldn't matter, hopefully, but yeah, agreed
14:31:58	gibi	stephenfin: thanks
14:32:16	mriedem	stephenfin: so...remove your +2 until their CI passes on it?
14:32:28	mriedem	it's just code cleanup, not really fixing a bug
14:32:31	mriedem	as far as i can tell
14:32:56	stephenfin	kewl
14:32:57	stephenfin	yeah
14:32:59	stephenfin	it is
14:34:46	openstackgerrit	Eric Fried proposed openstack/nova master: Allow versioned discovery unauthenticated https://review.opendev.org/685181
14:34:46	openstack	bug 1845530 in OpenStack Compute (nova) "Versioned discovery endpoint should not require authentication" [Undecided,In progress] https://launchpad.net/bugs/1845530 - Assigned to Eric Fried (efried)
14:34:46	openstackgerrit	Eric Fried proposed openstack/nova master: Repro bug 1845530: versioned discovery is authed https://review.opendev.org/685180
14:34:53	efried	resolved merge conflict ^
14:40:51	openstackgerrit	Silvan Kaiser proposed openstack/nova master: [WIP] Move Nova Quobyte driver to LibvirtMountedFileSystemVolumeDriver https://review.opendev.org/687066
14:47:48	openstackgerrit	Eric Fried proposed openstack/nova master: Allow versioned discovery unauthenticated https://review.opendev.org/685181
14:47:48	openstack	bug 1845530 in OpenStack Compute (nova) "Versioned discovery endpoint should not require authentication" [Undecided,In progress] https://launchpad.net/bugs/1845530 - Assigned to Eric Fried (efried)
14:47:48	openstackgerrit	Eric Fried proposed openstack/nova master: Repro bug 1845530: versioned discovery is authed https://review.opendev.org/685180
14:48:05	efried	mriedem: clarified those comments in the test case -^
14:49:57	mriedem	stephenfin: remember when you abandon a patch linked to a bug, e.g. https://review.opendev.org/#/c/585978/ - you should probably update the bug to say it's not in progress and remove the owner
14:53:55	mriedem	ack
14:54:29	mriedem	stephenfin: i know you're busy with cleaning up old patches, but now that you've got my attention on this giant docs rewrite https://review.opendev.org/#/c/667133/ it'd be nice to close that out
14:55:04	stephenfin	I can do both of those shortly, yup (y)
15:03:42	mriedem	thanks
15:04:18	mriedem_afk	gibi: dansmith: how are you feeling about this now? https://review.opendev.org/#/c/633293/

Earlier   Later