Posted	Nick	Remark
#openstack-nova - 2017-09-07
09:13:07	Dinesh_Bhor	stephenfin: Okay, np,
09:20:44	openstackgerrit	Rajesh Tailor proposed openstack/nova master: Host addition host-aggregate should be case-sensitive https://review.openstack.org/498334
09:22:56	stephenfin	mikal: Do you think this change could be restored now? https://review.openstack.org/#/c/397835/
09:23:46	openstackgerrit	Jianghua Wang proposed openstack/nova master: XenAPI: deprecate image handler class path option https://review.openstack.org/497201
09:23:47	openstackgerrit	Jianghua Wang proposed openstack/nova master: XenAPI: define a new image handler to use vdi streaming https://review.openstack.org/486475
09:53:40	openstackgerrit	Rodolfo Alonso Hernandez proposed openstack/os-vif master: Add abstract OVSDB API https://review.openstack.org/476612
09:58:46	openstackgerrit	Naichuan Sun proposed openstack/nova master: xenapi: Exception Error logs shown in Citrix XenServer CI https://review.openstack.org/478437
10:00:09	openstackgerrit	Rodolfo Alonso Hernandez proposed openstack/os-vif master: Add memoize function using oslo.cache https://review.openstack.org/472773
10:02:06	openstackgerrit	Rodolfo Alonso Hernandez proposed openstack/os-vif master: Add native implementation OVSDB API https://review.openstack.org/482226
10:28:04	gibi	What is using compute.api.HostAPI? I see that xen and vmware implements the backend but I don't see any usage of this compute.api
10:53:50	openstackgerrit	Andreas Jaeger proposed openstack/nova master: Fix broken URLs https://review.openstack.org/501402
12:39:10	awaugama	t
12:41:54	openstackgerrit	Takashi NATSUME proposed openstack/nova-specs master: Enable cold migration with target host https://review.openstack.org/489031
12:59:31	openstackgerrit	Stephen Finucane proposed openstack/nova master: doc: Add documentation for emulator_thread_policy https://review.openstack.org/501721
12:59:41	stephenfin	sahid, bauzas, jaypipes: Could ye look at that ^ simple doc fix
13:00:39	bauzas	ack
13:03:00	bauzas	stephenfin: https://en.wikipedia.org/wiki/RRRrrrr!!!
13:03:12	bauzas	stephenfin: https://review.openstack.org/#/c/501721/1
13:03:22	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Transform missing delete notifications https://review.openstack.org/410297
13:03:22	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Send soft_delete from context manager https://review.openstack.org/476459
13:03:26	stephenfin	Ha!
13:04:21	openstackgerrit	Stephen Finucane proposed openstack/nova master: doc: Add documentation for emulator_thread_policy https://review.openstack.org/501721
13:04:22	stephenfin	bauzas: Done :D
13:06:07	bauzas	stephenfin: it's a cult film y'know
13:07:16	stephenfin	bauzas: I figured as much. Had a Monty Python feel about
13:10:08	mriedem	this is blocked an upper-constraints change, can we get this in https://review.openstack.org/#/c/501400/ ?
13:10:11	mriedem	needs another +2
13:12:06	stephenfin	mriedem: Yup, looks grand to me
13:12:17	efried	'cept for the typos
13:13:46	stephenfin	efried: If you're brave enough to -1 it for that, I'll give you a cookie :P
13:13:59	stephenfin	efried: mriedem might give you something else though
13:14:09	efried	I only -1 for typos in docs
13:14:20	efried	("typo's in doc's"?)
13:14:35	stephenfin	Yup, likewise. I kid :)
13:18:58	mikal	stephenfin: yeah, I think we could take another pass at that now
13:18:59	sahid	stephenfin: https://review.openstack.org/#/c/501721/2 the explnation does not make a lot of sense
13:20:24	stephenfin	sahid: Yup, good call. I'll rework shortly
13:20:32	bauzas	sahid: I tend to fast-approve doc changes but it helps, but feel free to open a follow-up and I'll review it
13:20:49	bauzas	s/but/because
13:20:50	stephenfin	bauzas, sahid: It's actually blocked by a base patch so it won't merge :/
13:21:04	bauzas	stephenfin: ack, then push another rev and I'll +2
13:21:12	stephenfin	+2 to fast-approving docs too. Fast iteration is really helpful
13:21:28	bauzas	I mean, doc bugs don't hurt
13:21:45	bauzas	while doc amendments really help
13:21:48	stephenfin	Well, not as much as no docs does
13:21:57	bauzas	yup that
13:22:21	bauzas	unless of course if you state in the doc something like "rm -rf /"
13:22:59	sdague	mikal: ok, looking
13:23:31	stephenfin	mikal: Sweet. I'll revive that shortly so
13:24:40	bauzas	mriedem: new school year, new constraints. I'll need to bail out from the nova meeting around 1420UTC (ie. 20 mins after you start=
13:24:56	bauzas	but I should be back 20 mins after
13:25:26	bauzas	hopefully the daylight shift will solve the problem in a couple of weeks :)
13:25:42	mriedem	ok
13:33:25	openstackgerrit	Andreas Jaeger proposed openstack/nova master: Fix broken URLs https://review.openstack.org/501402
13:33:54	sdague	mikal: https://review.openstack.org/#/c/489486 ... that needs some more explaining, the the cryptsetup add isn't clear to me
13:37:54	dansmith	mriedem: you going to fix the typo in that second one or just steam on?
13:38:04	mriedem	doing it now
13:38:14	openstackgerrit	Matt Riedemann proposed openstack/nova master: Make ConductorTaskTestCase run with 2 cells https://review.openstack.org/501514
13:38:14	openstackgerrit	Matt Riedemann proposed openstack/nova master: Track which cell each instance is created in and use it consistently https://review.openstack.org/501452
13:38:15	openstackgerrit	Matt Riedemann proposed openstack/nova master: Ensure instance mapping is updated in case of quota recheck fails https://review.openstack.org/501408
13:39:31	mriedem	alex raised a good question here https://review.openstack.org/#/c/501408/8/nova/conductor/manager.py@1049
13:40:06	mriedem	in that if the build request is already deleted after we create the instance in the cell but before we actually send the instance to the host to build, we cleanup the instance (local delete) but don't cleanup dest node allocations
13:40:34	mriedem	separate bug, but a bug nonetheless
13:41:01	mikal	sdague: so I need a reno when I remove a command from rootwrap's filters? Or only when I add one?
13:41:04	openstackgerrit	Merged openstack/nova master: Fix test_rpc_consumer_isolation for oslo.messaging 5.31.0 https://review.openstack.org/501400
13:42:20	sdague	the adds are the critical bit. That's a config file which won't work if you don't add those lines
13:42:29	sdague	if people left old things around, meh
13:43:07	openstackgerrit	Andreas Jaeger proposed openstack/nova master: Fix broken URLs https://review.openstack.org/501402
13:43:38	mriedem	who wants to look at https://review.openstack.org/#/c/501514/ and up? regression in pike that needs to be backported
13:43:42	mriedem	vexxhost hit it yesterday
13:44:08	jaypipes	mriedem: just FYI, we're evacuating this evening so I'll be OOO tomorrow and part of this afternoon.
13:44:23	mriedem	jaypipes: ok, good luck
13:44:25	jaypipes	mriedem: and I do mean evacuate, not rebuild.
13:44:29	sdague	mikal: the general rule of thumb, assume someone is upgrading with pip install -U on the commit. Do they need to do something besides that to have usable environment
13:44:39	mriedem	jaypipes: i hear ohio still has room for you
13:44:44	mriedem	if you'll take it back
13:44:47	jaypipes	mriedem: that's where we're headed.
13:45:04	mriedem	nice
13:45:15	mriedem	just in time for your beloved browns season to begin
13:45:22	sdague	jaypipes: well, that means jeni's icecream when you arrive
13:45:24	dansmith	jaypipes: good luck
13:45:30	jaypipes	sdague: heh :)
13:45:43	jaypipes	dansmith: thx. still smoldering where you are?
13:45:56	dansmith	jaypipes: yeah
13:45:58	sdague	jaypipes: we stopped in columbus north market on our drive to TN for the eclipse. Could not leave without some jeni's
13:46:14	sdague	jaypipes: safe travels
13:46:21	jaypipes	sdague: indeed. Jeni's is awesome. Graeters is also very good :)
13:46:26	jaypipes	sdague: thx
13:46:36	jaypipes	dansmith: how close to you are the fires?
13:46:54	sdague	jaypipes: we going to see you in CO still?
13:47:08	dansmith	jaypipes: the fires are at portland's watershed, if that tells you, but that's like 15 miles by the crow I think
13:49:03	kashyap	dansmith: "by the crow" == shortest path I presume?
13:49:25	dansmith	kashyap: yes, as in "as the crow flies"
13:49:30	dansmith	apparently crows fly in straight lines
13:49:32	openstackgerrit	Matt Riedemann proposed openstack/nova stable/pike: Provide hints when nova-manage db sync fails to sync cell0 https://review.openstack.org/501745
13:49:36	kashyap	Yeah, I recall hearing that, just wanted to double-check :-)
13:49:47	openstackgerrit	Matt Riedemann proposed openstack/nova stable/ocata: Provide hints when nova-manage db sync fails to sync cell0 https://review.openstack.org/501746
13:49:48	kashyap	Oh dear, the wildfire looks insane
13:50:44	gibi	mriedem: hi! Is it OK for you If I rebase your patch https://review.openstack.org/#/c/498596 as mine top of it need to be adapted to a test case from master ?
13:56:33	mriedem	gibi: sure

Earlier   Later