Posted	Nick	Remark
#openstack-sdks - 2018-01-03
23:18:11	mordred	briancurtin: also - line 748 - yielded is reset in each loop iteration - seems like it should be moved to above line 734 - and that line 767 should be query_params['limit'] = limit - yielded
23:19:14	mordred	briancurtin: I'm going to keep it as is in this patch so we can fix it if needed in its own patch - but just wanted to double-check my reading of the logic there
#openstack-sdks - 2018-01-04
01:08:08	openstackgerrit	Jake Yip proposed openstack/python-openstackclient master: Fix error with image show when image name is None https://review.openstack.org/529464
02:06:01	liuyulong	hello python-openstackclient core team, can we get this https://review.openstack.org/#/c/517845/ merged, since neutron doc is depending on it.
02:06:29	liuyulong	neutron doc patch: https://review.openstack.org/#/c/526033/
03:07:47	dtroyer	liuyulong: I'll be able to verify that my concerns were addressed in the morning, I'd like amotoki to check his too…
03:09:34	liuyulong	dtroyer, OK, thank you.
06:56:21	openstackgerrit	OpenStack Proposal Bot proposed openstack/python-openstackclient master: Updated from global requirements https://review.openstack.org/531089
07:54:51	amotoki	liuyulong: I am in new year holidays till today. I am aware of yours and will check it tomorrow as I mentioned in the neutron team meeting this week.
07:57:13	liuyulong	amotoki: Great, thank you.
08:51:33	dtantsur	cdent: no, I haven't received any links. also likely to skip today's meeting due to personal stuff
08:52:32	cdent	dtantsur: no problem
17:06:25	openstackgerrit	Mike Fedosin proposed openstack/python-openstackclient master: Check that Glance returns image data before processing it https://review.openstack.org/531201
17:18:24	elmiko	edleafe: draft ready, https://etherpad.openstack.org/p/api-sig-newsletter
17:20:24	edleafe	elmiko: looking...
17:22:16	edleafe	elmiko: LGTM
17:22:33	cdent	lgtm too
17:22:45	elmiko	thanks, sending =)
20:23:12	openstackgerrit	Merged openstack/os-api-ref master: Updated from global requirements https://review.openstack.org/520469
21:14:53	openstackgerrit	Samuel Pilla proposed openstack/python-openstackclient master: Add project tags functionality https://review.openstack.org/481284
22:58:06	openstackgerrit	Merged openstack/python-openstacksdk master: Update the shade-merge document https://review.openstack.org/524391
#openstack-sdks - 2018-01-05
00:17:47	openstackgerrit	Monty Taylor proposed openstack/python-openstacksdk master: Prefer links dicts for pagination https://review.openstack.org/530769
00:17:47	openstackgerrit	Monty Taylor proposed openstack/python-openstacksdk master: Update for new docs PTI https://review.openstack.org/530978
00:17:48	openstackgerrit	Monty Taylor proposed openstack/python-openstacksdk master: WIP Make resource a dict subclass usable by shade layer https://review.openstack.org/530835
00:17:48	openstackgerrit	Monty Taylor proposed openstack/python-openstacksdk master: Use sdk for list_servers https://review.openstack.org/530770
00:17:49	openstackgerrit	Monty Taylor proposed openstack/python-openstacksdk master: Port wait_for_ methods to use iterate_timeout https://review.openstack.org/531268
00:17:49	openstackgerrit	Monty Taylor proposed openstack/python-openstacksdk master: Remove name from zuul project stanza https://review.openstack.org/531267
00:29:35	openstackgerrit	Monty Taylor proposed openstack/python-openstacksdk master: Use sdk for list_servers https://review.openstack.org/530770
00:29:36	openstackgerrit	Monty Taylor proposed openstack/python-openstacksdk master: WIP Make resource a dict subclass usable by shade layer https://review.openstack.org/530835
03:46:14	ldl	Is there any friend use the CentOS7.2 to develop openstack-sdk or openstack ? the official site docs is use Ubuntu as example.
09:41:21	openstackgerrit	Merged openstack/python-openstackclient master: Allow ports filtering with device_id https://review.openstack.org/526798
11:12:53	openstackgerrit	Akihiro Motoki proposed openstack/osc-lib master: Relocate utils tests to match the main code https://review.openstack.org/529956
13:24:11	amotoki	dtroyer: dhellmann: can we cut a new release of cliff?
13:24:37	amotoki	I would like to use the new sphinxext in neutronclient doc
16:09:55	dhellmann	amotoki : good idea. do you want to propose it in openstack/releases?
16:10:19	amotoki	dhellmann: sure. I will propose it
16:10:50	dhellmann	we'll get that csv fix in this release, too
16:14:27	jpich	Nice
18:18:31	cdent	elmiko: that dog face is not the face of gabbi excitement. I think the dog just has resting tripping balls face
18:18:41	cdent	he was perfectly relaxewd and chilled out
18:43:53	openstackgerrit	Mike Fedosin proposed openstack/python-openstackclient master: Check that Glance returns image data before processing it https://review.openstack.org/531201
18:49:24	johnsom	Do we have a timeline on a new openstacksdk release? octavia-dashboard has some patches waiting on a release with https://review.openstack.org/#/c/522962/ in it.
22:40:21	openstackgerrit	Michael Johnson proposed openstack/python-openstacksdk master: Add an octavia functional test gate https://review.openstack.org/531514
#openstack-sdks - 2018-01-06
00:28:04	johnsom	mordred Yee master of all things zuul, http://git.openstack.org/cgit/openstack/python-openstacksdk/tree/.zuul.yaml#n69 and http://logs.openstack.org/14/531514/1/check/openstacksdk-functional-devstack-octavia/14a627c/job-output.txt.gz#_2018-01-05_23_58_10_164433
00:28:40	johnsom	We usually disable cinder as it's not needed and saves gate time. Not sure how to override that in the parent zuul config
00:32:13	mordred	johnsom: looking - quick question ... any reason to not just add the octavia config to the other functional jobs?
00:32:33	mordred	like - does it increase the memory footprint a bunch and/or is enabling it flaky in any way?
00:32:50	johnsom	mordred Run time is the biggest reason we usually don't, but I'm open to it. It was before
00:33:02	johnsom	It boots VMs
00:34:51	mordred	johnsom: nod. well - I think we can get it going with a second job just to see - that might also make it nicer/easier for octavia to add the sdk functional test to octavia patches
00:34:56	johnsom	My thought on this would just push that config down to jobs that need it
00:35:44	johnsom	For a base job, it assumes you need a lot of infrastructure (heat, swift, cinder, etc.)
00:36:06	mordred	yah - well, so far those have been fairly standard - but I agree, I think we can restructure that a bit more
00:36:55	johnsom	Ok, so should I take that on tomorrow or just setup a separate octavia job replicating that parts of that I care about?
00:39:31	mordred	johnsom: I think refactoring that base job sounds like a good idea if you're up for it - we should probably add an OPENSTACKSDK_HAS_CINDER var and an entry https://review.openstack.org/#/c/531514/1/openstack/tests/functional/cloud/test_devstack.py like you did for octavia
00:39:59	mordred	johnsom: since this'll be the first time we'll have jobs that don't have a cinder - making sure we don't accidentally fail open at some point == good
00:40:31	johnsom	mordred Ok, I will look into that all tomorrow
00:40:42	mordred	thanks! I think this'll be a nice improvement
00:41:26	mordred	johnsom: fwiw, this took me down a rabbit hole of realizing that heat uses openstacksdk, that I need to make them a patch like I made you, and then all the way down the hole when I realized I could make the heat patch better by refactoring something in sdk
00:42:22	johnsom	lol, sorry/not sorry
00:42:51	mordred	hehe. it happens :)
00:43:09	mordred	I'm just looking forward to assaulting dtroyer and Shrews with the results
09:07:46	openstackgerrit	Rabi Mishra proposed openstack/osc-lib master: Fix find() interface when attr is not specified https://review.openstack.org/529934
#openstack-sdks - 2018-01-07
00:12:50	openstackgerrit	Michael Johnson proposed openstack/python-openstacksdk master: Add an octavia functional test gate https://review.openstack.org/531514
17:00:06	openstackgerrit	Monty Taylor proposed openstack/python-openstacksdk master: Update for new docs PTI https://review.openstack.org/530978
17:00:07	openstackgerrit	Monty Taylor proposed openstack/python-openstacksdk master: Port wait_for_ methods to use iterate_timeout https://review.openstack.org/531268
17:00:07	openstackgerrit	Monty Taylor proposed openstack/python-openstacksdk master: Remove name from zuul project stanza https://review.openstack.org/531267
17:00:08	openstackgerrit	Monty Taylor proposed openstack/python-openstacksdk master: Use sdk for list_servers https://review.openstack.org/530770
17:00:08	openstackgerrit	Monty Taylor proposed openstack/python-openstacksdk master: Prefer links dicts for pagination https://review.openstack.org/530769
17:00:09	openstackgerrit	Monty Taylor proposed openstack/python-openstacksdk master: Rename CloudConfig to CloudRegion https://review.openstack.org/531611
17:00:09	openstackgerrit	Monty Taylor proposed openstack/python-openstacksdk master: WIP Make resource a dict subclass usable by shade layer https://review.openstack.org/530835
17:00:10	openstackgerrit	Monty Taylor proposed openstack/python-openstacksdk master: Add function to make CloudRegion from session https://review.openstack.org/531612
20:56:11	openstackgerrit	Michael Johnson proposed openstack/python-openstacksdk master: Add an octavia functional test gate https://review.openstack.org/531514
22:06:36	openstackgerrit	Monty Taylor proposed openstack/python-openstacksdk master: Rename CloudConfig to CloudRegion https://review.openstack.org/531611
#openstack-sdks - 2018-01-08
01:44:47	openstackgerrit	Michael Johnson proposed openstack/python-openstacksdk master: Add an octavia functional test gate https://review.openstack.org/531514
02:43:04	openstackgerrit	Bar Elharar proposed openstack/osc-lib master: Suppress subTest() expected errors https://review.openstack.org/531025
02:56:22	openstackgerrit	Bar Elharar proposed openstack/osc-lib master: Suppress subTest() expected errors https://review.openstack.org/531025
06:20:41	openstackgerrit	Merged openstack/python-openstacksdk master: Updated from global requirements https://review.openstack.org/525689
14:25:46	openstackgerrit	Monty Taylor proposed openstack/python-openstacksdk master: Rename CloudConfig to CloudRegion https://review.openstack.org/531611
14:26:30	mordred	Shrews: if you have a sec, https://review.openstack.org/#/c/530978 and https://review.openstack.org/#/c/531267 are both super simple/mechanical
14:28:13	Shrews	mordred: YOU'RE super simple/mechanical
15:28:22	openstackgerrit	Merged openstack/python-openstacksdk master: Update for new docs PTI https://review.openstack.org/530978
15:45:36	openstackgerrit	Merged openstack/python-openstacksdk master: Remove name from zuul project stanza https://review.openstack.org/531267
17:04:02	mordred	briancurtin: responded to your note on the dict patch - thanks for the background! I've got one more thing to try, will ping you when its up- might be tomorrow
17:04:24	mordred	briancurtin: however, it's also entirely possible the results here will be "unpossible/bad idea"
17:48:29	briancurtin	mordred: ok cool, sounds good
17:49:23	mordred	edleafe, elmiko: was just chatting with notmyname verifying that I was understanding swift pagingation correctly
17:50:00	mordred	edleafe, elmiko: and in doing so I discovered that requests parses rfc5988 headers automatically: http://docs.python-requests.org/en/master/user/advanced/#link-headers
17:50:41	mordred	edleafe, elmiko: given the variation in how pagination links are returned today, and how unlikely it is that we'd be able to get all the services aligned on one approach - since backwards compat would make it living hell ...
17:51:19	mordred	what if we changed the recommendation to start publishing rfc5988 compliant links headers - the data is the same as what peopel are already putting in links bodies (slightly different format, but whatever)
17:53:18	mordred	since it's a standard header that's intended to be used for that, it shouldn't be a backwards compat issue - and clients could easily consume Links headers if they exist, and otherwise fallback to existing "look for resp.json()['links'] or resp.json['{resource}_links'] or resp.json()['next'] or an object count in resp.headers"
17:55:25	edleafe	mordred: so if I understand you correctly, projects could leave their assorted formats in the response body alone, and just add the necessary headers?
17:55:53	edleafe	mordred: and then clients could just check for the next/prev link from the headers?
18:27:12	elmiko	mordred: assuming what edleafe says is accurate, that makes entirely too much sense ;)
18:37:52	mordred	elmiko, edleafe: yes!
18:38:00	mordred	cdent: also ... ^^
18:38:19	cdent	mordred: yeah, was just reading through that
18:38:29	mordred	cdent: http://eavesdrop.openstack.org/irclogs/%23openstack-sdks/%23openstack-sdks.2018-01-08.log.html#t2018-01-08T17:49:23
18:38:31	mordred	oh - good
18:39:38	cdent	I've never been a huge fan of link headers, but it does provide a nice solution in this case

Earlier   Later