Posted	Nick	Remark
#openstack-sdks - 2018-03-13
06:49:16	openstackgerrit	OpenStack Proposal Bot proposed openstack/cliff master: Updated from global requirements https://review.openstack.org/548672
06:56:30	openstackgerrit	OpenStack Proposal Bot proposed openstack/keystoneauth master: Updated from global requirements https://review.openstack.org/549536
07:05:01	openstackgerrit	Peter Balogh proposed openstack/python-openstacksdk master: Fix TypeError in case of FloatingIP add and remove https://review.openstack.org/552112
07:18:53	openstackgerrit	OpenStack Proposal Bot proposed openstack/openstackclient master: Updated from global requirements https://review.openstack.org/521902
07:28:22	openstackgerrit	OpenStack Proposal Bot proposed openstack/python-openstackclient master: Updated from global requirements https://review.openstack.org/544222
07:28:32	openstackgerrit	OpenStack Proposal Bot proposed openstack/python-openstacksdk master: Updated from global requirements https://review.openstack.org/552385
09:59:31	openstackgerrit	Jens Harbott (frickler) proposed openstack-infra/shade master: Fix devstack jobs to use neutron-qos properly https://review.openstack.org/552459
10:28:05	openstackgerrit	Jens Harbott (frickler) proposed openstack/python-openstackclient master: Re-implement novaclient bits removed in 10.0 https://review.openstack.org/550105
10:28:06	openstackgerrit	Jens Harbott (frickler) proposed openstack/python-openstackclient master: Adapt service names to new neutron-* scheme https://review.openstack.org/552469
10:31:27	frickler	dtroyer: FYI, tried to fix your unittests a bit ^^ also hoping that devstack works better this way, though it might be we have to squash these two together, too
11:16:35	rcernin	hi, who is the right person to answer diff in design between legacy and new openstack cli? I wonder why we got rid of --fields?
13:07:06	openstackgerrit	Jens Harbott (frickler) proposed openstack/python-openstackclient master: Re-implement novaclient bits removed in 10.0 https://review.openstack.org/550105
13:43:00	dtroyer	frickler: thank you! I was working on a different approach, a bit simpler I think, but had not considered multiple inheritance for the test class, that simplifies it even more
13:43:23	dtroyer	I'm going to let the tests run first, then squash in the zuul fix too
13:52:09	frickler	dtroyer: we may not need the zuul fix immediately, I've agreed with andreaf to go back to using q-* services for now, see https://review.openstack.org/552503 .
13:53:15	dtroyer	ah, ok. damn, it's almost 3 years since that change-over started
13:55:04	frickler	yeah, would still be good to make the switch, just without blocking everyone else, that's why I proposed the dedicated job
13:56:45	frickler	dtroyer: did you also look at the "--insecure vs. sdk_connection" issue? I'm wondering what would be the best way to set up a functional test exposing it
13:57:44	dtroyer	frickler: I have not
14:06:16	openstackgerrit	Dean Troyer proposed openstack/python-openstackclient master: Re-implement novaclient bits removed in 10.0 https://review.openstack.org/550105
14:09:41	frickler	dtroyer: o.k., that is even nicer. now if you depend-on https://review.openstack.org/552503 it should hopefully even be passing all tests
14:10:40	dtroyer	doh, thanks
14:11:23	openstackgerrit	Dean Troyer proposed openstack/python-openstackclient master: Re-implement novaclient bits removed in 10.0 https://review.openstack.org/550105
16:27:26	openstackgerrit	Harald Jensås proposed openstack/python-openstackclient master: Allow setting network-segment on subnet update https://review.openstack.org/523974
17:58:34	frickler	dtroyer: if you are still there can you re-do your +W on https://review.openstack.org/550105 ? the dependency merged later, seems like zuul doesn't pick that up. would avoid a recheck and running the check queue again
18:13:09	dtroyer	frickler: done, there it goes
18:28:29	openstackgerrit	Hongbin Lu proposed openstack/cliff master: Remove the warning of getargspec removal https://review.openstack.org/552652
19:54:09	openstackgerrit	Merged openstack/python-openstackclient master: Re-implement novaclient bits removed in 10.0 https://review.openstack.org/550105
19:54:35	frickler	\o/
19:55:18	dtroyer	\o/
19:56:38	dtroyer	I think I have queens sorted too
20:02:44	dtroyer	cdent: my wife was not impressed with the OSAF squadron… although she did give you bonus points for the snow angel :)
20:03:08	cdent	Clearly she has excellent taste.
23:19:40	openstackgerrit	Merged openstack/python-openstackclient master: Updated from global requirements https://review.openstack.org/544222
#openstack-sdks - 2018-03-14
00:22:06	openstackgerrit	Dean Troyer proposed openstack/osc-lib master: Set defaults in ClientManager tests https://review.openstack.org/552716
00:37:31	openstackgerrit	Merged openstack/python-openstackclient master: Cleanup error messages on failure https://review.openstack.org/548656
01:33:30	openstackgerrit	Dongcan Ye proposed openstack/python-openstackclient master: Remove duplicated network attributes https://review.openstack.org/547703
02:16:37	openstackgerrit	Merged openstack/python-openstackclient master: Add CRUD support for application credentials https://review.openstack.org/536163
05:11:41	openstackgerrit	Merged openstack/osc-lib master: Set defaults in ClientManager tests https://review.openstack.org/552716
07:28:19	openstackgerrit	Pavlo Shchelokovskyy proposed openstack/osc-lib master: Allow to use the none auth plugin https://review.openstack.org/512699
08:35:30	openstackgerrit	Pavlo Shchelokovskyy proposed openstack/os-client-config master: Use 'none' auth plugin https://review.openstack.org/552837
13:06:35	openstackgerrit	Merged openstack/python-openstacksdk master: Run examples tests with functional tests https://review.openstack.org/540374
16:03:35	openstackgerrit	Dean Troyer proposed openstack/python-openstackclient master: Make Profile fallback go bye-bye https://review.openstack.org/540541
16:35:57	openstackgerrit	Andrey Volkov proposed openstack/cliff master: Allow finding command by partial name https://review.openstack.org/552970
18:11:08	cdent	edleafe, elmiko, dtantsur|afk: the just posted PTG cells summary includes the statement "An attempt to delete an instance in a "down" cell should result in a 500 or 503 error."
18:12:13	edleafe	oh dear
18:12:22	cdent	since cell being down is a sign of "broken" it _might_ be correct
18:12:48	cdent	but reflecting that broken-ness as the web server bring broken is not really what we have said is good
18:13:45	cdent	I'll respond as such unless somebody else wants to do it or something?
18:19:45	edleafe	I can do it
18:20:08	cdent	I've already started
18:20:34	edleafe	cool
18:28:54	cdent	sent
18:48:52	elmiko	cdent: ack, thanks for the update
21:30:27	openstackgerrit	Hongbin Lu proposed openstack/cliff master: Remove the warning of getargspec removal https://review.openstack.org/552652
#openstack-sdks - 2018-03-15
04:12:07	openstackgerrit	Lingxian Kong proposed openstack/service-types-authority master: Add qinling project https://review.openstack.org/550787
06:44:06	openstackgerrit	OpenStack Proposal Bot proposed openstack-infra/shade master: Updated from global requirements https://review.openstack.org/552305
06:49:31	openstackgerrit	OpenStack Proposal Bot proposed openstack/cliff master: Updated from global requirements https://review.openstack.org/548672
07:42:47	openstackgerrit	OpenStack Proposal Bot proposed openstack/openstackclient master: Updated from global requirements https://review.openstack.org/521902
07:43:16	openstackgerrit	OpenStack Proposal Bot proposed openstack/os-api-ref master: Updated from global requirements https://review.openstack.org/553242
07:44:07	openstackgerrit	OpenStack Proposal Bot proposed openstack/os-client-config master: Updated from global requirements https://review.openstack.org/551595
07:44:54	openstackgerrit	OpenStack Proposal Bot proposed openstack/os-service-types master: Updated from global requirements https://review.openstack.org/551598
07:46:45	openstackgerrit	OpenStack Proposal Bot proposed openstack/osc-lib master: Updated from global requirements https://review.openstack.org/551601
08:02:22	openstackgerrit	OpenStack Proposal Bot proposed openstack/python-openstackclient master: Updated from global requirements https://review.openstack.org/553257
08:02:45	openstackgerrit	OpenStack Proposal Bot proposed openstack/python-openstacksdk master: Updated from global requirements https://review.openstack.org/552385
08:08:32	openstackgerrit	Merged openstack/os-api-ref master: Updated from global requirements https://review.openstack.org/553242
08:35:35	openstackgerrit	Andrey Volkov proposed openstack/cliff master: Allow finding command by partial name https://review.openstack.org/552970
10:00:51	niraj_singh	hi i am getting AttributeError: Service Descriptors cannot be set error
10:01:04	niraj_singh	i dont understand the error please help
12:09:32	openstackgerrit	Chris Dent proposed openstack/microversion-parse master: Provide a helper method to get headers from environ https://review.openstack.org/495356
12:09:33	openstackgerrit	Chris Dent proposed openstack/microversion-parse master: Import functions from placement https://review.openstack.org/496212
12:09:33	openstackgerrit	Chris Dent proposed openstack/microversion-parse master: Improve Version.matches() method https://review.openstack.org/550251
12:09:34	openstackgerrit	Chris Dent proposed openstack/microversion-parse master: Add MicroversionMiddleware https://review.openstack.org/550252
12:09:34	openstackgerrit	Chris Dent proposed openstack/microversion-parse master: Allow passing a json_error_formatter to the middleware https://review.openstack.org/550263
12:09:35	openstackgerrit	Chris Dent proposed openstack/microversion-parse master: Clean up package version and python version support https://review.openstack.org/550264
12:10:20	cdent	elmiko, edleafe: I'd like to merge that ^ stuff to make a release of microversion parse that has it, but it needs some review and probably a bit of tuneup to dust the cdent-ness off it.
12:22:12	elmiko	cdent: ack, i'll add it to my queue for today
12:23:00	openstackgerrit	Emilien Macchi proposed openstack/python-openstackclient master: neutron: add --mtu for create/set network https://review.openstack.org/553374
12:23:28	cdent	thanks elmiko
12:23:42	cdent	I probably use %s a lot so you'll love it
12:57:20	edleafe	cdent: Can you refresh my memory about extract_version? https://review.openstack.org/#/c/496212/4/microversion_parse/__init__.py@165
12:57:37	edleafe	Where is the 'versions_list' supposed to come from?
12:58:59	cdent	see line 58 on this DNM of placement using these changes https://review.openstack.org/#/c/550265/3/nova/api/openstack/placement/deploy.py
12:59:32	cdent	in placement as it stands right now the VERSIONS are used directly, but since as a library we wouldn't have access to that global, we have to pass it in somehow
13:01:08	edleafe	What is the need to have this check in the library? IOW, why can't the lib return the version, and have the caller do whatever checking it might need?
13:04:35	cdent	It's a) a convenience, b) later the MicroversionMiddleware uses the method and there is no interaction with the caller
13:05:01	cdent	later == two patches further in the stack
13:06:17	cdent	edleafe: the patches you're looking at so far are what amount to a "middleware construction kit". Most people will not use any of these features if they use the middleware
13:06:44	edleafe	got it
13:07:10	edleafe	but I am still not clear on what is supposed to be authoritative when it comes to "acceptable versions"
13:07:31	edleafe	how do you see this middleware knowing that for a service?
13:08:05	cdent	that version_list, which is a thing like: https://github.com/openstack/nova/blob/master/nova/api/openstack/placement/microversion.py#L36
13:08:34	cdent	the DNM think I linked before passes VERSIONS into the middleware
13:10:14	edleafe	I saw that; I'm still not clear on the benefit of passing that from the service to the middleware
13:10:42	edleafe	With it, every service using the middleware has to implement a versions list and pass it during setup
13:11:07	edleafe	Without it, services that care about checking specific versions can do so locally
13:12:11	cdent	the job of the middleware is twofold:
13:12:31	cdent	extract a version and put it in the environ to be used by the app
13:12:51	cdent	to return a 4xx is the microversion is malformed or out of range
13:13:17	cdent	in order for it to "out of range" is has to know the range

Earlier   Later