Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2017-07-17
16:15:15 cdent yes, totally cool with that
16:15:38 mordred jamielennox: speaking of - when you get in, please ping me - I've got a 'fun' challenge - I want to use requests_mock and cachecontrol on the same library, and both are transport adapters which I can't stack
16:16:55 mordred jamielennox: (we want to wind up with requests.session being mocked by requests_mock and cachecontrol then being mounted on that so that cachecontrol is operating the whole time but when cachecontrol decides that it needs to make a remote call it hits requests_mock
16:18:21 openstackgerrit Gage Hugo proposed openstack/keystoneauth master: Enable sphinx todo extension https://review.openstack.org/484417
16:36:00 openstackgerrit Monty Taylor proposed openstack/service-types-authority master: Remove leading openstack/ from project name https://review.openstack.org/484422
17:34:07 openstackgerrit Monty Taylor proposed openstack/os-service-types master: Add support for getting OpenStack project information https://review.openstack.org/484435
17:35:37 mordred dhellmann, efried: ^^ that should help both the docs job (pulled in doug's method from that patch) and also the microversion header patch
17:37:58 efried mordred Is there a test for get_service_data?
17:38:18 mordred efried: thanks - I missed that one - one sec
18:20:33 openstackgerrit Monty Taylor proposed openstack/os-service-types master: Add support for getting OpenStack project information https://review.openstack.org/484435
18:20:34 openstackgerrit Monty Taylor proposed openstack/os-service-types master: Refactor bulk of data checking into mixin class https://review.openstack.org/484445
18:21:01 mordred efried: ok. added that test - and another missing test - then got annoyed and refactored a chunk of copy-pasta
18:23:54 openstackgerrit Monty Taylor proposed openstack/service-types-authority master: Remove leading openstack/ from project name https://review.openstack.org/484422
18:24:14 mordred efried: k - rebased that ^^ it's not actually related to the trailing-whitespace
19:14:17 lbragstad curious if i could get a quick api change sanity check here - https://review.openstack.org/#/c/482190/3
19:14:40 lbragstad i'm just wondering if ^ is ok to do or not and looking for some advice
20:02:28 openstackgerrit Monty Taylor proposed openstack/os-service-types master: Add support for getting a complete list of type names https://review.openstack.org/484473
20:03:31 openstackgerrit Monty Taylor proposed openstack/keystoneauth master: Add support for service type aliases https://review.openstack.org/484341
20:03:32 openstackgerrit Monty Taylor proposed openstack/keystoneauth master: Fix masked variable name https://review.openstack.org/484340
20:04:16 mordred efried, cmurphy: ^^ that should take care of service-type alias support AND supporting old-style microversion headers AND supporting services, like cinder, that have a different microversion name than official service-type
20:06:31 cmurphy mordred: o7
21:04:04 mordred cmurphy: oh - btw - I made you an initial core on os-service-types since you have context paged in for it
21:04:41 cmurphy mordred: oh neato
21:51:28 openstackgerrit Dean Troyer proposed openstack/python-openstackclient master: Fix column names for server list --no-name-lookup https://review.openstack.org/484500
22:52:29 johnsom Hi folks, FYI, I am going to try to pick up the octavia/load-balancer SDK patches
22:52:53 johnsom Reading the contributor docs now... grin
22:56:34 johnsom Should I use allow_retrieve or allow_get? I see the docs say allow_retrieve but some, neutron ports are using allow_get
23:05:44 johnsom Digging I see a bunch of stuff moving to resource2 so I guess that is the path forward.
23:24:58 openstackgerrit Monty Taylor proposed openstack/keystoneauth master: Remove deprecated_since parameter for interface https://review.openstack.org/484528
23:27:43 efried So mordred I've been banging my head against these same couple of test cases for the past couple of days. I'm at the point now where I'm actually working to understand all the logic in the new discovery algorithms.
23:28:27 efried mordred I'm pretty desperate not to have to support version+min_version+max_version in all these new APIs.
23:28:37 openstackgerrit Monty Taylor proposed openstack/keystoneauth master: Remove deprecated_since for interface and fix text https://review.openstack.org/484528
23:30:13 mordred efried: is the code that's up your latest?
23:30:20 mordred efried: if so, I can take a look first thing in the morning
23:31:09 efried mordred Well, nooo, not really. I've got a mess of a sandbox at this point which includes some things that are fixes, some things that are experiments, and a bunch of debug code.
23:31:16 mordred nod
23:31:32 efried mordred But I can try to work something up that represents only the first of those things and post it before morning.
23:31:57 mordred efried: cool - it may be that we're at the point where if you show me what you've got a second set of eyes can help move if along
23:32:50 efried mordred What I'm pretty sure I'm screwing up is the logic around https://review.openstack.org/#/c/483604/6/keystoneauth1/discover.py@905
23:33:09 efried I *thought* I didn't need all those weird and complex variables anymore.
23:33:46 efried But my _catalog_matches_version is a tad too loose, so I'm not getting latest when I should because a non-latest matches.
23:34:41 mordred efried: yah - I tihnk you still need them - but I think after you normalize version -> min/max you should be able to do update the definition of exact_match
23:35:31 efried mordred Yeah, I think that new def is making sure max_version[0] is equal, but max_version isn't exceeded.
23:36:28 efried So version_match, but with the inequality reversed.
23:39:19 mordred efried: actually - so version=2 -> min_version=2.0 max_version=2.latest right?
23:39:30 efried mordred Yes.
23:39:54 efried oh
23:39:59 mordred efried: exact_match could then be is_between and min_version[0] == max_version[0]
#openstack-sdks - 2017-07-18
00:24:30 openstackgerrit Michael Johnson proposed openstack/python-openstacksdk master: Update load_balancer for v2 API https://review.openstack.org/454410
01:32:10 openstackgerrit Qiming Teng proposed openstack/python-openstacksdk master: Support to node-adopt and node-adopt-preview https://review.openstack.org/484542
02:25:50 openstackgerrit Eric Fried proposed openstack/keystoneauth master: Discourage 'version' and accept 'M.latest' https://review.openstack.org/483604
02:26:08 efried mordred ^!^
02:26:20 efried Passing tox, locally anyway.
02:47:08 sbezverk hello, I am bringing up latest master in kolla-kubernetes env. I see nova comes up ok but nova service-list comes back completely empty, are you aware of any known issues or major changes planned in pike so nova service-list command would not work?
03:06:01 openstackgerrit Qiming Teng proposed openstack/python-openstacksdk master: Support to node-adopt and node-adopt-preview https://review.openstack.org/484542
06:27:25 openstackgerrit M V P Nitesh proposed openstack/python-openstackclient master: now we can add description for role creation in OSC https://review.openstack.org/484355
06:51:57 openstackgerrit Lingyong Xu proposed openstack/cliff master: Use assertIsNone(...) instead of assertIs(None,...) https://review.openstack.org/484644
09:02:46 openstackgerrit Stephen Finucane proposed openstack/cliff master: trivial: Fix comments in sphinxext module https://review.openstack.org/484680
10:15:16 openstackgerrit M V P Nitesh proposed openstack/python-openstackclient master: now we can add description for role creation in OSC https://review.openstack.org/484355
11:04:21 openstackgerrit Merged openstack/keystoneauth master: Expose valid_interfaces as a ListOpt for config files https://review.openstack.org/482743
11:04:32 openstackgerrit Merged openstack/keystoneauth master: Remove deprecated_since for interface and fix text https://review.openstack.org/484528
11:04:37 openstackgerrit Merged openstack/keystoneauth master: Remove kwargs from adapter.get_endpoint_data https://review.openstack.org/482744
11:28:26 amotoki does anyone working on the OSC functest breakage?
11:32:21 amotoki volume v1 API returns 404 and I wonder this is a cinder bug or we need some workaround for OSC side.
12:28:09 mordred efried: WOOT!
12:33:26 mordred efried: you have a py35 break - which I think is just going to need an is not None tossed in
12:47:07 efried mordred Looking...
12:48:25 efried mordred Oh - you can't compare Nones in py3? That's a pain.
12:50:49 efried mordred You know, I may as well voice this thing that's been in the back of my head: If someone decides to get clever and release a v0, we're effed.
12:50:58 efried Worth making a statement in the docs?
12:59:01 openstackgerrit Merged openstack/python-openstacksdk master: Support to node-adopt and node-adopt-preview https://review.openstack.org/484542
14:45:35 openstackgerrit Eric Fried proposed openstack/keystoneauth master: Discourage 'version' and accept 'M.latest' https://review.openstack.org/483604
14:45:41 efried mordred ^ fixed.
15:04:07 mordred efried: if someone decides to release a v0 I will personally kill them
15:05:30 efried mordred I will help. I know jiu jitsu.
15:07:04 mordred efried: ooh. I don't. you might actually be more useful than me there
15:08:06 efried mordred I generally try to use my powers for good rather than evil. But I think ridding the world of v0 counts as good.
16:11:41 openstackgerrit Sean Dague proposed openstack/keystoneauth master: Add ability to specify a microversion in a request https://review.openstack.org/482746
16:16:40 openstackgerrit Merged openstack/keystoneauth master: Minor cleanup https://review.openstack.org/478621
16:48:45 openstackgerrit Merged openstack/keystoneauth master: Add support for next_min_version and not_before https://review.openstack.org/482745
17:26:03 openstackgerrit Monty Taylor proposed openstack/keystoneauth master: Add ability to specify a microversion in a request https://review.openstack.org/482746
17:52:04 openstackgerrit Monty Taylor proposed openstack/keystoneauth master: Add ability to specify a microversion in a request https://review.openstack.org/482746
18:27:11 openstackgerrit Eric Fried proposed openstack/keystoneauth master: Discourage 'version' and accept 'M.latest' https://review.openstack.org/483604
18:27:17 efried mordred ^^ done (interesting side effect there)
18:42:47 openstackgerrit Michael Johnson proposed openstack/python-openstacksdk master: Update load_balancer for v2 API https://review.openstack.org/454410
19:33:37 openstackgerrit Michael Johnson proposed openstack/python-openstacksdk master: Update load_balancer for v2 API https://review.openstack.org/454410
20:17:04 openstackgerrit Dean Troyer proposed openstack/python-openstackclient master: DNM: functest check https://review.openstack.org/484209
20:33:50 openstackgerrit Michael Johnson proposed openstack/python-openstacksdk master: Update load_balancer for v2 API https://review.openstack.org/454410
20:44:08 openstackgerrit Michael Johnson proposed openstack/python-openstacksdk master: Update load_balancer for v2 API https://review.openstack.org/454410
23:01:28 openstackgerrit Dean Troyer proposed openstack/python-openstackclient master: Fix 'domain' filter not work well in some commands https://review.openstack.org/483278
#openstack-sdks - 2017-07-19
01:02:15 openstackgerrit Michael Johnson proposed openstack/python-openstacksdk master: Introduce Listener for Octavia (load balancing) https://review.openstack.org/451574
01:06:05 openstackgerrit Michael Johnson proposed openstack/python-openstacksdk master: Update load_balancer for v2 API https://review.openstack.org/454410
01:06:05 openstackgerrit Michael Johnson proposed openstack/python-openstacksdk master: Introduce Listener for Octavia (load balancing) https://review.openstack.org/451574
01:21:35 openstackgerrit Michael Johnson proposed openstack/python-openstacksdk master: Introduce Listener for Octavia (load balancing) https://review.openstack.org/451574
04:10:59 openstackgerrit Michael Johnson proposed openstack/python-openstacksdk master: Introduce Listener for Octavia (load balancing) https://review.openstack.org/451574
04:11:49 openstackgerrit Monty Taylor proposed openstack/os-service-types master: Remove the local cache implementation https://review.openstack.org/485026
04:33:36 openstackgerrit Monty Taylor proposed openstack/os-service-types master: Implement fetching and accessor methods https://review.openstack.org/484018
08:50:43 openstackgerrit Merged openstack/keystoneauth master: Enable sphinx todo extension https://review.openstack.org/484417
08:50:53 openstackgerrit Merged openstack/keystoneauth master: Update URLs in documents according to document migration https://review.openstack.org/483668
08:54:16 cmurphy mordred: efried I think I made it through the important stuff but I had concerns with https://review.openstack.org/#/c/483604/

Earlier   Later