Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2017-07-17
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: 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: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/
08:54:49 cmurphy but I'll be afk during US working hours today and tomorrow and then totally afk till Tuesday so if I'm offbase and/or it gets fixed don't wait on me
08:56:09 openstackgerrit liyi proposed openstack/python-openstacksdk master: Add config param for cluster object https://review.openstack.org/485100
08:57:45 openstackgerrit zhangdaolong proposed openstack/keystoneauth master: Fix print the password in the log https://review.openstack.org/483771
09:15:51 openstackgerrit liyi proposed openstack/python-openstacksdk master: Add config param for cluster object https://review.openstack.org/485100
12:43:13 amotoki Currently OSC doc gate job is broken by karborclient wrong entry points. bug 1705258
12:43:14 openstack bug 1705258 in python-karborclient "karborclient 0.4.0 breaks OSC docs gate due to wrong entry points" [Undecided,New] https://launchpad.net/bugs/1705258 - Assigned to Akihiro Motoki (amotoki)
12:43:53 amotoki I've proposed a fix to karborclient https://review.openstack.org/485193
12:44:41 amotoki we need to see the progress, but if it is not good we need to consider blocking karborclient 0.4.0 in g-r.
12:45:53 openstackgerrit Jan Gutter proposed openstack/python-openstackclient master: Add support for virtio-forwarder VNIC type https://review.openstack.org/483532
13:04:01 efried mordred I was about to start tackling cmurphy's comments on https://review.openstack.org/#/c/483604 - any disagreement or guidance before I do that?
13:07:52 openstackgerrit Harry Rybacki proposed openstack/python-openstackclient master: Implied Roles https://review.openstack.org/290253
13:36:02 efried mordred I can patch up one of those issues, but can you help with the other? I put in some test cases to demonstrate.
13:36:41 openstackgerrit Eric Fried proposed openstack/keystoneauth master: Discourage 'version' and accept 'M.latest' https://review.openstack.org/483604
13:39:19 efried mordred See my test comments ^^
13:40:40 dtroyer amotoki: good find! that is totally not where I have been looking...
13:41:57 amotoki dtroyer: i noticed this when I rechecked it just after your project-config change on cinder v1 was merged.
13:42:36 amotoki dtroyer: but it is a case we would like to detect in osc plugin reviews.
13:43:06 dtroyer so it looks like we had multiple problems than…
13:43:37 amotoki good news is karborclient patch has been merged.
14:22:46 openstackgerrit Merged openstack/python-openstacksdk master: Add config param for cluster object https://review.openstack.org/485100
15:48:42 mordred efried: morning!
15:48:53 efried mordred It has been that.
15:49:24 efried mordred TL;DR the new test cases I put in demonstrate a bug. I'm trying to fix it, would welcome your expertise & help there.
15:58:05 mordred efried: kk. will jump on it in just a sec
15:58:14 efried thx
16:23:22 openstackgerrit Monty Taylor proposed openstack/keystoneauth master: Fixed minor review comments and updated docs https://review.openstack.org/485267
16:39:13 openstackgerrit Merged openstack/keystoneauth master: Add ability to specify a microversion in a request https://review.openstack.org/482746
16:46:39 openstackgerrit Akihiro Motoki proposed openstack/python-openstackclient master: Disable karborclient until a fixed version is released https://review.openstack.org/485274
16:46:51 openstackgerrit Michael Johnson proposed openstack/python-openstacksdk master: Introduce Pool for Octavia (load balancing) https://review.openstack.org/449264
16:58:05 openstackgerrit Michael Johnson proposed openstack/python-openstacksdk master: Introduce Pool for Octavia (load balancing) https://review.openstack.org/449264
17:17:43 efried mordred Yeah, I can make both `latest` and `M.latest` work there.

Earlier   Later