Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2017-05-01
13:52:23 openstackgerrit OpenStack Proposal Bot proposed openstack/osc-lib master: Updated from global requirements https://review.openstack.org/460257
14:05:40 openstackgerrit OpenStack Proposal Bot proposed openstack/openstackclient master: Updated from global requirements https://review.openstack.org/440080
14:06:03 openstackgerrit OpenStack Proposal Bot proposed openstack/osc-lib master: Updated from global requirements https://review.openstack.org/460257
14:07:50 openstackgerrit OpenStack Proposal Bot proposed openstack/python-openstackclient master: Updated from global requirements https://review.openstack.org/461387
14:23:45 mordred cdent: I lke to flummox the color-blind
14:24:14 cdent sometimes you have to be cruel to be kind
14:24:45 mordred dtroyer: I believe I just figured out an end-to-end design and plan for the occ 2.0 stuff we chatted about in atlanta and how to do it without breaking anyone - I'm going to write it all down in narrative form for a sanity check - but I think it actually won't be bad to implement
14:50:52 EmilienM dtroyer: fyi https://review.openstack.org/#/c/461069/1/osc_lib/shell.py@a440
15:03:16 openstackgerrit Merged openstack/python-openstacksdk master: Updated from global requirements https://review.openstack.org/461388
15:04:46 openstackgerrit Merged openstack/python-openstacksdk master: Port metric v1 to resource2 https://review.openstack.org/460673
15:05:16 openstackgerrit Merged openstack/python-openstacksdk master: Port unversioned Version resources to resource2 https://review.openstack.org/460735
15:27:28 dtroyer EmilienM: thanks, commented
15:28:32 EmilienM dtroyer: ok thanks, have you seen https://review.openstack.org/#/c/461336/ ?
15:38:36 dtroyer commented there too. there's a bit of unnecessary stuff going on in there
15:57:58 EmilienM dtroyer: thx
16:10:29 openstackgerrit Robert van Leeuwen proposed openstack/python-openstackclient master: Fixes osc port set allowed-address for ports without previous set allowed address https://review.openstack.org/461364
16:23:49 openstackgerrit Merged openstack/python-openstackclient master: Updated from global requirements https://review.openstack.org/461387
19:39:04 openstackgerrit Dean Troyer proposed openstack/python-openstackclient master: Nova-net functional tests: aggregates https://review.openstack.org/461491
22:21:24 openstackgerrit Eric Fried proposed openstack/keystoneauth master: Introduce keystoneauth1.loading.adapter https://review.openstack.org/460337
23:11:24 openstackgerrit Merged openstack/os-client-config master: Fix interactions with keystoneauth from newton https://review.openstack.org/461114
#openstack-sdks - 2017-05-02
00:41:32 jamielennox efried: assume you've left, why use the oslo conf options instead of the keystoneauth options in https://review.openstack.org/#/c/460337/3/keystoneauth1/loading/adapter.py ?
00:42:07 efried jamielennox Is that not what I'm doing?
00:43:01 jamielennox efried: so in keystoneauth we define our own option type: https://github.com/openstack/keystoneauth/blob/master/keystoneauth1/loading/opts.py
00:43:28 jamielennox the idea being that these things can be available on CLI, oslo.config, and in the weird ways glance does things
00:44:01 efried jamielennox Ohh, I see now.
00:44:17 jamielennox i mean there might be no reason to provide adapter options via CLI
00:44:36 efried jamielennox I'm missing this line: https://review.openstack.org/#/c/460337/3/keystoneauth1/loading/session.py@159
00:45:08 efried jamielennox Yeah, not providing CLI options right now, but that doesn't mean we never will. And I should do it consistently with how loading.session is.
00:45:23 jamielennox efried: so session is not necessarily the best one to copy
00:45:37 jamielennox or maybe it is :)
00:46:05 efried Waidaminute - I am already doing cfg = _utils.get_oslo_config()
00:46:22 efried jamielennox https://review.openstack.org/#/c/460337/3/keystoneauth1/loading/adapter.py@51
00:46:50 efried So okay, you're saying I should *not* do that the same way as session does? /me confused.
00:47:01 jamielennox efried: so the idea with all of this loading as mentioned is that there are a number of ways you can do it, one of which is oslo.config
00:47:40 jamielennox if you look at the most simple plugin you see it returns keystoneauth opts from get_options: https://github.com/openstack/keystoneauth/blob/master/keystoneauth1/loading/_plugins/admin_token.py#L33
00:49:09 jamielennox then when dealing with oslo.config specifically it converts the ksa opt to an oslo opt: https://github.com/openstack/keystoneauth/blob/master/keystoneauth1/loading/conf.py#L128
00:49:29 jamielennox this means that os-c-c can use ksa loading without worrying about oslo.config
00:50:39 jamielennox now there are times where you want oslo.config to behave differently to just the standard ksa opts, and you can override it with the oslo_config_opts that you're doing - and that's fine, that's what session does because we matched old variables names to new variable names
00:51:28 jamielennox just in this case it looks like a really simple pass through (you're not manipulating variables), so you could probably just do it as ksa opts
00:51:48 jamielennox so i'm just wondering if that's planned or just the result of looking at session
00:52:05 efried jamielennox Result of looking at session, for sure.
00:52:59 efried jamielennox Though looking at loading.conf, it appears to be kinda auth-specific.
00:53:24 jamielennox efried: yea, i'm just having another look through that as well
00:53:35 jamielennox it was definitely designed around auth
00:54:27 jamielennox i never considered non-auth loading at the time
00:55:26 efried Okay.
00:55:52 efried jamielennox I do have to bail before my wife gets steamed. If you want to leave comments in the review, or email me, or something, we can hash it out.
00:56:13 jamielennox efried: yea, i'll try and have a closer look, just thought i'd see if you were here
00:56:21 jamielennox thanks
04:27:38 openstackgerrit Dean Troyer proposed openstack/python-openstackclient master: Funcional tests: quota list https://review.openstack.org/461618
05:08:59 openstackgerrit Robert van Leeuwen proposed openstack/python-openstackclient master: Fixes osc port set allowed-address for ports without previous set allowed address https://review.openstack.org/461364
12:34:49 efried jamielennox You still around?
13:26:46 mordred efried: he's in australia - so usually will only overlap early in the US morning or at the end of the day
13:27:07 efried mordred Yeah, figured as much, but was worth a shot ;-)
13:27:26 efried mordred If you have the cycles today, wanna take a look at those two changes?
13:27:45 efried ksa [https://review.openstack.org/460337] and nova [https://review.openstack.org/458257]
13:29:05 mordred efried: yes! I have a panel session in 45 minutes, but after that I totally will
13:29:20 efried mordred Thanks!
15:10:32 openstackgerrit Merged openstack/keystoneauth master: Introduce keystoneauth1.loading.adapter https://review.openstack.org/460337
15:32:55 openstackgerrit Dean Troyer proposed openstack/python-openstackclient master: Functional tests: Identity v2 and DevStack https://review.openstack.org/461826
16:57:50 openstackgerrit Ed Leafe proposed openstack/api-wg master: Microversions: add next_min_version field in version body https://review.openstack.org/446138
17:02:27 openstackgerrit Ed Leafe proposed openstack/api-wg master: Microversions: add signal when raising minimum version https://review.openstack.org/446138
19:09:26 openstackgerrit Dean Troyer proposed openstack/python-openstackclient master: Funcional tests: quota list https://review.openstack.org/461618
19:09:26 openstackgerrit Dean Troyer proposed openstack/python-openstackclient master: Nova-net functional tests: aggregates https://review.openstack.org/461491
20:17:53 openstackgerrit Merged openstack/python-openstackclient master: Functional tests: Identity v2 and DevStack https://review.openstack.org/461826
22:08:41 openstackgerrit Ed Leafe proposed openstack/api-wg master: Microversions: add next_min_version field in version body https://review.openstack.org/446138
#openstack-sdks - 2017-05-03
00:46:50 EmilienM dtroyer: I found a new issue, this time I'm not sure it's in tripleo
00:46:55 EmilienM http://logs.openstack.org/37/457037/7/check-tripleo/gate-tripleo-ci-centos-7-ovb-nonha/5f1ec27/console.html#_2017-05-02_22_26_46_061856
00:47:15 EmilienM I'm investigating it but if you know :)
00:50:11 EmilienM I'm not sure but I think about https://github.com/openstack/os-client-config/commit/e6755872ada4978f585bdf15edf623dbcf72c4ee
00:57:20 openstackgerrit Emilien Macchi proposed openstack/os-client-config master: DO NOT MERGE - Revert "Remove the keystoneclient auth fallback" https://review.openstack.org/461945
00:59:41 umbSublime Hi quick question, when using the python openstacksdk, what is equivalent to being an admin and running "nova list --all-tenants". When I use conn.compute.servers(), I get only the instances from my admin tenant_id
01:01:37 umbSublime same goes for conn.compute.find_server(name_or_id), it works fine no matter the tenant if I use a uuid for name_or_id param, but if I use name it can only find the instances from my own tenant_id
01:39:58 openstackgerrit Merged openstack/python-openstackclient master: Nova-net functional tests: aggregates https://review.openstack.org/461491
01:50:36 openstackgerrit Merged openstack/python-openstackclient master: Funcional tests: quota list https://review.openstack.org/461618
05:09:37 openstackgerrit Merged openstack/python-openstackclient master: Explicitly set 'builders' option https://review.openstack.org/457960
08:28:53 openstackgerrit Rodolfo Alonso Hernandez proposed openstack/python-openstacksdk master: Add direction field to QoS bandwidth limit. https://review.openstack.org/356512
09:31:36 ralonsoh briancurtin: hi, can you review again https://review.openstack.org/#/c/356512/?
09:31:47 ralonsoh briancurtin: thank you in advance
09:43:22 ralonsoh dtroyer: hi
09:44:06 ralonsoh dtroyer: https://review.openstack.org/#/c/356512/ depending patch is merged. And https://review.openstack.org/#/c/458056/ is also waiting for this one
09:44:21 ralonsoh dtroyer: can you take a look?
09:44:29 ralonsoh dtroyer: thank you in advance
11:01:47 openstackgerrit Rodolfo Alonso Hernandez proposed openstack/python-openstackclient master: Add direction field to QoS bandwidth limit. https://review.openstack.org/458056
11:04:01 openstackgerrit Rui Chen proposed openstack/python-openstackclient master: Create server with security group ID and name https://review.openstack.org/462076
11:26:42 briancurtin umbSublime: probably servers(all_tenants=True)
11:39:46 openstackgerrit Merged openstack/python-openstacksdk master: Add direction field to QoS bandwidth limit. https://review.openstack.org/356512
12:06:40 openstackgerrit Monty Taylor proposed openstack/os-client-config master: WIP Braindump thoughts and a tenantive plan for v2 https://review.openstack.org/462092
12:21:19 openstackgerrit OpenStack Proposal Bot proposed openstack/osc-lib master: Updated from global requirements https://review.openstack.org/462114
12:23:15 openstackgerrit OpenStack Proposal Bot proposed openstack/python-openstackclient master: Updated from global requirements https://review.openstack.org/462125
12:23:18 openstackgerrit OpenStack Proposal Bot proposed openstack/python-openstacksdk master: Updated from global requirements https://review.openstack.org/462126
12:34:24 openstackgerrit Monty Taylor proposed openstack/service-types-authority master: Add the rest of the services we know about https://review.openstack.org/461221
12:34:25 openstackgerrit Monty Taylor proposed openstack/service-types-authority master: Changes types of the three services that manage systems https://review.openstack.org/462137
12:36:17 openstackgerrit Monty Taylor proposed openstack/service-types-authority master: Change default swift name to object-storage https://review.openstack.org/462138
12:38:29 openstackgerrit Monty Taylor proposed openstack/service-types-authority master: Change rating to usage-report https://review.openstack.org/462139
12:41:37 openstackgerrit Monty Taylor proposed openstack/service-types-authority master: Add placement service https://review.openstack.org/462140
12:53:16 openstackgerrit Merged openstack/python-openstacksdk master: Updated from global requirements https://review.openstack.org/462126
13:11:11 openstackgerrit Jordan Pittier proposed openstack/os-client-config master: Revert "Use interface not endpoint_type for keystoneclient" https://review.openstack.org/462144
13:11:37 openstackgerrit Jordan Pittier proposed openstack/os-client-config master: Revert "Use interface not endpoint_type for keystoneclient" https://review.openstack.org/462144
13:16:05 openstackgerrit Jordan Pittier proposed openstack/os-client-config master: Revert "Use interface not endpoint_type for keystoneclient" https://review.openstack.org/462144

Earlier   Later