Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2017-05-22
19:29:47 openstackgerrit Merged openstack/openstackclient master: Explicitly depend on python-openstackclient https://review.openstack.org/439641
19:54:02 openstackgerrit Carlos Goncalves proposed openstack/python-openstackclient master: Add 'data_plane_status' option to Port classes https://review.openstack.org/458626
20:32:10 openstackgerrit Dean Troyer proposed openstack/python-openstackclient master: Create server with security group ID and name https://review.openstack.org/462076
20:41:43 openstackgerrit Merged openstack/python-openstackclient master: Make block-device-mapping more stable and clear https://review.openstack.org/438141
20:49:10 openstackgerrit Merged openstack/python-openstackclient master: Convert image functional tests into JSON format https://review.openstack.org/465203
21:28:01 openstackgerrit Merged openstack/cliff master: Updated from global requirements https://review.openstack.org/464363
22:08:59 openstackgerrit Merged openstack/python-openstackclient master: Correct the "extra spec" command openstack https://review.openstack.org/465122
22:19:38 openstackgerrit Dean Troyer proposed openstack/python-openstackclient master: JSON-ify image functional tests https://review.openstack.org/466926
22:48:31 openstackgerrit OpenStack Proposal Bot proposed openstack/openstackclient master: Updated from global requirements https://review.openstack.org/466932
23:09:03 openstackgerrit Merged openstack/python-openstackclient master: Create server with security group ID and name https://review.openstack.org/462076
23:35:59 openstackgerrit Merged openstack/python-openstackclient master: Refactor Extension show and list command https://review.openstack.org/449757
#openstack-sdks - 2017-05-23
00:53:37 openstackgerrit OpenStack Proposal Bot proposed openstack/python-openstackclient master: Updated from global requirements https://review.openstack.org/466954
02:53:26 openstackgerrit Dean Troyer proposed openstack/python-openstackclient master: JSON-ify image functional tests https://review.openstack.org/466926
03:17:25 openstackgerrit Reedip proposed openstack/python-openstackclient master: Add default-quota to subnet pool commands https://review.openstack.org/437367
09:00:00 openstackgerrit Hongbin Li proposed openstack/python-openstacksdk master: update params about cluster filter event https://review.openstack.org/467089
13:23:58 openstackgerrit Dean Troyer proposed openstack/python-openstackclient master: JSON-ify image functional tests https://review.openstack.org/466926
14:32:25 openstackgerrit Dean Troyer proposed openstack/python-openstackclient master: Add default-quota to subnet pool commands https://review.openstack.org/437367
15:17:00 openstackgerrit David Rabel proposed openstack/python-openstackclient master: Use _get_token_resource in role assignment list https://review.openstack.org/464684
15:52:43 openstackgerrit Dean Troyer proposed openstack/python-openstackclient master: JSON-ify image functional tests https://review.openstack.org/466926
16:52:09 openstackgerrit Merged openstack/python-openstackclient master: Updated from global requirements https://review.openstack.org/466954
18:45:18 hongbin hi osc team, a question. it looks nova has duplicated the floating ip api, i wonder if osc will keep or deprecate the related commands (i.e. "server add floating ip" and "server remove floating ip")
20:52:07 dtroyer hongbin: Have a look at the current release of OSC, we re-implemented the bits removed from novaclient 8.0 in OSC 3.10.0
20:52:27 hongbin dtroyer: ack
20:57:28 hongbin dtroyer: you mean this patch https://github.com/openstack/python-openstackclient/commit/763c8c5670f238920398165e670592e006213f32 ?
21:00:43 dtroyer No, I interpreted you to be asking about the bits removed in novaclient 8.0, those we did only a couple of weeks ago
21:02:11 hongbin dtroyer: sorry, i clarified
21:02:39 hongbin dtroyer: my question is about this api: https://github.com/openstack/python-novaclient/blob/603f0eae9f4c77b3e43a8c83eb797e857c75343d/novaclient/v2/servers.py#L908
21:03:23 hongbin dtroyer: it said it is deprecated, but it seems osc still uses it right now: https://github.com/openstack/python-openstackclient/blob/master/openstackclient/compute/v2/server.py#L266
21:03:47 hongbin dtroyer: i wonder how your team is going to deal with it
21:13:44 dtroyer Deprecation isn't removal, but we'll have to re-implement the library API similar to what we did for the other bits and call an appropriate microversion
22:01:29 hongbin dtroyer: ack. thanks for the information
22:22:47 openstackgerrit Dean Troyer proposed openstack/python-openstackclient master: Add default-quota to subnet pool commands https://review.openstack.org/437367
22:31:02 openstackgerrit Dean Troyer proposed openstack/python-openstackclient master: JSON-ify image functional tests https://review.openstack.org/466926
23:32:07 openstackgerrit Dean Troyer proposed openstack/python-openstackclient master: JSON-ify image functional tests https://review.openstack.org/466926
#openstack-sdks - 2017-05-24
00:53:17 openstackgerrit Akihiro Motoki proposed openstack/python-openstackclient master: volume functest: ensure snapshots deleted when volume delete https://review.openstack.org/466507
01:02:49 openstackgerrit Merged openstack/os-client-config master: Revert "Use interface not endpoint_type for keystoneclient" https://review.openstack.org/462144
01:14:28 openstackgerrit Merged openstack/osc-lib master: Updated from global requirements https://review.openstack.org/464442
01:18:39 openstackgerrit Le Hou proposed openstack/python-openstackclient master: test https://review.openstack.org/467429
01:21:12 openstackgerrit Ngo Quoc Cuong proposed openstack/keystoneauth master: Trivial fix typo in document https://review.openstack.org/467431
01:41:56 openstackgerrit Akihiro Motoki proposed openstack/python-openstackclient master: Network L3 Router Commands for OSC https://review.openstack.org/385729
01:45:46 openstackgerrit Akihiro Motoki proposed openstack/python-openstackclient master: Network L3 Router Commands for OSC https://review.openstack.org/385729
02:10:47 openstackgerrit chenyb4 proposed openstack/python-openstacksdk master: Fix cluster action list filter https://review.openstack.org/467444
10:26:27 openstackgerrit Merged openstack/keystoneauth master: Trivial fix typo in document https://review.openstack.org/467431
16:14:48 openstackgerrit Dean Troyer proposed openstack/python-openstackclient master: JSON-ify image functional tests https://review.openstack.org/466926
17:13:53 openstackgerrit Reedip proposed openstack/python-openstackclient master: Add default-quota to subnet pool commands https://review.openstack.org/437367
17:58:31 openstackgerrit Monty Taylor proposed openstack/api-wg master: Add guidelines on Version Discovery https://review.openstack.org/459710
17:58:31 openstackgerrit Monty Taylor proposed openstack/api-wg master: Add support for historical service type aliases https://review.openstack.org/460654
17:58:32 openstackgerrit Monty Taylor proposed openstack/api-wg master: Describe the publication of service-types-authority data https://review.openstack.org/462815
17:58:32 openstackgerrit Monty Taylor proposed openstack/api-wg master: Add document describing consuming version discovery https://review.openstack.org/459405
17:58:33 openstackgerrit Monty Taylor proposed openstack/api-wg master: Add guideline about consuming endpoints from catalog https://review.openstack.org/462814
19:17:03 openstackgerrit Merged openstack/python-openstackclient master: Use _get_token_resource in role assignment list https://review.openstack.org/464684
19:17:53 openstackgerrit Merged openstack/python-openstackclient master: JSON-ify image functional tests https://review.openstack.org/466926
19:29:04 cdent elmiko, edleafe : slim chance I might miss api-wg meeting tomorrow. definitely won't be able to be the newsletter guy; have an evening thing that I'm not sure when it starts
20:13:57 edleafe cdent: I should be able to handle things
20:14:05 cdent thanks edleafe
20:57:14 elmiko cdent: ack
#openstack-sdks - 2017-05-25
02:09:53 openstackgerrit Hong Hui Xiao proposed openstack/python-openstackclient master: Don't show hint about vlan transparent in network set https://review.openstack.org/466002
04:52:31 openstackgerrit Reedip proposed openstack/python-openstackclient master: Add default-quota to subnet pool commands https://review.openstack.org/437367
08:46:15 openstackgerrit Rui Chen proposed openstack/python-openstackclient master: Rework server migrate to support cold and live https://review.openstack.org/460059
10:43:56 openstackgerrit Sean Dague proposed openstack/keystoneauth master: Add global_request_id param to Adapter constructor https://review.openstack.org/467991
10:44:25 sdague jamielennox: if you happen to still be awake ^^^^ would be great to have eyes on
11:35:03 jamielennox sdague: that spec looks remarkably like what request id was supposed to be before it got all committeed
11:40:42 sdague jamielennox: yeh, well what is old is new again?
11:41:13 jamielennox it's nice to see some sanity being restored
11:41:29 sdague so, all that previous work predates me, but we seemed to get agreement at the summit, so I'm running at it as fast as I can while we have that
11:42:19 sdague having the 2 ids does end up being useful, so you can distinguish the fact that there are a lot of cross calls some times
11:42:53 sdague the oslo side changes are all landed and released, so now it's just plumbing clients and callers
11:44:12 jamielennox sdague: commented, code feels odd, i don't mind which way it's fixed
11:44:44 jamielennox sdague: it does feel weird to be setting this on the Adapter, adapters are created per client so you're not going to share that request_id across a lot of connections that way
11:45:51 jamielennox though with sessions lasting a long time there's not really a good spot to put a request level middleware in currently
11:47:38 sdague jamielennox: ok, I'm not sure I quite understand the concerns there
11:48:15 sdague because we don't want to be sharing request_id more broadly, the important thing is that it's relevant to the flow right now
11:49:06 jamielennox note the way i always hoped this would work is that auth_token middleware currently passes down an auth plugin. that auth plugin when used to contact another service does all the correct things like uses a service token and presents the right catalog and stuff
11:49:30 jamielennox and would be able to pass through a request-id that it received to the auth plugin and auto end up in other requests
11:49:48 jamielennox but it was super slow work getting people to adopt that and it didn't get very far
11:51:00 jamielennox sdague: yea, if it's useful there then that's fine. Re code, i just mean to store the variable in one place. I don't mind if it's a property around additional_headers or if you manually add it to headers later like we do for user_agent, just not stored twice
11:52:32 sdague ok
12:09:22 sdague jamielennox: for reference - https://review.openstack.org/#/c/467242/ is what the nova change looks like with some passing on to clients
12:10:06 sdague https://review.openstack.org/#/c/467417/ is it all glued together, you'll see some global_ids logged in cinder / neutron with that
12:10:10 jamielennox sdague: yea, you can get a fair way with additional_headers now, but i'm happy to make it a real thing
12:12:00 sdague jamielennox: yeh, I've been just direct plumbing into neutron/cinder/glance clients, but the neutron folks wanted it in keystonauth. I'm kind of fine either way, because it's pretty much just a single field to pop out on the constructor if it's totally at the top level
12:12:51 sdague even though we're setting global_request_id in constructors quite often in time, in code, it's typically consolidated to a single location in the caller
12:27:39 openstackgerrit Matthew Booth proposed openstack/os-client-config master: Don't pop from os.environ https://review.openstack.org/468020
12:47:13 openstackgerrit Sean Dague proposed openstack/keystoneauth master: Add global_request_id param to Adapter constructor https://review.openstack.org/467991
12:53:01 openstackgerrit Rui Chen proposed openstack/python-openstackclient master: Rework server migrate to support cold and live https://review.openstack.org/460059
13:31:55 openstackgerrit Monty Taylor proposed openstack/os-client-config master: Don't pop from os.environ https://review.openstack.org/468020
13:32:01 mordred Shrews, dtroyer: if you get time, I'd really like to get the top five patches from https://review.openstack.org/#/q/project:openstack/os-client-config+status:open+label:Workflow=0 landed and a new occ out - they're all small/easy
13:32:16 mordred Shrews, dtroyer: but there are a few behavior bugs that are biting people
13:38:10 Shrews mordred: k
13:42:07 Shrews mordred: is https://review.openstack.org/465195 still relevant with https://review.openstack.org/468020 ?
13:42:54 mordred Shrews: the reason for writing it isn't - but it's still a good idea I think - the OpenStackConfig object is really intended to be a singleton
15:19:49 openstackgerrit Reedip proposed openstack/python-openstackclient master: Introduce quota unset command https://review.openstack.org/376311
15:34:29 openstackgerrit Reedip proposed openstack/python-openstackclient master: Add extra dhcp option to 'port create/set/unset' https://review.openstack.org/356263
15:36:57 openstackgerrit Monty Taylor proposed openstack/api-wg master: Add guidelines on Version Discovery https://review.openstack.org/459710
15:36:58 openstackgerrit Monty Taylor proposed openstack/api-wg master: Describe the publication of service-types-authority data https://review.openstack.org/462815
15:36:58 openstackgerrit Monty Taylor proposed openstack/api-wg master: Add document describing consuming version discovery https://review.openstack.org/459405
15:37:32 mordred cdent, edleafe, elmiko: I believe I've gotten all current reviews taken care of
15:37:43 cdent yay!
15:38:18 edleafe mordred: on my list
15:46:52 elmiko mordred: sweet, i'm rounding the corner on my last lap for the reviews you posted

Earlier   Later