Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2017-04-29
14:08:23 openstackgerrit Monty Taylor proposed openstack/service-types-authority master: Add an attribute for aliases https://review.openstack.org/460539
14:40:07 openstackgerrit Akihiro Motoki proposed openstack/python-openstacksdk master: DNM: test functional test using OS_CLOUD=devstack-admin https://review.openstack.org/461211
15:42:15 openstackgerrit Akihiro Motoki proposed openstack/python-openstacksdk master: functional tests: minor cleanup https://review.openstack.org/461211
15:45:04 openstackgerrit Monty Taylor proposed openstack/service-types-authority master: Add the rest of the services we know about https://review.openstack.org/461221
15:45:04 openstackgerrit Monty Taylor proposed openstack/service-types-authority master: Add validation and publication infrastructure https://review.openstack.org/461209
15:52:28 openstackgerrit Monty Taylor proposed openstack/service-types-authority master: Add the rest of the services we know about https://review.openstack.org/461221
16:05:25 openstackgerrit Monty Taylor proposed openstack/service-types-authority master: Add the rest of the services we know about https://review.openstack.org/461221
16:28:23 openstackgerrit Akihiro Motoki proposed openstack/python-openstackclient master: Network L3 Router Commands for OSC https://review.openstack.org/385729
16:29:18 openstackgerrit Akihiro Motoki proposed openstack/python-openstacksdk master: Use constraints environment for testing https://review.openstack.org/461225
17:00:50 openstackgerrit Akihiro Motoki proposed openstack/python-openstacksdk master: Network tag support https://review.openstack.org/461194
17:09:13 openstackgerrit Akihiro Motoki proposed openstack/python-openstacksdk master: Network tag support https://review.openstack.org/461194
17:17:24 openstackgerrit Akihiro Motoki proposed openstack/python-openstacksdk master: Network tag support https://review.openstack.org/461194
17:33:06 openstackgerrit Akihiro Motoki proposed openstack/python-openstacksdk master: Fix document warnings https://review.openstack.org/461228
17:35:01 openstackgerrit Akihiro Motoki proposed openstack/python-openstacksdk master: Network tag support https://review.openstack.org/461194
18:47:16 EmilienM dtroyer: I was afk yesterday afternoon - I have more logs now: http://logs.openstack.org/37/457037/4/check-tripleo/gate-tripleo-ci-centos-7-ovb-nonha/5bf6daf/console.html#_2017-04-28_17_01_45_634230
18:47:20 EmilienM it's in tripleoclient, i'm investigating
#openstack-sdks - 2017-05-01
09:23:58 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/461354
09:58:09 leeuwenrjj Hi, I just hit a bug and created a fix but I have a short question on the unit test since that did not catch the issue
09:59:05 leeuwenrjj The unit test for the network/v2/fakes.py for the port creates empty dictionaries e.g. for allowed_address_pairs
10:00:10 leeuwenrjj Because those empty values are not created in the real world some of the unit tests will work but in the real world they will break. See my change: https://review.openstack.org/#/c/461354/
10:00:51 leeuwenrjj So my question: should those unit tests be modified to not include empty dictionaries?
11:32:31 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:08:01 EmilienM dtroyer: hey, just fyi about the issue we had last week: https://review.openstack.org/#/c/461336
12:37:21 openstackgerrit Merged openstack/python-openstacksdk master: Fix document warnings https://review.openstack.org/461228
13:30:27 cdent mordred: you must be very proud of your giant bikeshed. so big and in need of so many different colors
13:39:07 openstackgerrit OpenStack Proposal Bot proposed openstack/osc-lib master: Updated from global requirements https://review.openstack.org/460257
13:40:54 openstackgerrit OpenStack Proposal Bot proposed openstack/python-openstackclient master: Updated from global requirements https://review.openstack.org/461387
13:40:57 openstackgerrit OpenStack Proposal Bot proposed openstack/python-openstacksdk master: Updated from global requirements https://review.openstack.org/461388
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: Nova-net functional tests: aggregates https://review.openstack.org/461491
19:09:26 openstackgerrit Dean Troyer proposed openstack/python-openstackclient master: Funcional tests: quota list https://review.openstack.org/461618
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

Earlier   Later