Wednesday, 2017-07-19

*** gildub has joined #openstack-sdks00:24
openstackgerritMichael Johnson proposed openstack/python-openstacksdk master: Introduce Listener for Octavia (load balancing)  https://review.openstack.org/45157401:02
openstackgerritMichael Johnson proposed openstack/python-openstacksdk master: Introduce Listener for Octavia (load balancing)  https://review.openstack.org/45157401:06
openstackgerritMichael Johnson proposed openstack/python-openstacksdk master: Update load_balancer for v2 API  https://review.openstack.org/45441001:06
*** dave-mccowan has joined #openstack-sdks01:06
openstackgerritMichael Johnson proposed openstack/python-openstacksdk master: Introduce Listener for Octavia (load balancing)  https://review.openstack.org/45157401:21
*** bobh has quit IRC01:21
*** gouthamr has quit IRC01:25
*** annp has joined #openstack-sdks01:42
*** fzdarsky_ has joined #openstack-sdks01:44
*** bobh has joined #openstack-sdks01:45
*** fzdarsky has quit IRC01:48
*** gouthamr has joined #openstack-sdks02:21
*** markvoelker has quit IRC02:59
*** bobh has quit IRC03:11
*** hoangcx has quit IRC03:57
*** hoangcx has joined #openstack-sdks03:57
*** dave-mccowan has quit IRC04:01
openstackgerritMichael Johnson proposed openstack/python-openstacksdk master: Introduce Listener for Octavia (load balancing)  https://review.openstack.org/45157404:10
*** Dinesh_Bhor has joined #openstack-sdks04:11
openstackgerritMonty Taylor proposed openstack/os-service-types master: Remove the local cache implementation  https://review.openstack.org/48502604:11
*** gouthamr has quit IRC04:14
*** Dinesh_Bhor has quit IRC04:14
*** reedip has quit IRC04:14
*** Dinesh_Bhor has joined #openstack-sdks04:15
*** reedip has joined #openstack-sdks04:15
openstackgerritMonty Taylor proposed openstack/os-service-types master: Implement fetching and accessor methods  https://review.openstack.org/48401804:33
*** gouthamr has joined #openstack-sdks04:42
*** markvoelker has joined #openstack-sdks05:00
*** gouthamr has quit IRC05:07
*** Dinesh_Bhor has quit IRC05:17
*** frickler has quit IRC05:24
*** mvpnitesh has joined #openstack-sdks05:25
*** frickler has joined #openstack-sdks05:27
*** frickler has quit IRC05:33
*** markvoelker has quit IRC05:34
*** frickler has joined #openstack-sdks05:38
*** gildub has quit IRC05:48
*** markvoelker has joined #openstack-sdks06:31
*** frickler has quit IRC06:42
*** Dinesh_Bhor has joined #openstack-sdks06:50
*** Dinesh_Bhor has quit IRC06:50
*** annegentle has quit IRC07:01
*** markvoelker has quit IRC07:04
*** ralonsoh has joined #openstack-sdks07:46
*** mvpnitesh has quit IRC08:07
*** mvpnitesh has joined #openstack-sdks08:08
*** dtantsur|afk is now known as dtantsur08:09
*** mvpnitesh has quit IRC08:16
*** mvpnitesh has joined #openstack-sdks08:17
*** ralonsoh has quit IRC08:33
*** ralonsoh_ has joined #openstack-sdks08:33
*** dtantsur is now known as dtantsur|brb08:41
*** openstackgerrit has quit IRC08:49
*** openstackgerrit has joined #openstack-sdks08:50
openstackgerritMerged openstack/keystoneauth master: Enable sphinx todo extension  https://review.openstack.org/48441708:50
openstackgerritMerged openstack/keystoneauth master: Update URLs in documents according to document migration  https://review.openstack.org/48366808:50
*** itxaka has joined #openstack-sdks08:54
cmurphymordred: efried I think I made it through the important stuff but I had concerns with https://review.openstack.org/#/c/483604/08:54
cmurphybut 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 me08:54
openstackgerritliyi proposed openstack/python-openstacksdk master: Add config param for cluster object  https://review.openstack.org/48510008:56
openstackgerritzhangdaolong proposed openstack/keystoneauth master: Fix print the password in the log  https://review.openstack.org/48377108:57
*** markvoelker has joined #openstack-sdks09:02
*** annp has quit IRC09:07
openstackgerritliyi proposed openstack/python-openstacksdk master: Add config param for cluster object  https://review.openstack.org/48510009:15
*** markvoelker has quit IRC09:35
*** sdague has joined #openstack-sdks09:37
*** Dinesh_Bhor has joined #openstack-sdks10:27
*** markvoelker has joined #openstack-sdks10:32
*** jkilpatr has quit IRC10:36
*** markvoelker has quit IRC11:06
*** fzdarsky_ is now known as fzdarsky|lunch11:12
*** jkilpatr has joined #openstack-sdks11:12
*** cdent has joined #openstack-sdks11:18
*** dave-mccowan has joined #openstack-sdks11:26
*** fzdarsky|lunch is now known as fzdarsky11:31
*** chlong_ has quit IRC11:43
*** chenyb4_ has joined #openstack-sdks12:01
*** chenyb4_ has joined #openstack-sdks12:01
*** markvoelker has joined #openstack-sdks12:03
*** itxaka has quit IRC12:24
*** markvoelker has quit IRC12:25
*** markvoelker has joined #openstack-sdks12:26
*** chenyb4_ has quit IRC12:28
*** rate has joined #openstack-sdks12:31
*** ralonsoh_ has quit IRC12:32
*** dtantsur|brb is now known as dtantsur12:40
amotokiCurrently OSC doc gate job is broken by karborclient wrong entry points. bug 170525812:43
openstackbug 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
amotokiI've proposed a fix to karborclient https://review.openstack.org/48519312:43
amotokiwe need to see the progress, but if it is not good we need to consider blocking karborclient 0.4.0 in g-r.12:44
openstackgerritJan Gutter proposed openstack/python-openstackclient master: Add support for virtio-forwarder VNIC type  https://review.openstack.org/48353212:45
*** gouthamr has joined #openstack-sdks12:58
efriedmordred 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:04
openstackgerritHarry Rybacki proposed openstack/python-openstackclient master: Implied Roles  https://review.openstack.org/29025313:07
*** ralonsoh has joined #openstack-sdks13:19
*** ralonsoh has quit IRC13:20
*** ralonsoh has joined #openstack-sdks13:20
efriedmordred 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
openstackgerritEric Fried proposed openstack/keystoneauth master: Discourage 'version' and accept 'M.latest'  https://review.openstack.org/48360413:36
*** bobh has joined #openstack-sdks13:37
efriedmordred See my test comments ^^13:39
dtroyeramotoki: good find!  that is totally not where I have been looking...13:40
amotokidtroyer: i noticed this when I rechecked it just after your project-config change on cinder v1 was merged.13:41
amotokidtroyer: but it is a case we would like to detect in osc plugin reviews.13:42
dtroyerso it looks like we had multiple problems than…13:43
* dtroyer rubs eyes, still waking up here13:43
amotokigood news is karborclient patch has been merged.13:43
openstackgerritMerged openstack/python-openstacksdk master: Add config param for cluster object  https://review.openstack.org/48510014:22
*** annegentle has joined #openstack-sdks14:34
*** marst has joined #openstack-sdks14:35
*** rate has quit IRC15:36
*** rajathagasthya has joined #openstack-sdks15:46
mordredefried: morning!15:48
efriedmordred It has been that.15:48
efriedmordred 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:49
mordredefried: kk. will jump on it in just a sec15:58
efriedthx15:58
*** dtantsur is now known as dtantsur|afk16:03
*** aarefiev is now known as aarefiev_afk16:04
*** fzdarsky has quit IRC16:06
*** fzdarsky has joined #openstack-sdks16:08
*** annegentle has quit IRC16:19
openstackgerritMonty Taylor proposed openstack/keystoneauth master: Fixed minor review comments and updated docs  https://review.openstack.org/48526716:23
*** bobh has quit IRC16:24
*** bobh has joined #openstack-sdks16:30
*** fzdarsky is now known as fzdarsky|afk16:32
openstackgerritMerged openstack/keystoneauth master: Add ability to specify a microversion in a request  https://review.openstack.org/48274616:39
openstackgerritAkihiro Motoki proposed openstack/python-openstackclient master: Disable karborclient until a fixed version is released  https://review.openstack.org/48527416:46
openstackgerritMichael Johnson proposed openstack/python-openstacksdk master: Introduce Pool for Octavia (load balancing)  https://review.openstack.org/44926416:46
openstackgerritMichael Johnson proposed openstack/python-openstacksdk master: Introduce Pool for Octavia (load balancing)  https://review.openstack.org/44926416:58
*** efried has quit IRC16:59
*** efried has joined #openstack-sdks17:11
efriedmordred Yeah, I can make both `latest` and `M.latest` work there.17:17
mordredefried: the error I'm getting is TypeError: 'NoneType' object has no attribute '__getitem__'17:23
efriedmordred Yeah, because [versioned_]data_for is returning None when it should be returning the v3 object.17:24
mordredgotcha - I see it now17:24
efriedmordred I'm almost there - hold tight a sec.17:24
mordredkk17:24
*** bobh has quit IRC17:29
*** bobh has joined #openstack-sdks17:31
*** annegentle has joined #openstack-sdks17:37
openstackgerritEric Fried proposed openstack/keystoneauth master: Discourage 'version' and accept 'M.latest'  https://review.openstack.org/48360417:38
efriedmordred ^^ checka-checka-check-it-out.17:38
efriedcmurphy that was a great find!17:38
mordredefried: ooh, I'm excited17:42
*** ralonsoh has quit IRC17:42
efriedmordred 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
efried(Others may have antsiness of their own)17:42
mordredefried: beautiful!17:43
efriedmordred Okay, do we pester lbragstad edleafe cdent at this point?17:44
* cdent gives efried a coin17:44
* efried feeds it to the one-armed bandit and pulls the lever...17:45
mordredefried: and cmurphy :)17:46
efriedcmurphy said she was going to be afk today, and we should proceed without her.17:46
efriedmordred I believe I have addressed her concern.17:47
efriedmordred 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:47
mordredefried: I don't think that at all :)17:49
cdentmordred: efried is a gambling addict, you cannot believe what he says17:50
efriedhahahahaha17:50
cdentthat coin was for food!17:50
efriedcdent Sokay, when I hit the jackpot I'll buy ALL of us food.17:50
efriedGotta play to win.17:50
cdent\o/17:50
cdentso, where am I supposed to be looking?17:51
efriedcdent https://review.openstack.org/#/c/483604/ -- actually, wasn't sure whether you were involved or not (or are a ksa core)17:51
lbragstadksa version discovery bits?17:51
openstackgerritMonty Taylor proposed openstack/keystoneauth master: Fixed minor review comments and updated docs  https://review.openstack.org/48526717:51
efriedlbragstad Yes, specifically M.latest business https://review.openstack.org/#/c/48360417:51
lbragstadefried: neat17:52
mordredlbragstad: yup. and then that cleanup patch there whih addresses review conerns from cmurphy and samueldmq17:52
efriedOh, right, morgan too...17:52
lbragstadis that the last bit required to land before we can release ksa?17:52
mordredlbragstad: yes - although I'm still holding out hope for the alias patches - but these two are the last required required17:52
cdentefried: i’m not core, but I’ve been following along with that stuff so happy to review17:52
lbragstadour final release for ksa in Pike will be next week17:52
cdentI’m less and less core each day.17:53
mordredcdent, efried: while I have you here: https://review.openstack.org/#/c/484018/ - I pushed up a fifth patch to the series to address the review comments, but then just squashed it all into that one17:53
efriedlbragstad Will there be a ksa release this week too?17:53
mordredcdent: tl;dr - after digging further I have come to agree with your original review - and also realized that for our initial uses it's ok to not have it17:54
lbragstadwell - next week is the last week for client libraries17:54
lbragstadso for sure next week if not earlier17:54
mordredlbragstad: I thought this week was the freeze week17:54
lbragstadmordred: this week is non-client library freeze17:55
efriedlbragstad And ksa counts as a non-client library, yah?17:55
lbragstadhttps://releases.openstack.org/pike/schedule.html#p-final-lib17:55
lbragstadefried: i guess it does?17:55
efriedmordred So to be clear, https://review.openstack.org/#/c/484018/ is now sans cache, with test mixin refactor, etc. all in one?17:57
openstackgerritGage Hugo proposed openstack/python-openstackclient master: Add project tags functionality  https://review.openstack.org/48128417:57
mordredlbragstad: yah - I tihnk ksa falls into that bucket so that client libshave a week to incorporate changes from ksa before they freeze17:57
lbragstadmordred: ok - good point17:57
cdentstarting off without cache seems sound17:57
lbragstadi was just digging in to the definitions in releases17:57
mordredefried: yup. and the other patches will be anbandoned - left them there for now so that folks can look at the progression if it's helpful17:57
lbragstadyeah - so ksa should release this week17:57
mordredcdent: turns out the two pathches we have to use the lib woudldn't use the cache anyway17:58
mordredcdent: :)17:58
cdent:)17:58
openstackgerritXueFeng Liu proposed openstack/python-openstacksdk master: Fix `openstack cluster service list`  https://review.openstack.org/48529918:00
*** jkilpatr has quit IRC18:06
*** boris-42___ has joined #openstack-sdks18:17
*** jkilpatr has joined #openstack-sdks18:22
*** efried is now known as efried_lunch18:23
*** bobh has quit IRC18:24
*** jangutter has joined #openstack-sdks18:29
cdentmordred: gotta dine but will poke at that stuff by tomorrow morning at the latest18:29
*** cdent has quit IRC18:29
*** u_nuSLASHkm8 has joined #openstack-sdks18:31
*** u_nuSLASHkm8 has left #openstack-sdks18:33
jangutterIs the gate still broken with: Failed to load karborclient.osc.v1.protectables.ShowProtectable: No module named ShowProtectable?18:34
*** bobh has joined #openstack-sdks18:38
openstackgerritXueFeng Liu proposed openstack/python-openstacksdk master: Fix comment in services function  https://review.openstack.org/48529918:51
*** jkilpatr has quit IRC18:56
*** rajathagasthya has quit IRC19:03
*** jkilpatr has joined #openstack-sdks19:05
*** chlong_ has joined #openstack-sdks19:15
openstackgerritMerged openstack/python-openstackclient master: Disable karborclient until a fixed version is released  https://review.openstack.org/48527419:24
*** tinyurl_comSLASH has joined #openstack-sdks19:42
*** tinyurl_comSLASH has left #openstack-sdks19:44
*** rajathagasthya has joined #openstack-sdks19:46
*** annegentle has quit IRC19:55
*** efried_lunch is now known as efried19:59
*** chlong_ has quit IRC20:03
*** dtroyer has quit IRC20:05
*** rajathagasthya has quit IRC20:07
*** annegentle has joined #openstack-sdks20:15
*** jkilpatr has quit IRC20:18
*** annegentle has quit IRC20:26
*** rajathagasthya has joined #openstack-sdks20:27
*** annegentle has joined #openstack-sdks20:30
*** annegentle has quit IRC20:35
*** jkilpatr has joined #openstack-sdks20:36
*** dtroyer has joined #openstack-sdks20:46
*** chlong_ has joined #openstack-sdks20:48
*** chlong_ has quit IRC21:04
*** cdent has joined #openstack-sdks21:08
openstackgerritMonty Taylor proposed openstack/os-service-types master: Throw an exception on conflicting arguments  https://review.openstack.org/48536021:13
mordredefried: ^^ that should take care of your review feedback. you holding off on a +A of the first patch for a specific thing I should go track down?21:14
*** bobh has quit IRC21:36
*** boris-42___ has quit IRC21:41
openstackgerritDean Troyer proposed openstack/python-openstackclient master: Fix dynamic names in network functional tests  https://review.openstack.org/48536321:45
*** marst_ has joined #openstack-sdks21:48
*** gouthamr has quit IRC21:49
*** marst has quit IRC21:52
*** marst_ has quit IRC21:57
*** gouthamr has joined #openstack-sdks22:18
efriedmordred I'll +A https://review.openstack.org/#/c/484018/ if you're ready for it.22:27
mordredefried: j1522:27
mordredgah22:27
mordredefried: yes - I think it's good to go22:27
efriedmordred Done.22:27
mordredwoot22:28
efriedmordred https://review.openstack.org/#/c/485360/ reviewed22:33
*** sdague has quit IRC22:33
*** cdent has quit IRC22:34
openstackgerritMichael Johnson proposed openstack/python-openstacksdk master: Introduce Member for Octavia (load balancing)  https://review.openstack.org/45148822:41
openstackgerritMichael Johnson proposed openstack/python-openstacksdk master: Introduce Member for Octavia (load balancing)  https://review.openstack.org/45148822:46
openstackgerritMichael Johnson proposed openstack/python-openstacksdk master: Introduce Member for Octavia (load balancing)  https://review.openstack.org/45148822:53
*** sbezverk has quit IRC23:15
openstackgerritMerged openstack/os-service-types master: Implement fetching and accessor methods  https://review.openstack.org/48401823:29
openstackgerritMerged openstack/service-types-authority master: Remove leading openstack/ from project name  https://review.openstack.org/48442223:31

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!