Thursday, 2018-01-18

*** slaweq has quit IRC00:03
*** bobh has quit IRC00:11
*** bobh has joined #openstack-sdks00:11
*** bobh has quit IRC00:16
*** edmondsw has joined #openstack-sdks00:35
*** slaweq has joined #openstack-sdks00:36
*** edmondsw has quit IRC00:40
*** slaweq has quit IRC00:41
openstackgerritAdrian Turjak proposed openstack/python-openstacksdk master: Raise error when supplying invalid query params  https://review.openstack.org/53272300:50
*** fabian_ has joined #openstack-sdks00:51
*** fabian_ is now known as chenyb400:59
*** slaweq has joined #openstack-sdks01:12
*** slaweq has quit IRC01:16
*** thrash is now known as thrash|g0ne01:22
*** Kevin_Zheng has joined #openstack-sdks01:24
*** slaweq has joined #openstack-sdks01:51
*** openstackstatus has quit IRC01:55
*** johnsom has quit IRC01:55
*** zacksh_ has quit IRC01:55
*** Hazelesque has quit IRC01:55
*** mnaser has quit IRC01:55
*** vdrok has quit IRC01:55
*** reedip has quit IRC01:55
*** Hazelesque has joined #openstack-sdks01:55
*** zacksh has joined #openstack-sdks01:55
*** slaweq has quit IRC01:55
*** mnaser has joined #openstack-sdks01:56
*** vdrok has joined #openstack-sdks01:56
*** johnsom has joined #openstack-sdks01:56
*** openstackstatus has joined #openstack-sdks01:57
*** ChanServ sets mode: +v openstackstatus01:57
*** reedip has joined #openstack-sdks02:06
*** annp has joined #openstack-sdks02:19
*** edmondsw has joined #openstack-sdks02:24
*** ldl has joined #openstack-sdks02:26
*** edmondsw has quit IRC02:28
*** slaweq has joined #openstack-sdks02:30
*** slaweq has quit IRC02:35
*** fzdarsky_ has joined #openstack-sdks02:44
*** fzdarsky has quit IRC02:48
*** slaweq has joined #openstack-sdks03:10
*** slaweq has quit IRC03:10
*** gkadam has joined #openstack-sdks03:10
openstackgerritOpenStack Proposal Bot proposed openstack/openstackclient master: Updated from global requirements  https://review.openstack.org/52190203:20
openstackgerritOpenStack Proposal Bot proposed openstack/os-api-ref master: Updated from global requirements  https://review.openstack.org/53508803:20
openstackgerritOpenStack Proposal Bot proposed openstack/os-client-config master: Updated from global requirements  https://review.openstack.org/53399103:21
openstackgerritOpenStack Proposal Bot proposed openstack/os-service-types master: Updated from global requirements  https://review.openstack.org/53399203:21
openstackgerritOpenStack Proposal Bot proposed openstack/osc-lib master: Updated from global requirements  https://review.openstack.org/53399703:22
*** alex_xu has quit IRC03:30
*** RuiChen has quit IRC03:30
*** rm_work has quit IRC03:30
*** TheJulia has quit IRC03:30
*** csatari has quit IRC03:30
*** samueldmq has quit IRC03:30
*** briancurtin has quit IRC03:30
*** jamielennox has quit IRC03:30
*** gus has quit IRC03:30
*** gus has joined #openstack-sdks03:30
*** rm_work has joined #openstack-sdks03:30
*** rm_work has joined #openstack-sdks03:30
*** alex_xu has joined #openstack-sdks03:30
openstackgerritOpenStack Proposal Bot proposed openstack/python-openstackclient master: Updated from global requirements  https://review.openstack.org/53512303:30
*** samueldmq has joined #openstack-sdks03:30
*** RuiChen has joined #openstack-sdks03:31
*** csatari has joined #openstack-sdks03:31
*** TheJulia has joined #openstack-sdks03:31
*** briancurtin has joined #openstack-sdks03:31
*** jamielennox has joined #openstack-sdks03:37
*** slaweq has joined #openstack-sdks03:44
*** slaweq has quit IRC03:49
*** slaweq has joined #openstack-sdks04:15
*** slaweq has quit IRC04:20
*** bobh has joined #openstack-sdks04:24
*** zhongjun has quit IRC04:46
*** slaweq has joined #openstack-sdks04:51
*** gildub_ has joined #openstack-sdks04:52
*** gildub__ has quit IRC04:55
*** slaweq has quit IRC04:55
*** dave-mccowan has quit IRC04:59
*** chenyb4 has quit IRC05:01
*** chenyb4 has joined #openstack-sdks05:02
*** bobh has quit IRC05:12
*** slaweq has joined #openstack-sdks05:23
*** slaweq has quit IRC05:27
*** slaweq has joined #openstack-sdks06:01
*** slaweq has quit IRC06:05
*** gkadam has quit IRC06:21
*** slaweq has joined #openstack-sdks06:40
*** pzchen has quit IRC06:44
*** slaweq has quit IRC06:45
*** pzchen has joined #openstack-sdks06:45
*** zhongjun has joined #openstack-sdks06:49
*** ianw has quit IRC07:02
*** edmondsw has joined #openstack-sdks07:06
*** edmondsw has quit IRC07:11
*** dims has quit IRC07:12
*** dims has joined #openstack-sdks07:14
*** dims has quit IRC07:19
*** dims has joined #openstack-sdks07:20
*** slaweq has joined #openstack-sdks07:38
*** tulivu has joined #openstack-sdks07:52
*** annp has quit IRC07:56
*** gkadam has joined #openstack-sdks08:02
*** b_bezak has joined #openstack-sdks08:12
*** annp has joined #openstack-sdks08:13
*** chenyb4 has quit IRC08:13
*** chenyb4 has joined #openstack-sdks08:13
*** ralonsoh has joined #openstack-sdks08:17
*** ralonsoh has quit IRC08:28
*** ralonsoh has joined #openstack-sdks08:37
*** jpena|off is now known as jpena08:46
openstackgerritbrandon zhao proposed openstack/python-openstacksdk master: change spell error  https://review.openstack.org/53522708:48
*** Mr_Broken has joined #openstack-sdks08:49
*** edmondsw has joined #openstack-sdks08:54
*** ianw has joined #openstack-sdks08:58
*** edmondsw has quit IRC08:59
*** jpich has joined #openstack-sdks09:03
*** dtantsur|afk is now known as dtantsur09:07
*** e0ne has joined #openstack-sdks09:23
slaweqfrickler: hi09:37
slaweqfrickler: I want to ask You about https://storyboard.openstack.org/#!/story/200147509:37
slaweqfrickler: few days ago I said that I can handle it if You don't have time now09:37
slaweqfrickler: should I continue Your patch or we will first merge Your fix and then I should propose another one to preffer floating IPs always?09:38
fricklerslaweq: whatever suits you best. I've applied my patch locally, so I can continue my deployment, no pressure to merge that urgently09:42
slaweqfrickler: ok09:43
slaweqI think that it could be good to merge Your patch first and I will work on it then but I wanted to ask You about opinion :)09:43
*** chenyb4 has quit IRC10:22
openstackgerritMerged openstack/python-openstackclient master: Use Zuul v3 fetch-subunit-output  https://review.openstack.org/53413510:34
*** sdague has joined #openstack-sdks10:36
*** edmondsw has joined #openstack-sdks10:42
openstackgerritMerged openstack-infra/shade master: Fix address format used in find_best_address()  https://review.openstack.org/53430010:45
*** spredzy has quit IRC10:46
openstackgerritMerged openstack/os-api-ref master: Updated from global requirements  https://review.openstack.org/53508810:46
*** edmondsw has quit IRC10:47
*** spredzy has joined #openstack-sdks10:56
*** jkilpatr has joined #openstack-sdks11:52
*** jkilpatr has quit IRC11:53
*** jkilpatr_ has joined #openstack-sdks11:54
*** jpena is now known as jpena|lunch11:59
*** annp has quit IRC12:00
*** Mr_Broken has quit IRC12:07
openstackgerritColleen Murphy proposed openstack/keystoneauth master: Add support for application credentials  https://review.openstack.org/53445512:17
*** ldl has quit IRC12:25
*** ldl has joined #openstack-sdks12:25
*** ldl has quit IRC12:30
*** edmondsw has joined #openstack-sdks12:30
*** fabian_ has joined #openstack-sdks12:31
*** edmondsw has quit IRC12:35
*** fzdarsky_ is now known as fzdarsky|lunch12:45
*** fabian_ is now known as chenyb413:04
*** jpena|lunch is now known as jpena13:06
*** bobh has joined #openstack-sdks13:10
*** edmondsw has joined #openstack-sdks13:23
*** dave-mccowan has joined #openstack-sdks13:23
*** fzdarsky|lunch is now known as fzdarsky13:31
*** bobh has quit IRC13:33
*** thrash|g0ne is now known as thrash13:33
*** cleong has joined #openstack-sdks13:33
*** ldl has joined #openstack-sdks13:46
*** ldl has quit IRC13:50
openstackgerritMerged openstack/python-openstackclient master: Partially Revert "Update new documentation PTI jobs"  https://review.openstack.org/53481714:01
openstackgerritMerged openstack/python-openstackclient master: Updated from global requirements  https://review.openstack.org/53512314:01
openstackgerritHengqing Hu proposed openstack/python-openstacksdk master: Use version definition from openstack.version  https://review.openstack.org/53533614:05
*** nikhil has joined #openstack-sdks14:11
*** dayou has quit IRC14:30
dhellmanndtroyer: I would like to release https://review.openstack.org/#/c/534771/ today to unbreak the designate client gate14:36
*** dayou has joined #openstack-sdks14:44
openstackgerritMonty Taylor proposed openstack/python-openstacksdk master: Invert the attribute mapping  https://review.openstack.org/53382214:44
openstackgerritMonty Taylor proposed openstack/python-openstacksdk master: Clean up a bit after the mapping inversion  https://review.openstack.org/53382314:44
openstackgerritMonty Taylor proposed openstack/python-openstacksdk master: Update type conversion to handle subtypes more better  https://review.openstack.org/53382414:44
openstackgerritMonty Taylor proposed openstack/python-openstacksdk master: Remove resource and proxy  https://review.openstack.org/53382514:44
openstackgerritMonty Taylor proposed openstack/python-openstacksdk master: Rename resource2 and proxy2 to resource and proxy  https://review.openstack.org/53382614:44
openstackgerritMonty Taylor proposed openstack/python-openstacksdk master: Fix typo in the external service loader code  https://review.openstack.org/53445214:44
openstackgerritMonty Taylor proposed openstack/python-openstacksdk master: Fixed a few nits in the README  https://review.openstack.org/53445314:44
openstackgerritMonty Taylor proposed openstack/python-openstacksdk master: Remove deprecated wait_for methods from Proxy  https://review.openstack.org/53483214:44
dtroyerdhellmann: The commit message has it backwards but that is why we are changing it in this instance… mind if I update it so we don't record the wrong intention?14:50
dhellmanndtroyer : oh, no, go for it14:50
dhellmannsorry I missed that14:50
dhellmanntoo many irons in the fire yesterday :-/14:50
dtroyerme too…14:51
openstackgerritDean Troyer proposed openstack/cliff master: remove -s alias for --sort-columns  https://review.openstack.org/53477114:53
dtroyerdhellmann: wanna make sure I didn't get it wrong? ^^^^14:54
dhellmannyeah, that looks good14:54
dtroyer+W14:55
dhellmannI wasn't clear if we wanted a policy that cliff *not* use globals or if commands shouldn't14:55
dhellmannif we want the latter, we could probably enforce that, although doing it in a backwards-compatible way would be a nightmare14:55
dtroyerOSC has had the policy for commands to not use short options, because -s means too many different things, it seemed simpler to me to make those only used for global options, like those from cliff14:56
dhellmannok, that makes sense to me14:56
dtroyerThere are no short options in commands in the osc repo, the ones that do exist will all be in plugins14:56
dhellmannwe should probably publicize that somewhere14:56
dhellmannoh, how about this14:57
dhellmannwe could change cliff to register plugin options after command options14:57
dhellmannthen we could turn on that "resolve" flag14:57
dtroyerit is in our plugin guide, maybe needs to be bigger… then again the only people who tend to read that are the initial plugin devs,14:57
dtroyerthat's an idea14:57
dhellmannthat would have the effect of letting cliff and global plugins like formatters own the option namespace14:57
dtroyerin fact, I like that idea14:58
dhellmannok, I'll make a note about that and post to the ML to get some other feedback14:58
dtroyercool, thanks14:58
dhellmannimplementing that is going to be tricky, because we're relying on the inheritance right now15:04
dhellmannI'll have to think about the API changes needed15:04
dhellmanndtroyer : http://lists.openstack.org/pipermail/openstack-dev/2018-January/126348.html15:15
*** b_bezak has quit IRC15:27
openstackgerritSpencer Yu proposed openstack/python-openstackclient master: floatingip related should be removed from compute client  https://review.openstack.org/53536415:27
*** b_bezak has joined #openstack-sdks15:27
*** b_bezak has quit IRC15:32
*** slaweq has quit IRC15:39
*** chenyb4 has quit IRC15:40
openstackgerritSpencer Yu proposed openstack/python-openstackclient master: floatingip related should be removed from compute client  https://review.openstack.org/53536415:49
*** alex_xu has quit IRC15:50
*** alex_xu has joined #openstack-sdks15:52
*** fzdarsky is now known as fzdarsky|afk16:10
*** e0ne has quit IRC16:23
openstackgerritMerged openstack/api-wg master: Expand note about rfc5988 link header  https://review.openstack.org/53191416:28
edleafedtantsur: ready for your eyes: https://etherpad.openstack.org/p/api-sig-newsletter16:29
dtantsuredleafe: LGTM16:29
edleafedtantsur: ok, thanks!16:30
*** fzdarsky|afk is now known as fzdarsky17:02
*** gkadam has quit IRC17:06
openstackgerritDmitry Tantsur proposed openstack/api-wg master: Add guideline on exposing microversions in SDKs  https://review.openstack.org/53281417:28
dtantsuredleafe: slightly updated ^^^17:28
edleafedtantsur: ack17:33
*** jpich has quit IRC17:38
*** tulivu has quit IRC17:51
*** dtantsur is now known as dtantsur|afk17:58
*** ralonsoh has quit IRC18:05
*** nikhil has quit IRC18:31
mordreddtantsur|afk: left a note - I think we have a bikeshed we need to have about naming18:33
openstackgerritMerged openstack/cliff master: remove -s alias for --sort-columns  https://review.openstack.org/53477118:39
*** jpena is now known as jpena|off18:46
openstackgerritMerged openstack/python-openstackclient master: Fix indentation in authentication.rst  https://review.openstack.org/53484818:53
*** edleafe- has joined #openstack-sdks18:56
*** edleafe has quit IRC18:58
mordreddtroyer: if you have a sec, would you mind weighing in on https://review.openstack.org/#/c/533264 and https://review.openstack.org/#/c/533265 ?19:06
*** nikhil has joined #openstack-sdks19:08
*** edleafe- is now known as edleafe19:12
dtantsur|afkmordred: oh, yeah, it's going to be fun ;) will respond tomorrow19:20
*** dave-mccowan has quit IRC19:32
johnsomIs anyone looking at the neutron breakage in openstacksdk 0.10.0 that is blocking the GR update? https://review.openstack.org/#/c/534345/19:34
johnsomWe, octavia, really need that release in queens...19:34
mordredjohnsom: I am now :)19:50
johnsommordred Ok, thanks.  I just heard a report of this being broken in the field too.  Let me know if I can help in some way.19:50
mordredwait a second ... there's a new openstacksdk release?19:50
openstackgerritMerged openstack-infra/shade master: Updated from global requirements  https://review.openstack.org/52536819:51
johnsomEvidently19:51
mordredoh joy. that's ... unfortunate19:51
johnsomWell, I was hoping for one to get that sdk fix in for octavia. We really need that fix for the queens release.19:52
mordredwell, yes - I think a 0.10 release is a great idea - but there's a few more patches I'd wanted to get landed before we cut one this week19:53
johnsomI figured you knew, but I guess not: https://review.openstack.org/#/c/533624/19:53
johnsomI just saw it as I've been watching for it due to the octavia fix19:53
mordredwell - such is life - I'll look in to the neutron failure for you19:54
johnsomOk, thanks19:54
johnsomThe person I heard also saw it was just trying to create a floating IP via OSC and got a failure.19:55
mordredhttp://logs.openstack.org/45/534345/2/check/neutron-grenade/9831f38/logs/grenade.sh.txt.gz#_2018-01-17_18_04_33_42319:58
mordredseems to be the error19:58
mordredI can reproduce locally20:01
*** slaweq has joined #openstack-sdks20:03
dtroyermordred: +2 on both, ready to +W them?20:09
mordreddtroyer: yup!20:09
dtroyerdone20:09
mordreddtroyer: didn't we land a patch in osc to try importing new things from sdk if they're there?20:10
dtroyerHmmm… I actually don't remember for sure, but it wouldn't have been global, just getting Connection set up with OSC's sessiion or something20:11
mordredyah20:12
mordredwell - I'll look in to that *next*20:12
dtroyerthe semantics are different enough in most of the client libs that it'll be a bit more than s// to change over20:12
mordredoh - I mean in the network plugin where we make one for Neutron20:12
dtroyerah, yes, that is where we would have done that.  If it hasn't moved to clientManager it should though20:12
dtroyerI have unfortunately let most of OSC leave L1 cache in my brain the last few weeks20:13
*** dave-mccowan has joined #openstack-sdks20:13
*** e0ne has joined #openstack-sdks20:15
mordreddtroyer: I hear you20:18
mordredthere's only so much cache space20:18
mordredjohnsom: found bug - patch coming20:18
mordreddtroyer: I think we may want to add the neutron-grenade job to the openstacksdk gate - neutron-grenade caught an issue that the other sdk/osc shared jobs didn't20:20
dtroyermordred: is that something that we'll need to think about for the other APIs, and if it'll take N jobs to do N APIs?20:26
mordreddtroyer: no - this was just a bug in the backwards-compat code around Profile20:26
mordreddtroyer: it's also possible that adding neutron-grenade doesn't really gain us anything systemically and we just got lucky/unlucky this time20:27
johnsomAwesome you found the bug20:27
mordredjohnsom: yah - it's a silly likttle thing - but it's also easy enough to trigger, so making sure it's all gone20:28
openstackgerritMonty Taylor proposed openstack/python-openstacksdk master: Make sure we use config defaults in profile compat code  https://review.openstack.org/53547620:35
mordredjohnsom, dtroyer: ^^ that should fix the break in https://review.openstack.org/#/c/534345/20:35
*** e0ne has quit IRC20:39
*** fzdarsky is now known as fzdarsky|afk20:40
mordredShrews: ^^ you too, if you get a chance20:42
johnsomShouldn't there be a unit test update here?20:43
mordredjohnsom: yup. almost certainly20:52
mordred:)20:52
openstackgerritMonty Taylor proposed openstack/python-openstacksdk master: Invert the attribute mapping  https://review.openstack.org/53382220:53
openstackgerritMerged openstack/python-openstacksdk master: Rationalize logging helpers and docs  https://review.openstack.org/53263421:06
openstackgerritMerged openstack/python-openstacksdk master: Add ability to register non-official services  https://review.openstack.org/53326321:08
openstackgerritMonty Taylor proposed openstack/python-openstacksdk master: Make sure we use config defaults in profile compat code  https://review.openstack.org/53547621:10
mordredjohnsom, Shrews: ^^ k. now with a test. I verified that the test breaks without the patch21:11
mordred(and breaks in the way that osc was breaking)21:11
*** slaweq has quit IRC21:27
*** slaweq has joined #openstack-sdks21:32
*** openstackgerrit has quit IRC21:33
*** slaweq has quit IRC21:34
*** cleong has quit IRC21:37
johnsomNice21:42
*** dave-mccowan has quit IRC21:49
*** openstackgerrit has joined #openstack-sdks21:58
openstackgerritMonty Taylor proposed openstack/python-openstacksdk master: Make sure we use config defaults in profile compat code  https://review.openstack.org/53547621:58
*** nikhil has quit IRC22:01
*** edmondsw has quit IRC22:24
openstackgerritMonty Taylor proposed openstack/python-openstacksdk master: Use version definition from openstack.version  https://review.openstack.org/53533622:55
openstackgerritMonty Taylor proposed openstack/python-openstacksdk master: Move openstack_cloud helper functions  https://review.openstack.org/53551522:55
openstackgerritMonty Taylor proposed openstack/python-openstacksdk master: Invert the attribute mapping  https://review.openstack.org/53382223:14
*** edmondsw has joined #openstack-sdks23:23
*** gildub_ has quit IRC23:28
*** edmondsw has quit IRC23:29
*** gildub has joined #openstack-sdks23:41
*** sdague has quit IRC23:51

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