Posted	Nick	Remark
#openstack-nova - 2017-05-03
06:32:51	openstackgerrit	Andrey Volkov proposed openstack/nova master: [placement] Add api-ref for DELETE RP inventory https://review.openstack.org/455215
06:32:52	openstackgerrit	Andrey Volkov proposed openstack/nova master: [placement] Add api-ref for PUT RP inventory https://review.openstack.org/455214
06:32:52	openstackgerrit	Andrey Volkov proposed openstack/nova master: [placement] Add api-ref for GET RP inventory https://review.openstack.org/455213
06:32:53	openstackgerrit	Andrey Volkov proposed openstack/nova master: [placement] Add api-ref for GET RP inventories https://review.openstack.org/449765
06:32:53	openstackgerrit	Andrey Volkov proposed openstack/nova master: [placement] Add api-ref for DELETE RP inventories https://review.openstack.org/452708
06:32:54	openstackgerrit	Andrey Volkov proposed openstack/nova master: [WIP] [placement] Add api-ref resource provider https://review.openstack.org/456176
06:32:54	openstackgerrit	Andrey Volkov proposed openstack/nova master: [placement] Add api-ref for PUT RP inventories https://review.openstack.org/452707
06:50:35	openstackgerrit	Zhenyu Zheng proposed openstack/nova master: Send out notifications when instance tags changed https://review.openstack.org/459923
07:04:14	openstackgerrit	Zhenyu Zheng proposed openstack/nova master: Support tag instances when boot(3/4) https://review.openstack.org/459593
07:12:10	bauzas	p!spy
07:12:14	bauzas	good morning Nova
07:23:32	openstackgerrit	Huan Xie proposed openstack/nova master: WIP: Fix live migration on XS7.1 https://review.openstack.org/462017
07:25:05	openstackgerrit	Alex Xu proposed openstack/nova master: Add policy description for os-quota-classes https://review.openstack.org/461996
07:25:06	openstackgerrit	Alex Xu proposed openstack/nova master: Add policy description for extensions https://review.openstack.org/461995
08:39:55	openstackgerrit	Chris Dent proposed openstack/nova master: Optional separate database for placement API https://review.openstack.org/362766
09:04:44	openstackgerrit	jichenjc proposed openstack/nova master: Use plain routes list for os-instance-usage-audit-log endpoint instead of stevedore https://review.openstack.org/461647
09:19:35	openstackgerrit	Zhenyu Zheng proposed openstack/nova master: Use plain routes list for server-password endpoint instead of stevedore https://review.openstack.org/462049
09:27:29	openstackgerrit	Alex Xu proposed openstack/nova master: Add policy description for os-quota-classes https://review.openstack.org/461996
09:27:30	openstackgerrit	Alex Xu proposed openstack/nova master: Add policy description for extensions https://review.openstack.org/461995
09:37:02	johnthetubaguy	gmann: alex_xu: just looking at this ips API, seems like we should have deprecated that? https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/ips.py
09:41:55	openstackgerrit	Alex Szarka proposed openstack/nova master: Transform instance-live_migration_abort notification https://review.openstack.org/420453
09:42:14	asettle	Good morning \o/ it's been a whole week since I've been here
09:42:31	asettle	I've got a quick nova doc patch ifs omeone could take a wee lookie
09:42:34	asettle	https://review.openstack.org/#/c/462009/2
09:47:19	johnthetubaguy	asettle: seems like a good add to me
09:47:44	asettle	Thanks johnthetubaguy :) I just did my little edit, and +2
09:48:41	openstackgerrit	jichenjc proposed openstack/nova master: [placement] Add doc for resource provider usages https://review.openstack.org/450105
09:52:30	alex_xu	johnthetubaguy: I remember we talk about in the before
09:53:34	johnthetubaguy	alex_xu: ah, I forgot about that one completely
09:54:54	alex_xu	johnthetubaguy: I can't remeber the detail also, try to search the log
09:59:13	openstackgerrit	Gábor Antal proposed openstack/nova master: [WIP] Transform aggregate.update_metadata notification https://review.openstack.org/460625
10:01:32	alex_xu	johnthetubaguy: there is a roadmap for the interface API in the bottom of https://etherpad.openstack.org/p/nova-os-interfaces, it didn't have record about ips :( but I guess that isn't something we want
10:01:49	johnthetubaguy	alex_xu: ah, oops
10:02:29	alex_xu	johnthetubaguy: here is the talk, it is very long http://eavesdrop.openstack.org/irclogs/%23openstack-nova/%23openstack-nova.2017-04-11.log.html#t2017-04-11T14:02:10 if you are interesting
10:10:58	openstackgerrit	Abhishek Sharma M proposed openstack/nova master: Catching OverQuota Exception https://review.openstack.org/460305
10:47:16	openstackgerrit	Stephen Finucane proposed openstack/nova master: trivial: Remove dead code https://review.openstack.org/460662
10:50:41	openstackgerrit	Stephen Finucane proposed openstack/os-vif master: Require OVS 2.8 and kernel 4.8 for offload https://review.openstack.org/460093
10:55:22	openstackgerrit	Stephen Finucane proposed openstack/nova master: Ensure sample policy help text correctly wrapped https://review.openstack.org/460097
11:25:31	asettle	johnthetubaguy: could I Get your email with something? How much do you know about Compute schedulers?
11:26:51	sdague	melwitt: yeh, there was a reason this broke down on me before, and I really don't remember what it is
11:28:32	sfinucan	sdague: Could you take a punt at https://review.openstack.org/#/c/431037/ and successors today?
11:40:12	sdague	sfinucan: question on https://review.openstack.org/#/c/431037/10 (though +2 for now anyway)
12:01:41	mordred	efried: the keystoneauth change has landed - so I think next step is getting you a ksa patch cut, yeah?
12:19:09	openstackgerrit	OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/461446
12:23:07	openstackgerrit	OpenStack Proposal Bot proposed openstack/python-novaclient master: Updated from global requirements https://review.openstack.org/462124
12:46:43	alex_xu	nova api meeting is in 14 mins at #openstack-meeting-4
12:53:10	johnthetubaguy	asettle: I know bits, but there might be better folks to ask, whats up?
12:54:52	asettle	johnthetubaguy: I'm staring at this bug: https://bugs.launchpad.net/openstack-manuals/+bug/1684261
12:54:52	openstack	Launchpad bug 1684261 in openstack-manuals "AggregateImagePropertiesIsolation example doesn't actually indicate how it works" [Low,Confirmed] - Assigned to Alexandra Settle (alexandra-settle)
12:54:54	efried	mordred - lbragstad did that yesterday; 2.20.0 is out.
12:54:55	asettle	And I admit, I'm also very confused.
12:55:44	efried	It's working its way through requirements now. The bot proposed u-c bump, which merged; and I proposed g-r bump, which merged. Waiting for the bot to propose nova requirements.txt bump, but I think the nova change set will now verify, cause it uses u-c, right?
12:55:55	lbragstad	efried mordred https://pypi.python.org/pypi/keystoneauth1/2.20.0
12:58:37	johnthetubaguy	asettle: good question, sounds like we need a better worked example. We have more docs in here: https://docs.openstack.org/developer/nova/filter_scheduler.html
13:00:29	efried	mriedem Do you want me to include links to merged and/or proposed change sets in the use-service-catalog-for-endpoints edit? https://review.openstack.org/#/c/461481/
13:00:43	alex_xu	nova api meeting is running at #openstack-meeting-4
13:01:26	mriedem	efried: nope
13:01:53	asettle	johnthetubaguy: yeah I was reading that this morning. I couldn't, admittedly, find too many good examples to transfer across. The only thing I could find for AggregateImagePropertiesIsolation is the literal explanation about halfway down the page.
13:04:44	mriedem	dansmith: can you hit this simple grenade change for using wsgi on master https://review.openstack.org/#/c/460543/ ?
13:05:01	mriedem	i.e. the devstack change to use uwsgi on master depends on that https://review.openstack.org/#/c/457715/
13:06:33	mriedem	EmilienM: we've said in the past that when we make changes to grenade that we should give the deployment teams a heads up, do you think this change warrants that https://review.openstack.org/#/c/460543/ ?
13:06:46	mriedem	that's needed for using uwsgi for nova in devstack on master https://review.openstack.org/#/c/457715/
13:08:40	openstackgerrit	Eric Fried proposed openstack/nova master: nova.utils.get_service_url() https://review.openstack.org/458257
13:13:31	bauzas	sched.org, you <3
13:24:49	dansmith	mriedem: yep got it
13:26:15	sfinucan	sdague: One more review here for you, if you would be so kind https://review.openstack.org/460097
13:37:25	openstackgerrit	John Garbutt proposed openstack/nova master: Add policy description for fping https://review.openstack.org/461763
13:43:12	dansmith	sdague: so your native call logging hack suggestion...
13:43:38	dansmith	sdague: I dunno if you saw, but we're actually missing logs, and the two requests don't line up. we're not just seeing a big delay
13:43:43	dansmith	do you still think it's worth trying that?
13:46:11	openstackgerrit	melanie witt proposed openstack/nova master: WIP Use six.text_type() when logging Instance object https://review.openstack.org/461912
13:53:44	openstackgerrit	Dan Smith proposed openstack/nova master: WIP: Debug race in delete-while-booting test https://review.openstack.org/461513
13:54:36	sdague	dansmith: sure
13:54:47	dansmith	okay
13:54:50	sdague	dansmith: you want me to push it out there?
13:55:10	sdague	honestly, the only thing it loses us is the search by request-id, which would be nice
13:55:12	dansmith	sdague: I'll just slam it into my patch
13:55:16	sdague	but not hugely a thing
13:55:25	dansmith	I'm going to put something else to log in the delete path
13:55:32	sdague	any idea if there was an exception in this path?
13:55:33	dansmith	wait, what?
13:55:59	sdague	wait what what?
13:57:12	dansmith	commenting the use_journal thing means we don't log request_id?
13:57:14	dansmith	or you mean it doesn't do the rich field thing?
13:57:35	openstackgerrit	Dan Smith proposed openstack/nova master: WIP: Debug race in delete-while-booting test https://review.openstack.org/461513
13:58:00	openstackgerrit	melanie witt proposed openstack/nova master: WIP Use six.text_type() when logging Instance object https://review.openstack.org/461912
13:59:20	sdague	dansmith: it doesn't do the rich field thing
13:59:35	dansmith	okay gotcha
13:59:52	sdague	so with that on, the log path is through journal.send, which gives us all the extra metadata
14:00:00	sdague	with that off, it's just going to spew to stdout
14:00:12	dansmith	cool, just wanted to make sure it wasn't going to change more stuff
14:00:25	sdague	systemd will send it all to the journal but without any of the fun bits
14:00:47	sdague	dansmith: https://review.openstack.org/462163
14:01:05	sdague	I also tweaked the debug line, because processid is going to be in the systemd logs anyway
14:01:30	mriedem	melwitt: heh, display_name='tempest-Íñstáñcé-2027474072' is better than UnicodeDecodeError at least
14:01:36	dansmith	sdague: oh i just commented it out of my patch
14:02:03	sdague	mriedem: mriedem how did we get to that?
14:02:16	mriedem	sdague: https://review.openstack.org/#/c/461912/
14:02:40	melwitt	mriedem: true. it's correct when I download the .txt.gz file though and open it with the less command. not sure what's up with that considering it let me copy-paste the name into the review comment from the log file I had downloaded
14:02:40	sdague	dansmith: sure, honestly, for right now I think it's fine to turn it off if we have concerns, so I'll put the general patch out there as well

Earlier   Later