Wednesday, 2014-04-16

*** ChanServ changes topic to "Open for Juno development; submit design summit session proposals ASAP (deadline: April 20th) http://summit.openstack.org/"00:02
*** derek_c has quit IRC00:02
morganfainbergmarekd, you're going to be in ATL right?00:08
morganfainbergmarekd, cause... it would be terrible if you werent :)00:08
marekdmorganfainberg: yes, I got the ATL, since some of my patches got merged to the master :-)00:09
morganfainbergmarekd, yay!00:09
marekdbtw, ATL stands for...? :P00:09
marekd(i mean the full name)00:09
morganfainbergmarekd, atlanta00:09
morganfainbergATL is the airport code iirc00:09
marekdmorganfainberg: oh maaaan.00:10
marekdok it's late in here.00:10
morganfainbergATC you mean, ATC is active technical contributor00:10
marekdok, i thought you asked about ATC00:10
marekd:P00:10
morganfainbergnah.00:10
marekdthat's why i mentioned the patches.00:10
marekdi'd better go to bed now ;/00:10
morganfainbergmore concerned about people getting to the summit esp. those contributing a lot of time.00:10
morganfainbergmarekd, never a bad idea.00:11
morganfainbergmarekd, esp. if it's late00:11
marekd2 am00:11
morganfainberghave a good night dude. catch ya later00:11
marekdmorganfainberg: anyways, i have ATC status. I will be in Atlanta too :-)00:12
morganfainbergnice.00:12
marekdyep, gonna hit the bed now. good night!00:12
*** marekd is now known as marekd|away00:13
*** praneshp has quit IRC00:22
*** theocean154 has joined #openstack-keystone00:31
*** derek_c has joined #openstack-keystone00:34
*** marcoemorais has quit IRC00:39
*** jagee has quit IRC00:48
*** jagee has joined #openstack-keystone00:48
openstackgerritguang-yee proposed a change to openstack/python-keystoneclient: Implement endpoint filtering functionality on the client side.  https://review.openstack.org/8271301:01
*** gyee has quit IRC01:03
*** ilives has joined #openstack-keystone01:16
*** bknudson has joined #openstack-keystone01:28
*** dstanek has quit IRC01:29
*** dstanek has joined #openstack-keystone01:32
*** daneyon has joined #openstack-keystone01:32
*** derek_c_ has joined #openstack-keystone01:35
*** david-lyle has joined #openstack-keystone01:36
*** amcrn has quit IRC01:39
*** bknudson has quit IRC01:39
*** derek_c has quit IRC01:53
*** derek_c_ has quit IRC01:53
*** stevemar has joined #openstack-keystone02:01
*** david-lyle has quit IRC02:03
*** jzl-ctrip has joined #openstack-keystone02:07
jzl-ctriphi, guys, I just can't get through the test 'test_ipv6.py', is there any solution?02:08
*** jzl-ctrip has quit IRC02:09
openstackgerritA change was merged to openstack/keystone: Remove unnecessary dict copy  https://review.openstack.org/8743002:10
*** richm has quit IRC02:10
*** cloud has joined #openstack-keystone02:16
*** derek_c has joined #openstack-keystone02:16
*** cloud is now known as jzl-ctrip02:16
*** ls has joined #openstack-keystone02:20
*** ls is now known as jzl_ctrip02:20
*** jzl-ctrip has quit IRC02:20
jzl_ctripsorry for asking again as I just got disconnected, is there any solution for passing the test case test_ipv6.py?02:21
jzl_ctripnever mind, I just skipped that testcase02:29
*** derek_c_ has joined #openstack-keystone02:33
*** harlowja is now known as harlowja_away02:35
*** derek_c has quit IRC02:36
morganfainbergjzl_ctrip, i don't know what issue you're seeing. is ipv6 not available on the system you're using for testing?02:40
morganfainbergjzl_ctrip, if you provide a paste (paste.openstack.org) of the error I can see if I can help you.02:43
openstackgerritDavid Stanek proposed a change to openstack/keystone: More notification unit tests  https://review.openstack.org/8165902:44
openstackgerritDavid Stanek proposed a change to openstack/keystone: Refactor notifications  https://review.openstack.org/8166002:44
*** derek_c has joined #openstack-keystone02:51
*** mberlin has quit IRC02:58
*** zhiyan_ is now known as zhiyan03:01
*** mberlin has joined #openstack-keystone03:13
*** jimbaker has quit IRC03:20
*** jimbaker has joined #openstack-keystone03:21
*** jimbaker has quit IRC03:21
*** jimbaker has joined #openstack-keystone03:21
*** praneshp has joined #openstack-keystone03:29
openstackgerritA change was merged to openstack/keystone: Sync with oslo-incubator 2fd457b  https://review.openstack.org/8396604:11
*** praneshp_ has joined #openstack-keystone04:15
*** praneshp has quit IRC04:19
*** praneshp_ is now known as praneshp04:19
*** theocean154 has quit IRC04:21
*** saju_m has joined #openstack-keystone04:27
*** jamielennox is now known as jamielennox|away04:27
*** saju_m has quit IRC04:27
*** saju_m has joined #openstack-keystone04:28
*** saju_m has quit IRC04:30
*** marcoemorais has joined #openstack-keystone04:36
*** dstanek has quit IRC04:39
*** dstanek has joined #openstack-keystone04:40
openstackgerritJamie Lennox proposed a change to openstack/keystone: Move mutable parameter checking into federation  https://review.openstack.org/8784904:41
openstackgerritJamie Lennox proposed a change to openstack/keystone: Move hints building/filtering onto object  https://review.openstack.org/8785004:42
*** jagee has quit IRC04:43
*** jamielennox|away is now known as jamielennox04:45
jamielennoxayoung: you here?04:46
jamielennoxayoung, morganfainberg or anyone here, can you have a review of https://review.openstack.org/#/c/83630/ ?04:47
jamielennoxdolph wants to do a realease soon and that'd be a small nice to have04:47
*** dstanek has quit IRC04:55
*** Chicago has quit IRC04:57
*** Chicago has joined #openstack-keystone04:57
*** Chicago has joined #openstack-keystone04:57
*** nkinder has joined #openstack-keystone05:00
morganfainbergjamielennox, looking now05:08
morganfainbergjamielennox, +2/+A05:10
jamielennoxmorganfainberg: cheers05:10
morganfainbergnow i gotta get moving and get food :P05:10
morganfainberglate dinner05:10
openstackgerritLi Ma proposed a change to openstack/keystone: Password trunction makes password insecure  https://review.openstack.org/7732505:18
openstackgerritA change was merged to openstack/keystone: Removed unused code  https://review.openstack.org/8598405:20
*** morganfainberg is now known as morganfainberg_Z05:20
openstackgerritJamie Lennox proposed a change to openstack/keystone: Isolate backend loading  https://review.openstack.org/7429305:24
openstackgerritJamie Lennox proposed a change to openstack/keystone: Make Pecan the root routing framework  https://review.openstack.org/6542805:24
*** chandan_kumar has joined #openstack-keystone05:31
*** stevemar has quit IRC05:41
*** florentflament has quit IRC05:49
jzl_ctripthe variable 'repository' seems undesired here,06:14
jzl_ctripJiānróng xìng06:14
jzl_ctriphttps://github.com/openstack/keystone/blob/master/keystone/tests/test_sql_upgrade.py#L203-L21606:14
*** dstanek has joined #openstack-keystone06:22
*** dstanek has quit IRC06:27
*** jaosorior has joined #openstack-keystone06:38
openstackgerritA change was merged to openstack/python-keystoneclient: Add service name to catalog  https://review.openstack.org/7841006:39
*** tomoiaga has joined #openstack-keystone06:46
*** chandan_kumar has quit IRC07:02
*** chandan_kumar has joined #openstack-keystone07:08
*** ilives has quit IRC07:11
*** ilives has joined #openstack-keystone07:11
*** florentflament has joined #openstack-keystone07:27
*** marcoemorais has quit IRC07:31
*** leseb has joined #openstack-keystone07:42
*** marekd|away is now known as marekd07:53
*** henrynash has joined #openstack-keystone08:18
*** derek_c has quit IRC08:20
*** derek_c_ has quit IRC08:20
*** dstanek has joined #openstack-keystone08:24
*** henrynash has quit IRC08:25
*** andreaf has joined #openstack-keystone08:28
*** dstanek has quit IRC08:28
openstackgerritIlya Pekelny proposed a change to openstack/keystone: Sync test_migrations  https://review.openstack.org/8061808:35
openstackgerritIlya Pekelny proposed a change to openstack/keystone: Redundant unique constraint  https://review.openstack.org/8444708:35
openstackgerritIlya Pekelny proposed a change to openstack/keystone: Corresponding `nullable` value.  https://review.openstack.org/8444608:35
openstackgerritIlya Pekelny proposed a change to openstack/keystone: Compatible server default value in the models.  https://review.openstack.org/8444508:35
openstackgerritIlya Pekelny proposed a change to openstack/keystone: Explicit foreign key indexes.  https://review.openstack.org/8444408:35
openstackgerritIlya Pekelny proposed a change to openstack/keystone: Make it possible to use multiprocess file locks  https://review.openstack.org/8444808:35
openstackgerritIlya Pekelny proposed a change to openstack/keystone: Comparision of database models and migrations.  https://review.openstack.org/8063008:35
*** praneshp_ has joined #openstack-keystone08:46
*** praneshp has quit IRC08:46
*** praneshp_ is now known as praneshp08:46
openstackgerritwanghong proposed a change to openstack/keystone: delete the tokens when deleting ec2 credential  https://review.openstack.org/8745008:54
*** marcoemorais has joined #openstack-keystone09:00
*** marcoemorais has quit IRC09:05
*** praneshp has quit IRC09:05
*** chandan_kumar has quit IRC09:06
*** chandan_kumar has joined #openstack-keystone09:19
*** chandan_kumar has quit IRC09:26
*** chandan_kumar has joined #openstack-keystone09:26
openstackgerritJamie Lennox proposed a change to openstack/keystone: Make Pecan the root routing framework  https://review.openstack.org/6542809:34
*** jamielennox is now known as jamielennox|away09:40
*** dstanek has joined #openstack-keystone09:45
*** dstanek has quit IRC10:00
openstackgerritSergey Nikitin proposed a change to openstack/keystone: Code which gets and deletes elements of tree was moved to one method  https://review.openstack.org/8657810:14
*** florentflament has quit IRC10:26
*** dstanek has joined #openstack-keystone10:27
openstackgerritSergey Nikitin proposed a change to openstack/keystone: Some methods in ldap were moved to superclass  https://review.openstack.org/8625010:28
*** dstanek has quit IRC10:32
*** marekd is now known as marekd|away10:32
openstackgerritSergey Nikitin proposed a change to openstack/keystone: Some methods in ldap were moved to superclass  https://review.openstack.org/8625010:52
*** jaosorior has quit IRC11:21
*** dstanek has joined #openstack-keystone11:33
*** jaosorior has joined #openstack-keystone11:49
*** jzl_ctrip has quit IRC12:00
*** marcoemorais has joined #openstack-keystone12:02
*** zlji has joined #openstack-keystone12:02
*** marcoemorais has quit IRC12:06
*** zlji has quit IRC12:19
baffleRole names can contain spaces. Policies uses role names. Does role names with spaces work in Keystone and policies? Should they be escaped? Is space replaced with another character? :)12:42
*** snikitin has joined #openstack-keystone12:43
dolphmjamielennox|away: https://review.openstack.org/#/c/78878/12:52
*** marcoemorais has joined #openstack-keystone13:03
*** marcoemorais has quit IRC13:08
*** erecio has quit IRC13:08
*** tomoiaga has left #openstack-keystone13:09
*** erecio has joined #openstack-keystone13:14
*** dstanek_afk has joined #openstack-keystone13:21
*** dstanek has quit IRC13:23
*** dims has quit IRC13:32
openstackgerritDavid Stanek proposed a change to openstack/keystone: Fixes for in-code documentation  https://review.openstack.org/8796513:32
*** wchrisj has joined #openstack-keystone13:34
*** dstanek_afk is now known as dstanek13:35
dstanekdolphm: https://review.openstack.org/#/c/83630/ just doesn't want to work13:38
*** bknudson has joined #openstack-keystone13:40
*** vhoward has left #openstack-keystone13:43
*** wchrisj has left #openstack-keystone13:53
*** marcoemorais has joined #openstack-keystone14:04
*** nkinder has quit IRC14:07
*** marcoemorais has quit IRC14:08
*** topol has joined #openstack-keystone14:09
*** bach has joined #openstack-keystone14:10
openstackgerritIlya Pekelny proposed a change to openstack/keystone: Sync test_migrations  https://review.openstack.org/8061814:11
openstackgerritIlya Pekelny proposed a change to openstack/keystone: Redundant unique constraint  https://review.openstack.org/8444714:11
openstackgerritIlya Pekelny proposed a change to openstack/keystone: Corresponding `nullable` value.  https://review.openstack.org/8444614:11
openstackgerritIlya Pekelny proposed a change to openstack/keystone: Compatible server default value in the models.  https://review.openstack.org/8444514:11
openstackgerritIlya Pekelny proposed a change to openstack/keystone: Explicit foreign key indexes.  https://review.openstack.org/8444414:11
openstackgerritIlya Pekelny proposed a change to openstack/keystone: Make it possible to use multiprocess file locks  https://review.openstack.org/8444814:11
openstackgerritIlya Pekelny proposed a change to openstack/keystone: Comparision of database models and migrations.  https://review.openstack.org/8063014:11
*** dstanek has quit IRC14:14
*** dims has joined #openstack-keystone14:18
*** stevemar has joined #openstack-keystone14:19
openstackgerritBrant Knudson proposed a change to openstack/keystone: Sync with oslo-incubator 462e62d  https://review.openstack.org/8798014:25
*** bach has quit IRC14:28
*** dstanek has joined #openstack-keystone14:30
dolphmdstanek: seriously14:31
*** marcoemorais has joined #openstack-keystone14:33
*** daneyon has quit IRC14:34
*** dims has quit IRC14:34
*** daneyon has joined #openstack-keystone14:35
*** marcoemorais has quit IRC14:38
dolphmdstanek: https://review.openstack.org/#/c/81659/3/keystone/tests/test_notifications.py14:41
bknudsonthat's one way to resolve a merge conflict14:43
dstanekbknudson: yeah, it's the only way!14:45
*** dims has joined #openstack-keystone14:47
lbragstadwhoop whoop, gerrit powered meeting agendas for keystone? https://review.openstack.org/#/c/87759/1/meetings/keystone-team-meeting.yml14:47
openstackgerritDavid Stanek proposed a change to openstack/keystone: More notification unit tests  https://review.openstack.org/8165914:48
bknudsonI doubt that we could get the agenda reviewed in time for the meeting.14:49
lbragstadyeah, it's a work in progress.14:49
dstanekyeah, the agenda seems like a bad thing to add in there14:50
*** thedodd has joined #openstack-keystone14:50
bknudson-1 I don't want to talk about this14:50
dstaneksome neutron bug has been failing a ton of reviews recently14:51
*** jaosorior has quit IRC14:51
bknudsonhttp://status.openstack.org/elastic-recheck/14:52
bknudson2943 fails in 14 days14:52
*** ilives has quit IRC14:54
*** ilives has joined #openstack-keystone14:58
dstaneki'm looking to run tests against MySQL in parallel. i'm thinking right now that i should try to create the database before each test and drop it after.14:58
dstanekprobably named something like keystone_tests_#pid#. any reason that this would be bad?14:59
stevemarthat is an insane amount of failures in the last 14 days15:13
*** dstanek has quit IRC15:22
*** dstanek has joined #openstack-keystone15:23
*** ilives has quit IRC15:24
*** ilives has joined #openstack-keystone15:25
*** jagee has joined #openstack-keystone15:29
*** andreaf has quit IRC15:41
*** andreaf has joined #openstack-keystone15:41
*** vhoward has joined #openstack-keystone15:43
*** bach has joined #openstack-keystone15:45
*** chandan_kumar has quit IRC15:46
*** bach has quit IRC16:02
*** bach has joined #openstack-keystone16:02
openstackgerritMatthieu Huin proposed a change to openstack/keystone: Add missing import, remove trailing ":" in middleware example  https://review.openstack.org/8801416:03
bknudsondstanek: whatever you do make it extendable to other dbs...16:04
openstackgerritIlya Pekelny proposed a change to openstack/keystone: Migration DB_INIT_VERSION in common place  https://review.openstack.org/8801616:04
openstackgerritIlya Pekelny proposed a change to openstack/keystone: Sync test_migrations  https://review.openstack.org/8061816:04
openstackgerritIlya Pekelny proposed a change to openstack/keystone: Redundant unique constraint  https://review.openstack.org/8444716:04
openstackgerritIlya Pekelny proposed a change to openstack/keystone: Corresponding `nullable` value.  https://review.openstack.org/8444616:04
openstackgerritIlya Pekelny proposed a change to openstack/keystone: Compatible server default value in the models.  https://review.openstack.org/8444516:04
openstackgerritIlya Pekelny proposed a change to openstack/keystone: Explicit foreign key indexes.  https://review.openstack.org/8444416:04
bknudsondstanek: I'll want to add db2 support16:04
openstackgerritIlya Pekelny proposed a change to openstack/keystone: Make it possible to use multiprocess file locks  https://review.openstack.org/8444816:04
*** browne1 has joined #openstack-keystone16:04
openstackgerritIlya Pekelny proposed a change to openstack/keystone: Comparision of database models and migrations.  https://review.openstack.org/8063016:04
*** browne1 has left #openstack-keystone16:05
*** richm has joined #openstack-keystone16:05
*** browne has joined #openstack-keystone16:05
dstanekbknudson: absolutely16:06
*** bach has quit IRC16:07
*** zhiyan is now known as zhiyan_16:17
*** nkinder has joined #openstack-keystone16:22
dstanekshould we be nudging people toward matchers?16:25
*** zhiyan_ is now known as zhiyan16:34
*** ilives has quit IRC16:36
*** nkinder has quit IRC16:40
*** zhiyan is now known as zhiyan_16:41
*** gokrokve has joined #openstack-keystone16:47
*** leseb has quit IRC16:50
*** harlowja_away is now known as harlowja16:51
*** praneshp has joined #openstack-keystone16:54
*** praneshp has quit IRC16:54
*** praneshp has joined #openstack-keystone16:54
*** amcrn has joined #openstack-keystone16:57
*** amcrn has quit IRC17:10
bknudsondstanek: I'm trying.17:10
bknudsondstanek: it's not working well.17:10
bknudsonmaybe we'll get a critical mass of matchers code and then people will stop asking about it17:11
*** derek_c_ has joined #openstack-keystone17:15
*** marcoemorais has joined #openstack-keystone17:16
*** derek_c has joined #openstack-keystone17:16
*** david-lyle has joined #openstack-keystone17:20
*** david-lyle has quit IRC17:35
*** praneshp_ has joined #openstack-keystone17:36
dimsayoung, others, is this the latest instructions for keystone + mod_wsgi? is there anything in openstack doc? http://andymc-stack.co.uk/2013/06/apache2-mod_wsgi-openstack-pt1-keystone/17:37
ayoungdims, I'm like, the worst person to ask, cuz I'm the one that did it origianlly, and I do it all by trial and error.17:38
ayoungI just set on up  on Devstack, and I used mod_nss instead of mod_ssl, for example17:38
ayoungdims, for example, I hate port 5000 and 35357 with the white hot intensity of a thousand suns (to quote the bard)17:39
dimsa. ok :)17:39
ayoungso I put them both on 33417:39
ayounger17:39
ayoung44317:39
ayoungI got kerberos working, too, if you actually want something secure?17:40
*** praneshp has quit IRC17:40
*** praneshp_ is now known as praneshp17:40
dimsayoung, a bit of background, i am seeing a lockup of keystone when heat makes calls and peter's patch (resurrected) seems to help - https://review.openstack.org/#/c/85395/ - i was thinking of other ways to overcome the problem17:41
ayoungjust realized that both Kerberos and X509 would have been more resistant to Heartbleed.  But Kerberos would have been vulnerable to a replay attack,17:41
ayoungHe lies17:41
ayoung" keystone-all's throughput is limited to the17:41
ayoungthroughput of a single CPU core. "17:41
ayoungun troo17:41
ayoungNow, I am not one to sing the praises of Greenthreads, but each greenthread can spawn a process, and so the processes execute in parallel, and nothing lockes it to a single core17:42
dimshaven't dug deep into the problem yet. it could be some custom code that i may have causing trouble as well17:42
ayoungthe crypto is all done with a popoen17:42
dimsk17:43
ayounganyway, Apache is still better17:43
ayoungso you are on the right path, but I would not split out to ports 5000 and 3535717:43
ayoungInstead, run admin on  https://hostname/keystone/admin17:43
ayoungand main on17:43
ayoungInstead, run admin on  https://hostname/keystone/main17:43
dimsmakes sense17:43
ayoung WSGIDaemonProcess keystone_admin user=keystone group=nogroup17:43
ayoung WSGIDaemonProcess keystone_main user=keystone group=nogroup17:44
ayoungsame user but separate processes17:44
ayoungIf you don't, you get errors parsing the config file.17:44
*** amcrn has joined #openstack-keystone17:44
ayoungSeparate process groups, too17:44
dimsk. need to read up on Apache2/httpd config first. sounds like we don't have "Keystone under HTTPD" in devstack either17:47
dimsthanks ayoung let me see what i can find/do17:48
ayoungdims, not yet.  but you don't want it in devstack, trust me17:49
* ayoung just got nuked by that17:49
dimsah ok17:49
ayoungdevstack is awesome, but if you make changes to the live config...it gets over written if you need to rerun devstack17:49
ayoungand if you need to reboot the VM...you need to rerun devstack17:49
dimsright17:50
ayoungdims, so right now I am waiting on the RDO release of Icehouse RC packages to continue that effort.  However17:51
ayoungdevstack should be good for you. One thing:L17:51
ayoungdims, http://adam.younglogic.com/2014/04/teaching-horizon-to-share/17:52
ayoungyou can make those changes in17:52
*** thedodd has quit IRC17:53
ayoung /opt/stack/horizon/openstack_dashboard/local/local_settings.py17:53
*** d0ugal has quit IRC17:56
dimsgotcha. thanks17:57
*** gokrokve has quit IRC17:57
*** derek_c has quit IRC18:02
*** derek_c_ has quit IRC18:03
openstackgerritAndreas Jaeger proposed a change to openstack/keystone: Check that all po/pot files are valid  https://review.openstack.org/8421118:04
*** thedodd has joined #openstack-keystone18:15
*** praneshp is now known as praneshp_afk18:16
*** gokrokve has joined #openstack-keystone18:18
*** praneshp_afk has quit IRC18:26
*** morganfainberg_Z is now known as morganfainberg18:27
*** dstanek has quit IRC18:28
*** leseb has joined #openstack-keystone18:31
*** dstanek has joined #openstack-keystone18:42
*** praneshp has joined #openstack-keystone18:44
*** jimbaker has quit IRC18:44
*** jimbaker has joined #openstack-keystone18:45
*** jimbaker has quit IRC18:45
*** jimbaker has joined #openstack-keystone18:45
*** chandan_kumar has joined #openstack-keystone18:49
*** andreaf has quit IRC18:59
*** dstanek has quit IRC19:21
*** dstanek has joined #openstack-keystone19:22
*** derek_c has joined #openstack-keystone19:24
*** derek_c_ has joined #openstack-keystone19:24
*** bknudson has quit IRC19:28
*** derek_c has quit IRC19:28
*** derek_c_ has quit IRC19:28
*** derek_c has joined #openstack-keystone19:28
*** bknudson has joined #openstack-keystone19:29
*** chandan_kumar has quit IRC19:30
*** samuelmz has quit IRC19:31
openstackgerritAndreas Jaeger proposed a change to openstack/keystone: Check that all po/pot files are valid  https://review.openstack.org/8421119:38
*** marcoemorais has quit IRC19:39
*** leseb has quit IRC19:41
*** amcrn_ has joined #openstack-keystone19:46
*** amcrn has quit IRC19:47
*** amcrn_ is now known as amcrn19:48
*** dstanek has quit IRC19:53
*** Krsna has joined #openstack-keystone19:54
Krsnamorganfainberg: hope you got my message I sent to you when you were offline19:55
morganfainbergKrsna, i did19:55
morganfainbergKrsna, also let marekd|away know i pointed you at him as a resource for federation work19:55
morganfainbergKrsna, glad to have you working on this!19:55
*** chandan_kumar has joined #openstack-keystone19:56
Krsnamorganfainberg: well I have to clear a few tickets before then, but yes will be fun. I am looking forward to meeting you at the sumit19:57
*** browne has quit IRC19:57
*** leseb has joined #openstack-keystone19:57
morganfainbergKrsna, glad you'll be there! i know last summit was great for keystone, this next one should be awesome as well19:57
KrsnaI am excited!19:58
*** thedodd has quit IRC19:59
*** browne has joined #openstack-keystone20:00
Krsnamorganfainberg: Also, I am trying to get the code we have for multi-backends upstreamed. Hopefully soonish20:00
morganfainbergKrsna, very cool.20:00
*** thedodd has joined #openstack-keystone20:05
*** derek_c has quit IRC20:14
*** david-lyle has joined #openstack-keystone20:16
*** dstanek has joined #openstack-keystone20:19
*** gyee has joined #openstack-keystone20:21
*** topol has quit IRC20:24
*** marcoemorais has joined #openstack-keystone20:32
*** marcoemorais1 has joined #openstack-keystone20:35
*** marcoemorais has quit IRC20:37
*** gokrokve has quit IRC20:42
*** jamielennox|away is now known as jamielennox20:52
*** bach has joined #openstack-keystone20:59
*** gokrokve has joined #openstack-keystone21:02
*** Krsna has quit IRC21:10
openstackgerritJamie Lennox proposed a change to openstack/python-keystoneclient: Create a V3 Token Generator  https://review.openstack.org/7887821:12
jamielennoxdolphm, bknudson: ^^21:12
openstackgerritA change was merged to openstack/keystone: Add missing import, remove trailing ":" in middleware example  https://review.openstack.org/8801421:12
morganfainbergjamielennox, ooh cool.21:17
morganfainbergjamielennox, looked like the v2 one was merged, right?21:18
jamielennoxayoung: https://review.openstack.org/#/c/86727/ - why shouldn't kerberos be in requirements seperate from requests-kerberos?21:18
jamielennoxmorganfainberg: yep21:18
jamielennoxmorganfainberg: and there is a client release imminent so it's good to have the matched set21:18
morganfainbergjamielennox, ++ yeah21:18
morganfainbergjamielennox, i was opening this review to look at right now21:18
jamielennoxmorganfainberg: it's one of those client releases that will be immediately adopted by all the projects as well, so it's a good one to get things in to21:19
morganfainbergjamielennox, yep21:19
jamielennoxaww, my cross-project summit session on standardizing clients was rejected - thought that was a sure thing21:21
morganfainberg=/21:21
jamielennoxthey must be swamped21:21
bknudsonjamielennox: is there another session on openstack-sdk?21:21
jamielennoxthey had some proposed - i don't know if it was accepted or anything21:22
jamielennoxhttp://summit.openstack.org/cfp/details/33421:22
bknudsonthis cross-project workshop is super popular21:22
jamielennoxso there is a 2-parter on SDK and project libraries in general so i guess thats essentially the same21:23
bknudsonjamielennox: there's 2 not to be missed!21:23
bknudsonthey're going to need big rooms for the cross-project sessions21:24
morganfainbergjamielennox, there is no way we can make the v3 and v2 fixtures have less... duplicated code? or is that for future refactoring?21:24
morganfainbergjamielennox, or we don't care that much since v2 can probably die eventually.21:24
jamielennoxyea, so there is enough there to keep it interesting - and largely i think it's useful just to indentify who is interested21:24
jamielennoxmorganfainberg: it doesn't really work21:25
morganfainbergjamielennox, ok21:25
jamielennoxtheree are a lot of similar things but its all based on the token format21:25
morganfainbergjamielennox, i was thinking maybe there would be some way to align them... but tbh, i didn't see it off the bat21:25
morganfainbergfigured asking was the best bet21:26
morganfainbergi would love to get a .md that explicitly shows the token format(s)21:26
jamielennoxit's annoying with a few things where an endpoint in v2 is 1 for all 3 interfaces vs 3 endoints in v321:26
morganfainbergwould be useful to compare formats against... anyway21:27
jamielennoxbut it's testing infrastructure so i think people are ok to have to know a little baout what they are doing21:27
morganfainbergyeah21:27
*** jagee has quit IRC21:27
*** dstanek has quit IRC21:27
bknudsonhere's some code I had for converting v2 to v3 catalog: https://review.openstack.org/#/c/70630/5/keystone/catalog/backends/templated.py21:27
jamielennoxbknudson: ah - that's useful, i'm going to have to do the reverse of that in auth_token21:28
morganfainbergbknudson, cool.21:28
bknudsonthe templated backend only does v221:28
bknudsonso don't try to get a v3 token using templated backend21:28
morganfainbergbknudson, jamielennox, so i'm going to be building a construct for the token (object) that can be shared between v2/v3/vwhatever should this construct/object go in client or server? as in... should i put it in client first?21:29
jamielennoxthe backend there is a bit unusual because it's version specific21:29
morganfainbergbknudson, jamielennox, this is so we can make internal use of token non-version specific, take JSON/whatever and make it an object we can consistently work with.21:29
jamielennoxmorganfainberg: no, i don't think so - at least not initially21:29
morganfainbergjamielennox, ok so i'll develop it in server - was thinking auth_token could benefit from it as well.21:29
jamielennoxmorganfainberg: we have AccessInfo which is that abstraction in client21:30
morganfainbergbut we can shuffle it around i guess as needed.21:30
morganfainbergjamielennox, ah ok21:30
bknudsonmorganfainberg: use AccessInfo in the server21:30
jamielennoxbut it's an independent interface - not an independant representation21:30
morganfainbergbknudson, hm. i'll see if accessinfo does all i need.21:30
morganfainbergwe might need a largely expanded version of it.21:30
jamielennoxas in it keeps the orginal token format and provides different property accessors21:30
bknudsonwe might have a version of accessinfo in the server21:30
morganfainbergjamielennox, yeah that's not what i'm looking for in this case21:31
jamielennoxmorganfainberg: yep21:31
morganfainbergjamielennox, i'm looking for an object representation that we could use at the edge to emit a version of the token.21:31
jamielennoxmorganfainberg: i've been thinking about how to do that for all our models not just tokens21:31
morganfainbergbut would otherwise simplify accessing parts of the token (expires is the same place, trust is the same place, etc etc)21:32
morganfainbergjamielennox, well i'll start with token :) we can build from there. if i can do something more generic i'll build that then token on top of it21:32
jamielennoxmorganfainberg: i've been still looking at the pecan thing on and off, it's a bit harder without WSME so i was thinking about replacing that whole layer with jsonschema and doing models that way21:32
morganfainbergjamielennox, hm... well i was looking at possibly using jsonschema for this code anyway21:33
jamielennoxmorganfainberg: have you had a look at WSME and how it works?21:33
morganfainbergjamielennox, i'll ping you with the example/review since it's the next thing i'm working on21:33
jamielennoxmorganfainberg: i might have time to do a sketch up today of how i was thinking of doing the models with schema21:33
morganfainbergjamielennox, i have, but i thought WSME in general was dead for us because of the lack of arbitrary attrs21:34
jamielennoxif not i'm going down the coast for easter so it won't be till mid next week21:34
morganfainbergjamielennox, that would be useful if you have it - if not, i'll start working on some stuff and worst case we change it before we seriously consider merging it21:34
jamielennoxmorganfainberg: it is - but if you take the general concept of how it builds objects and such i think i can replicate that but do all the actual validation with jsonschema21:34
stevemarso many cross project workshop proposals21:34
jamielennoxstevemar: ++, it's going to be a busy week21:35
morganfainbergjamielennox, i'll revisit wsme when i'm working on this in either case.21:35
morganfainbergjamielennox, i want to get the code proposed (if not merged) prior to the summit since this is a requirement to move to ephemeral tokens and other such improvements21:36
jamielennoxmorganfainberg: so in general i was thinking one of the big issues with WSME is that it goes around the pecan.expose rather than inside21:36
jamielennoxas in WSME takes over JSON and XML rendering, i would try and do validation within pecan so that you can still use pecan's views layer21:37
morganfainbergjamielennox, the question then becomes the likelyhood of making pecan work with keystone21:37
jamielennoxmorganfainberg: pecan is fine i think21:37
jamielennoxhave you seen my patch on this?21:37
morganfainbergjamielennox, not recently21:37
jamielennoxhttps://review.openstack.org/#/c/65428/21:38
morganfainbergjamielennox, is it ready? as in, no real issues and i should build upon it.21:38
morganfainbergjamielennox, ah yeah i've seen these two. i'll base my work on top of them.21:38
jamielennoxit works for routing, the issue is all the @protected work is really intertwined with the current controller layout21:38
morganfainbergjamielennox, thankfully the @protected stuff isn't required for what i'm looking to do here (atm)21:39
jamielennoxi've been looking at ways of seperating them a little bit but i haven't found anything that's really easy to understand21:39
morganfainbergat least to start.21:39
jamielennoxtrue - for everything related to auth it won't matter21:39
morganfainbergyep21:39
morganfainbergmakes auth a great place to start21:40
jamielennoxyea, that would be good to start transitioning that i think21:40
morganfainbergcool.21:40
morganfainbergthanks.21:40
jamielennoxi'm looking at how to do common methods between new and old controllers so that there isn't much code change in controllers to start21:40
jamielennoxi'll let you know how that goes but you'll quickly figure out if you're missing things21:41
morganfainbergjamielennox, that would be good.21:41
morganfainbergjamielennox, hehe yeah i'm sure. anyway i'll point you at the code as i get through it.21:41
jamielennoxthat's what https://review.openstack.org/#/c/87850/ and https://review.openstack.org/#/c/87849/ are for - trying to simplify those controllers21:41
morganfainbergjamielennox, i think we also probably need to get to using stevedore for the backends.21:43
jamielennoxmorganfainberg: ++, that's on my list21:43
*** topol has joined #openstack-keystone21:48
mgagneAccording to my reading, tenant names are not unique. What's the boundary for such non-uniqueness? Can a single domain have 2 tenants with the same name? Or is non-uniqueness only possible with multiple domains?21:49
*** EmilienM has joined #openstack-keystone21:51
*** dims has quit IRC21:52
*** dstanek has joined #openstack-keystone21:54
*** larsks has joined #openstack-keystone21:54
*** dstanek has quit IRC21:59
*** bach has quit IRC22:02
morganfainbergmgagne, tenant/project names are unique within a domain22:03
morganfainbergmgagne, all tenants in the v2 api are unique names since it uses the "default" domain exclusivly22:03
mgagnemorganfainberg: cool, that clears things up then22:03
morganfainbergmgagne, sure thing22:03
mgagnemorganfainberg: thanks!22:04
*** dims has joined #openstack-keystone22:04
*** leseb has quit IRC22:09
jamielennoxmgagne: and tenant/project _id is globally unique so if you find yourself trying to use the tenant name for something you probably mean to be using the tenant_id22:10
openstackgerritBrant Knudson proposed a change to openstack/python-keystoneclient: Always configure logging  https://review.openstack.org/8809722:11
mgagnejamielennox: won't you use tenant name AND the domain when querying for it?22:12
jamielennoxmgagne: you can, but if you have the tenant id then you dont need the domain - it really depends what you are trying to do22:13
jamielennoxmgagne: if you are doing CLI and user facing stuff then names are often easier, if you're saving information in another service then you should always use ids22:13
mgagnejamielennox: right, I agree.22:14
jamielennoxmgagne: i only clarify because it came up with swift recently where they were using usernames as unique rather than user_ids and were attempting all sorts of hacks so that they could keep on using a username22:15
mgagnejamielennox: will take note of it for when I encounter such design choice =)22:17
*** bknudson has quit IRC22:27
*** marcoemorais1 has quit IRC22:31
*** ilives has joined #openstack-keystone22:32
*** marcoemorais has joined #openstack-keystone22:33
morganfainbergjamielennox, wait what?22:33
morganfainbergjamielennox, oh dear.... usernames unique22:34
*** nkinder has joined #openstack-keystone22:36
*** huats has quit IRC22:37
*** ilives has quit IRC22:38
*** derek_c has joined #openstack-keystone22:38
*** thedodd has quit IRC22:41
openstackgerritSteve Martinelli proposed a change to openstack/python-keystoneclient: Add request/access token and consumer support for keystoneclient  https://review.openstack.org/8198022:42
*** huats has joined #openstack-keystone22:51
*** huats has quit IRC22:51
*** huats has joined #openstack-keystone22:51
*** lnxnut_ has joined #openstack-keystone22:51
*** lnxnut has quit IRC22:51
*** lnxnut_ has quit IRC22:51
*** lnxnut has joined #openstack-keystone22:52
morganfainbergjamielennox, dolphm, stevemar, https://review.openstack.org/#/c/77325/8/keystone/common/utils.py curious on your opinion of my comment there.23:01
*** david-lyle has quit IRC23:01
morganfainbergi think WARNING is the wrong level and this should be downgraded to info... but kinda want other input on that23:01
morganfainbergtopol. ^ your input would also be welcome23:03
dolphmmorganfainberg: with configuration, i agree it should be info23:06
morganfainbergdolphm, i think previously warning was even a bit loud.23:07
dolphmmorganfainberg: that's what i meant -- it was already too loud23:09
morganfainbergdolphm, ok cool. we're on the same page.23:09
dolphmWARNING: The system is behaving exactly as you configured it, nothing to see here!23:10
morganfainbergdolphm, WARNING: don't worry, do nothing.23:10
morganfainbergwe should totally add more warning messages that everything is working as expected23:11
morganfainbergespecially every time someone logs in23:11
dolphmmorganfainberg: WARNING: CPU usage suddenly went up a bit.23:11
morganfainbergdolphm, WARNING: Someone used HTTP POST method23:12
dolphmWARNING: Non-zero load on auth service!23:12
morganfainbergdolphm, WARNING: This service runs in python.23:13
dolphmlol23:13
*** stevemar has quit IRC23:20
*** marcoemorais has quit IRC23:22
*** gokrokve has quit IRC23:23
topolmorganfainberg, what was the question?23:33
morganfainbergtopol, https://review.openstack.org/#/c/77325/8/keystone/common/utils.py the log message warning is a bit loud.23:34
morganfainbergtopol, was looking for other reviewer input23:34
morganfainbergas you can see the IRC channel got a little silly for non-warning log messages :P23:34
morganfainbergerm non-warning warnign messages23:35
topolwarning seems merited to me for truncating a password23:35
morganfainbergtopol, every time they authenticate?23:35
topolhmmm everytime23:35
dolphmtopol: even though they configured keystone to truncate?23:35
topolother choice is info?23:36
jamielennoxmorganfainberg: i'm ok to downgrade that to info23:36
morganfainbergtopol, i'd advocate info23:36
morganfainbergit allows you to still track occurences (you should be tracking things like that)23:36
morganfainbergbut it's nothing you need to act on as a operator/deployer23:36
jamielennoxmorganfainberg: it's not an operator issue, if you are looking through your keystone logs WARNING should really stand out as you've done something wrong23:36
morganfainbergjamielennox, ++23:36
topolSo the other choice is info, correct? It still gets placed in the logs everytime23:37
morganfainbergtopol, correct23:37
topolthey authenticate23:37
*** chandan_kumar has quit IRC23:37
morganfainbergtopol, well aslong as keystone is configured to emit info logs (min log level would apply)23:38
morganfainbergjamielennox, i think we have a number of logging lines that might need downgrade and probably some that need upgrade23:38
morganfainbergperhaps that should also be a target for Juno23:38
jamielennoxi know this has been in for a while, but it feels like this should be part of a larger 'password rules' component and it fails if you don't meet the rules - silent truncation is bad23:38
jamielennoxlike special chars and numbers required23:38
topolas long as they can turn on logging and be informed of the truncation. It doesnt matter to me whether its info or WARNING. As long as they get informed23:38
morganfainbergtopol, i think it is info, it's relevant for running a cloud, but nothing that needs immidiate action23:39
jamielennoxi'm hoping there is a library that could do that because i don't want to manage passwrd rules in keystone23:39
topolmorganfainberg agreed23:39
topolWARNING, keystone cores want to make token data binary23:39
topolthat one was worth a WARNING23:39
morganfainbergtopol, hehe23:40
morganfainbergjamielennox, isn't passlib capable of doing that?23:40
topolWARNING keystone cores dont use emoticons on IRC23:40
jamielennoxmorganfainberg: looking at that now23:40
morganfainbergtopol, HAH23:40
jamielennoxit's not mentioned on there front page23:40
topolWARNING, brad was about to have a stroke thinking about unusable curl examples from binary tokens cause brad is slow at keystone core humor23:40
topolso hard being the slow guy23:40
topollast one is an info23:41
jamielennoxmorganfainberg: i can't see it23:41
dolphmif they really cared deeply about this particular message, they could change the log level for this module23:42
morganfainbergdolphm, true23:43
dolphmwas someone going to tackle binary tokens in juno-1?23:45
topolso how did dolphm let the variable "bigboy" get into the test case for the password stuff. He likes the tests cases to have a more professional decorum :-)23:45
morganfainbergLOL23:45
morganfainbergi was going to let that var name slide23:46
morganfainbergi thought twice about it though23:46
topolso WARNING did we pick a bar yet to meet up at Sunday night in Atlanta.  I want to make sure I show up on time this time23:46
morganfainbergbut... i was amused23:46
dolphmtopol: i want to write a utf-8 generating method and take a pass at replacing all the arbitrary crap in tests with that (including uuid generation)23:46
morganfainbergdolphm, utf8_character_set[os.random-offset:length] ?23:47
topolmorganfainberg ++23:47
morganfainbergwith a little magic to do wrapping23:47
dolphmmorganfainberg: pretty much -- is utf8_character_set a thing?23:47
topolso are folks coming Sunday night?23:47
dolphmtopol: yes23:47
morganfainbergdolphm, probably somewhere23:48
morganfainbergOh hay, i think i'm arriving dinnertime on sunday23:49
dolphmpretty sure i'm there around 1123:50
dimstopol, i'll be there sun night too23:50
dolphmam23:51
* morganfainberg checks flight info23:51
morganfainbergi should be arriving 1730ish23:51
jamielennoxfriday afternoon :)23:51
ayoungjamielennox, ah...got confused.  You are right, I was thinking requests-kerberos.  But I don't want us pulling in the Kerberos library for another reason:   I want to push for straight HTTPD integation, not Kerberos in Eventlet.23:52
topolexcellent. I get in around dinner time on Sunday night as well23:53
jamielennoxayoung: i thought we decided we were going to support both in the same plugin?23:53
ayoungjamielennox, no, I only agreed to half of that23:53
ayoungI agreed to put "kerberos" in the "method" field23:53
topoldims you going to Vegas?23:53
ayoungbut...do you really think it makes sense to do in process kerberos?23:54
dimstopol, nope. just Atlanta23:54
ayoungI can see an argument for supporting it in conjunction with Token binding in a different service if getting it to run in HTTPD is too hard, but I would rather push for HTTPD23:55
jamielennoxayoung: i see that if we have the 'kerberos' plugin that it's really easy to check for HTTPD headers and authenticate easily and if you don't have them you can do it in process23:56
jamielennoxif we're getting rid of eventlet i don't see the problem with speed23:56
ayoungjamielennox, back in a bit..gotta battle a couple of bed-resistant pre-teens23:56
jamielennoxall of this crypto is done in C anyway - my understanding is it was mostly the threading issues that we didn't want it before23:56
topolmorganfainberg I get in at 6:42pm23:56
topoljamielennox take their phones. thats what I always do.23:56
morganfainbergtopol, cool23:57
morganfainbergi'm still trying to chase down the best place to do a whiskey night23:57
morganfainbergi'll propose something for one of the nights, post dinner likely23:57
topolmorganfainberg excellent23:57
topolis jamielennox correct? Are we getting rid of eventlet? What are we moving to?23:59
topoldolphm?23:59

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