Posted	Nick	Remark
#openstack-nova - 2017-11-14
21:00:37	openstackgerrit	Eric Fried proposed openstack/nova master: handle traits with sharing providers https://review.openstack.org/517119
21:00:37	openstackgerrit	Eric Fried proposed openstack/nova master: Add ProviderSummary.resource_class_names @property https://review.openstack.org/517646
21:00:38	openstackgerrit	Eric Fried proposed openstack/nova master: Test alloc_cands with one RP shared between two RPs https://review.openstack.org/519617
21:00:38	openstackgerrit	Eric Fried proposed openstack/nova master: Test alloc_cands with non overlapping sharing RPs https://review.openstack.org/519380
21:00:39	openstackgerrit	Eric Fried proposed openstack/nova master: Test helper: validate provider summaries https://review.openstack.org/518982
21:00:39	openstackgerrit	Eric Fried proposed openstack/nova master: Test allocation candidates: multiple aggregates https://review.openstack.org/518633
21:00:59	efried	jaypipes ^ Addressed most of the comments in this series.
21:01:12	efried	jaypipes There are still bugs; but I think alex_xu might be willing to +2 the bottom one at this point.
21:03:56	tonyb	mriedem: cool, I'll keep an eye out
21:04:47	openstackgerrit	OpenStack Proposal Bot proposed openstack/os-vif stable/pike: Updated from global requirements https://review.openstack.org/493146
21:15:58	efried	dansmith stephenfin When you get a chance, would you please put eyeballs back on https://review.openstack.org/#/c/377138/ -- at this point it's just in a sisyphean rebase cycle.
21:20:08	moshele	stephenfin: hi
21:22:50	oomichi	mriedem: scheduler-hints is one of gross API designs. Do we have more related patches except https://review.openstack.org/#/c/519717 ?
21:24:25	moshele	ramishra: hi
21:25:50	mriedem	oomichi: nope
21:26:05	mriedem	dansmith: here is my devstack newton-eol trickery https://review.openstack.org/519758
21:26:18	mriedem	thanks to google for providing bash substring tips
21:26:39	dansmith	mriedem: wow that's... special
21:26:44	mriedem	heh
21:27:23	mriedem	eol_tag=$(echo ${git_ref:7:${#git_ref}})-eol
21:27:27	mriedem	is the best piece of coding i've done in ages
21:27:30	dansmith	mriedem: heh
21:27:56	efried	mriedem What's the 7-char prefix there?
21:28:03	mriedem	stable/
21:28:04	dansmith	stable/
21:28:05	mriedem	strip that off
21:28:35	efried	Then eol_tag=${git_ref#stable/}-eol would work too
21:28:59	efried	You know, if you're a shell guy rather than a python guy.
21:29:21	mriedem	so it doies
21:29:27	mriedem	i am not a shell guy
21:29:36	efried	Furthermore, that syntax will work all the way back to ksh88
21:29:43	efried	Which is important.
21:29:50	efried	In case you're running this script on AIX
21:29:56	mriedem	fuck i wish
21:32:06	mriedem	updated, thanks for the tip
21:32:11	cburgess	mriedem OK sorry... things got.. crazy here. I'm reading what your wrote hours ago now.
21:33:43	efried	mriedem Sure thing. But also x=foo is the same as x=$(echo foo)
21:34:22	cburgess	mriedem OK so... back to the live migration cancel thing, I think its probably reasonable to be able to cancel queued or running.
21:45:58	mriedem	cburgess: ack
21:45:59	mriedem	good to know
21:46:10	cburgess	I agree there is a bit of race condition though.
21:46:17	mriedem	apparently our operators are annoyed that they queue up a bunch of live migrations and can't abort some of them before they hit the hypervisor
21:46:33	mriedem	tonyb: ok redoing this newton series with the depends-on
21:46:35	cburgess	As in you want the API to fast fail if the state is wrong, but that state can change by the time you hit the compute node.
21:46:43	mriedem	and had to slip another funtional test backport in the mix
21:47:04	mriedem	cburgess: yeah i think we'd just have to ignore it if we got to the compute and something changed,
21:47:08	cburgess	mriedem Yeah that makes sense. So I think queued and running are the 2 states that aren't really transitory. So just block action on the transitory states.
21:47:15	mriedem	although if it went from queued->running, we should be able to abort in the compute
21:47:27	mriedem	as normal
21:47:38	cburgess	mriedem Right thats my thought. Fast fail on the API, and then another check on compute and fail if you aren't in an allowed state on compute.
21:47:52	openstackgerrit	Matt Riedemann proposed openstack/nova stable/newton: Pass requested_destination in filter_properties https://review.openstack.org/516119
21:47:53	openstackgerrit	Matt Riedemann proposed openstack/nova stable/newton: Add security release note for OSSA-2017-005 https://review.openstack.org/519755
21:47:53	openstackgerrit	Matt Riedemann proposed openstack/nova stable/newton: Validate new image via scheduler during rebuild https://review.openstack.org/519684
21:47:54	openstackgerrit	Matt Riedemann proposed openstack/nova stable/newton: Fix non-parameterized service id in hypervisors sample tests https://review.openstack.org/519763
22:09:59	openstackgerrit	Merged openstack/nova master: placement: AllocCands.get_by_{filters => requests} https://review.openstack.org/514092
22:16:52	openstackgerrit	Merged openstack/nova master: Deduplicate server_group samples https://review.openstack.org/519629
22:20:27	openstackgerrit	Merged openstack/nova stable/pike: Add security release note for OSSA-2017-005 https://review.openstack.org/519752
22:20:33	openstackgerrit	Merged openstack/nova stable/ocata: Add security release note for OSSA-2017-005 https://review.openstack.org/519753
22:21:01	mriedem	i think the devstack hack is working
22:21:07	mriedem	tempest is running on that patch now
22:21:10	mriedem	so the install was good
22:24:35	openstackgerrit	Eric Fried proposed openstack/nova master: placement: adds REST API for nested providers https://review.openstack.org/384807
22:24:36	openstackgerrit	Eric Fried proposed openstack/nova master: Reformat _get_all_with_shared https://review.openstack.org/512799
22:24:36	openstackgerrit	Eric Fried proposed openstack/nova master: placement: update client to set parent provider https://review.openstack.org/385693
22:24:38	tonyb	mriedem: \o/ I +2'd the devstack change
22:25:22	efried	jaypipes Addressed the couple of comments remaining in here --^
22:25:39	mriedem	tonyb: you don't think that devstack change needs to go on master first and backported?
22:25:58	mriedem	ah i see your comment
22:26:08	tonyb	mriedem: Yeah that thing
22:27:59	mriedem	oh fo sho
22:28:33	mriedem	begin preparations for ocata and pike releases
22:37:25	tonyb	mriedem: You're just missing a "make it so" or "engage!" in there somewhere
22:37:39	tonyb	mriedem: perhaps you could take that as a challenge?
22:37:50	mriedem	i thought about it
22:37:56	mriedem	sdague said engage earlier today
22:38:14	mriedem	oh no he said the other
22:38:15	mriedem	make it so
22:38:41	mriedem	ok pike release https://review.openstack.org/#/c/519799/ and ocata release https://review.openstack.org/#/c/519808/
22:40:09	openstackgerrit	Matt Riedemann proposed openstack/nova stable/newton: Correct log message when removing a security group https://review.openstack.org/519586
22:41:33	mriedem	dansmith: tonyb: want to bless this https://review.openstack.org/#/c/519763/ ?
22:43:59	tonyb	mriedem: +2'd
22:44:26	dansmith	tonyb: there's one above it too
22:44:42	dansmith	(the important one)
22:47:55	tonyb	dansmith: Yeah I'm looking that the zuul consoles for that series and the pike/ocata releases
22:48:16	dansmith	okay
22:49:11	openstackgerrit	OpenStack Proposal Bot proposed openstack/python-novaclient master: Updated from global requirements https://review.openstack.org/519806
22:51:57	openstackgerrit	Merged openstack/nova master: Add error message on metadata API https://review.openstack.org/517152
22:52:56	mriedem	thanks - at this hour, i'll just fast re-approve anything that was already approved earlier todya
22:52:58	mriedem	once ci is happy
22:54:10	efried	jaypipes Taking on the latest merge conflict...
22:57:20	mriedem	tonyb: i think the devstack patch isn't actually doing the job
22:57:21	mriedem	http://logs.openstack.org/58/519758/3/check/legacy-tempest-dsvm-neutron-full/1676068/logs/devstacklog.txt.gz#_2017-11-14_21_45_19_885
22:57:56	mriedem	i'm not sure why the job passed though
22:58:34	tonyb	ergh moar debugging
22:58:44	mriedem	well, i assume my regex compare in bash is wrong
22:58:48	mriedem	i can update to what efried suggested
22:59:00	mriedem	although
22:59:01	mriedem	http://logs.openstack.org/58/519758/3/check/legacy-tempest-dsvm-neutron-full/1676068/logs/pip2-freeze.txt.gz
22:59:04	mriedem	neutron==9.4.2.dev21
22:59:26	mriedem	that is newton for neutron

Earlier   Later