Posted	Nick	Remark
#openstack-sdks - 2017-07-10
15:33:08	efried	Or do we keep those forever?
15:54:32	mordred	efried: there is not a plan - my current thinking is keep-them-forever - it shouldn't be many of them - hopefully we almost never update that data :)
15:56:19	efried	mordred Dig. Perhaps delete 'em all just before sta is considered truly primetime (maybe that's right now) to keep it pretty clean.
15:56:43	efried	mordred I mean, up to this point, we've mainly just been fleshing out entries, adding aliases, etc. There shouldn't be anyone relying on the older revs.
16:22:23	openstackgerrit	Eric Fried proposed openstack/keystoneauth master: Expand some discover.py docstrings https://review.openstack.org/482207
16:38:40	openstackgerrit	Eric Fried proposed openstack/keystoneauth master: Expand some discover.py docstrings https://review.openstack.org/482207
17:37:51	openstackgerrit	Eric Fried proposed openstack/keystoneauth master: normalize_version_number([1]) => (1, 0) and docs https://review.openstack.org/481309
19:06:35	openstackgerrit	Eric Fried proposed openstack/keystoneauth master: Make Discover.version_data accept null max_version https://review.openstack.org/482250
19:21:31	openstackgerrit	Eric Fried proposed openstack/keystoneauth master: Expand some discover.py docstrings https://review.openstack.org/482207
19:29:17	openstackgerrit	Merged openstack/service-types-authority master: Remove validate.py main() https://review.openstack.org/478562
19:45:09	openstackgerrit	Eric Fried proposed openstack/keystoneauth master: Nix EndpointData.get_versioned_data(authenticated) https://review.openstack.org/482260
19:56:21	openstackgerrit	Eric Fried proposed openstack/keystoneauth master: Fix _run_discovery caching https://review.openstack.org/481754
19:57:09	efried	mordred ^^ is the split-out version specifically for https://bugs.launchpad.net/keystoneauth/+bug/1703447
19:57:10	openstack	Launchpad bug 1703447 in keystoneauth "URL caching in EndpointData._run_discovery is busted" [Undecided,In progress] - Assigned to Eric Fried (efried)
19:59:53	efried	mordred Actually what I meant to call attention to was the previous one: Nix EndpointData.get_versioned_data(authenticated) https://review.openstack.org/482260
20:00:09	mordred	efried: cool. will read in just a sec
20:00:10	efried	mordred That's the one that needs to get into ksa before the next release.
20:00:36	mordred	efried: btw - I'm pushing up a initial shell for the os-service-types repo (to consume the service-types-authority data) I made you an initial core on it - enjoy :)
20:01:18	efried	mordred Oh? Is there a charter or mission statement of some kind therein that I can read?
20:16:17	openstackgerrit	Eric Fried proposed openstack/keystoneauth master: Miscellaneous cleanup in discover.py https://review.openstack.org/482271
20:17:47	openstackgerrit	Monty Taylor proposed openstack/os-service-types master: Add cookiecutter boilerplate https://review.openstack.org/482274
20:17:58	mordred	efried: there's one in the readme ^^
20:38:15	openstackgerrit	Eric Fried proposed openstack/keystoneauth master: normalize_version_number([1]) => (1, 0) and docs https://review.openstack.org/481309
20:54:54	efried	mordred Where does this boilerplate come from?
20:56:24	mordred	efried: the cookiecutter tool
20:56:38	efried	?
20:56:39	mordred	efried: http://git.openstack.org/cgit/openstack-dev/cookiecutter
20:56:52	mordred	those are templates for https://github.com/audreyr/cookiecutter
20:57:42	efried	mordred I gotta ask - why wouldn't we just put this function into sta?
20:58:33	mordred	efried: because - we want to keep sta a canonical locatoin for the data and not provide structural advantage to python over non-python
20:58:58	efried	okey.
20:59:03	mordred	if we put a library into sta itself, we'll also run in to a mismatch between version intent (data vs. api release)
20:59:21	mordred	it's - you konw - one of those things we're gonna spend extra time on and then likely never touch again
21:02:42	efried	mordred Okay, couple comments in there.
23:17:09	openstackgerrit	Boris Pavlovic proposed openstack/python-openstackclient master: Add server list -n and --no-name-lookup arguments https://review.openstack.org/479089
23:20:51	openstackgerrit	Boris Pavlovic proposed openstack/python-openstackclient master: Add server list -n and --no-name-lookup arguments https://review.openstack.org/479089
#openstack-sdks - 2017-07-11
06:48:59	openstackgerrit	NidhiMittalHada proposed openstack/python-openstackclient master: image-list should support filters 'name','status' https://review.openstack.org/479670
08:17:10	openstackgerrit	Akihiro Motoki proposed openstack/python-openstackclient master: Network L3 Router Commands for OSC https://review.openstack.org/385729
08:51:07	openstackgerrit	M V P Nitesh proposed openstack/python-openstackclient master: List images with --tags for 'openstack image list' https://review.openstack.org/481615
09:12:27	openstackgerrit	liyi proposed openstack/python-openstacksdk master: Unify style of 'domain' field https://review.openstack.org/482469
10:38:43	openstackgerrit	M V P Nitesh proposed openstack/python-openstackclient master: List images with --tags for 'openstack image list' https://review.openstack.org/481615
14:00:34	openstackgerrit	liyi proposed openstack/python-openstacksdk master: Unify style of 'domain' field https://review.openstack.org/482469
14:12:28	openstackgerrit	Monty Taylor proposed openstack/keystoneauth master: Use a specific logger for logging request ids https://review.openstack.org/482577
15:01:29	openstackgerrit	Merged openstack/python-openstacksdk master: Unify style of 'domain' field https://review.openstack.org/482469
16:47:36	openstackgerrit	Eric Fried proposed openstack/keystoneauth master: Miscellaneous cleanup in discover.py https://review.openstack.org/482271
17:47:46	openstackgerrit	Merged openstack/keystoneauth master: Nix EndpointData.get_versioned_data(authenticated) https://review.openstack.org/482260
19:14:35	openstackgerrit	Eric Fried proposed openstack/keystoneauth master: Make Discover.version_data accept null max_version https://review.openstack.org/482250
19:27:53	openstackgerrit	Merged openstack/keystoneauth master: Fix _run_discovery caching https://review.openstack.org/481754
20:32:59	openstackgerrit	Eric Fried proposed openstack/keystoneauth master: Update docs and add a release note https://review.openstack.org/477566
21:26:43	openstackgerrit	Monty Taylor proposed openstack/keystoneauth master: Add paragraph clarifying major and micro versions https://review.openstack.org/482710
21:52:24	openstackgerrit	Merged openstack/keystoneauth master: Make Discover.version_data accept null max_version https://review.openstack.org/482250
22:03:04	openstackgerrit	Eric Fried proposed openstack/keystoneauth master: Update docs and add a release note https://review.openstack.org/477566
22:03:13	openstackgerrit	Merged openstack/keystoneauth master: Expand some discover.py docstrings https://review.openstack.org/482207
22:03:24	efried	mordred -^
22:05:57	openstackgerrit	Monty Taylor proposed openstack/keystoneauth master: Add paragraph clarifying major and micro versions https://review.openstack.org/482710
22:16:11	openstackgerrit	Colleen Murphy proposed openstack/keystoneauth master: Add paragraph clarifying major and micro versions https://review.openstack.org/482710
22:20:04	openstackgerrit	Merged openstack/keystoneauth master: normalize_version_number([1]) => (1, 0) and docs https://review.openstack.org/481309
#openstack-sdks - 2017-07-12
00:01:56	openstackgerrit	Monty Taylor proposed openstack/keystoneauth master: Add min_version and max_version to adapter constructors https://review.openstack.org/482742
00:01:57	openstackgerrit	Monty Taylor proposed openstack/keystoneauth master: Expose interface as a ListOpt for config files https://review.openstack.org/482743
00:01:58	openstackgerrit	Monty Taylor proposed openstack/keystoneauth master: Remove kwargs from adapter.get_endpoint_data https://review.openstack.org/482744
00:01:59	openstackgerrit	Monty Taylor proposed openstack/keystoneauth master: Add support for next_min_version and not_before https://review.openstack.org/482745
00:02:00	openstackgerrit	Monty Taylor proposed openstack/keystoneauth master: Add ability to request a microversion in a request https://review.openstack.org/482746
07:24:29	openstackgerrit	Akihiro Motoki proposed openstack/python-openstackclient master: DNM: test https://review.openstack.org/482827
07:25:32	openstackgerrit	M V P Nitesh proposed openstack/python-openstackclient master: List images with --tags for 'openstack image list' https://review.openstack.org/481615
07:46:11	amotoki	hm... osc unit test is now broken
07:53:09	amotoki	when both os-client-config and osc-lib are downgraded, osc unit test succeeded.
07:53:22	amotoki	is anyone investigating the detail?
07:53:47	amotoki	Good: osc-lib===1.6.0 and os-client-config===1.27.0
07:53:55	amotoki	Bad: osc-lib===1.7.0 and os-client-config===1.28.0
08:06:16	openstackgerrit	Rui Chen proposed openstack/python-openstackclient master: Clean up the changes of os.environ in functional tests https://review.openstack.org/471341
08:06:22	openstackgerrit	Rui Chen proposed openstack/python-openstackclient master: Rework server migrate to support cold and live https://review.openstack.org/460059
08:06:31	openstackgerrit	Rui Chen proposed openstack/python-openstackclient master: [Compute]Make column content readable for both human and machine https://review.openstack.org/445794
08:17:04	amotoki	the failure patterns related to osc-lib 1.7.0, osc-lib 1.7.0 is good. osc volume find_resource unit tests look bad.
08:18:10	amotoki	RuiChen: are you looking into the ut failure?
08:23:53	RuiChen	amotoki: not yet, that impact all the volume unit tests?
08:24:25	amotoki	RuiChen: it only affects openstackclient.tests.unit.volume.test_find_resource
08:25:16	RuiChen	amotoki: let me try it in my local env with libs
08:25:17	amotoki	RuiChen: looking at osc-lib change, it drops unnecessary try-except. according to osc-lib comments, it was a dirty-hack to hide test errors
08:25:51	amotoki	RuiChen: i am now looking at os-client-config failure mode to determine we need to block it in g-r.
08:50:35	amotoki	I filed bugs on the ut failures: bug 1703782 and bug 1703783 for tracking them.
08:50:36	openstack	bug 1703782 in python-openstackclient "OSC unit test volume.test_find_resource fails with osc-lib 1.7.0" [Undecided,New] https://launchpad.net/bugs/1703782
08:50:37	openstack	bug 1703783 in python-openstackclient "OSC unit fails with os-client-config 1.27.0" [Undecided,New] https://launchpad.net/bugs/1703783
09:16:17	amotoki	hm.... openstack command calls OpenStackConfig.__init__() TWICE... why?
09:17:11	amotoki	it hits os-client-config check and causes the UT failures. tricky :(
09:18:32	amotoki	I have no time to investigate more on OSC UT failure today.
09:32:51	amotoki	RuiChen: i identified the cause of the failure related to os-client-config. will upload a patch soon.
09:50:43	openstackgerrit	Akihiro Motoki proposed openstack/python-openstackclient master: Fix os-client-config related unit tests for occ 1.28.0 https://review.openstack.org/482880
10:01:46	openstackgerrit	Akihiro Motoki proposed openstack/python-openstackclient master: Fix os-client-config related unit tests for occ 1.28.0 https://review.openstack.org/482880
10:06:31	openstackgerrit	Akihiro Motoki proposed openstack/python-openstackclient master: Fix os-client-config related unit tests for occ 1.28.0 https://review.openstack.org/482880
10:08:07	RuiChen	amotoki: it's great, thank you pushing the patch, about the volume ut failures, I think it's related with https://review.openstack.org/#/c/476523/ changes
10:08:52	amotoki	RuiChen: it is not related to volume ut failures. it's related to the new os-client-config.
10:09:17	amotoki	RuiChen: ah. I misunderstood your comment.
10:09:40	amotoki	RuiChen: all are delimited by a comma. I got confused
10:10:12	RuiChen	I think your patch to fix the openstackclient.tests.unit.integ.cli.test_shell.TestIntegShellCliPrecedenceOCC failures, right?
10:10:53	amotoki	RuiChen: right
10:11:11	amotoki	also skips volume find_resource to pass py27.
10:13:07	RuiChen	amotoki: ah, I got it, thank you
10:18:50	openstackgerrit	Monty Taylor proposed openstack/keystoneauth master: Add min_version and max_version to adapter constructors https://review.openstack.org/482742
10:18:51	openstackgerrit	Monty Taylor proposed openstack/keystoneauth master: Expose interface as a ListOpt for config files https://review.openstack.org/482743
10:18:52	openstackgerrit	Monty Taylor proposed openstack/keystoneauth master: Add paragraph clarifying major and micro versions https://review.openstack.org/482710

Earlier   Later