Posted	Nick	Remark
#openstack-nova - 2017-07-10
12:03:27	bhagyashris	cdent: Hi, Thanks for new release of gabbi. Actually I am just trying to chcek response object with sample.json (expected response) for the placement functional gabbi test "confirm the correct post" but i am getting the error as "Unable to match $ as <@sample.json, got" after running the test.
12:04:05	bhagyashris	cdent: I am checking like: response_json_paths:
12:04:06	bhagyashris	$: <@sample.json
12:44:19	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Send soft_delete from context manager https://review.openstack.org/476459
12:44:20	openstackgerrit	Balazs Gibizer proposed openstack/nova master: use context mgr in instance.delete https://review.openstack.org/443764
12:44:20	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Transform missing delete notifications https://review.openstack.org/410297
12:57:59	bhagyashris	cdent: Hi, Thanks for new release of gabbi. Actually I am just trying to chcek response object with sample.json (expected response) for the placement functional gabbi test "confirm the correct post" but i am getting the error as "Unable to match $ as <@sample.json, got" after running the test.
12:58:05	bhagyashris	cdent: I am checking like: response_json_paths:
12:58:05	bhagyashris	<bhagyashris> $: <@sample.json
12:59:06	cdent	the file sample.json needs to be in the same directory as the yaml file, and it of course needs to have the correct content in it
12:59:19	cdent	can you paste your yaml file and sample.json somewhere?
12:59:54	bhagyashris	yamal
12:59:56	bhagyashris	- name: confirm the correct post
12:59:57	bhagyashris	GET: /resource_providers/$ENVIRON['RP_UUID']
12:59:57	bhagyashris	request_headers:
12:59:57	bhagyashris	content-type: application/json
12:59:57	bhagyashris	response_json_paths:
12:59:57	bhagyashris	$.: <@sample.json
13:01:01	bhagyashris	cdent: sample.json http://paste.openstack.org/show/614909/
13:03:23	cdent	bhagyashris you’ll want to change sample.json so that name and uuid are “$ENVIRON[‘RP_UUID’]” and $ENVIRON[‘RP_NAME’] and also use the environ in the url paths too?
13:04:56	cdent	bhagyashris: but also, based on the error message it looks like you don’t have the latest gabbi installed
13:05:58	bhagyashris	cdent: I have pulled new gabbi test and install on my machine
13:06:37	cdent	bhagyashris: are you running the tests with tox? if so, it will be using the gabbi in the virtualenv
13:07:31	bhagyashris	cdent: and regarding the name and uuid temporarily I have hard coded the name and uuid in the fixtures.py
13:07:45	cdent	ah, okay
13:08:06	bhagyashris	cdent: yeah like tox -e functional
13:08:28	cdent	then in that case you need to install the new gabbi with '.tox/functional/bin/pip install -U gabbi’
13:10:49	bhagyashris	cdent: ok. trying
13:11:51	bhagyashris	cdent: got error: raise AssertionError('unable to extract JSON from test results')
13:11:51	bhagyashris	AssertionError: unable to extract JSON from test results
13:13:58	cdent	bhagyashris: instead of $. you $
13:14:00	bhagyashris	cdent: working :)
13:14:07	bhagyashris	cdent: yeah
13:14:10	cdent	:)
13:15:23	bhagyashris	cdent: I just one question How should I access name and uuid "$ENVIRON[‘RP_UUID’]" and $ENVIRON[‘RP_NAME’] in sample.json
13:15:48	cdent	bhagyashris: I’ll change your paste to show you, one sec
13:15:58	bhagyashris	cdent: ok
13:20:31	cdent	bhagyashris: http://paste.openstack.org/show/614916/
13:22:56	bhagyashris	cdent: Thanks a lot. :)
13:23:47	cdent	bhagyashris: keep me posted on how it’s going, will be interesting to see how it works out
13:28:13	bhagyashris	cdent: ok. and one more thing I guess keeping the sample.json in other folder will cause problem right so is it ok that response.json in same folder?
13:29:54	cdent	the way the file reading works is that it needs to be in the same directory as the yaml file or any directory within that directory
13:31:24	bhagyashris	cdent: ok. Thanks for info.
13:36:30	andreykurilin	sdague: thanks
13:38:06	sdague	andreykurilin: no prob, thanks for building the patch
13:42:37	openstackgerrit	Alex Szarka proposed openstack/nova master: Transform instance-evacuate notification https://review.openstack.org/480119
13:44:53	openstackgerrit	Alex Szarka proposed openstack/nova master: Transform instance-evacuate notification https://review.openstack.org/482148
13:45:35	openstackgerrit	Alex Szarka proposed openstack/nova master: Transform instance-live_migration_post notification https://review.openstack.org/480119
13:55:39	edleafe	Scheduler subteam meeting in 5 minutes in #openstack-meeting-alt
14:01:01	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Stop using mox stubs in cast_as_call.py https://review.openstack.org/477730
14:01:35	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Stop using mox stubs in test_remote_consoles.py https://review.openstack.org/478423
14:01:47	openstackgerrit	Takashi NATSUME proposed openstack/nova master: api-ref: Verify parameters in os-migrations.inc https://review.openstack.org/474879
14:01:59	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Use PCIAddressField in oslo.versionedobjects https://review.openstack.org/466201
14:02:14	openstackgerrit	Takashi NATSUME proposed openstack/nova master: api-ref: Fix parameters in server-security-groups https://review.openstack.org/476434
14:02:28	openstackgerrit	Takashi NATSUME proposed openstack/nova master: api-ref: Add missing parameters in limits.inc https://review.openstack.org/475640
14:02:40	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Fix a missing classifier https://review.openstack.org/478733
14:03:15	openstackgerrit	Takashi NATSUME proposed openstack/nova master: api-ref: Add X-Openstack-Request-Id description https://review.openstack.org/474847
14:03:27	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Stop using mox stubs in test_console_auth_tokens.py https://review.openstack.org/477374
14:03:43	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Add 'updated_at' field to InstancePayload in notifications https://review.openstack.org/475276
14:03:56	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Use URIOpt https://review.openstack.org/479215
14:04:44	openstackgerrit	Takashi NATSUME proposed openstack/nova-specs master: Amend the "list-show-all-server-migration-types" spec https://review.openstack.org/460847
14:05:39	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Fix quota class set APIs https://review.openstack.org/467999
14:16:02	mriedem	o/
14:21:16	openstackgerrit	sean mooney proposed openstack/os-vif master: set mtu on all code paths https://review.openstack.org/469951
14:31:48	dansmith	jaypipes: can you legitimify my comment here? https://review.openstack.org/#/c/476631/19
14:32:37	mriedem	dansmith: we're talking about it in the scheduler meeting
14:32:42	mriedem	#openstack-meeting-alt
14:32:43	dansmith	oh
14:54:03	openstackgerrit	Merged openstack/python-novaclient master: Fix cropping the endpoint url https://review.openstack.org/479818
14:59:10	artom	mriedem, could you remove the procedural -2 on https://review.openstack.org/#/c/467321/?
15:00:11	mriedem	done, but i haven't really looked at everything else going on there
15:00:31	artom	mriedem, cheers!
15:00:41	artom	mriedem, bah, that can be done in its own time
15:00:49	artom	Getting the -2 off is a good first step ;)
15:03:29	mriedem	i was a -W but who's counting
15:03:53	mriedem	artom: not sure if you saw before going on PTO, but i opened a docs bug for putting something in the feature support matrix for device tags,
15:03:57	mriedem	is that something you can work on?
15:04:13	artom	mriedem, I saw that, yeah
15:04:26	artom	mriedem, didn't someone already submit a patch?
15:05:00	mriedem	ah yes https://review.openstack.org/#/c/481478/
15:05:19	artom	mriedem, err, twice, even
15:05:24	artom	https://review.openstack.org/#/c/481479/ as well
15:05:34	artom	Or are those different?
15:05:43	mriedem	once is for the server create, the other is attach
15:06:02	artom	Aha! I'll put those on my to-review list, in a call right this instant
15:08:34	openstackgerrit	KaiLi proposed openstack/nova master: Add "default" rule when generate policy.yaml.sample https://review.openstack.org/481443
15:14:47	mkucia	Hi, I have question for qeens as I believe this is beyond pike. Is there some work planned that is related to extracting ResourceProvider handling from nova's driver code? For example when libvirt xml is being assembled there are multiple interations over different kinds of resources. In the RPs perspective it seems for me that those resources can be generalized.
15:15:10	openstackgerrit	sean mooney proposed openstack/os-vif master: add host port profile info class https://review.openstack.org/441590
15:18:30	mriedem	mkucia: there are no plans for that
15:18:55	mriedem	mkucia: there are only plans to extract the placement service/api/data model into a separate project
15:19:27	mkucia	Ok. Thanks!
15:27:05	openstackgerrit	sean mooney proposed openstack/os-vif master: set mtu on all code paths https://review.openstack.org/469951
15:27:06	bauzas	dumb question, but can someone point me if/where we have privsep commands in our codebase?
15:29:28	mriedem	we don't
15:29:41	bauzas	nevermind, got the fully monty https://blueprints.launchpad.net/nova/+spec/privsep
15:29:41	mriedem	os-brick and os-vif use privsep
15:29:50	bauzas	full*
15:29:59	bauzas	mriedem: that's what I discovered
15:30:03	mriedem	you can also check out https://review.openstack.org/#/c/459166/
15:30:05	bauzas	the whiteboard is pretty explicit

Earlier   Later