Posted	Nick	Remark
#openstack-sdks - 2018-07-26
01:51:08	openstackgerrit	Nguyen Hai proposed openstack/openstackclient master: add lower-constraints job https://review.openstack.org/556037
02:58:41	openstackgerrit	Ryan Brady proposed openstack-infra/shade master: Add support for static routes https://review.openstack.org/580661
05:02:39	openstackgerrit	Nguyen Hai proposed openstack/openstackclient master: add lower-constraints job https://review.openstack.org/556037
05:03:45	openstackgerrit	Nguyen Hai proposed openstack/openstackclient master: add lower-constraints job https://review.openstack.org/556037
07:11:51	openstackgerrit	yanpuqing proposed openstack/python-openstackclient master: Add error message when network resources use "set" "unset" command https://review.openstack.org/579515
07:24:24	usr2033	hi, does official openstack-sdk support quota operations?
08:58:15	openstackgerrit	OpenStack Release Bot proposed openstack/python-openstackclient master: Update reno for stable/rocky https://review.openstack.org/586006
09:01:02	openstackgerrit	Josephine Seifert proposed openstack/python-openstackclient master: osc-included image signing (using openstacksdk) https://review.openstack.org/580086
09:02:57	openstackgerrit	Josephine Seifert proposed openstack/python-openstackclient master: osc-included image signing (using openstacksdk) https://review.openstack.org/580086
11:54:44	openstackgerrit	Ryan Brady proposed openstack-infra/shade master: Add support for static routes https://review.openstack.org/580661
12:00:01	openstackgerrit	Monty Taylor proposed openstack/openstacksdk master: Update create_object to handled chunked data https://review.openstack.org/585532
12:34:05	openstackgerrit	OpenStack Release Bot proposed openstack/keystoneauth master: Update reno for stable/rocky https://review.openstack.org/586083
12:35:07	openstackgerrit	OpenStack Release Bot proposed openstack/openstacksdk master: Update reno for stable/rocky https://review.openstack.org/586094
12:35:37	openstackgerrit	OpenStack Release Bot proposed openstack/os-client-config master: Update reno for stable/rocky https://review.openstack.org/586100
12:36:32	openstackgerrit	Monty Taylor proposed openstack/os-service-types master: Add flag for returning unofficial types https://review.openstack.org/585411
12:36:33	openstackgerrit	OpenStack Release Bot proposed openstack/osc-lib master: Update reno for stable/rocky https://review.openstack.org/586112
12:48:32	odyssey4me	mordred: I thought you might like to know a new and exciting and totally useful error I got the other day when I tried to do a compute thing with the openstack client. :) http://logs.openstack.org/15/582315/9/check/openstack-ansible-functional-ubuntu-xenial/911cdf7/job-output.txt.gz#_2018-07-21_09_14_00_043030
12:49:09	odyssey4me	It turns out, when you do not have an endpoint for the service, you get the rather oscure "'list' object has no attribute 'get'" error.
13:18:10	mordred	odyssey4me: well! that certainly is exciting
13:24:33	mordred	odyssey4me: I'm not 100% sure if that's coming from osc or novaclient- how hard would it be to recreate that with --debug ?
13:25:31	odyssey4me	mordred: hmm, let me do that and get back to you - I might already have one up that I can break
13:25:55	odyssey4me	oh bother, I don't - ok, let me kick off a build
13:36:14	mordred	odyssey4me: woot. thanks!
16:07:40	odyssey4me	mordred: it turns out that it's far more fun than anticipated - to replicate it you have to setup the compute endpoint with the wrong service backing it
16:08:09	odyssey4me	in my test I put the placement service endpoints into the catalog, saying it was the compute service
16:08:11	odyssey4me	https://gist.github.com/odyssey4me/fdfb2474bdd6fde4d1368af2af4044c6
16:08:45	odyssey4me	while of course this is not someone one would expect someone to do, it is perhaps better to handle the error somewhat more elegantly
16:12:18	dtroyer	odyssey4me: this sounds like it might be a good integration test for OSC. Those drop commands in the top and use requests-mock so we let the entire stack have a chance to get it wrong, but still wouldn't require a specially-configured could to test against...
16:27:58	cdent	thanks elmiko, was rather distracted during all that
16:28:36	elmiko	cdent: no worries =)
16:28:51	openstackgerrit	Merged openstack/api-wg master: Add links to errors-example.json https://review.openstack.org/578369
16:28:52	openstackgerrit	Merged openstack/api-wg master: Expand schema for error.codes to reflect reality https://review.openstack.org/580703
16:33:32	edleafe	cdent, elmiko, dtantsur: https://etherpad.openstack.org/p/api-sig-newsletter is ready for your review
16:34:17	edleafe	Oops - hold on a minute
16:34:56	edleafe	ok, it's ready now. Forgot to paste the links
16:35:07	cdent	lgtm
16:35:50	elmiko	edleafe: maybe reference that notmyname suggested we look at the draft?
16:36:12	edleafe	elmiko: sure. You wanna add that?
16:36:18	elmiko	k
16:37:23	elmiko	edleafe: how's that look?
16:37:37	edleafe	lgtm
16:37:39	edleafe	ship it?
16:37:45	elmiko	+1
16:38:03	elmiko	thanks again edleafe !
16:39:03	edleafe	sent
16:39:18	elmiko	\o/
19:10:20	mordred	dtroyer, odyssey4me: yes to integration test!
19:11:20	mordred	odyssey4me: and thanks for making that traceback ... it also looks like the base issue is in novaclient with an assumption on its part on how error payloads look. when we shift stuff over to sdk the error handler there is more permissive because it has to handle all of the services- so it's coded defensively to check and see what exists
19:11:35	mordred	in any case, yes - I do think that error could be handled more elegantly
20:02:55	openstackgerrit	Monty Taylor proposed openstack/openstacksdk master: Add method for returning a raw response for an object https://review.openstack.org/586262
20:02:55	openstackgerrit	Monty Taylor proposed openstack/openstacksdk master: Update create_object to handled chunked data https://review.openstack.org/585532
20:02:56	openstackgerrit	Monty Taylor proposed openstack/openstacksdk master: Add support for streaming object responses https://review.openstack.org/586250
22:10:35	openstackgerrit	Merged openstack/os-service-types master: Add flag for returning unofficial types https://review.openstack.org/585411
#openstack-sdks - 2018-07-27
06:02:03	openstackgerrit	Chen proposed openstack/python-openstackclient master: Compute: Add description support for server https://review.openstack.org/568549
06:22:00	openstackgerrit	tianhui proposed openstack/python-openstackclient master: Compute: Add tag support for server add fixed ip https://review.openstack.org/581325
06:35:58	openstackgerrit	Fan Zhang proposed openstack/python-openstackclient master: Add --key-name and --key-unset option for server rebuild API. https://review.openstack.org/576422
06:56:32	openstackgerrit	YAMAMOTO Takashi proposed openstack/osc-lib master: Replaced a few references to LP bugs with storyboard https://review.openstack.org/586453
07:54:46	openstackgerrit	Dmitry Tantsur proposed openstack/openstacksdk master: Support for microversions in base Resource https://review.openstack.org/583491
08:01:54	openstackgerrit	Dmitry Tantsur proposed openstack/openstacksdk master: [WIP] Support for microversions in baremetal resources https://review.openstack.org/583532
08:23:47	openstackgerrit	tianhui proposed openstack/python-openstackclient master: Compute: Add tag support for server add network https://review.openstack.org/581573
08:28:27	openstackgerrit	tianhui proposed openstack/python-openstackclient master: Compute: Add tag support for server add port https://review.openstack.org/580977
08:35:53	openstackgerrit	tianhui proposed openstack/python-openstackclient master: Compute: Add tags support for server https://review.openstack.org/569386
09:19:22	openstackgerrit	Fan Zhang proposed openstack/python-openstackclient master: Add --key-name and --key-unset option for server rebuild API. https://review.openstack.org/576422
10:51:42	dtantsur	mordred: hey! can I get some reviews on https://review.openstack.org/#/c/583491/ please? this is an important step in microversioning support
11:43:42	mordred	dtantsur: yes! sorry - was focused on stuff for the freeze
11:54:25	mordred	dtantsur: lgtm. Shrews - you wanna look at https://review.openstack.org/#/c/583491 ?
11:54:38	dtantsur	thanks!
11:55:33	Shrews	sure. lemme finish coffeeing first
12:46:17	openstackgerrit	Monty Taylor proposed openstack/openstacksdk master: Add method for returning a raw response for an object https://review.openstack.org/586262
12:46:18	openstackgerrit	Monty Taylor proposed openstack/openstacksdk master: Add support for streaming object responses https://review.openstack.org/586250
13:00:26	openstackgerrit	Fan Zhang proposed openstack/python-openstackclient master: Add --key-name and --key-unset option for server rebuild API. https://review.openstack.org/576422
13:47:20	openstackgerrit	Dmitry Tantsur proposed openstack/openstacksdk master: Support for microversions in baremetal resources https://review.openstack.org/583532
13:48:20	dtantsur	mordred: how hard is it to add functional tests for ironic to the sdk?
13:50:26	mordred	dtantsur: it shouldn't be too hard to add the tests - but I don't think we currently run against any devstacks configured for ironic (or any other deployments - doesn't have to be devstack)
13:50:38	mordred	so we probably need to add a job
13:50:58	mordred	right now our ironic coverages is basically through the shade layer and then through bifrost
13:51:04	mordred	and the bifrost job
13:51:44	dtantsur	mordred: but we don't have a bifrost job on openstacksdk itself, do we?
13:54:18	mordred	oh - no we don't - we should add one
13:54:29	mordred	seeing as how the ansible modules use openstacksdk now
13:57:52	openstackgerrit	Monty Taylor proposed openstack/openstacksdk master: Run bifrost integration test jobs https://review.openstack.org/586564
13:57:56	mordred	dtantsur: ^^
13:58:14	dtantsur	mordred: looking already, thanks!
13:58:31	mordred	dtantsur: we should probably update bifrost itself to just use openstacksdk directly and not via shade ... but maybe that's a step
13:59:15	dtantsur	oh, right, I did not realize it also uses shade from python
14:00:17	mordred	dtantsur: we added openstacksdk as a shade depend last cycle, so it should be a fairly easy switch
14:00:31	dtantsur	right
14:01:09	dtantsur	mordred: okay, I'll ponder creating a simple functional job for baremetal bits this afternoon. testing it locally does not scale well :)
14:06:03	dtantsur	mordred: will I have to do something to use admin credentials?
14:09:54	mordred	dtantsur: the functional test suite looks for clouds.yaml files already - and with a particular name
14:09:58	mordred	dtantsur: TEST_CLOUD_NAME = os.getenv('OS_CLOUD', 'devstack-admin')
14:10:55	mordred	dtantsur: oh - well - bother - the sdk-layer and shade-layer functional tests are still a little different
14:11:00	mordred	that above is for the sdk layer ones
14:11:12	mordred	for the shade layer (openstack/tests/functional/cloud) - it's
14:11:17	mordred	self._demo_name = os.environ.get('OPENSTACKSDK_DEMO_CLOUD', 'devstack')
14:11:19	mordred	self._op_name = os.environ.get(
14:11:21	mordred	'OPENSTACKSDK_OPERATOR_CLOUD', 'devstack-admin')
14:11:24	mordred	but basically the same idea
14:11:53	mordred	it's also worth looking at openstack/tests/functional/cloud/test_devstack.py ...
14:12:21	mordred	we have a little system where you can set an env var in the job definition, like OPENSTACKSDK_HAS_MAGNUM: 1

Earlier   Later