Friday, 2015-01-30

*** abhirc has quit IRC00:00
*** abhirc has joined #openstack-keystone00:00
openstackgerritShraddha Pandhe proposed openstack/python-keystoneclient: Keystone tries to save the auth_ref in the keyring in store_auth_ref_in_keyring and the auth_ref retrieval happens in get_auth_ref_from_keyring during authentication. Please note that both these methods are part of httpclient.py and not shell.py. Hence, i  https://review.openstack.org/15147800:02
openstackgerritShraddha Pandhe proposed openstack/python-keystoneclient: Keystoneclient doesn't look at cached auth_ref in keyring  https://review.openstack.org/15147800:03
morganfainberghehe00:03
*** oomichi has joined #openstack-keystone00:05
*** nellysmitt has joined #openstack-keystone00:09
openstackgerritJamie Lennox proposed openstack/python-keystoneclient: Add get_communication_params interface to plugins  https://review.openstack.org/14126700:12
openstackgerritJamie Lennox proposed openstack/python-keystoneclient: Add get_headers interface to authentication plugins  https://review.openstack.org/14089400:13
*** nellysmitt has quit IRC00:14
*** joesavak has quit IRC00:15
*** SpamapS has left #openstack-keystone00:16
*** david-lyle is now known as david-lyle_afk00:31
*** dims has quit IRC00:32
openstackgerritNathan Kinder proposed openstack/oslo.policy: Improve policy documentation  https://review.openstack.org/15095300:44
openstackgerritMerged openstack/keystone: do parameter check before updating endpoint_group  https://review.openstack.org/14604000:45
*** spandhe has quit IRC00:46
*** tellesnobrega_ has quit IRC00:47
*** _cjones_ has quit IRC00:51
*** _cjones_ has joined #openstack-keystone00:52
nkindermorganfainberg: what's the process around getting +A on specs?  Is that only done after discussion in the weekly meeting?00:55
nkindermorganfainberg: was wondering about this one - https://review.openstack.org/#/c/148229/00:55
rodrigodsnkinder, think this one didn't receive a +A because both +2 were from engineers from the same company00:56
nkinderrodrigods: ah, I see00:56
*** fifieldt has quit IRC01:00
rodrigodsstevemar2, maybe you can approve this change? https://review.openstack.org/#/c/148080/ (has already 2 +2)01:00
rodrigodsstevemar2, or we never should +A something that has not approved dependencies?01:01
jamielennoxmorganfainberg: so swift doesn't use oslo.config - AT ALL - how do we handle that and not have people configure auth_token from paste?01:01
* jamielennox thinks that he didn't break devstack - swift broke devstack01:01
*** avozza is now known as zz_avozza01:05
*** jaosorior has quit IRC01:14
*** zzzeek has quit IRC01:18
*** lhcheng has quit IRC01:20
*** zz_avozza is now known as avozza01:26
*** markvoelker_ has quit IRC01:34
*** markvoelker has joined #openstack-keystone01:35
*** markvoelker has quit IRC01:40
*** ayoung has joined #openstack-keystone01:41
*** ChanServ sets mode: +v ayoung01:41
stevemar2rodrigods, yeah, no point in +A'ing something that has a dependency01:43
*** _cjones_ has quit IRC01:44
*** lhcheng has joined #openstack-keystone01:44
*** lhcheng_ has joined #openstack-keystone01:46
*** ayoung has quit IRC01:46
*** lhcheng has quit IRC01:48
*** tellesnobrega_ has joined #openstack-keystone01:53
*** ljfisher has quit IRC01:56
*** samueldmq_ has quit IRC02:00
*** lhcheng_ has quit IRC02:00
*** ayoung has joined #openstack-keystone02:01
*** ChanServ sets mode: +v ayoung02:01
*** packet has quit IRC02:07
*** nellysmitt has joined #openstack-keystone02:10
*** abhirc has quit IRC02:11
*** nellysmitt has quit IRC02:14
morganfainbergjamielennox, uhm..02:16
morganfainbergjamielennox, no good idea02:16
openstackgerritBrant Knudson proposed openstack/keystone: Move eventlet server options to a config section  https://review.openstack.org/13096202:17
openstackgerritBrant Knudson proposed openstack/keystone: Regenerate sample config file  https://review.openstack.org/15150502:17
jamielennoxi think it essentially means i don't get to deprecate passing options via paste02:19
*** erkules_ has joined #openstack-keystone02:19
jamielennoxand i need to come up with some way of supporting both02:19
*** erkules has quit IRC02:22
openstackgerritMerged openstack/keystone-specs: IDP ID registration and validation  https://review.openstack.org/14822902:23
nkindermorganfainberg: thanks for reviewing/approving that spec!02:32
openstackgerritBrant Knudson proposed openstack/keystone: Move eventlet server options to a config section  https://review.openstack.org/13096202:32
openstackgerritBrant Knudson proposed openstack/keystone: Cleanup tests to not set multiple workers.  https://review.openstack.org/15151102:32
morganfainbergjamielennox, yeah :(02:42
morganfainberghrm02:42
*** rushiagr_away is now known as rushiagr02:43
*** rwsu is now known as rwsu-afk02:48
*** harlowja_ is now known as harlowja_away02:58
openstackgerritSteve Martinelli proposed openstack/keystone: Add CADF notifications for most resources  https://review.openstack.org/15113703:10
*** KanagarajM has joined #openstack-keystone03:17
*** rushiagr is now known as rushiagr_away03:17
*** adam_g_out is now known as adam_g03:29
*** topol has joined #openstack-keystone03:33
*** ChanServ sets mode: +v topol03:33
*** topol has quit IRC03:51
*** rushiagr_away is now known as rushiagr03:56
*** rushiagr has quit IRC04:00
*** rushiagr has joined #openstack-keystone04:00
*** richm has quit IRC04:03
*** nellysmitt has joined #openstack-keystone04:11
*** nellysmitt has quit IRC04:15
*** zzzeek has joined #openstack-keystone04:37
*** ncoghlan has joined #openstack-keystone04:54
stevemar2anyone know why we set public=False to the notifications for region/service/endpoint/policy??05:05
stevemar2by doing that we only limit them to being used for callback handling05:06
stevemar2oh maybe it's for the endpoint policy stuff...05:06
*** zzzeek has quit IRC05:08
openstackgerritSteve Martinelli proposed openstack/pycadf: Add a new CADF type for keystone trusts  https://review.openstack.org/15153605:16
*** haneef has joined #openstack-keystone05:32
*** erkules_ is now known as erkules05:32
haneefjamielennox: question - keystoneclient.auth.identity.v3.Password.get_options -- Why don't we register project_id, project_name etc. How does  does this plugin get project info?05:40
jamielennoxhaneef: we do05:40
jamielennox.. we pretty much have to05:40
haneefI done see :  https://github.com/openstack/python-keystoneclient/blob/master/keystoneclient/auth/identity/v3.py#L27305:41
jamielennoxhaneef: it's in the super()05:42
jamielennoxhttps://github.com/openstack/python-keystoneclient/blob/master/keystoneclient/auth/identity/v3.py#L14405:42
haneefok, iI see in baseclass05:42
*** afazekas_ has joined #openstack-keystone05:43
haneefThanks05:43
jamielennoxyep, so all v3 auth plugins are going to need that information to know what to scope against05:43
jamielennoxhaneef: np05:43
*** jaosorior has joined #openstack-keystone05:56
*** nellysmitt has joined #openstack-keystone06:12
*** nellysmitt has quit IRC06:16
openstackgerritOpenStack Proposal Bot proposed openstack/keystone: Imported Translations from Transifex  https://review.openstack.org/15154706:21
openstackgerritSteve Martinelli proposed openstack/keystone: Add CADF notifications for most resources  https://review.openstack.org/15113706:28
*** EmilienM is now known as EmilienM|afk06:29
*** quack_quack_ has quit IRC06:29
openstackgerritSteve Martinelli proposed openstack/keystone: Add CADF notifications for most resources  https://review.openstack.org/15113706:31
*** ajayaa has joined #openstack-keystone06:34
openstackgerritSteve Martinelli proposed openstack/keystone: Add CADF notifications for most resources  https://review.openstack.org/15113706:39
openstackgerritSteve Martinelli proposed openstack/keystone: Refactor _send_audit_notification  https://review.openstack.org/15155106:39
openstackgerritSteve Martinelli proposed openstack/keystone: Refactor _send_audit_notification  https://review.openstack.org/15155106:40
openstackgerritSteve Martinelli proposed openstack/keystone: Add CADF notifications for most resources  https://review.openstack.org/15113706:40
*** stevemar2 is now known as stevemar06:41
openstackgerritMerged openstack/python-keystoneclient: Add validate token for v2.0  https://review.openstack.org/14194406:48
*** afazekas_ has quit IRC07:16
openstackgerritSteve Martinelli proposed openstack/keystone: Add CADF notifications for most resources  https://review.openstack.org/15113707:18
*** stevemar has quit IRC07:24
*** afazekas has quit IRC07:25
*** avozza is now known as zz_avozza07:37
*** ncoghlan has quit IRC07:37
*** zz_avozza is now known as avozza07:52
*** ajayaa has quit IRC08:01
openstackgerritwanghong proposed openstack/keystone: make trust manager raise formatted message exception  https://review.openstack.org/14955008:06
*** chlong has quit IRC08:08
*** nellysmitt has joined #openstack-keystone08:13
*** nellysmitt has quit IRC08:14
*** nellysmitt has joined #openstack-keystone08:14
openstackgerritMerged openstack/keystone: Imported Translations from Transifex  https://review.openstack.org/15154708:18
*** pnavarro has joined #openstack-keystone08:27
*** avozza is now known as zz_avozza08:35
*** KanagarajM has quit IRC08:35
*** wanghong has quit IRC08:35
*** zhiyuan has quit IRC08:35
*** breton_ is now known as bretom08:36
*** bretom is now known as breton08:36
*** oomichi has quit IRC08:39
*** zhiyuan has joined #openstack-keystone08:41
openstackgerritYuriy Taraday proposed openstack/keystone: Add a module to work with LDAP filters and DNs  https://review.openstack.org/11748408:41
*** zhiyuan has quit IRC08:42
*** zhiyuan has joined #openstack-keystone08:43
*** zhiyuan has quit IRC08:45
*** zhiyuan has joined #openstack-keystone08:46
*** bjornar has quit IRC08:54
*** KanagarajM has joined #openstack-keystone09:03
*** bjornar has joined #openstack-keystone09:09
*** afazekas has joined #openstack-keystone09:17
*** fmarco76 has joined #openstack-keystone09:26
*** jistr has joined #openstack-keystone09:31
openstackgerritMarek Denis proposed openstack/keystone: Implement Service Providers API for OS-FEDERATION  https://review.openstack.org/10462309:35
*** fmarco76 has left #openstack-keystone09:55
*** fmarco76 has joined #openstack-keystone09:58
*** fmarco76 has left #openstack-keystone09:58
*** fmarco76 has joined #openstack-keystone10:02
openstackgerritwanghong proposed openstack/keystone: add missing links for v3 OS-EC2 API response  https://review.openstack.org/15159210:14
*** tellesnobrega_ has quit IRC10:16
*** tsufiev_ is now known as tsufiev10:26
*** henrynash has joined #openstack-keystone10:30
*** ChanServ sets mode: +v henrynash10:30
*** tellesnobrega_ has joined #openstack-keystone10:32
*** jamielennox is now known as jamielennox|away10:36
*** jacer_huawei has joined #openstack-keystone10:37
*** chlong has joined #openstack-keystone10:43
*** tellesnobrega_ has quit IRC10:44
*** jamielennox|away is now known as jamielennox10:44
*** krykowski has joined #openstack-keystone10:46
*** nellysmitt has quit IRC10:50
*** breton has quit IRC10:54
*** krykowski has quit IRC10:55
*** henrynash has quit IRC10:55
*** jamielennox is now known as jamielennox|away10:56
*** breton has joined #openstack-keystone10:58
*** andreaf has joined #openstack-keystone11:01
*** krykowski has joined #openstack-keystone11:04
*** jaosorior has quit IRC11:06
*** jaosorior has joined #openstack-keystone11:06
*** gtt116_ has joined #openstack-keystone11:07
*** gtt116 has quit IRC11:11
openstackgerritMarek Denis proposed openstack/keystone: Service Providers API for OS-FEDERATION  https://review.openstack.org/10462311:14
*** henrynash has joined #openstack-keystone11:17
*** ChanServ sets mode: +v henrynash11:17
*** jacer_huawei has quit IRC11:19
*** jacer_huawei has joined #openstack-keystone11:20
*** krykowski has quit IRC11:25
*** krykowski has joined #openstack-keystone11:26
*** KanagarajM has quit IRC11:28
openstackgerritMarco Fargetta proposed openstack/keystone: Multiple IDP authentication URL  https://review.openstack.org/14274311:29
*** obutenko_ has joined #openstack-keystone11:30
*** aix has joined #openstack-keystone11:34
*** zz_avozza is now known as avozza11:41
openstackgerritMarco Fargetta proposed openstack/keystone: Multiple IDP authentication URL  https://review.openstack.org/14274311:50
*** gabriel-bezerra has joined #openstack-keystone11:51
*** BobBall has left #openstack-keystone11:53
*** diegows has joined #openstack-keystone11:55
*** diegows has quit IRC11:55
*** diegows has joined #openstack-keystone11:56
*** fmarco76 has left #openstack-keystone12:00
*** krykowski has quit IRC12:08
*** nellysmitt has joined #openstack-keystone12:10
*** jasondotstar has quit IRC12:12
*** andreaf has quit IRC12:13
*** andreaf has joined #openstack-keystone12:14
*** krykowski has joined #openstack-keystone12:14
*** aix has quit IRC12:22
openstackgerrithenry-nash proposed openstack/keystone: Support data-driven backend assignment testing  https://review.openstack.org/14917812:28
*** nellysmitt has quit IRC12:31
openstackgerrithenry-nash proposed openstack/keystone: Add support for data-driven backend assignment testing  https://review.openstack.org/14917812:38
openstackgerrithenry-nash proposed openstack/keystone: Add support for effective & inherited mode in data driven tests  https://review.openstack.org/15162312:39
openstackgerrithenry-nash proposed openstack/keystone: Add support for effective & inherited mode in data driven tests  https://review.openstack.org/15162312:40
*** krykowski has quit IRC12:56
*** aix has joined #openstack-keystone13:03
*** rushiagr is now known as rushiagr_away13:06
*** henrynash has quit IRC13:07
*** ljfisher has joined #openstack-keystone13:13
*** tellesnobrega_ has joined #openstack-keystone13:16
*** ljfisher has quit IRC13:16
*** nellysmitt has joined #openstack-keystone13:18
*** henrynash has joined #openstack-keystone13:19
*** ChanServ sets mode: +v henrynash13:19
*** bdossant has joined #openstack-keystone13:22
*** krykowski has joined #openstack-keystone13:23
*** tellesnobrega_ has quit IRC13:25
*** EmilienM|afk is now known as EmilienM13:28
*** radez_g0n3 is now known as radez13:28
rodrigodshenrynash, updated the reseller spec with the content of our discussion yesterday :)13:34
henrynashrodigods: ok…will look later13:35
rodrigodshenrynash, thx!13:36
raildohenrynash, thanks :)13:37
*** markvoelker has joined #openstack-keystone13:37
*** henrynash has quit IRC13:37
*** rushiagr_away is now known as rushiagr13:48
*** markvoelker has quit IRC13:51
*** markvoelker has joined #openstack-keystone13:52
*** markvoelker has quit IRC13:57
*** mflobo1 has quit IRC13:58
*** henrynash has joined #openstack-keystone14:02
*** ChanServ sets mode: +v henrynash14:02
*** krykowski has quit IRC14:08
*** mattfarina has joined #openstack-keystone14:10
*** raildo has quit IRC14:10
*** richm has joined #openstack-keystone14:13
*** angelamolock has quit IRC14:13
*** krykowski has joined #openstack-keystone14:14
*** raildo has joined #openstack-keystone14:15
*** jasondotstar has joined #openstack-keystone14:15
*** openstackstatus has joined #openstack-keystone14:20
*** ChanServ sets mode: +v openstackstatus14:20
-openstackstatus- NOTICE: zuul isn't running jobs since ~10:30 utc, investigation underway14:24
*** ChanServ changes topic to "zuul isn't running jobs since ~10:30 utc, investigation underway"14:24
*** henrynash_ has joined #openstack-keystone14:27
*** ChanServ sets mode: +v henrynash_14:27
*** henrynash has quit IRC14:28
*** henrynash_ is now known as henrynash14:28
*** EmilienM is now known as EmilienM|afl14:35
*** EmilienM|afl is now known as EmilienM|afk14:35
*** dims has joined #openstack-keystone14:36
*** henrynash has quit IRC14:36
*** bknudson has joined #openstack-keystone14:38
openstackgerritDavid Stanek proposed openstack/keystone: Support for running functional federation tests  https://review.openstack.org/13913714:38
openstackgerritDavid Stanek proposed openstack/keystone: adds a devstack plugin for setting up federation  https://review.openstack.org/15131114:38
*** ChanServ sets mode: +v bknudson14:38
*** gtt116_ has quit IRC14:45
*** gtt116_ has joined #openstack-keystone14:45
*** joesavak has joined #openstack-keystone14:50
openstackgerritayoung proposed openstack/keystone-specs: Merge role id and role name  https://review.openstack.org/15165714:52
*** diegows has quit IRC14:53
*** gordc has joined #openstack-keystone15:09
*** avozza is now known as zz_avozza15:14
openstackgerritAlexander Makarov proposed openstack/keystone: Redis token backend  https://review.openstack.org/15084415:18
*** nellysmitt has quit IRC15:18
*** r-daneel has joined #openstack-keystone15:22
*** markvoelker has joined #openstack-keystone15:22
*** stevemar has joined #openstack-keystone15:24
*** ChanServ sets mode: +v stevemar15:24
*** markvoelker has quit IRC15:25
*** markvoelker has joined #openstack-keystone15:25
*** zz_avozza is now known as avozza15:29
*** timcline has joined #openstack-keystone15:30
*** angelamolock has joined #openstack-keystone15:32
*** packet has joined #openstack-keystone15:33
*** dims has quit IRC15:39
*** krykowski has quit IRC15:41
*** topol has joined #openstack-keystone15:42
*** ChanServ sets mode: +v topol15:42
*** carlosmarin has joined #openstack-keystone15:44
marekdHm, I just noticed mappings and protocols don't have 'enabled' attribute.15:44
*** avozza is now known as zz_avozza15:44
*** dims has joined #openstack-keystone15:46
*** kfox1111 has joined #openstack-keystone15:48
*** dims has quit IRC15:50
*** kragniz is now known as zingark15:53
marekdmorganfainberg: what status code should keystone response if an user tries to do the action on disabled object?15:55
marekd403?15:55
*** abhirc has joined #openstack-keystone15:55
*** timcline has quit IRC15:56
*** timcline has joined #openstack-keystone15:57
openstackgerritBrant Knudson proposed openstack/python-keystoneclient: Add validate token for v3  https://review.openstack.org/14214715:58
openstackgerritBrant Knudson proposed openstack/python-keystoneclient: Tests use keep_blank_values when parse_qs  https://review.openstack.org/15147115:58
*** zz_avozza is now known as avozza16:05
openstackgerritMarek Denis proposed openstack/keystone: During authentication validate if IdP is enabled  https://review.openstack.org/15168316:06
*** EmilienM|afk is now known as EmilienM16:07
ayoungdstanek, how do I trigger -f (fail fast) option with tox for -ep27?16:08
dstanekayoung: i think it's 'tox -e py27 -- --failfast'16:09
ayoungdstanek, nope16:09
dstanekayoung: what happens when you do that?16:09
ayoungdstanek, the -- -failfast gets passed to setup.py16:10
ayoungdstanek, http://paste.fedoraproject.org/178444/22634244/16:10
ayoungmaybe -- -- ?16:11
dstanekayoung: hmm...let me look. that looks like it gets passed to testr OK, but it doesn't know what it is16:11
ayoungdstanek, and doing tox -e py27 -- -- --failfast16:12
ayoung  just gets ignored an all the tests run16:12
*** ChanServ changes topic to "Release Blockers: https://gist.github.com/dolph/651c6a1748f69637abd0 << please review for client release on Feb 1st | http://opensax.com/ | Reviews Guarantee Citizenship </starship troopers>"16:12
-openstackstatus- NOTICE: zuul is running again and changes have been reenqueud. seehttp://status.openstack.org/zuul/ before rechecking if in doubt16:12
ayoungdstanek, I'm doing this on python-keystoneclient btw16:12
*** bjornar has quit IRC16:18
dstanekayoung: i got the command to work by using 'tox -e py27 -- " -- --failfast"'16:18
ayoungjoy16:18
dstanekayoung: but it doesn't seem to actually stop - it just doesn't complain about the args16:18
ayoungdstanek, that still seems to run multiple tests16:19
dstanektestr is so much more painful than just running nose16:19
ayoungmaybe due to parallelization?16:19
*** david-lyle_afk is now known as david-lyle16:19
dstanekayoung: yep that was it16:20
dstanekTEST_RUN_CONCURRENCY=1 tox -e py27 -- " -- --failfast"16:20
dstanekand that's my final answer!16:20
ayoungdstanek,  nope16:20
dstanekadd -- that actually works for me16:21
ayoung$ TEST_RUN_CONCURRENCY=1 tox -e py27 -- " -- --failfast"  2>&1 | fpaste16:21
ayoungUploading (10.0KiB)...16:21
ayounghttp://ur1.ca/jlafn -> http://paste.fedoraproject.org/178453/1422634816:21
dstaneks/add/odd/16:21
dstaneki wonder why you env isn't picking up the env variable16:22
dstanekthat variable is used in .testr.conf16:22
ayoungdstanek, too many layers of cruft here.16:23
dstanekayoung: yes, the openstack way!16:24
ayoungdstanek, its not my way16:24
ayoungmy way is to hide behind a boulder and hit him in the head with a rock16:24
ayoungmy way's not very sportsmanlike16:24
dstanekthis is why i still use nose to run tests16:24
*** nellysmitt has joined #openstack-keystone16:25
ayoungdstanek, yeah, but that means we are not communicating with the community about how best to do things16:25
ayoungrun_test.sh still exsits, but we all now use tox for the venv...16:25
ayoungI work with this crap all day and I still get hung, imagine the casual developers plight16:26
dstanektestr actually makes me angry :-(16:26
dstanekwe use tox to run a command under different versions of Python16:27
dstanekin tox the command we run is testr16:27
dstanektestr is really just a wrapper about subunit - it first call a list command and feeds the results in the a run command16:27
dstanektestr is configured in .testr.conf16:28
dstanekall of this means that command line args are carried through the various commands in an insane and hard to follow way16:28
ayoungdstanek, so what *should* we  be doing?16:29
*** avozza is now known as zz_avozza16:30
*** abhirc has quit IRC16:30
*** dims has joined #openstack-keystone16:31
*** dims has quit IRC16:31
*** dims has joined #openstack-keystone16:32
*** zz_avozza is now known as avozza16:32
*** zzzeek has joined #openstack-keystone16:33
*** dims is now known as dimsum__16:33
dstanekayoung: i'd rather be on on nose, but i don't think openstack will ever go back16:34
ayoungdstanek, what, specifically, would nose replace?  all of testr, or just the subcommand part?16:34
dstanekall of testr16:35
dstanekso far it has no provided any value to me ... only pain16:35
ayoungdstanek, it is all pain to me16:36
*** jasondotstar has quit IRC16:36
dstanekwe use nose for python3 tests now because testr must crawl and parse the entire code tree even if you only want to run a single test16:36
ayoungcould we at least have alternate tox rules to run nose?16:36
dstaneki do that now for myself - otherwise i'd be going nuts16:37
*** andreaf has quit IRC16:37
dstaneki also use nose directly in my vim config to run tests, which i can't do with testr16:37
*** andreaf has joined #openstack-keystone16:38
ayoungdstanek, I'm just realizing how dreadful out sample data was in the client tests16:39
ayounglatest one seems to be endpoints-without-ids16:39
dstanekmarekd: what kind of operation?16:40
raildoayoung, ping, we updated the reseller spec with what we discuss yesterday, if you have some time to take a look, i appreciate :) https://review.openstack.org/#/c/139824/25/specs/kilo/reseller.rst16:41
ayoungraildo, at some point...but I have more basic errors to fix first16:41
raildoayoung, ok16:42
*** rwsu-afk is now known as rwsu16:49
openstackgerritBrant Knudson proposed openstack/keystone: Change oslo.utils to oslo_utils  https://review.openstack.org/14801916:49
openstackgerritBrant Knudson proposed openstack/keystone: Change oslo.serialization to oslo_serialization  https://review.openstack.org/14802516:49
openstackgerritBrant Knudson proposed openstack/keystone: Change oslo.config to oslo_config  https://review.openstack.org/14525016:49
openstackgerritBrant Knudson proposed openstack/keystone: Change oslo.db to oslo_db  https://review.openstack.org/14802916:49
openstackgerritBrant Knudson proposed openstack/keystone: Change oslo.messaging to oslo_messaging  https://review.openstack.org/14802816:49
* stevemar patiently waits for someone to review his notification work *whistles*16:54
*** packet has quit IRC16:54
*** packet has joined #openstack-keystone16:56
*** KanagarajM2 has joined #openstack-keystone16:59
*** bdossant has quit IRC17:00
*** angelamo_ has joined #openstack-keystone17:00
*** angelamolock has quit IRC17:04
dstanekare we still keeping catalog KVS around?17:05
*** zingark is now known as krangiz17:07
*** krangiz is now known as kragniz17:07
*** chuckcarmack has joined #openstack-keystone17:08
stevemardstanek, i think so, isn't it needed for templated?17:11
*** gyee has joined #openstack-keystone17:11
*** ChanServ sets mode: +v gyee17:11
openstackgerritBrant Knudson proposed openstack/keystone: Consistently use oslo_config.cfg.CONF  https://review.openstack.org/14736717:12
dstanekstevemar: it does use it now, but there is no reason that it can't be collapsed17:13
stevemardstanek, kill it with fire?17:13
stevemarthe catalog backend is a mess17:13
*** obutenko_ has quit IRC17:16
*** afazekas has quit IRC17:23
*** lhcheng has joined #openstack-keystone17:24
openstackgerritMerged openstack/python-keystoneclient: Tests use keep_blank_values when parse_qs  https://review.openstack.org/15147117:30
dstanekstevemar: yeah, looking into some weird test behavior now17:30
stevemardstanek, currently the templated catalog doesn't return IDs for endpoints17:35
*** nellysmitt has quit IRC17:35
stevemardstanek, and doesn't enforce it either17:35
stevemarrefer to bugs here: https://review.openstack.org/#/c/131663/17:35
stevemarso consumers (like pycadf, and even ksc/ksm) make assumptions about what's returned, and left throwing exceptions17:36
stevemaror weird round-about code17:36
*** _cjones_ has joined #openstack-keystone17:39
*** dimsum__ has quit IRC17:40
*** harlowja_away is now known as harlowja_17:50
*** kfox1111 has quit IRC17:52
*** alex_xu has quit IRC17:52
*** alex_xu has joined #openstack-keystone17:55
*** abhirc has joined #openstack-keystone17:55
morganfainbergmarekd, i think we 404 on disabled17:56
morganfainbergmarekd, but i'd look at other APIs to be sure17:56
raildomorganfainberg, ping, do you have some time to review the reseller spec?  we updated with what we discuss yesterday :D https://review.openstack.org/#/c/139824/25/specs/kilo/reseller.rst17:56
morganfainbergmarekd, i'll see what i can dig up, but it also depends on the operation17:56
morganfainbergraildo, i am just getting going today but yes.17:56
dstanekmorganfainberg: what's the final word on the kvs backend for catalogs? should be removed right?17:56
raildomorganfainberg, ok, thanks :)17:57
morganfainbergdstanek, if we can17:57
morganfainbergdstanek, it's a mes.17:57
morganfainbergmess*17:57
dstanekmorganfainberg: but templated stays?17:57
morganfainbergdstanek, yes, it's actually used by a number of deployers17:57
morganfainbergCMS vs API configuration17:57
morganfainbergi don't like it, but i can't justify removing it17:57
*** kfox1111 has joined #openstack-keystone17:58
dstanekmorganfainberg: what's awesome is that since the base class is deprecated both kvs and templated catalogs log deprecation warnings17:58
morganfainbergdstanek, yeah we need to fix that17:58
dstanekmorganfainberg: i started working on a patch earlier that i can finish up today17:59
morganfainbergcool17:59
*** harlowja_ has quit IRC18:04
*** _cjones_ has quit IRC18:04
*** TempLPBugBot has quit IRC18:06
*** TempLPBugBot has joined #openstack-keystone18:06
*** _cjones_ has joined #openstack-keystone18:07
*** charz has quit IRC18:07
*** harlowja has joined #openstack-keystone18:07
*** dobson has quit IRC18:07
*** dimsum__ has joined #openstack-keystone18:07
*** mkoderer has quit IRC18:07
morganfainbergraildo, reviewed - minor comments that should be easy to address before this goes in18:08
*** samueldmq has quit IRC18:08
*** amakarov has quit IRC18:08
raildomorganfainberg, thanks! I'll fix and send a new patch now :)18:08
morganfainbergraildo, added 1 more comment18:09
morganfainbergre: marking the new API calls experimental18:09
*** Qlawy has quit IRC18:09
raildomorganfainberg, ok, I will do that.18:09
*** alex_xu has quit IRC18:09
*** radez is now known as radez_g0n318:10
*** Qlawy has joined #openstack-keystone18:10
*** alex_xu has joined #openstack-keystone18:11
*** radez_g0n3 is now known as radez18:12
*** mkoderer has joined #openstack-keystone18:13
*** charz has joined #openstack-keystone18:14
morganfainbergstevemar, is there anything besides the pycadf lib we need to document taxonomy?18:16
morganfainbergstevemar, re: https://review.openstack.org/#/c/151536/118:16
*** dobson has joined #openstack-keystone18:19
*** amakarov has joined #openstack-keystone18:20
*** samueldmq has joined #openstack-keystone18:22
openstackgerritMerged openstack/oslo.policy: Explicit configuration object  https://review.openstack.org/15096918:23
*** markvoelker has quit IRC18:25
openstackgerritMerged openstack/oslo.policy: Improve policy documentation  https://review.openstack.org/15095318:27
*** KanagarajM2 has quit IRC18:37
*** rushiagr is now known as rushiagr_away18:37
*** spandhe has joined #openstack-keystone18:40
-openstackstatus- NOTICE: Gerrit and Zuul will be offline from 1900 to 1930 UTC for project renames18:42
*** angelamo_ has quit IRC18:47
*** angelamolock has joined #openstack-keystone18:51
*** henrynash has joined #openstack-keystone18:53
*** ChanServ sets mode: +v henrynash18:53
*** angelamolock has quit IRC18:53
*** angelamolock has joined #openstack-keystone18:54
openstackgerritMarco Fargetta proposed openstack/keystone: Multiple IDP authentication URL  https://review.openstack.org/14274318:59
*** jaosorior has quit IRC19:06
-openstackstatus- NOTICE: Gerrit and Zuul are offline until 1930 UTC for project renames19:07
*** ChanServ changes topic to "Gerrit and Zuul are offline until 1930 UTC for project renames"19:07
*** jistr has quit IRC19:08
*** jasondotstar has joined #openstack-keystone19:10
*** bknudson has quit IRC19:14
*** tqtran_afk has joined #openstack-keystone19:17
*** tqtran_afk is now known as tqtran19:18
*** tqtran is now known as tqtran_afk19:20
*** henrynash has quit IRC19:22
*** ChanServ changes topic to "Release Blockers: https://gist.github.com/dolph/651c6a1748f69637abd0 << please review for client release on Feb 1st | http://opensax.com/ | Reviews Guarantee Citizenship </starship troopers>"19:30
-openstackstatus- NOTICE: Gerrit is back online19:30
*** vhoward has left #openstack-keystone19:30
*** zzzeek has quit IRC19:34
*** nellysmitt has joined #openstack-keystone19:35
*** nellysmitt has quit IRC19:40
raildomorganfainberg, just one doubt about the henrynash question in the reseller spec (line 311), what do you think? maybe we need use the domain id instead the domain name?19:43
raildohttps://review.openstack.org/#/c/139824/25/specs/kilo/reseller.rst19:43
*** zzzeek has joined #openstack-keystone19:44
morganfainbergIt's a real issue.19:45
morganfainbergWe will need to solve that in either case (domain name)19:45
morganfainbergAnd how to reference the hierarchy.19:45
*** vhoward has joined #openstack-keystone19:45
morganfainbergayoung: ^ thoughts?19:46
*** EmilienM is now known as EmilienM|afk19:47
ayoungmorganfainberg, so domain_id is globally unique.  domain name is unique in the namespace...what is the issue?19:47
morganfainbergReferencing a hierarchy.19:47
ayoungI don't think we "turn projects into domains"19:47
morganfainbergBy name.19:47
morganfainbergDelimiter? Since we have no reserved characters atm.19:48
*** dimsum__ has quit IRC19:48
ayoungthe name of a domain will be an URL19:48
ayoungthe delimiter will be the /19:48
ayoung(fragemtn of an url really)19:49
morganfainbergayoung: I think you can use that today though in a domain name19:49
morganfainbergMeaning we just broke potential users doing that.19:49
morganfainbergThis is a hard compatibility issue.19:49
ayoungI would think that roles are not inherited across domain boundaries by default19:49
* morganfainberg checks schema for domains.19:49
ayoungmorganfainberg, ?19:49
ayoungdoimain names can't have / in them today19:50
ayoungcan they?19:50
morganfainbergYou sure?19:50
morganfainbergI think they can. Checking the json schema.19:50
ayoungdo we use them in URLs anywhere?19:50
morganfainbergDon't think so19:51
morganfainbergAnd schema says "string". So if string includes '/'....19:51
ayoungso lets not allow nested domains under the existing API.19:52
ayoungif we can't do it cleanly, we need to provide a better mech19:52
morganfainbergayoung: so only new domains can be in a hierarchy?19:52
*** thedodd has joined #openstack-keystone19:53
ayoungif names cannot be URL safe due to backwards compat reasons, we need a differnt name mechanism.  Names become an array or something19:53
morganfainbergI wonder if we can special-case existing domains with a / in them.19:53
rodrigodswhy we don't allow request a token using the entity name for not roots?19:53
rodrigodsbad english ^19:53
ayoungNo.  Let's explicitly not allow domains with / in it to start19:54
morganfainbergrodrigods: I think that is a worse solution.19:54
ayounggive people a chance to migrate19:54
ayoungand then do them as URLs19:54
morganfainbergayoung: my point was no one can make new domains with / and old domains can't participate in a hierarchy of they have a /19:54
morganfainbergThat kind of special case.19:54
ayoungthe alternative is doman : name : ['a','b','c']19:55
morganfainbergayoung: gross. :P this isn't c :P19:55
ayoungmorganfainberg, no, this is JSON19:55
rodrigodsI liked ayoung's idea19:55
morganfainbergNo this is Sparta19:55
raildohaha19:56
rodrigodslol19:56
*** angelamolock has quit IRC19:56
ayoungthe alternative is doman : { name : ['a','b','c']  }  instead of       domain: { name : 'a/b/c'  }19:56
rodrigodsif we could use lists in the field name, they could represent hierarchies19:56
raildosounds good to me19:56
morganfainbergayoung: that will definitely break current api.19:56
rodrigodsas ayoung is proposing19:56
ayoungI know19:56
rodrigodsmorganfainberg, true :(19:56
morganfainbergayoung: the answer is Id rather exempt / as a delimiter.19:57
ayoungfine by me19:57
morganfainbergAnd domains with / in the name can't be in a hierarchy (but can be renamed).19:57
raildomorganfainberg, ok, I'll put this in the spec19:57
ayoungwe need to start using URLs as the primary way we refer to objects.  Our current approach is too custom19:57
morganfainbergayoung: until now we've used ids due to a lack of hierarchy to solve that issue (or dodge it more appropriately)19:58
ayoungmorganfainberg, if there is a / in the name at all it will probably make parsing ... bad19:58
morganfainbergSame will need to apply to projects.19:58
morganfainbergOh boy, not going to be fun :(19:59
*** aix has quit IRC19:59
morganfainbergayoung: alternative. Is a mid ground19:59
morganfainbergSpecifying the hierarchy is an array.20:00
rodrigodsmorganfainberg, thx for the review in the parent_as_ids patch, ayoung lost the opportunity to review it20:00
morganfainbergName is *never* used as the flat string in the hierarchy.20:00
morganfainbergBut that becomes weird20:00
morganfainbergBah.20:00
ayoungmorganfainberg, flat string is the degenerate case, and is treated the same as a singe element array ["doma"]20:06
*** openstackgerrit has quit IRC20:06
*** openstackgerrit has joined #openstack-keystone20:06
morganfainbergayoung the issue is what our API returns to the client20:07
ayoungbut dropping "/" as a valid char in a domain name is a better approach20:07
morganfainbergthat would be the breaking part.20:07
ayoungin the token?20:07
morganfainberganywhere20:07
morganfainbergif you need to reference the hierarchy20:08
morganfainberganyway20:08
morganfainbergmaking '/' a reserrved seems much better20:08
ayoungyes20:08
ayounglets proposed for it everywhere, and then people will object to the real cases they have20:08
morganfainbergayoung, x-project spec?20:11
ayounghuh?20:12
morganfainberg"propose everywhere"20:12
morganfainbergor just everywehre in keystone?20:12
stevemarmorganfainberg, hey, question about notifications.... according to http://docs.openstack.org/developer/keystone/event_notifications.html we emit notifications for region/endpoint/service/policy, but those are actually *not* emitted publicly20:13
morganfainbergstevemar, sounds like we should be emitting them20:13
morganfainbergthen20:13
stevemari think those 4 are only used for callback handling20:13
morganfainberghere is the deal20:14
morganfainbergwould you want to audit on an action happening to a region, endpoint, service, or policy?20:14
morganfainbergif the answer is yes - [or take an action based on that event]20:14
morganfainbergit should emit20:14
stevemarlet me think about it20:14
stevemartopol, do you have an opinion on that ^20:15
topolstevemar, on what?20:15
stevemartopol, so according to http://docs.openstack.org/developer/keystone/event_notifications.html we emit notifications on region/endpoint/service CRUD - but we actually don't, we just use them for internal callbacks.20:16
stevemartopol, so do you think a region/endpoint/service CRUD event is audit-worthy?20:17
stevemarthose are typically admin only events20:17
topolstevemar onesec20:17
*** david-lyle has quit IRC20:25
*** harlowja is now known as harlowja_away20:29
*** tqtran_afk is now known as tqtran20:29
openstackgerritayoung proposed openstack/python-keystoneclient: Access Info  https://review.openstack.org/13851920:30
openstackgerritayoung proposed openstack/python-keystoneclient: add issued_at and id values to sample data  https://review.openstack.org/15176120:30
*** harlowja_away is now known as harlowja20:34
openstackgerritRaildo Mascena de Sousa Filho proposed openstack/keystone-specs: Reseller  https://review.openstack.org/13982420:34
topolstevemar, so two options.  My gut says I think  region,endpoint, service is all audit worthy. In an audit those are things you would want to know20:36
*** avozza is now known as zz_avozza20:36
topolstevemar, but at the same time its okay to be stakeholder driven. So if needed you can defer it if we don't have a stakeholder asking for it yet20:37
topolstevemar, but I think eventually someone will want audit records for that20:37
topolstevemar, make sense?20:38
*** richm has quit IRC20:40
*** dimsum__ has joined #openstack-keystone20:41
stevemartopol, yep, makes sense... i'm asking our stakeholder now20:46
morganfainbergtopol is a steak holder?20:46
topolgreat minds think alike20:46
stevemartopol, I am just wondering *why* they were not displayed publicly in the first place20:46
morganfainbergstevemar, doc shuffling?20:46
stevemarmorganfainberg, no, i don't think so, my gut says it was something to do with the endpoint policy extension work20:47
*** bknudson has joined #openstack-keystone20:47
*** ChanServ sets mode: +v bknudson20:47
morganfainbergtopol, watch out the vampires will be scared of you with all those steaks you're holding20:47
openstackgerritSteve Martinelli proposed openstack/keystone: Add CADF notifications for most resources  https://review.openstack.org/15113720:47
morganfainbergoh.. not that kind of steak?20:47
morganfainbergprime cuts?20:47
morganfainbergchoice?20:48
morganfainberg>.>20:48
stevemarhar-dee-har-har20:48
*** g2` is now known as [OO]20:48
morganfainbergstevemar, ಠ_ಠ20:48
stevemaroh damn, henrys stuff is going through!20:49
stevemarHENRYS STUFF IS GOING THROUGH!20:49
*** [OO] is now known as g2`20:49
morganfainbergstevemar, SHHH!20:49
*** spandhe has quit IRC20:50
*** pnavarro has quit IRC20:51
*** radez is now known as radez_g0n320:51
rodrigodsmorganfainberg, stevemar omg where is he?20:51
rodrigodswe have a party in UK today20:51
morganfainbergrodrigods, in the UK >.>20:51
openstackgerritRaildo Mascena de Sousa Filho proposed openstack/keystone-specs: Reseller  https://review.openstack.org/13982420:52
rodrigodsmorganfainberg, ^20:53
rodrigods:)20:53
stevemarhttps://github.com/openstack/keystone/commit/8612a54f91a6d5e4afe1fe2dc484c3fe063bbe5820:53
stevemarinteresting20:53
* morganfainberg needs to find someone with a OnePlus invite20:53
* morganfainberg is getting tired of ios8 bugs.20:54
* morganfainberg is *REALLY* tired of Yosemite sucking as an OS20:54
raildomorganfainberg, come to android world :D20:54
morganfainbergsee my first coment20:54
rodrigodsis morganfainberg an apple fan?20:54
morganfainbergneed a oneplus invite20:54
*** richm has joined #openstack-keystone20:55
morganfainbergi am very anti-carrier phone20:55
stevemarmorganfainberg, i think i can get one for you20:55
morganfainbergiphone, oneplus, and nexus 6 are the best options to be anti-carrier20:55
morganfainbergespecially if you hate.. i mean i wont ever own a phone based on it hate... touchwiz20:55
morganfainbergit's an absolutely bloody awful android skin20:56
*** abhirc has quit IRC20:56
rodrigodsOnePlus fame hasn't arrived in south america yet20:56
stevemarmorganfainberg, touchwiz really is awful20:56
morganfainbergstevemar, i'd place my order today with an invite for OnePlus20:56
rodrigodsmorganfainberg, stevemar ++20:56
rodrigodsI once had a nexus 420:56
rodrigodshad a tragic death :(20:56
morganfainbergand frankly nexus 6 is too pricy for what it is.20:56
stevemarmorganfainberg, i have a buddy that received his a week ago, i think he has an invite, bugging him now - no reply yet20:57
morganfainbergbeen hoping mordred gets his invites soon20:57
morganfainbergbut he doens't have invite(s) yet iirc20:57
*** zz_avozza is now known as avozza20:57
*** spandhe has joined #openstack-keystone20:59
*** andreaf has quit IRC21:02
openstackgerritSteve Martinelli proposed openstack/keystone: Publicize region/endpoint/policy/service events  https://review.openstack.org/15177421:04
stevemarmorganfainberg, topol ^21:05
morganfainbergstevemar, -421:05
morganfainbergstevemar, i mean >.>21:05
stevemar:O21:05
morganfainbergstevemar, easy review: https://review.openstack.org/#/c/151381/21:08
*** abhirc has joined #openstack-keystone21:08
*** david-lyle has joined #openstack-keystone21:08
morganfainbergsmall code review, but not easy: https://review.openstack.org/#/c/148354/21:08
*** chuckcarmack has left #openstack-keystone21:10
morganfainbergrodrigods, this https://review.openstack.org/#/c/148567/ is going to merge conflict21:11
morganfainbergbut henry's wont21:12
morganfainbergso. -- yeah i know headache21:12
morganfainbergso basically not really fixable until henry's assignment patch merges21:12
*** dimsum__ has quit IRC21:15
gyeemorganfainberg, true or fase? keystone CLI has been deprecated?21:21
morganfainberggyee, keystoneclient CLI is frozen.21:21
topolstevemar, that notifications decorator you are using sure looks really handy!21:21
morganfainberggyee, it is not deprecated because $badexperienceforusersonolddeployments$21:22
gyeemorganfainberg, thank you sir, I'll convey the message21:24
morganfainberggyee, in short a security fix could go into keystoneclient cli. no new functionality. use OSC for $new hotness$21:24
morganfainberg(not to be confused with old busted hotness /MiB reference)21:24
gyeeheh21:25
topolmorganfainberg, QQ  We have an awesome video in chinese that covers how to contribute to OpenStack.  It is Film Studio production quality.  Would it be approrpiate for me to advertise it on the OpenStack dev mailing list?  http://www.kaikeba.com/courses/27421:25
topolgyee ^^^^21:25
morganfainbergtopol, QQ means something very different for a former game-system-dev :P21:25
gyeenice21:25
morganfainbergtopol, >.>21:25
morganfainbergtopol, i uh.. dunno21:25
morganfainbergtopol, i think i'm the wrong person to ask.21:26
topolmorganfainberg. No worries, I sent it to Stefano21:26
topolthe graphics are really good!21:26
morganfainbergtopol, to a former blizz or lol dev QQ would be:21:27
morganfainbergQQ is an acronym often used in online communication to signify two crying eyes. It is usually used in the pejorative such as "QQ more nub" meaning "Cry more, newbie." "QQ" has become a popular term, and can also be used to reply to a poster that the person thinks is whining.21:27
morganfainbergso i always have to go "wait what" when someone says QQ21:27
morganfainberg:P21:27
stevemarmorganfainberg, i knew what you were talking about :)21:27
gyeetranslation: if you use openstack, you are in bad with a lot of people21:27
gyee:)21:27
topolmorganfainberg, wow I learned something new.21:27
morganfainbergstevemar, i figured you did, but i didn't expect topol to know :P he doesn't seem like a StarCraft or WoW type person21:27
morganfainbergor LoL21:28
gyeeactually, he say "you are not along"21:28
gyeemy bad21:28
topolmorganfainberg, stevemar my comment on Henry's megapatch:  In addition to opensax.com I think we need to start biggestkeystonepatch.com And we can list this patch as the one to beat. /me BRAD DUCKS!!!! :-)21:29
morganfainbergtopol, so, i think my response to the LDAP Assignment driver is going to be "i'd rather this be out of tree unless we have a documented series of users "not 'i can't tell you but i swear we use it'" and a serious maintainer"21:29
morganfainbergtopol, mostly for the same reason(s) EC2 API is likely going to end up on stackforge21:30
topolmorganfainberg +100021:30
topolmakes a lot of sense21:30
morganfainbergand i'm all for helping them split that driver out to stackforge21:30
stevemartopol, now do the dependent patches for the notifications changes :P21:30
morganfainbergas in, making it easier to load a driver for assignment backend21:30
topolmorganfainberg I love it!21:30
*** radez_g0n3 is now known as radez21:34
*** nellysmitt has joined #openstack-keystone21:36
*** david-lyle_ has joined #openstack-keystone21:37
*** david-lyle has quit IRC21:37
openstackgerritSteve Martinelli proposed openstack/keystone: Add CADF notification handling for policy/region/service/endpoint  https://review.openstack.org/15178621:40
openstackgerritMerged openstack/oslo.policy: Add API documentation  https://review.openstack.org/15095621:41
*** nellysmitt has quit IRC21:41
openstackgerritMerged openstack/oslo.policy: Fix project metadata  https://review.openstack.org/15095721:44
openstackgerritSteve Martinelli proposed openstack/keystone: Add a test for create_domain in notifications  https://review.openstack.org/15179121:46
*** david-lyle_ has quit IRC21:51
morganfainbergtopol, there is my response21:51
morganfainbergi might have been pretty harse21:51
morganfainbergharsh21:51
morganfainbergbut it really feels as if it was "oh yeah we use that OMG YOU MAINTAIN IT BUT WE DONT WANT TO COMMIT ANYTHING TO IT"21:52
stevemarhow does one recheck xenserver21:53
morganfainbergrecheck xen?21:53
morganfainbergi dunno21:53
openstackgerritMerged openstack/python-keystoneclient: Add validate token for v3  https://review.openstack.org/14214721:55
*** jasondotstar has quit IRC21:56
openstackgerritGeorge Peristerakis proposed openstack/python-keystoneclient: Removed deprecated oslo modules.  https://review.openstack.org/15179421:58
*** packet has quit IRC21:58
morganfainbergi just realized i change the end of my emails depending on how i feel about the topic22:01
morganfainberg"cheers" = normal22:01
morganfainberg"regards" = yeah not happy about this22:01
morganfainberg"sincerely" = corporate / scary / politically charged email.22:01
morganfainberg:P22:01
topolmorganfainberg. Dont play poker. you clealry have a tell :-)22:01
openstackgerritMerged openstack/keystone: Move projects and domains to their own backend  https://review.openstack.org/14482422:01
morganfainbergtopol, knowing your tell is important22:02
morganfainbergcause you can abuse that other people think you don't know it22:02
morganfainberg>.>22:02
topolso true22:02
openstackgerritMerged openstack/keystone: Remove unused pointer to assignment in identity driver  https://review.openstack.org/14502222:03
openstackgerritMerged openstack/keystone: Make controllers and managers reference new resource manager  https://review.openstack.org/13352522:03
morganfainbergtopol, so checked. we even got a response from mirantis that they don't use LDAP Assignment.22:03
openstackgerritMerged openstack/keystone: Make unit tests call the new resource manager  https://review.openstack.org/13095422:03
*** abhirc has quit IRC22:03
morganfainbergtopol, made that email even more fun to write.22:03
topolmorganfainberg WTF???22:03
morganfainbergor at least someone in mirantis said that on the survey22:03
morganfainbergyeah22:03
topolthats insane22:04
morganfainbergi think my response was very realistic in what it would take for us to keep it in tree22:04
morganfainbergi am lookingforward to the patch that removed LDAP assignment22:07
morganfainbergthat will be a lot less code to maintain22:07
gyeemorganfainberg, for LDAP, look for an email from me today22:08
morganfainbergLDAP assignment?22:08
gyeeLDAP assignment22:08
gyeeI just got fart in the face by a customer22:08
gyeethere are two serious bugs we need to fix22:08
morganfainbergsecurity?22:09
gyeesorry LDAP identity22:09
gyeeno, performance22:09
morganfainbergok22:09
morganfainbergyep no issue there we should fix ldap identity22:09
morganfainbergtotally fix it :)22:09
gyeek, I am typing up an email22:09
*** abhirc has joined #openstack-keystone22:09
*** mattfarina has quit IRC22:11
*** topol has quit IRC22:11
*** spandhe has quit IRC22:11
kfox1111what kind of support does keystone have for maping users assigned to a group to roles in a tenant?22:13
*** joesavak has quit IRC22:13
kfox1111specifically in juno or kilo?22:13
morganfainbergkfox1111, you mean group assignments?22:13
*** gordc has quit IRC22:14
morganfainbergkfox1111, you apply the assignments to the group22:14
morganfainbergkfox1111, any users in that group get those assignments [has been the case for a while]22:14
morganfainbergpredating juno22:14
kfox1111how do you do that?22:15
kfox1111there is no keystone group-list, etc.22:15
morganfainberginstead of assigning the role on a tenant to a user, you do assign it to the group.22:15
morganfainbergdon't use keystoneclient22:15
morganfainberguse openstackclient for anything v3 related22:16
kfox1111is it all just api?22:16
kfox1111OH. its a v3 thing...22:16
morganfainbergkeystoneclient doesn't support v322:16
*** stevemar has quit IRC22:16
kfox1111ah.22:16
morganfainbergform a cli22:16
morganfainbergas a python lib it does very well w/ v3 support ;)22:16
kfox1111cool. I'll see what I can figure out. thanks for the tip.22:17
*** thedodd has quit IRC22:17
*** morganfainberg is now known as NotLurkingHere22:17
kfox1111does the client drop options? I see no group stuff in there either. maybe my api endpoint is wrong..22:18
NotLurkingHerekfox1111, hm let me summon the expert22:18
kfox1111yeah. its a v2 url.22:18
NotLurkingHereoh darn it stevemar isn't here.22:18
NotLurkingHereah yeah you can't do v3 operations on a v2 url22:18
kfox1111I just change v2.0 to v3?22:19
*** NotLurkingHere is now known as morganfainberg22:19
kfox1111openstack --help | grep group still only shows sever groups and security groups. nothing else.22:19
kfox1111is there another term for it?22:19
morganfainbergi think there is22:19
morganfainbergmaybe22:20
morganfainbergwe might have a gap in OSC here22:20
kfox1111hmm.... wait. this particular box is icehouse. let me try on a juno box.22:20
nkinderkfox1111, morganfainberg: you can do group based role assignment in OSC22:21
nkinder...but you have to connect via v3 for the commands to show up22:21
morganfainbergnkinder, thanks.22:21
morganfainbergkfox1111, yeah it's there: https://github.com/openstack/python-openstackclient/blob/master/openstackclient/identity/v3/group.py lots of group stuff22:21
nkinderkfox1111, morganfainberg: here's an example - https://github.com/nkinder/rdo-vm-factory/blob/master/rdo-federation-setup/vm-post-cloud-init-rdo.sh#L14622:22
nkinderthat has group creation and role assignment22:22
kfox1111ah. perfect thanks. :)22:24
nkinderkfox1111: the "--os-identity-api-version 3" is key, though you can set that via environment variable instead22:24
nkinderkfox1111: you also need the v3 URL of course22:24
*** abhirc has quit IRC22:25
kfox1111hmm... changing http://172.20.96.65:5000/v2.0 to http://172.20.96.65:5000/v3 doesn't work.22:25
kfox1111or --os-identity-api-version 3 with either of those two exports.22:26
nkinderkfox1111: what OS_* environment variables do you have set?22:26
kfox1111username, tenant name, auth_url, region and pw.22:27
kfox1111do I have to specify a domain with v3?22:27
nkinderok, some of that stuff is v2 specific (like tenant_name)22:27
nkinderand yes, you need a domain22:27
kfox1111k22:28
nkinderkfox1111: those commands I linked to assume no OS_* stuff is set22:28
kfox1111ok. I'll unset everything then try the commands.22:28
nkinderkfox1111: a v3 style keystonerc is here - https://github.com/nkinder/rdo-vm-factory/blob/master/rdo-federation-setup/vm-post-cloud-init-rdo.sh#L21522:28
nkinder...but you need a domain scoped token to create groups (and that keystonerc is for project scoped tokens)22:29
kfox1111yup. that did the trick.22:30
kfox1111so I probably need to update all my openstackrc files to be v3.22:30
kfox1111bummer.... unfortunatly the dashboard still generates a v2 one. :/22:31
*** dimsum__ has joined #openstack-keystone22:32
*** dimsum__ has quit IRC22:33
*** dimsum__ has joined #openstack-keystone22:33
*** jacer_huawei has quit IRC22:35
*** jacer_huawei has joined #openstack-keystone22:36
*** jacer_huawei has quit IRC22:36
openstackgerritRodrigo Duarte proposed openstack/keystone: Implements parents_as_ids query param  https://review.openstack.org/14856722:37
openstackgerritRodrigo Duarte proposed openstack/keystone: Implements subtree_as_ids query param  https://review.openstack.org/14861822:37
nkinderkfox1111: I usually create multiple rc files22:37
nkinderkfox1111: at least for admin users, you will sometimes need domain scoped tokens, and sometimes project scoped22:37
rodrigodsmorganfainberg, https://review.openstack.org/#/c/148567/ needed a rebase22:37
*** jacer_huawei has joined #openstack-keystone22:38
*** jacer_huawei has quit IRC22:40
*** jacer_huawei has joined #openstack-keystone22:41
*** jacer_huawei has quit IRC22:42
*** jacer_huawei has joined #openstack-keystone22:44
*** jacer_huawei has quit IRC22:44
*** jacer_huawei has joined #openstack-keystone22:45
*** morganfainberg is now known as outforteaorcoffe22:47
openstackgerritBrant Knudson proposed openstack/keystone: Internal notifications for cleanup domain  https://review.openstack.org/12552122:47
*** zzzeek has quit IRC22:49
*** tellesnobrega_ has joined #openstack-keystone22:51
openstackgerritNathan Kinder proposed openstack/oslo.policy: Correct docstring references  https://review.openstack.org/15181322:57
*** htruta has quit IRC23:02
*** samueldmq has quit IRC23:02
*** raildo has quit IRC23:02
*** tellesnobrega has quit IRC23:03
*** harlowja is now known as harlowja_away23:05
*** jasondotstar has joined #openstack-keystone23:05
*** jasondotstar has quit IRC23:05
*** abhirc has joined #openstack-keystone23:08
*** carlosmarin has quit IRC23:12
bknudsonnkinder: u around? re https://review.openstack.org/#/c/151813/23:14
bknudsonI figured out why no docs for some of the classes -- there's no docstring.23:14
bknudsonadding the docstring makes the links work so I'm fine with  https://review.openstack.org/#/c/151813/ as-is.23:17
*** zzzeek has joined #openstack-keystone23:17
*** _cjones_ has quit IRC23:18
*** timcline has quit IRC23:23
*** timcline has joined #openstack-keystone23:23
*** _cjones_ has joined #openstack-keystone23:25
*** harlowja_away is now known as harlowja23:27
*** timcline has quit IRC23:28
*** tellesnobrega_ has quit IRC23:28
nkinderbknudson: ok, so want me to just wait until this merges and add the docstrings in another commit?23:29
*** henrynash has joined #openstack-keystone23:29
*** ChanServ sets mode: +v henrynash23:29
*** harlowja has quit IRC23:29
*** harlowja has joined #openstack-keystone23:29
*** tellesnobrega has joined #openstack-keystone23:34
*** tellesnobrega has quit IRC23:34
*** nellysmitt has joined #openstack-keystone23:37
*** nellysmitt has quit IRC23:42
nkinderbknudson: what do you think about simply moving the __call__ docstrings from those classes to be class docstrings instead?23:42
nkinderbknudson: I'm not really sure that they are needed on the __call__ methods23:43
*** timcline has joined #openstack-keystone23:44
*** timcline_ has joined #openstack-keystone23:46
*** timcline has quit IRC23:49
*** timcline_ has quit IRC23:51
bknudsonnkinder: adding the docstrings doesn't need to wait at all, can be an unrelated commit... there wouldn't be a conflict23:52
nkinderyeah, preparing it now (was just building docs)23:52
bknudsonmoving the __call__ docstring makes sense.23:52
nkinderthat's what I think too23:52
*** timcline has joined #openstack-keystone23:54
openstackgerritNathan Kinder proposed openstack/oslo.policy: Add docstrings for check classes  https://review.openstack.org/15182223:56
*** timcline has quit IRC23:58

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