Posted	Nick	Remark
#openstack-nova - 2017-07-26
22:14:27	edleafe	dansmith: agreed
22:15:26	openstackgerrit	Eric Fried proposed openstack/nova master: Centralize keystoneauth1 Adapter conf option setup https://review.openstack.org/487581
22:19:23	openstackgerrit	Jay Pipes proposed openstack/nova master: placement: account for move operations in claim https://review.openstack.org/487589
22:19:23	openstackgerrit	Jay Pipes proposed openstack/nova master: claim resources in placement API during schedule() https://review.openstack.org/483566
22:19:25	jaypipes	mriedem, bauzas, dansmith: ding ding ding. round 2. ^^
22:19:51	mriedem	heh, can i be glass joe?
22:22:48	openstackgerrit	Nicolas Simonds proposed openstack/nova master: libvirt: add support for virtio-net rx/tx queue sizes https://review.openstack.org/484997
22:23:17	jaypipes	mriedem: I'll work on my eye tattoo.
22:24:02	mriedem	heh i guess we forgot about safe_connect on claim_resources
22:24:40	openstackgerrit	Merged openstack/nova master: Remove the code related to extension loading from APIRouterV21 https://review.openstack.org/486414
22:25:03	openstackgerrit	Eric Fried proposed openstack/nova master: nova.utils.get_service_url() https://review.openstack.org/458257
22:30:05	openstackgerrit	Matt Riedemann proposed openstack/nova master: Fix example in _serialize_allocations_for_consumer https://review.openstack.org/487614
22:30:20	mriedem	jaypipes: dansmith: so we don't need to handle anything different from the compute RT perspective for a move operation?
22:36:47	openstackgerrit	Merged openstack/nova master: Suppress some test warnings https://review.openstack.org/486700
22:37:32	openstackgerrit	Merged openstack/nova master: Remove the useless FakeExt https://review.openstack.org/486415
22:51:59	openstackgerrit	Nicolas Simonds proposed openstack/nova master: libvirt: add support for virtio-net rx/tx queue sizes https://review.openstack.org/484997
22:59:11	dansmith	mriedem: seems like not
23:14:53	mikal	melwitt: you still around?
23:15:34	mikal	melwitt: tonyb and I are talking about making a quick list of things we'd like to see cleaned up in nova, because I keep forgetting things because I am an old man. tonyb suggested you had an etherpad like this already somewhere?
23:17:18	melwitt	I think you probably mean mriedem :) ^
23:17:37	mikal	nah
23:17:40	mikal	Its all about you
23:17:48	melwitt	haha
23:18:29	mikal	Well, it depends if tonyb is lying about you already having an etherpad
23:18:50	melwitt	yeah, if I ever knew of one in the past, I've forgotten about it. I'm not aware of a general nova todos etherpad
23:19:04	mikal	Ok, I'll just call tonyb a liar in public then
23:19:05	melwitt	I wouldn't be surprised if we had one at some point though
23:19:07	mikal	I'm ok with that
23:19:11	melwitt	haha :)
23:19:30	mikal	https://etherpad.openstack.org/p/nova-warts is my very long list of things I'd like to potter around fixing now that I am a hobbyist again
23:19:58	melwitt	coolness
23:22:00	mikal	melwitt: speaking of which, it would be cool to get https://review.openstack.org/#/c/486831 merged before pike releases too from that series. Its only comments but it gives us a cleanish slate for the cleanup in queens.
23:22:08	openstackgerrit	Jay Pipes proposed openstack/nova master: claim resources in placement API during schedule() https://review.openstack.org/483566
23:22:08	openstackgerrit	Jay Pipes proposed openstack/nova master: placement: account for move operations in claim https://review.openstack.org/487589
23:22:11	mikal	melwitt: i.e. please make with the rubber stamping, kthxbye
23:25:11	melwitt	mikal: k, will look
23:28:20	jaypipes	edleafe: consolidated the dict construction to a single line for you.
23:29:30	mikal	melwitt: ta
23:35:01	openstackgerrit	Eric Fried proposed openstack/nova master: WIP: Use service catalog (and discovery) for cinder endpoints https://review.openstack.org/487621
23:35:32	efried	mordred ^^ *very* WIP, but could you look and see if this is a remotely sane approach to the cinder conversion?
23:35:44	efried	mordred thanks in advance.
23:38:05	smcginnis	Ooh, nice.
23:39:53	openstackgerrit	Merged openstack/nova master: Netronome SmartNIC Enablement https://review.openstack.org/483459
23:40:54	openstackgerrit	Merged openstack/nova master: Remove the useless extension block_device_mapping_v1 object https://review.openstack.org/486069
23:41:44	openstackgerrit	Merged openstack/nova master: Remove the useless fake ExtensionManager from API unittests https://review.openstack.org/486416
23:42:41	openstackgerrit	Merged openstack/nova master: Skip boot from encrypted volume on Xen+libvirt https://review.openstack.org/481627
23:43:24	openstackgerrit	Merged openstack/nova master: Add helper method for waiting migrations in functional tests https://review.openstack.org/486537
23:43:42	tonyb	melwitt, mikal: let me see if I can find the one I was thinking of.
23:44:08	openstackgerrit	Merged openstack/nova master: docstring and unused code removal https://review.openstack.org/487492
23:44:49	openstackgerrit	Merged openstack/nova master: Move the note about '/os-volume_boot' to the correct place https://review.openstack.org/486071
23:49:37	melwitt	tonyb: in rootwrap.d, do the comments do anything to scope the commands or are they just for humans to read?
23:51:08	tonyb	melwitt: just for humans
23:51:18	melwitt	tonyb: cool, thanks
23:52:02	tonyb	melwitt: when doing the privsep stuff in later patches it was annoying to identify things that could be pruned, so I thought I'd mkae it a little easier
23:52:26	melwitt	tonyb: yeah, I think I understand now
23:53:05	tonyb	melwitt: I'll address your review feedback after the last_bytes patch lands to make the rebase slightly easier if that's cool with you
23:54:04	melwitt	tonyb: okay. if the stuff I mentioned is going to be a later cleanup, that's cool. just wanted to make sure I understood
23:55:21	tonyb	well one of you comments is taken care of in a later patch the other was on my mental todo list I haven't done more that that.
23:55:34	tonyb	I'll reply and then you can make a call.
23:57:48	melwitt	tonyb: cool thanks
#openstack-nova - 2017-07-27
00:11:30	openstackgerrit	Moshe Levi proposed openstack/nova master: hardware offload support for openvswitch https://review.openstack.org/398265
00:13:01	tonyb	mikal, melwitt: the etherpad I was thinking of was https://etherpad.openstack.org/p/nova-low-hanging-fruit which has been migrated to the wiki
00:13:27	tonyb	So I guess start a new one?
00:13:51	tonyb	I know low-hanging-fruit isn't exactly what we were thinking of but I'm easily confused
00:15:12	mikal	tonyb: I have a tweak to the localfs patch I'd like to upload before you do any rebasing please
00:15:53	mikal	tonyb: I might upload that now unless you shout no in the next ten seconds
00:16:00	tonyb	mikal: I was kinda using the 'I' in the sense mikal or tony sense
00:16:00	tonyb	mikal: Sure
00:16:05	mikal	LOL
00:16:12	mikal	We are one entity now?
00:16:18	mikal	That must be very embarassing for you
00:16:25	tonyb	mikal: as long as you don't touch the last_bytes review go nuts ;P
00:16:41	moshele	mriedem: hi
00:16:52	tonyb	mikal: only here where privsep is concerend
00:17:09	mikal	Oh sigh. I need to rebase. Please hold.
00:17:13	tonyb	mikal: It's more of that 'tonyb took my keyboard and wont give it back pair programming'
00:17:26	mikal	tonyb: true that. tonyb doesn't know how to share.
00:17:40	tonyb	mikal: that's what my teachers said a school
00:17:52	mikal	Heh
00:18:09	tonyb	mikal: you might want to hold off until last_bytes merged unless that also needs a fix
00:18:19	mikal	That didn't
00:18:24	mikal	But it seems to hate merging
00:19:03	tonyb	the py35 sdvm job seems to fail lots but perhaps it just hates us
00:19:06	mikal	Hmmm, 472228 is in the check queue not the gate queue?
00:19:20	tonyb	mikal: we had to recheck
00:19:21	mikal	Looks like its going to pass there though
00:19:29	tonyb	which does both queues
00:19:33	mikal	Does it go to the gate queue automagically?
00:19:39	mikal	Oh, so it runs all the tests twice?
00:19:46	mikal	Thus maximizing the changes of a flakey test?
00:19:55	tonyb	if it still has +W it will just move to gate when it gets a +1 from jenkins
00:20:35	tonyb	mikal: Yeah but if anything has merged since the check was run then the results could now be wrong so it has to do bith
00:20:40	tonyb	*both*
00:21:10	mikal	So why not just re-run only the gate job?
00:22:48	tonyb	'cause gate and check have different configs and you could effectivle bypass a job that will fail but not catch it
00:24:34	mikal	I hate everything
00:47:59	openstackgerrit	yuanyue proposed openstack/nova master: Add a periodic task to destroy ReqSpecs of deleted instances https://review.openstack.org/484694
01:08:15	mikal	tonyb: check passed
01:08:29	rm_work	anyone know what the var for the nova-cpu config file is in devstack
01:08:30	rm_work	like

Earlier   Later