Monday, 2017-11-27

*** masuberu has joined #openstack-keystone00:28
*** masber has quit IRC00:29
*** dave-mccowan has joined #openstack-keystone00:41
*** dave-mccowan has quit IRC01:09
*** dave-mccowan has joined #openstack-keystone01:24
*** masuberu has quit IRC01:43
*** gmann_afk is now known as gmann02:37
*** threestrands_ has joined #openstack-keystone02:49
*** threestrands_ has quit IRC02:49
*** threestrands_ has joined #openstack-keystone02:49
openstackgerritwangxiyuan proposed openstack/keystone master: Remove member role assignment  https://review.openstack.org/52300502:49
*** threestrands_ has quit IRC02:50
*** threestrands has quit IRC02:50
*** threestrands_ has joined #openstack-keystone02:50
*** masber has joined #openstack-keystone02:54
openstackgerritwangxiyuan proposed openstack/keystonemiddleware master: Remove kwargs_to_fetch_token  https://review.openstack.org/51327302:54
*** dave-mccowan has quit IRC03:01
*** dave-mccowan has joined #openstack-keystone03:07
*** masber has quit IRC03:09
*** dave-mccowan has quit IRC03:18
openstackgerritwangxiyuan proposed openstack/keystone-specs master: Limits API  https://review.openstack.org/45570903:23
*** masber has joined #openstack-keystone03:29
*** daidv has joined #openstack-keystone04:00
*** hoonetorg has quit IRC04:11
*** Dinesh_Bhor has joined #openstack-keystone04:21
*** links has joined #openstack-keystone04:26
*** jaosorior has joined #openstack-keystone05:19
*** sticker has quit IRC05:33
*** dklyle has joined #openstack-keystone05:47
*** david-lyle has quit IRC05:48
*** d0ugal has quit IRC06:35
*** annp has joined #openstack-keystone06:39
openstackgerritwangxiyuan proposed openstack/keystone master: Expose a bug when create trust with roles  https://review.openstack.org/52270506:39
openstackgerritwangxiyuan proposed openstack/keystone master: Fix 500 error when create trust with invalid role key  https://review.openstack.org/52270606:39
*** spectr has joined #openstack-keystone06:44
*** namnh has joined #openstack-keystone06:47
*** rcernin has quit IRC06:47
*** d0ugal has joined #openstack-keystone06:48
*** rcernin has joined #openstack-keystone06:49
*** spectr has quit IRC06:55
openstackgerritwangxiyuan proposed openstack/keystone master: Remove member role assignment  https://review.openstack.org/52300507:00
*** rcernin has quit IRC07:28
openstackgerritwangxiyuan proposed openstack/keystone master: Add new tables for unified limits  https://review.openstack.org/52304107:30
*** AlexeyAbashkin has joined #openstack-keystone07:57
*** aojea has joined #openstack-keystone08:01
*** aojea has quit IRC08:06
*** pcaruana has joined #openstack-keystone08:11
*** magicboiz has joined #openstack-keystone08:21
*** belmoreira has joined #openstack-keystone08:22
*** magicboiz has quit IRC08:28
*** magicboiz has joined #openstack-keystone08:28
*** josecastroleon has joined #openstack-keystone08:40
*** gmann is now known as gmann_afk09:13
openstackgerritwangxiyuan proposed openstack/keystone master: Add new tables for unified limits  https://review.openstack.org/52304109:30
*** spectr has joined #openstack-keystone09:37
*** spectr has quit IRC09:37
*** aloga has quit IRC09:41
*** aloga has joined #openstack-keystone09:41
*** magicboiz has quit IRC09:55
*** annp has quit IRC10:03
*** mvk has joined #openstack-keystone10:05
*** magicboiz has joined #openstack-keystone10:11
*** magicboiz has quit IRC10:17
*** magicboiz has joined #openstack-keystone10:17
*** spectr has joined #openstack-keystone10:22
*** spectr has quit IRC10:23
*** namnh has quit IRC10:25
*** pcaruana has quit IRC10:55
*** pcaruana has joined #openstack-keystone10:59
*** ianw has quit IRC11:03
*** ianw_ has joined #openstack-keystone11:03
*** ianw_ is now known as ianw11:04
openstackgerritwangxiyuan proposed openstack/keystone master: Expose a bug when create trust with roles  https://review.openstack.org/52270511:25
openstackgerritwangxiyuan proposed openstack/keystone master: Fix 500 error when create trust with invalid role key  https://review.openstack.org/52270611:25
*** spectr has joined #openstack-keystone11:35
*** spectr has quit IRC11:35
*** threestrands_ has quit IRC11:42
*** kong has quit IRC11:53
*** samueldmq has quit IRC11:53
*** ying_zuo has quit IRC11:53
*** kmalloc has quit IRC11:54
*** hrybacki has quit IRC11:54
*** ildikov has quit IRC11:54
*** Chealion has quit IRC11:54
*** magicboiz has quit IRC11:54
*** samueldmq has joined #openstack-keystone11:55
*** hrybacki has joined #openstack-keystone11:55
*** kong has joined #openstack-keystone11:55
*** Chealion has joined #openstack-keystone11:56
*** ildikov has joined #openstack-keystone11:56
*** cloudnull has quit IRC11:56
*** kmalloc has joined #openstack-keystone11:56
*** magicboiz has joined #openstack-keystone11:57
*** cloudnull has joined #openstack-keystone11:59
*** magicboiz has quit IRC12:01
*** magicboiz has joined #openstack-keystone12:02
*** raildo has joined #openstack-keystone12:05
*** dave-mccowan has joined #openstack-keystone12:08
*** dave-mcc_ has joined #openstack-keystone12:09
*** dave-mccowan has quit IRC12:12
*** fried_turkey is now known as efried12:18
*** magicboiz has quit IRC12:21
*** szaher has quit IRC12:22
*** panbalag has joined #openstack-keystone12:25
*** panbalag has left #openstack-keystone12:25
*** szaher has joined #openstack-keystone12:25
*** links has quit IRC12:29
*** links has joined #openstack-keystone12:43
*** jdennis has quit IRC13:01
*** jdennis has joined #openstack-keystone13:02
*** markvoelker has quit IRC13:16
*** markvoelker has joined #openstack-keystone13:16
*** magicboiz has joined #openstack-keystone13:33
*** daidv has quit IRC13:38
*** magicboiz has quit IRC13:38
*** magicboiz has joined #openstack-keystone13:39
openstackgerritwangxiyuan proposed openstack/keystone master: Fix 500 error when create trust with invalid role key  https://review.openstack.org/52270613:47
*** ying_zuo has joined #openstack-keystone13:51
lbragstado/14:20
cmurphy\o14:23
openstackgerritColleen Murphy proposed openstack/keystone master: Expose a bug when create trust with roles  https://review.openstack.org/52270514:25
*** josecastroleon1 has joined #openstack-keystone14:30
*** josecastroleon has quit IRC14:30
*** links has quit IRC14:32
*** belmoreira has quit IRC14:33
*** panbalag has joined #openstack-keystone14:38
*** panbalag has left #openstack-keystone14:38
*** rmascena has joined #openstack-keystone14:45
*** raildo has quit IRC14:48
openstackgerritLance Bragstad proposed openstack/keystone master: Fix 500 error when create trust with invalid role key  https://review.openstack.org/52270615:11
knikollao/15:11
*** panbalag has joined #openstack-keystone15:18
*** panbalag has left #openstack-keystone15:19
*** rmascena has quit IRC15:25
knikollalbragstad: we're offering the cloud computing course again this spring. any ideas/desire for proposing a keystone related project we can mentor?15:25
lbragstadknikolla: that's a good question - we have plenty of work to do15:26
knikollalbragstad: jan-may, that would be early in the rocky cycle.15:30
lbragstadyeah15:31
*** phalmos has joined #openstack-keystone15:31
gagehugoo/15:50
knikollagagehugo: o/15:56
*** josecastroleon1 has quit IRC16:04
lbragstadknikolla: we do any stuff on our roadmap that will push into rocky i'm sure16:07
*** belmoreira has joined #openstack-keystone16:09
*** alex_xu has quit IRC16:11
*** magicboiz has quit IRC16:14
*** magicboiz has joined #openstack-keystone16:16
*** magicboiz has quit IRC16:21
*** magicboiz has joined #openstack-keystone16:21
*** alex_xu has joined #openstack-keystone16:27
*** raildo has joined #openstack-keystone16:32
*** belmoreira has quit IRC16:35
*** AlexeyAbashkin has quit IRC16:59
*** gyee_ has joined #openstack-keystone17:00
*** pcaruana has quit IRC17:09
*** jmlowe has quit IRC17:16
openstackgerritMerged openstack/keystone master: Expose a bug when create trust with roles  https://review.openstack.org/52270517:30
*** efried is now known as efried_rollin17:38
*** harlowja has joined #openstack-keystone17:47
*** belmoreira has joined #openstack-keystone18:04
*** markvoelker has quit IRC18:10
*** spilla has joined #openstack-keystone18:31
*** AlexeyAbashkin has joined #openstack-keystone18:52
*** belmoreira has quit IRC18:52
*** belmoreira has joined #openstack-keystone18:53
*** AlexeyAbashkin has quit IRC18:56
*** jmlowe has joined #openstack-keystone18:59
*** linkmark has joined #openstack-keystone19:04
cmurphylbragstad: mordred kmalloc could you take another look at https://review.openstack.org/#/c/512505 ?19:12
lbragstadcmurphy: yep - i have that slated for my things to do this afternoon19:13
cmurphyty19:13
*** pcaruana has joined #openstack-keystone19:16
*** AlexeyAbashkin has joined #openstack-keystone19:17
*** AlexeyAbashkin has quit IRC19:22
openstackgerritLance Bragstad proposed openstack/oslo.policy master: Add scope_types to RuleDefault objects  https://review.openstack.org/51022219:23
openstackgerritGage Hugo proposed openstack/keystone master: WIP - Add JSON Web Token functionality  https://review.openstack.org/52320019:29
lbragstadlol - nice19:32
lbragstadgagehugo: i was just about to send a follow-up email regarding JWT from the summit :)19:32
gagehugolbragstad I threw that together at the SF airport :)19:33
gagehugodidn't have much time to work on it since, but now that the holiday is over should be more available19:35
kmalloccmurphy: it looks pretty good19:48
kmalloci'll do another longer pass in a bit19:48
kmallocand score it19:48
cmurphythanks kmalloc19:52
openstackgerritLance Bragstad proposed openstack/keystone-specs master: Remove role check from middleware from specs  https://review.openstack.org/52321020:02
*** dave-mcc_ is now known as dave-mccowan20:04
*** AlexeyAbashkin has joined #openstack-keystone20:17
lbragstadis anyone here interested in the totp auth stuff?20:20
lbragstadhttps://review.openstack.org/#/q/topic:totp-auth+(status:open+OR+status:merged)20:20
lbragstada couple of those patches are coming up on a year without an update20:21
*** AlexeyAbashkin has quit IRC20:21
lbragstadand i figured i'd ask if anyone wants to take them over before abandoning them20:21
openstackgerritLance Bragstad proposed openstack/keystone master: Validate disabled domains and projects online  https://review.openstack.org/25327320:26
cmurphymissing context on what the end goal was there20:26
cmurphyi remember adriant had worked on totp stuff at some point20:28
lbragstadyeah - i think that was to make it so end users could update their totp secrets, versus having an admin do it20:30
lbragstad^ that was one of the problems i remember with it, not sure if that is what dstanek was working on though20:30
gagehugoyeah I have no idea what those were going for20:31
*** jmlowe has quit IRC20:31
lbragstadit could have been just general cleanup, too20:31
*** threestrands_ has joined #openstack-keystone20:32
*** threestrands_ has quit IRC20:32
*** threestrands_ has joined #openstack-keystone20:32
*** threestrands_ has quit IRC20:33
*** threestrands_ has joined #openstack-keystone20:34
*** threestrands has joined #openstack-keystone20:34
*** threestrands has quit IRC20:34
*** threestrands has joined #openstack-keystone20:34
kmallocin theory anyone can update their totp secrets via the credentials api -- IIRC20:36
*** efried_rollin is now known as efried20:38
*** threestrands_ has quit IRC20:44
lbragstadupdated unified limits spec is up https://review.openstack.org/#/c/455709/20:49
*** spilla has quit IRC20:56
adriantlbragstad, cmurphy: good timing! Just got to my computer21:10
adriantyeah, I've been sort of doing some totp stuff, but we're mostly doing it right now with a auth plugin for keystone to enforce password + totp21:11
adriantand I'm doing user setup of totp (or will soon) via workflow in our admin tasks service (adjutant)21:12
adriantMy plan is to submit a spec to keystone for some things I'm missing for real MFA in keystone in the next few days.21:13
adriantEven if the spec isn't approved for Queens I'll probably start playing with some prototyping for it.21:13
adriantkmalloc: given the current default policy, you can list your own, but not create.21:17
adriantthat said for totp creds... it's not as easy as just create. You need to realistically confirm you can generate a passcode for it otherwise you will give keystone something unsafe, or something that stops you from being able to auth.21:18
adriantbecause totp as a single auth method is useless by itself, it only really makes sense in context of MFA, and if you turn that on without confirming you can actually generate a passcode, then you're kind of screwed. That's why we are doing it as a workflow where you have to confirm a valid passcode before we turn it on for you in keystone.21:20
*** aselius has joined #openstack-keystone21:21
kmalloclbragstad: commented on limits21:22
openstackgerritGage Hugo proposed openstack/keystone master: Migrate legacy-tempest-dsvm-py35 job to zuulV3  https://review.openstack.org/52323121:34
*** belmoreira has quit IRC21:34
*** Tahvok has quit IRC21:35
openstackgerritGage Hugo proposed openstack/keystone master: Migrate legacy-tempest-dsvm-py35 job to zuulV3  https://review.openstack.org/52323121:36
*** pcaruana has quit IRC21:41
*** raildo has quit IRC21:47
openstackgerritMerged openstack/keystone master: Fix 500 error when create trust with invalid role key  https://review.openstack.org/52270621:50
kmalloclbragstad: i'm going to abandon any patch that hasn't been touched in > 1 year with a comment.21:53
kmalloclbragstad: FYI21:53
kmalloclbragstad: looks like it was 1 patch in keystone.21:53
lbragstadkmalloc: yeah - i spent a bunch of time last week cleaning up the review queue21:55
kmalloci think anything > 7 mo should be abandoned21:55
kmallocbut i'm just nuking 1yr + now21:55
kmalloclbragstad: ok all patchsets a 12mo or more were abandoned21:58
*** markvoelker has joined #openstack-keystone22:09
*** aojea has joined #openstack-keystone22:10
*** d0ugal has quit IRC22:21
openstackgerritGage Hugo proposed openstack/keystone master: Add New in Pike note to using db_sync check  https://review.openstack.org/52323822:32
*** rcernin has joined #openstack-keystone22:34
kmalloclbragstad: https://review.openstack.org/#/c/253273/44 needs correct exceptions raised, otherwise +2.22:36
*** phalmos has quit IRC22:42
kmalloccmurphy: reveiwed, no score. but responded to your and lbragstad's comments on identity-acting-app-creds22:43
*** jmlowe has joined #openstack-keystone22:43
lbragstadkmalloc: quick question22:47
lbragstadkmalloc: is DomainNotFound accurate if the domain is disabled?22:47
lbragstadthe domain technically exists, but it isn't enabled22:49
lbragstadah - i think i see what you mean,22:51
cmurphykmalloc: are you saying something like have POST /v3/users/{}/application_credentials and a special POST /v3/users/{}/identity_application_credentials ? re https://review.openstack.org/#/c/51250522:52
lbragstadso as long as it's still a 404 - we should be able to change from a TokenNotFound to DomainNotFOund22:52
kmalloccmurphy: yes22:59
kmalloclbragstad: yes, 404 still22:59
kmalloccmurphy: that was my thinking on how we can address the issue.23:00
*** sticker has joined #openstack-keystone23:00
kmalloccmurphy: if we differentiate or disallow normal app-creds from working on identity... we pretty much need a new path so we can have policy that allows correct access to it23:00
cmurphykmalloc: so what is the result of a POST /v3/users/{}/identity_app_cred? an object with a special flag?23:00
kmalloccmurphy: the same exact cred, just with the "identity-actionable-whatever-we-call-it" flag23:01
cmurphyokay23:01
kmallocs/exact/same object type/23:01
kmalloccmurphy: if we go with the "shutup and let me do the insecure thing" for any user (not default), my comment is not relevant23:01
kmallocit's just an alternative we could use to allow locking down app-creds for keystone to known classes of uses w/o specific IDs (et al) in config23:02
cmurphykmalloc: aha, so what do you think about allowing users to do that?23:02
kmalloci prefer normal users don't get that by default23:02
kmalloci don't feel it is worthy of changing from a +2 once we address that specifically23:02
kmallocin either way23:02
cmurphywell - normal users won't have roles that can really do anything on the identity api, so they wouldn't be able to grant their app creds anything special23:03
kmallocso, in short, i'd prefer normal user not acting on identity with the "shut up and let me do it" flag (again, not default). but i would still +2 it23:03
*** jmlowe has quit IRC23:03
kmalloc"normal" being <whatever role can create an app-cred, given they have an identity actionable role>23:03
kmalloci just want to be clear we are either allowing differentiation so identity is not actionable with "normal" app creds. or we're not23:04
kmallocright now it feels like we're saying "uh, maybe...? something"23:04
kmalloci'd be happy enough to accept it in either case. but if we do explicitly differentiate (aka, need a specific user/role to make an app-cred that can identity things), we need to think hard about how that works.23:05
openstackgerritLance Bragstad proposed openstack/keystone master: Validate disabled domains and projects online  https://review.openstack.org/25327323:06
kmalloclbragstad: +223:07
lbragstadfastest +2 ever!23:07
kmalloclbragstad: i also +A'd the spec (rbac thing) removal23:07
kmalloclbragstad: we can revert/add it back in if needed at anypoint23:07
lbragstadoh nice23:07
lbragstadright23:07
cmurphykmalloc: so in this revision i narrowed down the blocklist from blocking the whole identity API to just blocking the app cred creation for the reason i gave above, a non-admin user won't be able to grant the ability to do those things anyway23:07
cmurphykmalloc: before talking about how to implement it do you think that's a mistake? (differentiating all identity API from app cred creation)23:08
* lbragstad dangles https://review.openstack.org/#/c/464763/ out for people to review23:08
kmalloccmurphy: i simply want to say we can't differentiate an ADMIN vs say normal user on the same api path23:08
*** panbalag has joined #openstack-keystone23:09
kmalloccmurphy: so we can't block for everyone but, say, heat to create more app-creds23:09
kmallocif we allow no one to create app-creds with app-creds.. it means heat and similar use cases are unhappy/need another path we can lock down to specific users23:09
kmallocif we allow anyone to create an app-cred that can make app-creds, it's fine as is.23:10
kmalloci simply want to ensure we're not trying to do magic based upon a ... role or user that allows for this special circumstance23:10
kmalloccmurphy: if that helps narrow down what i want.23:10
*** panbalag has left #openstack-keystone23:10
kmalloci am also 100% ok with simply not allowing app-creds being made with app-creds (Similar to how trusts work)23:11
kmallocbut i know it will disappoint heat23:11
*** gmann_afk is now known as gmann23:11
cmurphykmalloc: what i'm wondering is if you care whether e.g. POST /v3/projects is lumped in with POST /v3/users/{}/app_creds on the allow/block list, however that gets implemented23:12
openstackgerritMerged openstack/keystone-specs master: Remove role check from middleware from specs  https://review.openstack.org/52321023:12
kmallocoh no i don't care really.23:12
cmurphyokay23:12
kmalloci don't feel strongly either direction23:12
kmalloci care at the basic level of what we block/allow23:12
kmallocalso... if we're blocking app-creds... we should block trusts from being created via app-creds23:12
kmallocfwiw23:12
kmallocbecause in theory you could then create a trust, then create an app-cred to do whatever you want23:13
kmalloc(or vice-versa, no app-creds from trusts)23:13
cmurphythat's true...23:13
lbragstadyeah....23:13
kmallocthere are a lot of edge cases we have addressed in trusts =/23:13
kmallocand we have to do all the same work for app-creds really.23:14
kmallocthe easiest solution is: anyone can create an appcred that makes an app-cred23:14
openstackgerritMerged openstack/pycadf master: Updated from global requirements  https://review.openstack.org/52050223:18
*** dave-mccowan has quit IRC23:24
kmalloccmurphy: Nice summary in the notes23:25
kmalloccmurphy: thanks I was about to circle back and comment about the IRC convo! :P you beat me to it23:25
cmurphy\o/23:26
*** linkmark has quit IRC23:26
*** aojea has quit IRC23:31
*** jmlowe has joined #openstack-keystone23:35

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