Posted	Nick	Remark
#openstack-nova - 2017-07-06
07:37:25	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Use URIOpt https://review.openstack.org/479215
07:42:00	openstackgerrit	Silvan Kaiser proposed openstack/nova master: [WIP] Add truncate for images if prealloc is true https://review.openstack.org/480464
07:52:24	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Stop using mox stubs in test_console_auth_tokens.py https://review.openstack.org/477374
08:41:01	openstackgerrit	Naichuan Sun proposed openstack/nova master: xenapi: Exception Error logs shown in Citrix XenServer CI https://review.openstack.org/478437
08:50:59	openstackgerrit	Zhenyu Zheng proposed openstack/nova master: Support tag instances when boot(3/4) https://review.openstack.org/459593
08:53:16	openstackgerrit	Andrey Volkov proposed openstack/nova master: [placement] Add api-ref for RP traits https://review.openstack.org/474550
08:53:17	openstackgerrit	Andrey Volkov proposed openstack/nova master: [placement] Add api-ref for allocations https://review.openstack.org/470933
08:53:17	openstackgerrit	Andrey Volkov proposed openstack/nova master: [placement] Add api-ref for usages https://review.openstack.org/480563
08:53:18	openstackgerrit	Andrey Volkov proposed openstack/nova master: [placement] Add api-ref for RP allocations https://review.openstack.org/450550
08:53:18	openstackgerrit	Andrey Volkov proposed openstack/nova master: [placement] Add api-ref for RP usages https://review.openstack.org/450105
08:59:31	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Stop using mox stubs in test_console_auth_tokens.py https://review.openstack.org/477374
09:12:39	openstackgerrit	Andrey Volkov proposed openstack/nova master: [placement] Make placement_api_docs.py failing https://review.openstack.org/480924
09:20:04	openstackgerrit	Andreas Scheuring proposed openstack/nova master: DO NOT MERGE! Dummy patch https://review.openstack.org/478186
09:38:02	bauzas	sfinucan: very old bug https://review.openstack.org/#/c/391060/
09:38:41	stephenfin	bauzas: I feel like I already reviewed that?
09:39:13	bauzas	I know, you did +1
09:39:33	bauzas	but it was a long time ago :p
09:40:20	stephenfin	bauzas: Was there something similar to that a few weeks back though?
09:40:33	stephenfin	with requestspec objects
09:41:28	stephenfin	Oh yeah, commit e211fca55a11c80058d5d78e31dc3ad466d7edfd
09:41:55	stephenfin	Cool. I'll take a look once I've finished resolving the merge conflict on https://review.openstack.org/#/c/415921/
09:45:16	openstackgerrit	Alex Szarka proposed openstack/nova master: Transform instance-live_migration_post notification https://review.openstack.org/480119
09:59:44	bauzas	sorry, was afk for getting my daughter at school
10:17:24	_jarvis	need help in this bug
10:17:25	_jarvis	https://bugs.launchpad.net/nova/+bug/1698253
10:17:25	openstack	Launchpad bug 1698253 in OpenStack Compute (nova) "InvalidInput: Invalid input received: vif type bridge not supported" [Undecided,In progress] - Assigned to PRAVIN (jarvisopenstack)
10:19:06	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Add sample test for instance audit https://review.openstack.org/480955
10:19:17	_jarvis	these was working in liberity
10:20:29	openstackgerrit	Stephen Finucane proposed openstack/nova master: placement: adds REST API for nested providers https://review.openstack.org/384807
10:20:31	openstackgerrit	Stephen Finucane proposed openstack/nova master: placement: allow filter providers in tree https://review.openstack.org/377215
10:20:32	openstackgerrit	Stephen Finucane proposed openstack/nova master: placement: set/check if inventory change in tree https://review.openstack.org/470575
10:20:32	openstackgerrit	Stephen Finucane proposed openstack/nova master: placement: integrate ProviderTree to report client https://review.openstack.org/415921
10:20:33	openstackgerrit	Stephen Finucane proposed openstack/nova master: placement: update client to set parent provider https://review.openstack.org/385693
10:20:33	openstackgerrit	Stephen Finucane proposed openstack/nova master: placement: add nested resource providers https://review.openstack.org/377138
10:20:34	openstackgerrit	Stephen Finucane proposed openstack/nova master: placement: add ProviderTree.is_inventory_empty https://review.openstack.org/480957
11:05:43	openstackgerrit	Alex Szarka proposed openstack/nova master: Add method for verify multiple versioned notifications https://review.openstack.org/465526
11:05:53	openstackgerrit	Alex Szarka proposed openstack/nova master: Transform instance.exists notification https://review.openstack.org/403660
11:20:38	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Make notification publisher_id consistent https://review.openstack.org/480984
11:30:58	openstackgerrit	Arundhati Surpur proposed openstack/python-novaclient master: Removed extra 'method' from the NOTE https://review.openstack.org/480988
11:33:45	openstackgerrit	Béla Vancsics proposed openstack/nova master: Transform keypair.import notification https://review.openstack.org/467514
11:41:43	alex_xu	bauzas: another fix for placement https://review.openstack.org/#/c/480130/, this should be easy :)
11:51:35	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Use enum value instead of string service name https://review.openstack.org/476538
12:01:16	openstackgerrit	OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.openstack.org/477091
12:04:57	openstackgerrit	sudhir agarwal proposed openstack/nova master: Replaces uuid.uuid4 with uuidutils.generate_uuid() https://review.openstack.org/481007
12:07:55	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Add BDM to InstancePayload https://review.openstack.org/448779
12:09:21	bauzas	alex_xu: heh
12:15:52	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Add sample test for instance audit https://review.openstack.org/480955
12:17:10	openstackgerrit	Andrey Kurilin proposed openstack/python-novaclient master: Fix cropping the endpoint url https://review.openstack.org/479818
12:19:29	openstackgerrit	zhangdaolong proposed openstack/python-novaclient master: Fix the inappropriate parameter name https://review.openstack.org/480826
12:33:37	jaypipes	ralonsoh: no idea what the deal is on that "missing link" unit test that's failing on your openstack/releases patch for os-traits 0.3.2....
12:34:39	ralonsoh	jaypipes: uhmmm I'll check this now. Thanks for the heads up
12:35:03	jaypipes	ralonsoh: yeah, I have no idea why that is failing. :(
12:36:49	cdent	morning jaypipes: i've spiked that gabbi feature you were looking for while doing allocation_candates: https://github.com/cdent/gabbi/pull/215
12:37:08	jaypipes	cdent: w00tness. :)
12:37:28	cdent	i'm not sure it is quite good enough yet, so if you can looksee that'd be keen
12:37:50	jaypipes	cdent: though, to be honest, the feature I'd *really* love is to be able to represent in YAML the expected HTTP response without using JSONPath at all.
12:38:02	cdent	i'm not sure what you mean by that
12:38:22	jaypipes	cdent: I mean, I'd like to be able to something like:
12:38:26	jaypipes	http_response_expect:
12:38:36	jaypipes	allocation_requests:
12:38:45	jaypipes	- allocations:
12:38:51	jaypipes	resource_provider:
12:39:00	jaypipes	uuid: <blah>
12:39:02	jaypipes	etc etc
12:39:11	jaypipes	cdent: make sense?
12:39:25	cdent	full response, or piecemeal?
12:39:36	jaypipes	cdent: would be awesome to have options.
12:39:58	jaypipes	cdent: like http_response_full_expect: or something like that which would fail if not exact
12:40:27	jaypipes	cdent: and otherwise, just fail if the listed parts of the http_response_expect: were not included in the response.
12:40:32	cdent	piecemeal is hard because you need some way to say "this" allocations, and that's what led to the json paths
12:40:41	jaypipes	understood.
12:40:43	jaypipes	code is hard.
12:40:45	jaypipes	:P
12:40:49	cdent	the fully query stuff see this thread: http://lists.openstack.org/pipermail/openstack-dev/2017-July/119286.html
12:42:18	cdent	in any case, you can fake the full stuff by matching the root ($) key, see last half of: https://gabbi.readthedocs.io/en/latest/jsonpath.html
12:42:42	jaypipes	cdent: sure, but that's ugly as f**k :)
12:43:01	cdent	that looks almost exactly what you have above (which is also ugly as fuck)
12:43:10	jaypipes	ok, I'm done.
12:43:18	cdent	heh
12:44:31	jaypipes	cdent: oh, I see you're referring to the final example in that page
12:44:35	cdent	yes
12:44:42	jaypipes	cdent: and not the one above it..
12:44:51	cdent	yes
12:45:05	cdent	yeah, the filter stuff is ghastly, I agree
12:45:07	jaypipes	cdent: didn't know I could do that.
12:45:36	cdent	it just falls out of the nature of json path '$' is 'everything'
12:45:43	jaypipes	cdent: that would work. though a little hackery would allow one to remove the need for the $:
12:46:14	jaypipes	cdent: if, say, Gabbit saw a "response_yaml_exact" tag in the gabbit.
12:46:55	cdent	jaypipes: if you read around in that thread, especially the message I linked to, there's some discussion on options for how to do it. A little more discussino ought to lead to some reasonable outcomes
12:47:31	cdent	the main thrust of the thread is to be able to read in files that are the json, but it's the same deal in the abstract
12:47:49	bauzas	cdent: out of curiosity because I had the problem on Monday, how can I introspect a gabbi functional test?
12:48:50	cdent	bauzas: depends on the kind of introspection you want to be doing, but you can make an individual test 'verbose' with 'verbose: True' but frequently the output will be swallowed, depending on the environment
12:49:03	cdent	another option is to add a response_strings:\n- something that won't match
12:49:10	cdent	and then it will fail and you can see the response
12:49:14	bauzas	cdent: basically debugging
12:49:19	bauzas	ie. pdb'ing
12:49:34	bauzas	since tox can't do that, I generally use testtools.run directly
12:50:08	bauzas	but since placement tests are generated by an helper, I can't directly use it
12:50:26	bauzas	because testtools.run can't find the test I wanted to debug
12:50:36	cdent	subunit.run can

Earlier   Later