Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2017-07-18
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/
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.
17:23:59 mordred efried: the error I'm getting is TypeError: 'NoneType' object has no attribute '__getitem__'
17:24:33 efried mordred Yeah, because [versioned_]data_for is returning None when it should be returning the v3 object.
17:24:40 efried mordred I'm almost there - hold tight a sec.
17:24:40 mordred gotcha - I see it now
17:24:44 mordred kk
17:38:23 openstackgerrit Eric Fried proposed openstack/keystoneauth master: Discourage 'version' and accept 'M.latest' https://review.openstack.org/483604
17:38:36 efried mordred ^^ checka-checka-check-it-out.
17:38:47 efried cmurphy that was a great find!
17:42:05 mordred efried: ooh, I'm excited
17:42:39 efried mordred And looks like the rest of the chain is now merged, so this is the last thing I'm really antsy about getting into the release.
17:42:49 efried (Others may have antsiness of their own)
17:43:33 mordred efried: beautiful!
17:44:18 efried mordred Okay, do we pester lbragstad edleafe cdent at this point?
17:46:35 mordred efried: and cmurphy :)
17:46:49 efried cmurphy said she was going to be afk today, and we should proceed without her.
17:47:05 efried mordred I believe I have addressed her concern.
17:47:56 efried mordred http://eavesdrop.openstack.org/irclogs/%23openstack-sdks/%23openstack-sdks.2017-07-19.log.html#t2017-07-19T08:54:16 just so you don't think I'm making that up :)
17:49:34 mordred efried: I don't think that at all :)
17:50:00 cdent mordred: efried is a gambling addict, you cannot believe what he says
17:50:05 efried hahahahaha
17:50:22 cdent that coin was for food!
17:50:44 efried cdent Sokay, when I hit the jackpot I'll buy ALL of us food.
17:50:51 efried Gotta play to win.
17:50:53 cdent \o/
17:51:08 cdent so, where am I supposed to be looking?
17:51:26 lbragstad ksa version discovery bits?
17:51:26 efried cdent https://review.openstack.org/#/c/483604/ -- actually, wasn't sure whether you were involved or not (or are a ksa core)
17:51:27 openstackgerrit Monty Taylor proposed openstack/keystoneauth master: Fixed minor review comments and updated docs https://review.openstack.org/485267
17:51:49 efried lbragstad Yes, specifically M.latest business https://review.openstack.org/#/c/483604
17:52:02 lbragstad efried: neat
17:52:05 mordred lbragstad: yup. and then that cleanup patch there whih addresses review conerns from cmurphy and samueldmq
17:52:15 lbragstad is that the last bit required to land before we can release ksa?
17:52:15 efried Oh, right, morgan too...
17:52:45 cdent efried: i’m not core, but I’ve been following along with that stuff so happy to review
17:52:45 mordred lbragstad: yes - although I'm still holding out hope for the alias patches - but these two are the last required required
17:52:47 lbragstad our final release for ksa in Pike will be next week
17:53:04 cdent I’m less and less core each day.

Earlier   Later