Thursday, 2016-01-07

*** dims_ has quit IRC00:07
*** Qiming has quit IRC00:19
*** pratikmallya has quit IRC00:22
*** devth_ is now known as devth00:24
*** dims has joined #openstack-sdks00:56
*** jaypipes has quit IRC00:59
*** gouthamr_ has joined #openstack-sdks01:07
*** gouthamr has quit IRC01:10
*** Qiming has joined #openstack-sdks01:12
*** boris-42 has quit IRC01:13
*** devth is now known as devth_afk01:42
openstackgerritMerged openstack/python-openstackclient: Updated from global requirements  https://review.openstack.org/26440602:22
*** lhcheng_ has quit IRC02:23
*** salv-orl_ has quit IRC02:23
*** salv-orlando has joined #openstack-sdks02:23
openstackgerritMerged openstack/keystoneauth: Updated from global requirements  https://review.openstack.org/26442702:35
*** devth_afk is now known as devth02:48
openstackgerritMerged openstack/python-openstackclient: Delete the unused LOG configure code  https://review.openstack.org/26361202:56
*** dims has quit IRC02:58
*** boris-42 has joined #openstack-sdks03:02
*** salv-orlando has quit IRC03:26
*** salv-orlando has joined #openstack-sdks03:27
openstackgerritSteve Martinelli proposed openstack/python-openstackclient: experimental: test the exp. job to detect naming conflicts  https://review.openstack.org/26453403:37
*** thrash is now known as thrash|g0ne03:42
openstackgerritEinst Crazy proposed openstack/cliff: Fix wrong use of 'an'  https://review.openstack.org/26453503:45
*** gouthamr_ has quit IRC04:05
*** gouthamr has joined #openstack-sdks04:06
*** salv-orl_ has joined #openstack-sdks04:09
*** salv-orlando has quit IRC04:11
*** gouthamr_ has joined #openstack-sdks04:28
*** gouthamr has quit IRC04:31
openstackgerritSteve Martinelli proposed openstack/python-openstackclient: experimental: test the exp. job to detect naming conflicts  https://review.openstack.org/26453404:56
openstackgerritSai Kiran proposed openstack/python-openstackclient: Add support to delete the ports  https://review.openstack.org/26056805:21
*** gouthamr_ has quit IRC05:30
*** Yanyanhu has joined #openstack-sdks05:36
*** salv-orl_ has quit IRC06:08
*** salv-orlando has joined #openstack-sdks06:08
*** markvoelker has quit IRC06:11
openstackgerritEinst Crazy proposed openstack/python-openstackclient: Replace assertEqual(*, None) with assertIsNone in tests  https://review.openstack.org/26460106:39
openstackgerritMerged openstack/python-openstacksdk: Remove status property from LBaaS resources  https://review.openstack.org/26433606:56
openstackgerritOpenStack Proposal Bot proposed openstack/python-openstacksdk: Updated from global requirements  https://review.openstack.org/26440706:57
openstackgerritSwapnil Kulkarni (coolsvap) proposed openstack/python-openstackclient: Use assertTrue/False instead of assertEqual(T/F)  https://review.openstack.org/26462407:10
*** markvoelker has joined #openstack-sdks07:11
openstackgerritSwapnil Kulkarni (coolsvap) proposed openstack/python-openstacksdk: Use assertTrue/False instead of assertEqual(T/F)  https://review.openstack.org/26462607:12
*** markvoelker has quit IRC07:16
openstackgerritLiuNanke proposed openstack/os-client-config: Remove openstack-common.conf  https://review.openstack.org/26463507:22
*** fzdarsky has joined #openstack-sdks08:00
*** salv-orlando has quit IRC08:04
*** salv-orlando has joined #openstack-sdks08:04
*** salv-orlando has quit IRC08:30
*** salv-orlando has joined #openstack-sdks08:30
openstackgerritMerged openstack/python-openstackclient: Replace assertEqual(*, None) with assertIsNone in tests  https://review.openstack.org/26460108:52
*** markvoelker has joined #openstack-sdks09:12
*** markvoelker has quit IRC09:17
openstackgerritMerged openstack/python-openstackclient: Use assertTrue/False instead of assertEqual(T/F)  https://review.openstack.org/26462409:21
*** fzdarsky is now known as fzdarsky|afk09:22
openstackgerritJude Augustine Job proposed openstack/python-openstackclient: change owner to project for image commands  https://review.openstack.org/26468409:27
*** salv-orl_ has joined #openstack-sdks10:08
*** salv-orlando has quit IRC10:11
*** fzdarsky|afk is now known as fzdarsky10:15
*** Yanyanhu has quit IRC10:16
*** Qiming has quit IRC10:22
*** cdent has joined #openstack-sdks10:49
*** lhcheng has joined #openstack-sdks11:07
*** salv-orl_ has quit IRC11:07
*** salv-orlando has joined #openstack-sdks11:08
*** dims has joined #openstack-sdks11:10
*** markvoelker has joined #openstack-sdks11:13
*** Qiming has joined #openstack-sdks11:14
*** markvoelker has quit IRC11:18
*** lucas-dinner is now known as lucasagomes11:20
openstackgerritSai Kiran proposed openstack/python-openstackclient: Refactor abstract columns and datalist out in compute test cases  https://review.openstack.org/26473111:38
*** enthurohini has joined #openstack-sdks11:50
*** markvoelker has joined #openstack-sdks12:14
*** markvoelker has quit IRC12:18
*** thrash|g0ne is now known as thrash12:20
*** rtheis has joined #openstack-sdks12:22
*** woodster_ has joined #openstack-sdks12:54
*** salv-orlando has quit IRC13:03
*** salv-orlando has joined #openstack-sdks13:03
*** enthurohini has quit IRC13:03
*** markvoelker has joined #openstack-sdks13:18
*** gouthamr has joined #openstack-sdks13:35
*** cdent has quit IRC14:02
*** cdent has joined #openstack-sdks14:24
*** lucasagomes is now known as lucas-hungry14:32
*** jose4183 has joined #openstack-sdks14:41
*** petertr7_away is now known as petertr714:43
*** petertr7 is now known as petertr7_away14:55
*** woodster_ has quit IRC15:06
*** enthurohini has joined #openstack-sdks15:14
*** jaypipes has joined #openstack-sdks15:15
*** navidp has left #openstack-sdks15:23
*** lucas-hungry is now known as lucasagomes15:34
*** dims_ has joined #openstack-sdks15:35
*** dims has quit IRC15:36
*** sigmavirus24_awa is now known as sigmavirus2415:42
openstackgerritMerged openstack/python-openstacksdk: Use assertTrue/False instead of assertEqual(T/F)  https://review.openstack.org/26462615:53
openstackgerritMerged openstack/python-openstacksdk: Correct response value in resource unittests  https://review.openstack.org/26196915:54
*** devth_ has joined #openstack-sdks16:05
*** devth has quit IRC16:05
*** devth has joined #openstack-sdks16:06
*** salv-orl_ has joined #openstack-sdks16:08
*** devth_ has quit IRC16:10
*** salv-orlando has quit IRC16:11
*** Qiming has quit IRC16:12
*** woodster_ has joined #openstack-sdks16:13
*** devth has quit IRC16:26
*** devth has joined #openstack-sdks16:26
openstackgerritOpenStack Proposal Bot proposed openstack/python-openstacksdk: Updated from global requirements  https://review.openstack.org/26440716:49
*** enthurohini has quit IRC16:59
*** dims has joined #openstack-sdks16:59
*** dims_ has quit IRC17:02
*** sigmavirus24 is now known as sigmavirus24_awa17:27
openstackgerritMerged openstack/python-openstackclient: Further improve output for "os security group show"  https://review.openstack.org/26384317:35
openstackgerritMerged openstack/os-client-config: Replace assertEqual(None, *) with assertIsNone in tests  https://review.openstack.org/26420617:40
openstackgerritMerged openstack/os-client-config: Remove openstack-common.conf  https://review.openstack.org/26463517:40
*** breton has quit IRC17:44
openstackgerritMerged openstack/python-openstacksdk: Updated from global requirements  https://review.openstack.org/26440717:47
etoewsbriancurtin: terrylhowe: i'm looking at the exception mapping issue. now that we use ksa session, there isn't one single place where we can try/except and make sure everything is mapped properly. i'm not totally sure what's the best design here.17:51
etoewssome stuff i'm considering...17:51
etoews1. move all calls to ksa session methods into our own *very thin* session so we can wrap calls to ksa session in try/except.17:52
etoews2. decorate the calls in ksa session to wrap them in a try/except (not sure how feasible this is)17:53
*** breton has joined #openstack-sdks17:53
etoews3. something i haven't thought of17:53
briancurtinetoews: on 2, that’s pretty much how we could solve this from within Resource, decorating any calls that go out via the session and just have a handler that knows how to translate17:55
briancurtinstill thinking about how that works compared to #1, though17:57
*** cdent has quit IRC17:59
*** breton has quit IRC18:01
*** breton has joined #openstack-sdks18:01
briancurtinetoews: we probably want #1. get the transition done at the earliest time and inside of the SDK we work with our own exceptions18:04
etoewsbriancurtin: okay. i'll take a stab at that one18:06
*** elmiko has quit IRC18:14
rtheisI'm running into problems with proxy methods for PoolMember...  These proxy methods need path_args to set pool_id in URL, but neutron doesn't like pool_id also being passed on the request.  It appears the base proxy updates the resource to include path_args.18:14
rtheisnot sure the best way to fix this...18:14
*** markvoelker has quit IRC18:14
briancurtinrtheis: i think that’s one of the places that is the basis for needing to fix how we do path_args IIRC. i’ll take a look18:15
rtheisok18:15
*** elmiko has joined #openstack-sdks18:16
*** e0ne has joined #openstack-sdks18:23
briancurtinrtheis: what exactly do you mean “also being passed on the request”? do you mean it’s coughing on pool_id showing up in the body as well as in the URI?18:23
rtheisbriancurtin: yes, exactly18:23
rtheisPUT http://127.0.0.1:9696/v2.0/lbaas/pools/57234610-760a-47ce-bfe2-179c789e24cc/members/42db89a5-2f96-4d5c-bd00-d25b89bcf43018:24
briancurtinah, ok, yeah there’s a review out that sort of fixes some of that but needs to be fixed up as we have this issue in a few other areas.18:24
rtheis{"member": {"protocol_port": "81", "pool_id": "57234610-760a-47ce-bfe2-179c789e24cc"}}18:24
briancurtini’ve been meaning to get back into that one as path_args in general is one of the bigger things left that we need to get figured out. i’ll take a look after i finish what i’m on right now18:25
rtheisbriancurtin: cool thanks ... I'll take a look at that review18:25
rtheisbriancurtin: Is https://review.openstack.org/#/c/262367/ the review that you are referring to?18:28
briancurtinrtheis: https://review.openstack.org/#/c/219840/ is the old one, and i’m going to probably revive it there if i can and incorporate ideas from my comment18:29
rtheisthanks18:29
*** fzdarsky has quit IRC18:47
*** fzdarsky has joined #openstack-sdks18:48
*** sigmavirus24_awa is now known as sigmavirus2418:54
*** markvoelker has joined #openstack-sdks19:03
*** markvoelker_ has joined #openstack-sdks19:05
*** e0ne has quit IRC19:08
*** markvoelker has quit IRC19:09
*** markvoelker has joined #openstack-sdks19:14
*** markvoelker_ has quit IRC19:17
openstackgerritHardik Italia proposed openstack/python-openstacksdk: Update router's functional tests to validate is_ha property  https://review.openstack.org/26493219:17
*** petertr7_away is now known as petertr719:20
*** rtheis_ has joined #openstack-sdks19:29
*** nikhil_k has joined #openstack-sdks19:31
sigmavirus24etoews: briancurtin did either of you see the message on the ML from Qiming Teng about namespace conflicts?19:31
*** breton_ has joined #openstack-sdks19:31
*** tangchen__ has joined #openstack-sdks19:32
*** dims_ has joined #openstack-sdks19:33
sigmavirus24woops missed that dhellmann already responded. Mail.app is being not great today19:33
*** dims has quit IRC19:34
*** ameade_ has joined #openstack-sdks19:35
*** etoews_ has joined #openstack-sdks19:35
*** stevemar_znc has joined #openstack-sdks19:37
*** breton has quit IRC19:38
*** rtheis has quit IRC19:38
*** nikhil has quit IRC19:38
*** ameade has quit IRC19:38
*** briancurtin has quit IRC19:38
*** tangchen_ has quit IRC19:38
*** etoews has quit IRC19:38
*** stevemar has quit IRC19:38
*** stevemar_znc is now known as stevemar19:40
*** lhcheng_ has joined #openstack-sdks19:40
*** ameade_ is now known as ameade19:40
*** etoews_ is now known as etoews19:42
*** lhcheng has quit IRC19:43
*** sigmavirus24 is now known as sigmavirus24_awa19:45
*** briancurtin has joined #openstack-sdks19:46
*** sigmavirus24_awa is now known as sigmavirus2419:48
*** lucasagomes is now known as lucas-dinner19:49
*** dhellmann_ has joined #openstack-sdks19:55
*** dims_ has quit IRC20:00
stevemardtroyer: whelp, no meeting today i suppose :)20:06
dtroyerstevemar: jeez…20:08
dtroyermy bad there20:08
*** dhellmann has quit IRC20:11
*** dhellmann_ is now known as dhellmann20:11
*** dhellmann has quit IRC20:12
stevemardtroyer: np20:12
*** dims has joined #openstack-sdks20:12
*** cdent has joined #openstack-sdks20:13
*** dhellmann has joined #openstack-sdks20:14
*** dims has quit IRC20:19
etoewsbriancurtin: looking at things a bit more closely. the sdk already has https://github.com/openstack/python-openstacksdk/blob/master/openstack/session.py20:20
etoewsso for 1. it would be a matter of overriding the methods and wrapping the calls there.20:20
etoewsbut that can potentially break if ksa session methods change or are deleted or added.20:21
*** dims has joined #openstack-sdks20:22
etoewsfor 2 i think it would look something like this http://stackoverflow.com/questions/16697545/wrap-calls-to-methods-of-a-python-class20:23
*** openstackgerrit has quit IRC20:23
*** dims has quit IRC20:23
*** openstackgerrit has joined #openstack-sdks20:25
etoewsi think i'm going to take a stab at 2. just to see how it works.20:28
briancurtinetoews: yeah, sort of, but we’d know which ones we’re using since we’re calling through them. i’m interested in seeing 2 though so go for it20:38
*** dims has joined #openstack-sdks20:39
*** petertr7 is now known as petertr7_away20:48
*** petertr7_away is now known as petertr720:51
*** fzdarsky has quit IRC21:03
*** petertr7 is now known as petertr7_away21:04
*** petertr7_away is now known as petertr721:05
openstackgerritBrant Knudson proposed openstack/keystoneauth: Switch saml2 from lxml to built-in xml  https://review.openstack.org/24251221:21
*** nikhil_k is now known as nikhil21:29
*** gouthamr has quit IRC21:48
*** jaypipes has quit IRC21:50
*** boris-42 has quit IRC21:50
openstackgerritRichard Theis proposed openstack/python-openstacksdk: Fix create_pool_member and update_pool_member  https://review.openstack.org/26498021:51
*** jaypipes has joined #openstack-sdks21:51
*** boris-42 has joined #openstack-sdks21:51
*** cdent has quit IRC21:54
*** rtheis_ has quit IRC22:05
*** salv-orlando has joined #openstack-sdks22:09
*** salv-orl_ has quit IRC22:11
*** gouthamr has joined #openstack-sdks22:19
*** petertr7 is now known as petertr7_away22:22
*** jose4183 has quit IRC22:23
*** thrash is now known as thrash|g0ne22:45
*** lhcheng_ has quit IRC22:48
jamielennoxetoews, briancurtin: not sure if either of you is still here, but what's the probably with the ksa excs?22:52
briancurtinjamielennox: no problem per se with the exceptions, but i don’t want SDK end-users to have to be handling keystonauth1.exceptions.* or something like that. we shouldn’t be leaking things from below22:59
*** dims_ has joined #openstack-sdks23:00
jamielennoxbriancurtin: that makes sense for most things - are you going to end up replicating everything though?23:01
*** dims has quit IRC23:01
briancurtinjamielennox: what i think it’ll end up being is just catching the base HttpError and re-raising as SDK’s HTTPException, using the status and code and stuff. SDK doesn’t have an exception per code (though it does on some common ones we use, such as 404)23:02
briancurtinmostly coalescing down to that one, though we haven’t worked out exactly how it’s being done yet so that may change23:02
jamielennoxbriancurtin: that makes sense, i agree ksa should mostly be an implementation detail23:03
*** sigmavirus24 is now known as sigmavirus24_awa23:12
*** Qiming has joined #openstack-sdks23:32
*** dims_ has quit IRC23:39
*** dims_ has joined #openstack-sdks23:45
*** dims_ has quit IRC23:50

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