Posted	Nick	Remark
#openstack-nova - 2018-02-20
19:08:52	openstackgerrit	Mohammed Naser proposed openstack/nova stable/pike: Detach volumes when deleting a BFV server pre-scheduling https://review.openstack.org/546275
19:09:09	mnaser	(i dun goof'd the first set because i cherry pick'd the wrong functional test, oops.)
19:09:11	sean-k-mooney	cool just checking as i was reading the 400 error bit i assumed the db statement covered them
19:12:16	sean-k-mooney	efried: just one observation. required for /resource_providers will work differently then required for /allocation_candidates as the later with out granular requests looks at the union of all traits in the tree not just on an indivigual resouce provider
19:12:27	cfriesen	mriedem: the test_rebuild_with_new_image() testcase is still failing. I've patched nova/compute/manager.py and restarted devstack@n-cpu.service, am I missing something?
19:12:48	sean-k-mooney	efried: they are two different apis however so that should be fine
19:13:04	efried	sean-k-mooney: In fact in more than just the tree - in the tree plus any sharing providers it's considering for the request.
19:13:29	sean-k-mooney	efried: yep
19:13:33	efried	sean-k-mooney: And yes, a valid observation. Same applies to ?resources=
19:14:05	openstackgerrit	Mohammed Naser proposed openstack/nova stable/queens: Drop extra loop which modifies Cinder volume status https://review.openstack.org/546200
19:14:05	openstackgerrit	Mohammed Naser proposed openstack/nova stable/queens: Store block device mappings in cell0 https://review.openstack.org/546201
19:14:06	openstackgerrit	Mohammed Naser proposed openstack/nova stable/queens: Add functional tests to ensure BDM removal on delete https://review.openstack.org/546202
19:14:06	openstackgerrit	Mohammed Naser proposed openstack/nova stable/queens: Clean up ports and volumes when deleting ERROR instance https://review.openstack.org/546203
19:14:07	openstackgerrit	Mohammed Naser proposed openstack/nova stable/queens: Add functional recreate test of deleting a BFV server pre-scheduling https://review.openstack.org/546204
19:14:07	openstackgerrit	Mohammed Naser proposed openstack/nova stable/queens: Detach volumes when deleting a BFV server pre-scheduling https://review.openstack.org/546277
19:14:14	mnaser	someone teach me git at the ptg, thanks
19:14:55	mnaser	but at least gerrit can tell its a rebase so it didnt lose the +verified
19:18:49	mriedem	mnaser: git review -R -y
19:18:59	mriedem	cfriesen: not sure what you mean, the functional test passes for me
19:19:07	mriedem	remove the code fix and the test fails as expected
19:21:49	edleafe	mnaser: I always liked this talk: https://www.youtube.com/watch?v=yCh6TSLIQBQ
19:23:36	mnaser	edleafe: cool, i'll give it a listen tonight
19:25:04	cfriesen	mriedem: my bad, I had "if recreate is not None" in the test
19:26:14	cfriesen	all is good now
19:26:16	openstackgerrit	Dan Smith proposed openstack/nova master: Add AggregateList.get_by_metadata() query method https://review.openstack.org/544728
19:26:16	openstackgerrit	Dan Smith proposed openstack/nova master: Add aggregates list to Destination object https://review.openstack.org/544729
19:26:17	openstackgerrit	Dan Smith proposed openstack/nova master: Add request filter functionality to scheduler https://review.openstack.org/544730
19:26:17	openstackgerrit	Dan Smith proposed openstack/nova master: Add require_tenant_aggregate request filter https://review.openstack.org/545002
19:26:18	openstackgerrit	Dan Smith proposed openstack/nova master: WIP: Honor availability_zone hint via placement https://review.openstack.org/546282
19:39:34	openstackgerrit	Mohammed Naser proposed openstack/nova stable/pike: Clean up ports and volumes when deleting ERROR instance https://review.openstack.org/546221
19:39:35	openstackgerrit	Mohammed Naser proposed openstack/nova stable/pike: Add functional recreate test of deleting a BFV server pre-scheduling https://review.openstack.org/546222
19:39:35	openstackgerrit	Mohammed Naser proposed openstack/nova stable/pike: Detach volumes when deleting a BFV server pre-scheduling https://review.openstack.org/546275
19:48:52	openstackgerrit	Dan Smith proposed openstack/nova master: Add AggregateList.get_by_metadata() query method https://review.openstack.org/544728
19:48:53	openstackgerrit	Dan Smith proposed openstack/nova master: Add aggregates list to Destination object https://review.openstack.org/544729
19:48:53	openstackgerrit	Dan Smith proposed openstack/nova master: Add request filter functionality to scheduler https://review.openstack.org/544730
19:48:54	openstackgerrit	Dan Smith proposed openstack/nova master: Add require_tenant_aggregate request filter https://review.openstack.org/545002
19:48:54	openstackgerrit	Dan Smith proposed openstack/nova master: WIP: Honor availability_zone hint via placement https://review.openstack.org/546282
20:14:37	mnaser	mriedem: https://review.openstack.org/#/c/545123/ has coverage case using new attach flow but not with the old one, so it fails in stable/pike. would you like me to update it to use the old flow fixture?
20:14:59	mnaser	i don't think we have to do it for queens or master, because technically we can only have compute releases that are behind by one version
20:15:44	mnaser	or actually that might be unrelated. regardless, i'm open to hearing the options
20:18:00	openstackgerrit	Dan Smith proposed openstack/nova master: Add require_tenant_aggregate request filter https://review.openstack.org/545002
20:18:01	openstackgerrit	Dan Smith proposed openstack/nova master: WIP: Honor availability_zone hint via placement https://review.openstack.org/546282
20:18:33	dansmith	ah jeez
20:20:18	openstackgerrit	Dan Smith proposed openstack/nova master: Add AggregateList.get_by_metadata() query method https://review.openstack.org/544728
20:20:19	openstackgerrit	Dan Smith proposed openstack/nova master: Add aggregates list to Destination object https://review.openstack.org/544729
20:20:19	openstackgerrit	Dan Smith proposed openstack/nova master: Add request filter functionality to scheduler https://review.openstack.org/544730
20:20:20	openstackgerrit	Dan Smith proposed openstack/nova master: Add require_tenant_aggregate request filter https://review.openstack.org/545002
20:20:20	openstackgerrit	Dan Smith proposed openstack/nova master: WIP: Honor availability_zone hint via placement https://review.openstack.org/546282
20:25:49	mriedem	mnaser: yeah need to use the CinderFixture in pike
20:25:58	mriedem	mnaser: i realized that would be the case
20:26:06	openstackgerrit	Merged openstack/nova stable/pike: Rollback instance.image_ref on failed rebuild https://review.openstack.org/539003
20:26:25	mriedem	dansmith: you've gone full bore on converting all of the post-placement filters to pre-placement filters haven't you :)
20:26:28	mnaser	mriedem: ok ill change it up to use cinderfixture and modify the code (as it would depend on if volume_id in self.reserved_volumes)
20:26:38	mriedem	mnaser: thanks
20:27:12	dansmith	mriedem: no, I just wanted another example
20:27:22	mriedem	guh why doesn't ctrl+c work inside a cirros shell
20:42:36	mriedem	woot got this multiattach demo working
20:43:03	mriedem	mounting and umounting like a mothertrucker
20:45:21	smcginnis	mriedem: Are you recording a demo?
20:48:37	mriedem	that's the plan
20:48:43	mriedem	finally got the run through working
20:48:56	smcginnis	mriedem: Nice!
20:49:03	mriedem	https://docs.google.com/presentation/d/1bp7IwEubvXqQf7mUE3nlUNW_T04L4c1-AMXELZ45jgg/edit?usp=sharing
20:50:28	mriedem	also uses device tags
20:50:29	mriedem	artom: ^
20:50:56	mriedem	and shows how to boot with multiple volumes using device tags, then mount the config drive in the guest to read the disk device tags from metadata
20:51:45	mriedem	throughout this whole thing i've found a new appreciation for openstackclient and the unified CLI, and ability to print out specific columns and parse the results for storing into local shell variables
20:51:51	mriedem	which is really really nice
20:52:17	mriedem	so much so that i think we should seriously think about deprecating the nova CLI
20:52:41	mriedem	melwitt: ^ might be a topic, albeit old, for the ptg
20:52:56	mriedem	except for things that we can't do in osc (yet)
20:53:13	artom	mriedem, noice
20:53:36	mriedem	one thing osc doesn't do is the super complicated --block-device params that nova boot does
20:53:47	mriedem	for specifying things like bootindex and tags
20:54:03	melwitt	mriedem: yeah. that's a tough one because we'll need to do an audit on what all osc doesn't do. I learned the hard way the other day that you can't boot from volume in osc with one command anymore
20:54:08	artom	Does it do microversions yet?
20:54:14	melwitt	no, it does not
20:54:19	mriedem	well, it does
20:54:28	mriedem	but not by default, you have to opt in
20:54:29	melwitt	well, it passes them through if specified manually
20:54:31	mriedem	right
20:54:34	mriedem	so i have been doing;
20:54:35	melwitt	but it doesn't do any discovery
20:54:42	mriedem	export OS_COMPUTE_API_VERSION=2.60
20:55:14	mriedem	melwitt: you can bfv with osc by doing: openstack server create --flavor 1 --volume <volume name or id> myserver
20:55:30	melwitt	that's if the volume has already been created by you earlier
20:55:36	melwitt	you have to do two steps
20:55:45	melwitt	you can't bfv from image with one command
20:55:47	mriedem	there is a --block-device option to server create
20:55:48	melwitt	anymore
20:55:54	melwitt	that doesn't work anymore
20:56:03	mriedem	but it's a bit limited, i think it only allows source volume or snapshot?
20:56:14	mriedem	so yeah, the osc --block-device param is definitely less robust than the nova boot one
20:56:19	melwitt	yeah, maybe. I was trying to do from image
20:56:25	mriedem	but to be fair,
20:56:32	mriedem	every time i need to use nova boot --block-device,
20:56:35	mriedem	i have to parse the help first
20:58:27	melwitt	yeah. we'd just have to turn our attention to making sure commands like those can be one step and keep them working
20:59:03	melwitt	long ago I wondered if we could make the novaclient stuff be an osc plugin, that way we can take care of the test coverage/regression coverage for our commands
21:00:08	melwitt	in our tree

Earlier   Later