Thursday, 2015-08-20

jamielennoxhow do we roll forward the existing federation CLI plugins?00:00
ayoungjamielennox, we have to deal with what we have for now.00:00
*** mylu_ has quit IRC00:00
ayoungWe'll do the two blocks, config the Keystone values before apache, and so on00:00
*** dave-mccowan has quit IRC00:00
ayoungbut let's do the websso thing outside of /v300:01
ayoungclient doesn't need to know about it00:01
ayoungjust make sure that Federation without webssso has a place to live, too.00:01
jamielennoxlbragstad: either way i think it should be /auth/OS-FEDERATION/identity_providers/{idp_id}/protocols/{protocol}/websso rather than /auth/OS-FEDERATION/websso/...00:01
*** mylu has joined #openstack-keystone00:01
jamielennoxbecause otherwise your apache modules are going to step all over each other00:02
ayoungjamielennox, I could see telling the CLI that the AUTH URL is  httpsd://hostname:5000/auth/idp/<>/protocols/  even00:02
ayoungjamielennox, ++00:02
openstackgerritLance Bragstad proposed openstack/keystone-specs: IDP specific websso  https://review.openstack.org/19933900:02
ayoungI think we can frop OS-FEDERATION00:02
lbragstadjamielennox: ah, I think that makes sense?00:02
ayoungmake sure token auth can work in there somehow too00:03
ayoungactually, that should be00:03
lbragstadbecause you'll have some apache directive that could be on /auth/OS-FEDERATION/websso/00:03
ayoung /auth/OS-FEDERATION/identity_providers/keystone/protocols/password00:03
ayounger00:03
ayoung /auth/identity_providers/keystone/protocols/password00:03
ayoungor00:03
ayoung /auth/identity_providers/keystone/protocols/token  ?00:03
ayoungjamielennox, what is done now for enumerating projects etc?00:04
*** geoffarn_ has quit IRC00:04
jamielennoxlbragstad: generally they'd probably be on /websso/{protocol} but still gets confusing00:04
jamielennoxayoung: latest is /auth/projects00:04
ayoungthat is why we can't put /auth/idp ,right?00:04
ayoungso that should have been /auth/token/projects00:05
jamielennoxi'm not sure how that fits into morgan_2549's auth split00:05
ayoungto avoid cluttering the namespace00:05
ayoungyeah, it should not be implicit00:05
ayoungit should be a minimal service catalog like you proposed00:05
jamielennoxthat exists /auth/catlog00:05
*** geoffarnold has joined #openstack-keystone00:06
jamielennoxmy intent there was that /auth should be everything that can be performed without a service catalog00:06
jamielennoxbecause putting a service catalog in the unscoped token got killed00:06
jamielennoxor give me information relevant to my current authentication00:06
ayoungyep00:07
jamielennoxi would have no issue with that being part of the auth split00:07
ayoungwell, we are not going to solve this tonight, are we?00:07
jamielennoxi would say it might even have to be00:07
jamielennoxayoung: it depends - i feel if we don't have something we're happy with for the idp specific websso by next meeting it won't make this cycle00:07
ayoungjamielennox, then for now, maybe just keep  /auth/OS-FEDERATION/identity_providers/{idp_id}/protocols/{protocol}/websso and allow  /auth/OS-FEDERATION/identity_providers/{idp_id}/protocols/{protocol} to work, too?00:08
ayoung /auth/OS-FEDERATION/identity_providers/{idp_id}/protocols/{protocol}  to work for CLI, too00:08
jamielennoxayoung: currently there is .../{protocol}/auth for CLI authentication00:09
ayoungjamielennox, ah...that will work00:10
ayoungso00:10
ayoung.../{protocol}/websso00:10
*** geoffarnold has quit IRC00:10
ayoungand then the matching rule can be00:10
jamielennoxayoung: is there a reason to distinguish CLI and websso login here though?00:10
ayoung~ .../{protocol}/*00:10
jamielennoxayoung: how do you configure shib for the difference?00:10
ayoungyeah, webssso does the redirect00:10
jamielennoxhow do you configure mellon for the difference00:10
jamielennoxayoung: right but it's the apache module that does the redirect00:11
jamielennoxif you present it with an ECP assertion up front it wouldn't right?00:11
jamielennoxthis is part of why i want to get this environment set up so i can see what the configuration differences are for ECP and websso00:11
ayoungjamielennox, http://httpd.apache.org/docs/2.4/mod/core.html#locationmatch00:11
*** lhcheng has quit IRC00:12
jamielennoxright00:12
dstanekmorgan_2549: have you been paying attention to the interface discussion?00:12
ayoungjamielennox, I think we can move the keystone-sssd.yml before the keystone.yml, no?00:12
jamielennoxayoung: probably, i don't know, different discussion for different channel00:13
ayoungah, not without breaking the HTTPS00:13
jamielennoxbut if you have ECP set up can we not simply run bot websso and ECP via /auth/OS-FEDERATION/identity_providers/{idp_id}/protocols/{protocol}00:13
jamielennoxis there a reason for the /websso and /auth distinguishers00:14
*** dave-mccowan has joined #openstack-keystone00:14
lbragstadI think the federated_sso call returns a web form with the redirect back to the origin host (horizon)00:14
lbragstadstraight federated cli auth doesn't do that I don't think00:14
jamielennoxlbragstad: but is that only in the situation where you didn't come with an assertion already00:14
jamielennoxbecause i don't see any configuration difference for ECP vs websso00:14
jamielennoxi guess there is no harm to having seperate routes and it might help in future if we come across something that does need to be handled seperateyl00:15
jamielennoxlbragstad: so i think we append /auth to that newest spec review00:17
jamielennoxand we'll sort out CLI in the future00:17
lbragstadappend to the end of the call?00:18
lbragstadand do OS-FEDERATION/identity_provider/ ?00:18
jamielennoxahh, prepend - i do that a lot00:18
morgan_2549dstanek: some of it00:18
morgan_2549dstanek: but not this week00:18
jamielennox /auth/OS-FEDERATION/identity_providers/{idp_id}/protocols/{protocol}/websso00:18
*** shoutm has quit IRC00:18
jamielennoxmaybe we can come up with some regexp or something that will then allow us to match both CLI and websso00:19
jamielennoxbut it should be /auth if we expect it to be part of the auth split which this definetly would be00:19
lbragstadok00:21
*** shoutm has joined #openstack-keystone00:21
jamielennoxlbragstad: are you looking at a DOA that will let us test this?00:21
lbragstadI haven't started on that yet00:22
*** shadower has quit IRC00:23
*** shadower has joined #openstack-keystone00:23
jamielennoxok, i want to play with this patch anyway, i'll push something and email you if i start on it today00:23
lbragstadjamielennox: want me to push a new patch with the path changes?00:23
*** mylu has quit IRC00:24
lbragstadjamielennox: I don't want to overwrite anything you're working on if we're both pushing patches00:24
* lbragstad and dolphm had issues with that when we were working on the fernet stuff 00:24
jamielennoxlbragstad: if you're still around sure, if you're supposed to be home then i can push something as i get to it00:25
*** mylu has joined #openstack-keystone00:25
lbragstadjamielennox: I worked from home today (does that count?)00:25
jamielennoxlbragstad: it depends how much trouble you get in from working late whilst at home00:26
lbragstadjamielennox: and *that* depends on when the wife's mood starts to deteriorate00:26
lbragstadjamielennox: so, we're not going to be extending wsgi.V3ExtensionRouter anymore for this specific call, right?00:28
jamielennoxcompletely understand00:28
jamielennoxi don't think so00:28
jamielennoxhmm00:28
lbragstadok, because I think the '/v3/' part is tacked on there..00:28
jamielennoxyou may as well use whatever the current federation stuff uses00:28
lbragstadcurrent federation paths look like they start with00:29
jamielennoxno point trying to add all new modules for this00:29
lbragstad'auth'00:29
*** mylu has quit IRC00:29
jamielennoxlbragstad: the current patch looks good from a glance if you update the route00:29
jamielennoxi think smallest possible change is good for this one with the SFE00:30
lbragstadjamielennox: ok00:30
*** dims has joined #openstack-keystone00:30
lbragstadso /v3/auth/OS-FEDERATION/websso/identity_providers/{idp_id}/protocols/{protocol_id}/ will change to,00:31
*** tiny-hands has joined #openstack-keystone00:31
lbragstadthis /v3/auth/OS-FEDERATION/identity_providers/{idp_id}/protocols/{protocol_id}/websso00:31
lbragstadjamielennox: is that what you mean by update the route?00:31
jamielennoxyep00:32
lbragstadok, I can respin that quick00:32
*** gyee has quit IRC00:33
*** dave-mccowan has quit IRC00:37
*** piyanai has quit IRC00:38
jamielennoxergh, DOA might be uglier than anticipated00:43
*** sigmavirus24 is now known as sigmavirus24_awa00:46
openstackgerritLance Bragstad proposed openstack/keystone: Add federated auth for idp specific websso  https://review.openstack.org/21476600:52
*** fangzhou has quit IRC00:57
*** dave-mccowan has joined #openstack-keystone00:59
*** browne has quit IRC01:06
*** lhcheng has joined #openstack-keystone01:17
*** ChanServ sets mode: +v lhcheng01:17
*** dims has quit IRC01:25
*** fangzhou has joined #openstack-keystone01:25
*** mylu has joined #openstack-keystone01:26
*** mylu has quit IRC01:30
*** qiaowei has joined #openstack-keystone01:37
*** mpmsimo has joined #openstack-keystone01:43
*** ankita_wagh has joined #openstack-keystone01:46
*** woodster_ has quit IRC01:49
*** boris-42 has quit IRC01:50
*** piyanai has joined #openstack-keystone01:51
*** ankita_wagh has quit IRC01:57
*** ankita_wagh has joined #openstack-keystone01:57
*** dsirrine has quit IRC02:01
*** ankita_wagh has quit IRC02:01
*** _cjones_ has quit IRC02:02
*** davechen has joined #openstack-keystone02:04
*** fangzhou has quit IRC02:12
*** mpmsimo has quit IRC02:19
*** ankita_wagh has joined #openstack-keystone02:20
*** mpmsimo has joined #openstack-keystone02:21
*** dims has joined #openstack-keystone02:21
*** lhcheng has quit IRC02:26
*** mylu has joined #openstack-keystone02:27
*** tiny-hands has quit IRC02:29
*** dims has quit IRC02:29
*** dims has joined #openstack-keystone02:30
*** shoutm_ has joined #openstack-keystone02:30
*** mylu has quit IRC02:31
*** shoutm has quit IRC02:31
*** lhcheng has joined #openstack-keystone02:33
*** ChanServ sets mode: +v lhcheng02:33
*** dims has quit IRC02:34
*** tiny-hands has joined #openstack-keystone02:35
*** markvoelker has quit IRC02:46
*** lhcheng has quit IRC02:46
*** hakimo has joined #openstack-keystone02:52
*** piyanai has quit IRC02:54
*** hakimo_ has quit IRC02:54
*** nkinder has quit IRC03:03
*** shoutm has joined #openstack-keystone03:06
*** shoutm_ has quit IRC03:07
*** browne has joined #openstack-keystone03:10
*** lhcheng has joined #openstack-keystone03:14
*** ChanServ sets mode: +v lhcheng03:14
openstackgerritMerged openstack/keystone: Add is_domain field in Project Table  https://review.openstack.org/21327303:23
*** narengan has joined #openstack-keystone03:26
openstackgerritOpenStack Proposal Bot proposed openstack/keystone: Updating sample configuration file  https://review.openstack.org/21433903:29
*** piyanai has joined #openstack-keystone03:32
*** Kennan2 is now known as Kennan03:36
*** tiny-hands has quit IRC03:42
*** markvoelker has joined #openstack-keystone03:46
*** markvoelker has quit IRC03:51
*** mylu has joined #openstack-keystone03:56
*** piyanai has quit IRC03:58
*** shoutm_ has joined #openstack-keystone04:00
*** shoutm has quit IRC04:01
*** lhcheng has quit IRC04:03
*** ankita_wagh has quit IRC04:18
*** hrou has quit IRC04:18
*** ayoung has quit IRC04:20
*** dave-mccowan has quit IRC04:22
*** ankita_wagh has joined #openstack-keystone04:22
*** mflobo has quit IRC04:24
*** mflobo has joined #openstack-keystone04:36
*** ankita_wagh has quit IRC04:43
*** ankita_wagh has joined #openstack-keystone04:44
*** darrenc is now known as darrenc_afk04:46
*** darrenc_afk is now known as darrenc05:06
*** mylu has quit IRC05:07
*** mylu has joined #openstack-keystone05:08
*** hafe has joined #openstack-keystone05:08
*** afazekas has joined #openstack-keystone05:08
*** mylu has quit IRC05:12
*** mylu has joined #openstack-keystone05:12
*** afazekas has quit IRC05:15
*** kiran-r has joined #openstack-keystone05:18
*** hafe has quit IRC05:33
*** geoffarnold has joined #openstack-keystone05:39
*** geoffarnold has quit IRC05:43
*** geoffarnold has joined #openstack-keystone05:43
*** mylu has quit IRC05:45
*** geoffarnold is now known as geoffarnoldX05:45
*** mylu has joined #openstack-keystone05:45
*** lhcheng has joined #openstack-keystone05:52
*** ChanServ sets mode: +v lhcheng05:52
*** shoutm_ has quit IRC05:52
*** lhcheng has quit IRC05:56
*** kiran-r has quit IRC05:59
*** mpmsimo has quit IRC06:01
*** topol has joined #openstack-keystone06:05
*** ChanServ sets mode: +v topol06:05
*** shoutm has joined #openstack-keystone06:07
*** topol has quit IRC06:09
*** narengan has quit IRC06:13
*** narengan has joined #openstack-keystone06:13
*** ajayaa has joined #openstack-keystone06:14
*** mpmsimo has joined #openstack-keystone06:15
*** narengan has quit IRC06:18
*** afazekas has joined #openstack-keystone06:21
*** mflobo has left #openstack-keystone06:29
*** afazekas has quit IRC06:37
*** ajayaa has quit IRC06:37
*** urulama has quit IRC06:39
*** mpmsimo has quit IRC06:39
*** urulama has joined #openstack-keystone06:39
*** mpmsimo has joined #openstack-keystone06:40
*** mpmsimo has left #openstack-keystone06:41
*** ajayaa has joined #openstack-keystone06:50
*** henrynash has joined #openstack-keystone06:59
*** ChanServ sets mode: +v henrynash06:59
*** browne has quit IRC06:59
*** mylu has quit IRC07:00
*** mylu has joined #openstack-keystone07:00
*** Nirupama has joined #openstack-keystone07:03
*** mylu has quit IRC07:05
*** marekd_404 is now known as marekd07:08
*** mylu has joined #openstack-keystone07:11
*** mflobo has joined #openstack-keystone07:13
mfloboHi there, question for the community, Is possible to unset project metadata? How should be the CURL call?07:14
openstackgerritOpenStack Proposal Bot proposed openstack/keystone: Updated from global requirements  https://review.openstack.org/21450907:17
*** yottatsa has joined #openstack-keystone07:18
*** ankita_w_ has joined #openstack-keystone07:27
*** ankita_w_ has quit IRC07:27
*** ankita_w_ has joined #openstack-keystone07:28
*** ankita_wagh has quit IRC07:28
*** ankita_wagh has joined #openstack-keystone07:30
*** ankita_wagh has quit IRC07:30
*** ankita_wagh has joined #openstack-keystone07:30
*** yottatsa has quit IRC07:31
*** ankita_wagh has quit IRC07:32
*** ankita_w_ has quit IRC07:33
*** ankita_wagh has joined #openstack-keystone07:33
*** yottatsa has joined #openstack-keystone07:34
*** fhubik has joined #openstack-keystone07:40
*** lhcheng has joined #openstack-keystone07:41
*** ChanServ sets mode: +v lhcheng07:41
*** fhubik is now known as fhubik_brb07:44
*** lhcheng has quit IRC07:45
*** markvoelker has joined #openstack-keystone07:48
*** markvoelker has quit IRC07:53
*** fhubik_brb is now known as fhubik07:59
*** fhubik is now known as fhubik_brb08:00
*** lhinds has joined #openstack-keystone08:01
*** fhubik_brb is now known as fhubik08:02
mfloboHere I can not see anything http://developer.openstack.org/api-ref-identity-v3.html about metadata08:04
*** lhcheng has joined #openstack-keystone08:05
*** ChanServ sets mode: +v lhcheng08:05
*** lhcheng has quit IRC08:10
*** hafe has joined #openstack-keystone08:11
*** mylu has quit IRC08:14
*** afazekas has joined #openstack-keystone08:18
*** qiaowei has quit IRC08:19
*** jistr has joined #openstack-keystone08:28
openstackgerritJamie Lennox proposed openstack/keystone: Add federated auth for idp specific websso  https://review.openstack.org/21476608:29
*** shoutm has quit IRC08:29
*** fhubik is now known as fhubik_brb08:29
*** hafe has left #openstack-keystone08:29
*** fhubik_brb is now known as fhubik08:33
*** pnavarro has joined #openstack-keystone08:36
*** aix has joined #openstack-keystone08:41
*** shoutm has joined #openstack-keystone08:44
*** ankita_wagh has quit IRC08:45
*** shoutm has quit IRC08:54
*** shoutm has joined #openstack-keystone08:55
*** jamie_h has joined #openstack-keystone09:02
marekdrodrigods: https://review.openstack.org/#/c/190361/28 so i am really looking forward to see code and capabilities of those puppet module. It's still full of inconsistencies...09:08
marekdrodrigods: they talk about supporting saml and oidc and then don't give a ** about oidc09:08
marekdand want to do this from one module?09:08
*** tsubic has quit IRC09:13
*** shoutm has quit IRC09:15
*** jistr has quit IRC09:19
*** jistr has joined #openstack-keystone09:20
*** yottatsa has quit IRC09:26
*** fhubik is now known as fhubik_brb09:28
*** fhubik_brb is now known as fhubik09:38
openstackgerritDave Chen proposed openstack/keystone: WIP - Should return no result for unexpected query  https://review.openstack.org/21504109:38
davechenmarekd: hi,09:38
*** yottatsa has joined #openstack-keystone09:38
davechenmarekd, henrynash: hi, are you there?09:39
marekddavechen: hello i am here09:40
davechenmarekd, henrynash: I am working on this bug #1479837, after some investigation, I think there should be some change in keystone and will modify the original design.09:40
openstackbug 1479837 in Keystone "improper handling non existing identity providers " [Medium,In progress] https://launchpad.net/bugs/1479837 - Assigned to Dave Chen (wei-d-chen)09:40
davechenmarekd, hi, this bug is filed by you, I did some investigation today,09:40
davechenand propose a initial patch.09:41
marekdhttps://review.openstack.org/215041 ?09:41
davechenmarekd: yes.09:41
marekdshouldnw we expand it to the whole project?09:42
davechenmarekd: I think the previous logic is implemented by henry.09:42
marekddavechen: what are hints for ?09:42
davechenmarekd: yes, I think so.09:42
davechenmarekd: hint is the query for the backend. such as name=...09:43
davechenand in that bug is id=donexist.09:43
* marekd was 1 week away and when looks at a pile of reviews and fixes pending just wants to cry09:43
davechen:)09:43
*** fhubik is now known as fhubik_brb09:44
davechenmarekd: I think to address that bug I need to change a lot in keystone, so I am not quite sure.09:44
davechenmarekd: can I add you to the review list, so when you get a chance you can take a look?09:45
davechenreviewer list.09:45
*** lhinds has quit IRC09:47
*** yottatsa has quit IRC09:48
*** yottatsa has joined #openstack-keystone09:49
*** markvoelker has joined #openstack-keystone09:49
*** yottatsa has quit IRC09:50
davechenmarekd: I need to take the shuttle, talk to you later. :)09:50
marekddavechen: thanks09:51
marekdi will review today09:51
*** aix has quit IRC09:51
davechenmarekd: anytime is okay, no need today.09:53
davechenmarekd: just want to know if there is big mistake there.09:54
*** markvoelker has quit IRC09:54
davechenmarekd: talk to you later, have a good day!09:54
*** davechen has left #openstack-keystone09:54
marekdthanks :-)09:55
*** yottatsa has joined #openstack-keystone09:55
*** dims has joined #openstack-keystone09:56
*** lhcheng has joined #openstack-keystone09:59
*** ChanServ sets mode: +v lhcheng09:59
*** fhubik_brb is now known as fhubik09:59
*** Kennan2 has joined #openstack-keystone10:03
*** Kennan has quit IRC10:04
*** tiny-hands has joined #openstack-keystone10:06
*** tiny-hands has quit IRC10:08
* marekd feels like Federation is destroying Fernet tokens10:09
*** yottatsa has quit IRC10:14
odyssey4memarekd ?10:16
marekdodyssey4me: heh, so fernet was meant to be non persistent, yet small token and step after step we find use cases where someting is missing and need to stuff more and more into token payloads10:17
marekdwhich completely kills idea of fernet tokens.10:17
*** henrynash has quit IRC10:21
odyssey4memarekd well, it's making them larger but the lack of persistence is still good - it cuts down the DB load dramatically and as long as the tokens are less onerous than PKI tokens this is still a win in my books. :)10:25
marekdodyssey4me: yeah, but there is a hardlimit - 255B10:26
marekdeverything over that size will start making them not-to-cool10:26
marekdodyssey4me: besides...we still query db for projects users, domains, roles etc.10:27
odyssey4memarekd hmm, that hard limit may become a problem depending on how the mappings are done10:30
odyssey4methe db queries for persistent data like projects, users, domains, etc is ok in my mind - the token storage is a pita for ops10:31
marekdpita?10:31
marekdodyssey4me: well, ok i can imagine that querying project or user is much faster than 1000s of tokens10:32
marekdand this is what defends fernet tokens10:32
odyssey4mepita = pain in the ass :p10:32
marekdodyssey4me: oh10:33
marekdwhat is the other 'pita' reason rather than number of tokens?10:33
marekdor it's the only thing?10:33
odyssey4methe queries in large environments, the constant synchronisation - which is a pain when you're looking at a global cluster for keystone, the need for the background process to clean the database of expired tokens10:34
odyssey4meit makes keystone's db more write heavy than it needs to be10:34
marekdodyssey4me: right.10:35
dstanekshould be an easy one to merge: https://review.openstack.org/#/c/201738/10:37
marekddstanek: is admin_request appropriate there?10:50
dstanekmarekd: which one?10:52
marekddstanek: in test you pasted10:52
dstanekmarekd: which admin_request? the one to get the token or the one to delete?10:55
*** gpanda has joined #openstack-keystone10:56
*** gpanda has quit IRC10:56
dolphmdstanek: why are you up so early11:01
dstanekdolphm: you're up earlier!11:05
dolphmoh right, timezones work in that direction11:06
*** topol has joined #openstack-keystone11:07
*** ChanServ sets mode: +v topol11:07
hugokuohttps://youtu.be/duRBlm9RtCw watching Fernet Token.11:07
*** henrynash has joined #openstack-keystone11:09
*** ChanServ sets mode: +v henrynash11:09
*** topol has quit IRC11:11
*** urulama has quit IRC11:12
*** lhcheng has quit IRC11:13
*** urulama has joined #openstack-keystone11:13
*** jamie_h has quit IRC11:14
*** dims has quit IRC11:15
*** dims has joined #openstack-keystone11:16
*** henrynash has quit IRC11:16
*** aix has joined #openstack-keystone11:16
*** dims has quit IRC11:20
openstackgerritMehdi Abaakouk (sileht) proposed openstack/keystonemiddleware: Allow to use oslo.config without global CONF  https://review.openstack.org/20896511:21
openstackgerritOpenStack Proposal Bot proposed openstack/keystone: Updated from global requirements  https://review.openstack.org/21450911:21
*** yottatsa has joined #openstack-keystone11:21
openstackgerritMehdi Abaakouk (sileht) proposed openstack/keystonemiddleware: Allow to use oslo.config without global CONF  https://review.openstack.org/20896511:22
*** yottatsa has quit IRC11:22
*** yottatsa_ has joined #openstack-keystone11:22
*** yottatsa_ has quit IRC11:27
*** dims has joined #openstack-keystone11:33
*** markvoelker has joined #openstack-keystone11:35
*** yottatsa has joined #openstack-keystone11:35
*** yottatsa has quit IRC11:37
*** markvoelker has quit IRC11:40
*** gordc has joined #openstack-keystone11:45
*** samueldmq has joined #openstack-keystone11:46
*** afazekas has quit IRC11:46
*** woodster_ has joined #openstack-keystone11:48
samueldmqmorning12:01
*** piyanai has joined #openstack-keystone12:10
*** topol has joined #openstack-keystone12:15
*** ChanServ sets mode: +v topol12:15
marekddstanek: https://review.openstack.org/#/c/201738/3/keystone/tests/unit/test_v3_auth.py the one from line 39212:19
dstanek marekd that's how we get a v2 token in most of these tests12:21
*** doug-fish has joined #openstack-keystone12:23
*** edmondsw has joined #openstack-keystone12:23
*** topol has quit IRC12:24
iurygregorymarekd, what do you mean by "full of inconsistencies"?12:27
openstackgerrithenry-nash proposed openstack/keystone: Support project hierarchies in data driver tests  https://review.openstack.org/15448512:31
iurygregorymarekd, what problems do you see in http://specs.openstack.org/openstack/puppet-openstack-specs/specs/liberty/enabling-federation.html ?12:33
*** henrynash has joined #openstack-keystone12:34
*** ChanServ sets mode: +v henrynash12:34
marekdiurygregory: for instance you seem to make references that you will be able to setup saml2 and oidc and later you mention only shibd and mellon modules.12:36
marekdiurygregory: but since the spec was approved you are good to go and implement modules :-)12:37
iurygregorywe are going to support shib mellon and oidc12:38
*** markvoelker has joined #openstack-keystone12:39
iurygregoryi have only give examples with shib and mellon because I am familiar with they12:39
marekdiurygregory: line 60 - if available modules are shibboleth and mellon how are you going to support oidc then?12:40
iurygregoryoidc is protocol like SAML12:41
iurygregorythis are module available for SAML12:41
iurygregory=)12:41
marekdbesides to me saying "OpenID Connect, Shibboleth and Mellon"  is like saying "truck, Fiat and Porsche"12:42
marekdiurygregory: ok, so how are you going to support OpenID Connect if the only (acording to spec) modules are those capable of handling SAML2?12:42
marekdunless one of those started supporting oidc?12:43
iurygregory"protocol: The protocol used to provide Federation. There is support for two protocols: OpenID Connect and SAML"12:43
iurygregoryyou need to choose one12:43
iurygregoryif you choose SAML you need to choose the module (shib/mellon)12:43
marekdiurygregory: ok, so what is going to happend if i choose "OIDC" as protocol and module set to mellon ?12:44
iurygregorythe module will not allow12:44
iurygregoryor just ignore the "module"12:45
marekdis the protocol going to be used to configure protocol via Keystone API ?12:45
iurygregorycan you make it more clear?12:45
*** chlong has joined #openstack-keystone12:46
marekdwhat is 'protocol' going to do in your puppet module?12:46
marekdwhere are you going to use this value?12:46
iurygregorywill be used to verify and install the necessary packages12:46
iurygregoryand set the configuration in keystone files12:46
marekdso it will not actually execute any API calls ?12:47
marekdiurygregory: next - line 40 - where do i specify those choices?12:49
marekdFWIW protocol can be only "OpenID Connect" or "SAML"12:50
marekdand module only 'shibboleth' and 'mellon'12:50
*** dsirrine has joined #openstack-keystone12:51
iurygregoryyou can specify in your "site.pp" ^^12:51
EmilienMit's called Puppet parameters12:52
marekdiurygregory: but what parameter is it going to be?12:52
marekdprotocol ?12:52
marekdmodule?12:52
marekdsomething that you didn't mention?12:52
*** dave-mccowan has joined #openstack-keystone12:52
marekdanyways, go ahead and implement.12:52
iurygregoryyou can take a look at the example12:53
EmilienMI suggest iurygregory adding marekd as a reviewer when code is pushed12:53
iurygregoryline 17512:53
*** jistr is now known as jistr|mtg12:53
*** thiagop has joined #openstack-keystone12:54
*** aix has quit IRC12:54
marekdiurygregory: ok, and which parameter should i choose to specify my preference from line 40?12:54
marekdi have three options12:54
marekdaccording to line 4012:54
iurygregorycan you go to line 180 and 181?12:56
*** henrynash has quit IRC12:57
marekdso there is protocol and module12:58
marekdso you flatten protocol and modules and make them list (OpenID Connect, Mellon and Shibbboleth) ?12:58
*** fhubik has quit IRC12:58
samueldmqmarekd, so basically you are asking for clarifying the difference in protocol and module, like saying if saml is used, there are two possible modules: shib and mellon ?12:59
samueldmqmarekd, and if openidc is used, that's all (no module to specify)12:59
*** aix has joined #openstack-keystone13:00
*** chlong has quit IRC13:00
rodrigodsmarekd, hi, just arrived13:00
marekdsamueldmq: i am basically asking why all those explanations must be posted here are they are not in the spec...13:00
iurygregorymarek you really have only three coices13:00
iurygregoryprotocol = OpenID Connect module is ignored13:01
iurygregoryprotocol = SAML and module = Shib13:01
iurygregoryprotocol = SAML and module = mellon13:01
*** richm has joined #openstack-keystone13:02
*** fhubik has joined #openstack-keystone13:04
*** chlong has joined #openstack-keystone13:04
*** tiny-hands has joined #openstack-keystone13:05
marekdiurygregory: it's really not clear there. anyway, i am not going back to this.13:06
marekdlets now make it work13:06
iurygregoryok13:06
iurygregoryIt may have been a failure in the spec as you say,  but the code will be ok. i don't think it's worth to send a change now for the spec.13:08
*** jecarey has joined #openstack-keystone13:08
*** fhubik has quit IRC13:09
*** raildo-afk is now known as raildo13:10
*** hrou has joined #openstack-keystone13:14
*** ajayaa has quit IRC13:18
marekdi didnt say it's a failure13:21
*** ayoung has joined #openstack-keystone13:24
*** ChanServ sets mode: +v ayoung13:24
iurygregoryi know you didn't say marek ^^13:24
openstackgerritLance Bragstad proposed openstack/keystone: Add federated auth for idp specific websso  https://review.openstack.org/21476613:28
*** jamielennox is now known as jamielennox|away13:30
*** henrynash has joined #openstack-keystone13:32
*** ChanServ sets mode: +v henrynash13:32
*** dims_ has joined #openstack-keystone13:35
*** zzzeek has joined #openstack-keystone13:36
*** davechen has joined #openstack-keystone13:38
*** dims has quit IRC13:39
openstackgerrithenry-nash proposed openstack/keystone: Support project hierarchies in data driver tests  https://review.openstack.org/15448513:42
*** geoffarnoldX is now known as geoffarnold13:42
*** dims_ has quit IRC13:44
*** dims has joined #openstack-keystone13:46
samueldmqdstanek, in my policy code, I was using oslo timeutils and doing : timeutils.utcnow() - datetime.datetime.min to get the timestamp, instead of timeutils.utcnow_ts()13:53
davechenhenrynash: ping?13:53
*** piyanai has quit IRC13:53
samueldmqdstanek, when I converted it back to a datetiem object, my policy was valid until the year 3xxx :-)13:53
henrynashdavechen: hi13:53
samueldmqdstanek, hehe13:53
davechenhenrynash: want to ask a quesiton :)13:54
davechenhttps://github.com/openstack/keystone/blob/master/keystone/common/controller.py#L613-L61413:54
henrynashdavechen: sure13:54
davechenpls take a look at this.13:54
openstackgerritLance Bragstad proposed openstack/keystone-specs: IDP specific websso  https://review.openstack.org/19933913:54
davechenwhy we need ignore the key not in the  query_dict?13:54
davechenhenrynash: I saw some comment from you, so I assume this feature is implemented by you. :)13:55
henrynashdavechen: (thinking…it was a long time ago!)13:55
davechenhenrynash: yeah,  i think so.13:55
samueldmqhenrynash, the ibm wildduck, making impressive code since .. a long time ago13:56
samueldmq:-)13:56
henrynashdavechen: well I think it was to ensure we only supported filtering on the items we state in the api spec13:56
davechenhenrynash: this will lead the issue when the query key is not in query_dict will return the entire entries in the DB.13:56
henrynashdavechen: yes, if you specify a query we don’t support, we ignore that filter13:57
davechenhenrynash: I am currently work on the bug related with this, https://bugs.launchpad.net/python-openstackclient/+bug/147983713:57
openstackLaunchpad bug 1479837 in Keystone "improper handling non existing identity providers " [Medium,In progress] - Assigned to Dave Chen (wei-d-chen)13:57
henrynashdavechen: I seem to remember us discussion that…i.e. do you show nothing or at least give the user something13:57
*** Nirupama has quit IRC13:58
davechenhenrynash: I propose an inital patch here (https://review.openstack.org/#/c/215041/) to change a little bit, but I am not sure if this is by design.13:58
*** browne has joined #openstack-keystone13:59
davechenhenrynash: so, the bug desc as what 'openstack identity provider show idontexist' returns is correct by this design?13:59
davechenand, when we specify a random key, then the API return all of the entries in the DB is correct?14:00
henrynashdavechen: so what does the show command on opentsack do…is it meant to show by name, or ID or what?14:00
*** narengan has joined #openstack-keystone14:00
henrynashshow by name, I assume14:01
davechenhenrynash: osc will try to use invoke get_** and then it will try again with list_** with the filter.14:02
henrynashdavechen: ok, I see14:02
davechenthe issue is name is not always an existing column14:02
henrynashdavechen: yep14:03
*** sigmavirus24_awa is now known as sigmavirus2414:03
davechenso, it will try to use list and filter by the filter, but when the 'name' is not a column, it will return all of them.14:04
*** aix has quit IRC14:04
henrynashdavechen: so neither the get or the list is going to work…so no sure what osc shoud do!14:04
davechenthis is not limited to this, it's existing in any APIs rather than identity provider.14:04
henrynashdavechen: well, it’s an entity that doesn’t have a name atribute (as far as an osc problem)14:05
henrynash(any entity…)14:05
*** lhinds has joined #openstack-keystone14:06
davechenhenrynash: If curl is used with an random key specified as the filter, then we will get all...14:06
henrynashdavechecn: origionally we specified that entities had to have certain attributes as mandatory - I think id and name were the two everythig had to have14:06
henrynashdavechen: yes, and that really was by design14:06
henrynashdavechen: not saying the design is necessarily right, mind you (!)14:07
*** afazekas has joined #openstack-keystone14:07
henrynashdavechen: we can obviosly change it…but we would have to deprecate the old fucntionality14:07
davechenhenrynash: can we relax it? not ignore the key not in the query_dict, then it will not return anything?14:08
henrynashdavechen: I don’t think we can just change it without a deprecation preiod14:08
*** jistr|mtg is now known as jistr14:08
davechenhenrynash: got you, I am thinking how to deprectate the old behaviour.14:08
henrynashdavechen: and I’d want wider discussion in terms of what the best approach would be14:08
*** yottatsa has joined #openstack-keystone14:09
*** fhubik has joined #openstack-keystone14:09
davechenhenrynash: maybe add a topic in our meeting. :)14:09
*** aix has joined #openstack-keystone14:09
henrynashdavechen: the actual problem here is that osc assumes that all entities conform to the origional standard (of everything having an id and a name) but IDP entities do not follow that standard14:09
davechenhenrynash: not every entites in keystone both has ID and name, it's okay if we do some DB migration, but if we use the CURL, what's return is still make user confuse.14:10
henrynashdavechen: it is a separate issue as to whether specifying a filter we do not support should return you nothing or teh filter shoudl be ignored14:10
*** geoffarnold has quit IRC14:10
henrynashdavechen: I really think they are two separate issues14:11
*** yottatsa has quit IRC14:11
henrynashdavechen: first class entities in keystone were MEANT to always have id and anme14:11
henrynashname14:11
*** ajayaa has joined #openstack-keystone14:11
*** yottatsa has joined #openstack-keystone14:11
*** yottatsa has quit IRC14:12
davechenhenrynash: yeah, osc seems okay since I cannot figure out a way to change it.14:12
henrynashdavechen: and just for clarity it is not true to say that if you specify a filter we do not suppor that we return everything…it is just that we ignore the filter (you may, for instance, be specifiy more than one filter)14:13
davechenhenrynash: yep, only that filter is ignored, and return everything only in case there is just this one filter.14:14
*** kiran-r has joined #openstack-keystone14:14
henrynashdavechen: agreed14:14
*** kiran-r has quit IRC14:15
davechenhenrynash: whats' the best approach for this issue per your understanding?14:15
*** doug-fish has quit IRC14:16
davechenhenrynash: db migraiton for all of these table which don't have name column?14:16
henrynashdavechen: even if we changed how we did filtering, osc would still be borken14:16
*** doug-fish has joined #openstack-keystone14:16
davechenhenrynash: or deprecate this old functionlity and return nothing.. or just keep it as it? :)14:16
henrynashdavechen: either osc needs be more flexible in what attribute the show coammnd uses...14:16
henrynashdavechen: or we have to name to teh IDP table14:16
henrynashdavechen: iI still think the filtering is a separate issue - magine we did that…what would osc do if you said show?14:17
henrynashdavechen: it still won’t get you enything14:17
davechenhenrynash: osc dont know what's attribute in the table from each endpoing.14:17
*** yottatsa has joined #openstack-keystone14:18
henrynashdavechen: so osc is assume ‘name’ exists….so either we must chaneg that, or we chaneg our idp entitty14:18
*** petertr7_away is now known as petertr714:19
henrynashdavechen: I woudl so first up is a discusion with dean/stevemar on what osc should and should not be assuming14:19
davechenhenrynash: but if we not ignore the key not in the query_dict, osc will works well.14:19
*** narengan has quit IRC14:20
*** narengan has joined #openstack-keystone14:20
henrynashdavechen: will it?  the list will retunr nothing14:20
*** lhinds_ has joined #openstack-keystone14:21
henrynashdavechen: am I missign something?14:21
davechenhenrynash: list call list_***, why return nothing? I didn't get it.14:21
henrynashdavechen: well won’t osc issue a GET /idp?name=xyz ?14:23
davechenhenrynash: this is for show not for list, I think.14:24
*** narengan has quit IRC14:24
henrynashdavechen: well won’t it try and list using the filter too?  ie.. GET /idps?name=xyz14:24
davechenhttps://github.com/openstack/python-openstackclient/blob/master/openstackclient/common/utils.py#L66-L8314:25
henrynashdavechen: soory need to go offline for a bit..wil be back14:25
*** henrynash has quit IRC14:25
davechenhenrynash: talk to you later, thanks for talking...14:26
*** lhinds_ has quit IRC14:29
*** vivekd has joined #openstack-keystone14:29
*** albertom-afk is now known as albertom14:31
*** davechen has left #openstack-keystone14:31
*** doug-fish has quit IRC14:33
*** doug-fish has joined #openstack-keystone14:33
*** doug-fish has quit IRC14:38
*** geoffarnold has joined #openstack-keystone14:40
*** samueldmq has quit IRC14:40
*** yottatsa has quit IRC14:44
*** yottatsa has joined #openstack-keystone14:47
*** csoukup has joined #openstack-keystone14:47
*** piyanai has joined #openstack-keystone14:49
*** raildo is now known as raildo-afk14:51
*** yottatsa has quit IRC14:51
openstackgerritRodrigo Duarte proposed openstack/keystone: Some fixes in the is_domain field creation  https://review.openstack.org/21516714:52
*** dsirrine has quit IRC14:52
rodrigodsbknudson, ayoung ^14:53
*** narengan has joined #openstack-keystone14:54
*** raildo-afk has quit IRC14:55
*** raildo-afk has joined #openstack-keystone14:56
*** raildo-afk has quit IRC14:57
*** raildo-afk has joined #openstack-keystone14:57
*** raildo-afk is now known as raildo14:58
*** yottatsa has joined #openstack-keystone14:59
*** yottatsa has quit IRC14:59
*** fhubik has quit IRC15:04
*** shoutm has joined #openstack-keystone15:05
*** alejandrito has joined #openstack-keystone15:08
*** yottatsa has joined #openstack-keystone15:09
ayoungrodrigods, bad commit title.  What did you fix?15:15
rodrigodsayoung, in the approved review bknudson made some comments and gave a -115:15
rodrigodshave any suggestions to make the title clearer?15:16
*** yottatsa has quit IRC15:16
openstackgerritMehdi Abaakouk (sileht) proposed openstack/keystonemiddleware: Allow to use oslo.config without global CONF  https://review.openstack.org/20896515:16
*** urulama has quit IRC15:18
*** yottatsa has joined #openstack-keystone15:18
*** urulama has joined #openstack-keystone15:19
*** shoutm has quit IRC15:20
*** stevemar has joined #openstack-keystone15:21
*** ChanServ sets mode: +v stevemar15:21
*** slberger has joined #openstack-keystone15:21
*** browne has quit IRC15:21
*** jistr is now known as jistr|mtg15:22
*** topol has joined #openstack-keystone15:28
*** ChanServ sets mode: +v topol15:28
*** samueldmq has joined #openstack-keystone15:28
openstackgerritMerged openstack/python-keystoneclient: Deprecate ServiceCatalog(region_name)  https://review.openstack.org/20580915:31
*** piyanai has quit IRC15:33
*** piyanai has joined #openstack-keystone15:34
*** r-daneel has joined #openstack-keystone15:36
*** arunkant_ has joined #openstack-keystone15:39
*** pnavarro has quit IRC15:39
slberger@lbragstad @dolphm  with this bug https://bugs.launchpad.net/keystone/+bug/1477600 is there potential to enter a loop of sorts for token requests, after we implemented fernet tokens people noticed a spike in keystone node usage and more than 50% of keystone token validation requests returned 40115:40
openstackLaunchpad bug 1477600 in Keystone kilo "Token Validation API returns 401 not 404 on invalid fernet token" [Medium,Fix committed] - Assigned to Dolph Mathews (dolph)15:40
*** geoffarnold has quit IRC15:41
dstanekvivekd: i gave the  interface review a quick look this morngin15:41
*** tjcocozz has joined #openstack-keystone15:42
*** petertr7 is now known as petertr7_away15:42
bretonslberger: afaik keystoneclient tries to authenticate only once after receiving 40115:44
vivekdthanks a lot for your review comments dstanek;15:44
vivekdhelps a lot for beginner like me.15:44
vivekdi'm working on addressing all your comments.15:45
vivekddstanek: will post an updated patch in a while.15:45
dstanekvivekd: i'm going to ask about this in the next meeting because this isn't what we discussed at the summit15:45
dstanekvivekd: i forgot to push up my hacky alternative...15:46
vivekdoh ok dstanek i'll participate in the next meeting15:46
openstackgerritDavid Stanek proposed openstack/keystone: WIP: WIPier and most WIPs - stable ifc design alternative  https://review.openstack.org/21520215:46
dstanekvivekd: ^15:46
dstanekvivekd: i'm just not sure how valuable doing only methods will be because we still won't have a stable interface15:47
*** urulama has quit IRC15:47
*** petertr7_away is now known as petertr715:48
*** belmoreira has joined #openstack-keystone15:49
vivekddstanek: i submitted my patch based on my understanding of the spec @ specs.openstack.org/openstack/keystone-specs/specs/liberty/stable-driver-interfaces.html so i thought i was going inline with what was discussed in the summit15:51
vivekddstanek: i'm happy to accept corrections from u if my 'doing only methods' approach is not inline with what was discussed in the summit15:51
*** doug-fish has joined #openstack-keystone15:52
dstanekvivekd: the discussion was more about how to document and enforce the inputs/outputs like i mentioned yesterday15:52
lbragstadslberger: I think dolphm wrote about the keystone node usage part15:52
*** hogepodge has quit IRC15:53
stevemarslberger: yay you found the keystone channel!15:54
*** btully has joined #openstack-keystone15:54
lbragstadslberger: http://dolphm.com/benchmarking-openstack-keystone-token-formats/15:54
stevemardolphm: lbragstad, this is slberger, he's trying to make fernet tokens work for us :)15:54
lbragstadslberger: o/15:54
lbragstadslberger: I have an item on my todo list to implement some sort of caching solution for rebuilding the catalog api15:55
lbragstads/catalog api/catalog from the token api/15:55
dolphmslberger: o/15:56
dolphmlbragstad: that should be a one liner!15:56
lbragstadslberger: on every token request (auth or validate) the catalog is reconstructed15:56
lbragstaddolphm: ++ yeah it should15:56
*** yottatsa has quit IRC15:57
*** AlexeyElagin has joined #openstack-keystone15:57
dstanekdolphm: really? i would have though our methods weren't that fine grained15:57
*** belmoreira has quit IRC15:58
*** yottatsa has joined #openstack-keystone15:58
*** btully has quit IRC16:00
*** narengan has quit IRC16:01
*** narengan has joined #openstack-keystone16:02
lbragstaddolphm: it looks like we MEMOIZE get_region, get_service, and get_endpoint already?16:03
dolphmdstanek: add a @MEMOIZE here https://github.com/openstack/keystone/blob/master/keystone/catalog/core.py#L27316:03
dolphmlbragstad: ^16:03
openstackgerritMerged openstack/keystone: Updating sample configuration file  https://review.openstack.org/21433916:03
dolphmlbragstad: i don't think we're caching authorization stuff, but the trick there is to refactor all calls into ones that don't have optional arguments16:05
openstackgerritLance Bragstad proposed openstack/keystone: Add caching to get_catalog  https://review.openstack.org/21521216:05
*** narengan has quit IRC16:06
lbragstaddolphm: optional arguments where? in get_service, get_endpoint, and get_region ?16:06
slbergerdolphm: with your benchmarks did you do any testing with cpu load?16:07
slbergerdolphm: when changing to fernet16:07
dstaneklbragstad: that's actually surprising. i would have thought that it depended on a context object16:08
lbragstadslberger: I'm not sure we saved metrics from CPU load, mostly based on response times16:08
*** piyanai has quit IRC16:09
dolphmslberger: not specifically, but it should certainly increase16:09
*** jistr|mtg is now known as jistr16:10
*** narengan has joined #openstack-keystone16:10
dolphmslberger: in each benchmark, CPU time was our bottleneck though16:10
*** henrynash has joined #openstack-keystone16:12
*** ChanServ sets mode: +v henrynash16:12
*** lsmola has quit IRC16:12
*** _cjones_ has joined #openstack-keystone16:13
*** piyanai has joined #openstack-keystone16:13
*** piyanai has quit IRC16:18
openstackgerritMarek Denis proposed openstack/keystone: Ensure ephemeral user's user_id is url-safe  https://review.openstack.org/21522116:18
*** yottatsa has quit IRC16:20
*** kiran-r has joined #openstack-keystone16:20
*** david8hu has quit IRC16:22
*** hogepodge has joined #openstack-keystone16:23
*** vivekd has quit IRC16:23
*** petertr7 is now known as petertr7_away16:25
*** mylu has joined #openstack-keystone16:27
*** geoffarnold has joined #openstack-keystone16:27
*** roxanaghe has joined #openstack-keystone16:29
*** piyanai has joined #openstack-keystone16:29
*** _kiran_ has joined #openstack-keystone16:30
*** kiran-r has quit IRC16:31
*** browne has joined #openstack-keystone16:33
*** doug-fish has quit IRC16:33
*** doug-fish has joined #openstack-keystone16:33
*** piyanai has quit IRC16:34
lbragstadmfisch: I'm curious if this will help your token validation problem https://review.openstack.org/#/c/215212/16:34
lbragstadproblem = token response time16:35
dolphmlbragstad: i wonder if morgan_2549 knows why that @MEMOIZE wasn't there already? ^16:35
lbragstaddolphm: maybe he thought it was too edge case if we were already caching the service, regions, and endpoints?16:35
morgan_2549Uh. Because it was probably just missed16:36
*** piyanai has joined #openstack-keystone16:37
*** geoffarnold has quit IRC16:37
openstackgerritSamuel de Medeiros Queiroz proposed openstack/keystone: Calculate validity and control caching  https://review.openstack.org/20969516:40
arunkant_gordc and stevemar: Barbican taxonomy change is merged in pycadf. I need a new version so I can use it in Barbican side. How do I request that?16:41
gordcarunkant_: i can create one for you.16:42
gordcit probably won't be released this week (we tend not to release late in week).16:43
*** narengan_ has joined #openstack-keystone16:44
*** jistr has quit IRC16:46
*** narengan has quit IRC16:47
*** vivekd has joined #openstack-keystone16:48
*** lhinds has quit IRC16:48
*** piyanai has quit IRC16:49
arunkant_gordc, thanks. Next week is fine.16:49
*** Kennan2 has quit IRC16:50
gordcarunkant_: https://review.openstack.org/#/c/215232/16:52
gordchmmm... spelled something wrong.16:52
*** btully has joined #openstack-keystone16:54
*** piyanai has joined #openstack-keystone16:54
arunkant_gordc, looks good now.16:56
*** narengan has joined #openstack-keystone16:56
*** piyanai has quit IRC16:56
*** tjcocozz_ has joined #openstack-keystone16:58
*** piyanai has joined #openstack-keystone16:59
*** narengan_ has quit IRC17:00
slbergerdoes keystone support oauth v2.0?17:00
*** tjcocozz has quit IRC17:02
*** afazekas has quit IRC17:02
*** Kennan has joined #openstack-keystone17:05
ayoungslberger, define support?17:06
openstackgerritOpenStack Proposal Bot proposed openstack/keystone: Updated from global requirements  https://review.openstack.org/21450917:06
*** henrynash has quit IRC17:06
openstackgerritOpenStack Proposal Bot proposed openstack/python-keystoneclient: Updated from global requirements  https://review.openstack.org/21389817:10
slbergerI guess is there any way to federate with an oauth v2.0 based backend17:10
*** ankita_wagh has joined #openstack-keystone17:13
*** piyanai has quit IRC17:13
*** piyanai has joined #openstack-keystone17:13
*** Kennan has quit IRC17:14
stevemarslberger: i think chris casey was working on trying to get that figured out17:17
stevemarit should, mod_auth_openidc should have oauthv2.0 support17:17
*** Kennan has joined #openstack-keystone17:20
*** csoukup has quit IRC17:21
*** aix has quit IRC17:22
*** geoffarnold has joined #openstack-keystone17:22
*** mylu has quit IRC17:22
*** mylu has joined #openstack-keystone17:23
*** narengan has quit IRC17:23
*** doug-fish has quit IRC17:23
*** jeffDeville has joined #openstack-keystone17:23
*** narengan has joined #openstack-keystone17:24
*** doug-fish has joined #openstack-keystone17:24
*** Kennan has quit IRC17:24
*** narengan has quit IRC17:28
*** david8hu has joined #openstack-keystone17:29
*** piyanai has quit IRC17:32
*** narengan has joined #openstack-keystone17:33
*** narengan has quit IRC17:34
*** narengan has joined #openstack-keystone17:34
*** doug-fish has quit IRC17:35
*** tsymanczyk has quit IRC17:35
*** doug-fish has joined #openstack-keystone17:35
*** geoffarnold has quit IRC17:38
*** narengan has quit IRC17:39
*** piyanai has joined #openstack-keystone17:39
*** Kennan has joined #openstack-keystone17:40
openstackgerritRodrigo Duarte proposed openstack/keystone: Some fixes in the is_domain field creation  https://review.openstack.org/21516717:40
*** lhcheng has joined #openstack-keystone17:41
*** ChanServ sets mode: +v lhcheng17:41
*** piyanai has quit IRC17:41
*** yottatsa has joined #openstack-keystone17:43
*** stevemar has quit IRC17:44
*** stevemar has joined #openstack-keystone17:46
*** ChanServ sets mode: +v stevemar17:46
lhchengstevemar: ping17:50
stevemarlhcheng: pong17:50
lhchengstevemar: https://bugs.launchpad.net/keystone/+bug/148277217:50
openstackLaunchpad bug 1482772 in Keystone "Region filtering for endpoints does not work" [Medium,In progress] - Assigned to Lin Hua Cheng (lin-hua-cheng)17:50
*** _kiran_ has quit IRC17:50
lhchengwe have the region filter in OSC and KSC, but it is passing region instead of region_id :(17:51
stevemaroh noes17:51
stevemarmy bad, re-open them17:51
lhchengdo we make the API accept both region and region_id?17:51
lhchengI am just about to add the filter, so I can still add it we want..17:52
lhcheng*if we want17:52
stevemarregion & region_id seems pointless17:53
*** petertr7_away is now known as petertr717:54
stevemari think the standard is just the ID17:54
lhchengokay, sounds good to me.17:54
*** tsymanczyk has joined #openstack-keystone17:55
*** tsymanczyk is now known as Guest9076717:55
lbragstaddolphm: any suggestions on how we get these kinds of tests to pass with caching enabled on get_catalog? https://github.com/openstack/keystone/blob/a42db6085bde6bbbe5ba35fa8823a7b1ef5b3742/keystone/tests/unit/test_backend_templated.py#L69-L7917:57
*** jasonsb has quit IRC17:57
*** csoukup has joined #openstack-keystone17:58
*** csoukup has quit IRC17:58
openstackgerritDavid Stanek proposed openstack/keystone: Remove all traces of olso incubator  https://review.openstack.org/19934317:58
dstanekhow do people feel about https://blueprints.launchpad.net/keystone/+spec/remove-oslo-incubator ?17:59
dstaneklbragstad: do those tests fail?17:59
*** gyee has joined #openstack-keystone18:00
*** ChanServ sets mode: +v gyee18:00
*** piyanai has joined #openstack-keystone18:00
*** gyee has quit IRC18:00
*** gyee has joined #openstack-keystone18:03
*** ChanServ sets mode: +v gyee18:03
lbragstaddstanek: yeah, just the one18:04
lbragstadhttp://logs.openstack.org/12/215212/1/check/gate-keystone-python27/f224531/testr_results.html.gz18:04
lbragstaddstanek: I'm sure the update to the templated catalog works, but since it's caching, it doesn't pickup that change18:05
dstanekwow, that's odd that it would fail18:05
lbragstaddstanek: I think it's caching on arguments18:05
*** mpmsimo has joined #openstack-keystone18:06
lbragstaddstanek: so, 'foo' and 'bar'18:06
*** mpmsimo has quit IRC18:06
openstackgerritVivek Dhayaal proposed openstack/keystone: Stable Keystone Driver Interfaces  https://review.openstack.org/20952418:07
dstaneklbragstad: that reminds me that i have to fix a conflict in my review that decouples templated from kvs18:07
*** jdennis has quit IRC18:08
*** bapalm has quit IRC18:11
*** urulama has joined #openstack-keystone18:13
*** bapalm has joined #openstack-keystone18:14
*** jdennis has joined #openstack-keystone18:17
*** vivekd has quit IRC18:21
*** ayoung has quit IRC18:23
*** samleon has joined #openstack-keystone18:23
*** yottatsa has quit IRC18:27
*** geoffarnold has joined #openstack-keystone18:28
*** kiran-r has joined #openstack-keystone18:33
*** henrynash has joined #openstack-keystone18:35
*** ChanServ sets mode: +v henrynash18:35
*** dave-mcc_ has joined #openstack-keystone18:36
*** jeffDeville has quit IRC18:38
*** dave-mccowan has quit IRC18:39
*** jeffDeville has joined #openstack-keystone18:39
*** piyanai has quit IRC18:46
*** kiran-r has quit IRC18:53
*** thiagop is now known as thiagop_afk18:54
*** afazekas has joined #openstack-keystone18:55
*** piyanai has joined #openstack-keystone18:56
*** geoffarnold has quit IRC19:00
*** narengan has joined #openstack-keystone19:01
*** afazekas has quit IRC19:03
*** piyanai has quit IRC19:06
openstackgerritRoxana Gherle proposed openstack/python-keystoneclient: Deprecate default admin endpoint type for v3 client  https://review.openstack.org/18520019:08
*** piyanai has joined #openstack-keystone19:10
*** Guest90767 has quit IRC19:15
*** btully has quit IRC19:16
*** jasonsb has joined #openstack-keystone19:18
*** urulama has quit IRC19:18
*** urulama has joined #openstack-keystone19:18
*** dims_ has joined #openstack-keystone19:26
*** lhcheng_ has joined #openstack-keystone19:27
*** Protux has quit IRC19:27
*** tsymanczyk has joined #openstack-keystone19:27
*** woodster_ has quit IRC19:27
*** dolphm has quit IRC19:27
*** jamielennox|away has quit IRC19:27
*** tsymanczyk is now known as Guest8849819:28
*** dims has quit IRC19:28
*** piyanai has quit IRC19:28
*** lhcheng has quit IRC19:28
*** r-daneel has quit IRC19:28
*** serverascode has quit IRC19:28
*** piyanai has joined #openstack-keystone19:29
*** dave-mccowan has joined #openstack-keystone19:30
gyeehenrynash, question for ya if you still awake19:30
henrynashgyee: yep!19:30
gyeeyay!19:30
gyeeso for per-domain config in sql19:30
gyeehow do we handle custom certificates for LDAP servers?19:30
gyeemeaning we have LDAP server certs that are not issued by a commercial CA19:31
gyeethose we still have to go through CMS right?19:31
*** woodster_ has joined #openstack-keystone19:31
*** btully has joined #openstack-keystone19:31
*** rm_work is now known as rm_work|away19:32
*** ankita_w_ has joined #openstack-keystone19:32
gyeeright now we specify the cert location in the config19:32
*** ayoung has joined #openstack-keystone19:32
*** ChanServ sets mode: +v ayoung19:32
*** dave-mcc_ has quit IRC19:33
henrynashso is cert config one ofteh ldap config options?19:33
openstackgerrithenry-nash proposed openstack/keystone: Enable listing of role assignments in a project hierarchy  https://review.openstack.org/20815219:33
gyeehenrynash, yes, it is an requirement for passing password to LDAP19:33
gyeemost be done over TLS19:34
henrynashgyee: let me see it is in our white list19:34
*** serverascode has joined #openstack-keystone19:34
gyeehenrynash, how does out LDAP code handle that though? we have to write it out to a file first I would think19:35
*** ankita_wagh has quit IRC19:35
*** Protux has joined #openstack-keystone19:35
gyeelet me see if it takes a file descriptor instead19:36
*** dolphm has joined #openstack-keystone19:36
henrynashgyee: so tls_cretdiir and tls_certfile are certainly supported by out config on a domain by domain basis19:36
gyeeright19:36
gyeebut its still a combination of SQL and CMS19:37
gyeeI was trying to see if we can avoid CMS19:37
*** narengan_ has joined #openstack-keystone19:37
henrynashguee: yep, we don’t offer storing the actual cert in teh DB19:37
henrynashgyee: isn’t that barbican?19:37
gyeehenrynash, blueprint time :)19:37
gyeehenrynash, yes, ideally we want to be able to use URLs to pull the certs19:38
gyeefile://...19:38
gyeehttps://...19:38
gyeeinstead of dir path19:38
*** richm has quit IRC19:38
henrynashgyee: yep…or is barbican the “openstack cms” for this?19:38
gyeeif we can load a cert from a URL, it can be barbican or any API server19:39
henrynashgyee: true19:39
*** darrenc has quit IRC19:39
gyeehenrynash, you want to start a blueprint or do you want me to start one?19:40
henrynashsamueldmq, rodigods: see https://review.openstack.org/208152 for some heavy weight hierarchy testing!19:40
gyeewe'll have to enhance the LDAP driver code as well19:40
samueldmqhenrynash, nice!19:40
*** narengan has quit IRC19:40
henrynashgyee: sounds liek you would be a good “customer” to write the bp…I’d be happy to take onteh work19:40
*** darrenc has joined #openstack-keystone19:40
samueldmqhenrynash, love those ascii art there :-)19:41
gyeehenrynash, lemme write one then19:41
*** Ephur_ has joined #openstack-keystone19:41
henrynashsamueldmq: :-)19:41
* gyee puts on his operator hat19:41
henrynashsamueldmq: space bar…tap, tap , tap ,tap19:41
samueldmqhenrynash, heheh very nice :)19:42
samueldmqhenrynash, I am going to take a better look later (doing somehting right now), but definitely added here as high-priority todo19:42
samueldmqhenrynash, I own you those reviews :)19:42
*** Ephur has quit IRC19:43
gyeesamueldmq, ascii art reminds me of the NNTP days :)19:43
*** Ephur has joined #openstack-keystone19:43
openstackgerrithenry-nash proposed openstack/keystone: Enable listing of role assignments in a project hierarchy  https://review.openstack.org/20815219:43
*** r-daneel has joined #openstack-keystone19:44
rodrigodshenrynash, haha nice!19:44
samueldmqhenrynash, :) (had to google NNTP tbh haha)19:45
*** slberger has left #openstack-keystone19:45
gyeein the old days, we got our 'ascii arts' via nntp19:46
*** Ephur_ has quit IRC19:46
*** alejandrito has quit IRC19:47
*** jamielennox|away has joined #openstack-keystone19:47
*** jamielennox|away is now known as jamielennox19:47
*** ChanServ sets mode: +v jamielennox19:47
openstackgerrithenry-nash proposed openstack/keystone: Enable listing of role assignments in a project hierarchy  https://review.openstack.org/20815219:48
*** alejandrito has joined #openstack-keystone19:49
*** dsirrine has joined #openstack-keystone19:51
*** richm has joined #openstack-keystone19:54
*** doug-fis_ has joined #openstack-keystone19:56
*** topol has quit IRC19:57
*** topol has joined #openstack-keystone19:57
*** ChanServ sets mode: +v topol19:57
*** doug-fish has quit IRC19:59
*** doug-fish has joined #openstack-keystone20:00
*** doug-fis_ has quit IRC20:01
*** alejandrito has quit IRC20:09
*** browne has quit IRC20:11
*** pnavarro has joined #openstack-keystone20:11
*** browne has joined #openstack-keystone20:12
*** alejandrito has joined #openstack-keystone20:13
openstackgerrithenry-nash proposed openstack/keystone: Remove manager-driver assignment metadata construct  https://review.openstack.org/14899520:15
samueldmqgyee, yes, 'N      T      T      P' ftw20:15
samueldmq:)20:15
openstackgerritDavid Stanek proposed openstack/keystone: Remove all traces of oslo incubator  https://review.openstack.org/19934320:16
*** ajayaa has quit IRC20:19
openstackgerrithenry-nash proposed openstack/keystone: Enable listing of role assignments in a project hierarchy  https://review.openstack.org/20815220:22
openstackgerritSamuel de Medeiros Queiroz proposed openstack/keystone: Calculate validity and control caching  https://review.openstack.org/20969520:26
*** urulama has quit IRC20:27
*** urulama has joined #openstack-keystone20:27
*** e0ne has joined #openstack-keystone20:31
dstanekhenrynash: you still around?20:31
henrynashdstanek: yep….on phone..will be free in bit20:35
dstanekhenrynash: np... i was just wondering if there is anything we can/should do with https://review.openstack.org/#/c/15353520:36
*** boris-42 has joined #openstack-keystone20:41
*** ankita_wagh has joined #openstack-keystone20:44
henrynashdstanek: so I don’t think there is an real big deal on this one….it was just that I was working with a group experimenting with alternate assignment engines…and they could bolt it in for expermientation…with no changes…except these couple of fixes20:45
henrynashdstanek: I don’t thikn they are pressing anything now, so if we like this fine, if not, killing it is fine too!20:45
*** geoffarnold has joined #openstack-keystone20:47
*** ankita_w_ has quit IRC20:47
dstanekhenrynash: i don't mind killing it! only because i don't want people coming in and thinking that we like drivers that don't implement the full driver api20:48
henrynashdstanek: consider it dead, sir! (like a parrot)20:48
*** jeffDeville has quit IRC20:49
dstanekhenrynash: is it dead or just resting :-)20:50
henrynashdstanek: now, that, is the question!20:51
*** geoffarnold has quit IRC20:52
*** e0ne has quit IRC20:58
*** tjcocozz__ has joined #openstack-keystone20:59
openstackgerritSamuel de Medeiros Queiroz proposed openstack/keystone: Calculate validity and control caching  https://review.openstack.org/20969520:59
*** tomas_c has joined #openstack-keystone21:00
*** geoffarnold has joined #openstack-keystone21:00
*** lhcheng has joined #openstack-keystone21:01
*** ChanServ sets mode: +v lhcheng21:01
*** lhcheng_ has quit IRC21:01
*** tjcocozz_ has quit IRC21:02
tomas_cHey, can someone please explain me this error? http://pastebin.com/sAaAF16w It happens on ./stack in Devstack.21:04
lbragstadtomas_c: it looks like the user creation failed (in a previous command) and the value was never persisted to a variable/21:05
lbragstadtomas_c: you might find some more information around the ERROR: openstack Internal Server Error (HTTP 500) on the keystone screen (if you're running devstack)21:05
*** urulama has quit IRC21:08
tomas_clbragstad: thank you for responding. i'm not sure if you mean by keystone screen a web-interface, because unfortunatelly i can't access it. Do you think it would be sufficient to back-out commits at keystone project?21:09
*** urulama has joined #openstack-keystone21:09
lbragstadtomas_c: oh sorry21:09
lbragstadtomas_c: what I means was the keystone-all process that is started in a screen session21:09
lbragstadmeant*21:10
lbragstadtomas_c: you should be able to attach to that screen session and find the window that is running the keystone process.21:10
lbragstadtomas_c: that might give you some more information around the 500s21:10
openstackgerritDavanum Srinivas (dims) proposed openstack/keystone: No more openstack.common  https://review.openstack.org/21533721:11
*** rm_work|away is now known as rm_work21:13
tomas_clbragstad: sounds good, i will try that. i'm wondering if you think that a commit merged in keystone should cause this?21:14
tomas_c* could21:15
lbragstadtomas_c: I would hope not, since it would hopefully be caught by the gate21:15
lbragstadand the gate uses devstack to stand up the environments that we use for testing21:15
*** urulama has quit IRC21:17
tomas_clbragstad: hm, i haven't known about that. at this time it looks to me more like there might be stgh wrong in my local.conf21:17
*** tiny-hands has left #openstack-keystone21:21
*** ankita_w_ has joined #openstack-keystone21:24
lbragstaddo we have triggers in keystone somewhere that allow us to invalidate a cache if something specific happens?21:25
*** Guest88498 is now known as tsymanczyk21:25
lbragstadlike "i'm going to add  a new endpoint, so call this trigger that invalidates the endpoint cache?"21:25
*** ayoung has quit IRC21:25
*** ankita_wagh has quit IRC21:27
*** thiagop_afk has quit IRC21:28
*** stevemar has quit IRC21:28
dstaneklbragstad: not really triggers21:29
dstaneklbragstad: http://git.openstack.org/cgit/openstack/keystone/tree/keystone/resource/core.py#n22421:30
lbragstaddstanek: sorry, triggers probably isn't the right word, do we have a way of invoking an invalid cache if we change stuff?21:30
*** dguerri` is now known as dguerri21:30
lbragstadah... interesting21:30
lbragstadI was wondering what those were!21:31
lbragstaddstanek: and .invalidate() is a cache thing?21:31
*** dguerri is now known as dguerri`21:31
dstaneklbragstad: i believe that dogpile adds that on there21:31
*** pnavarro has quit IRC21:31
dstaneklbragstad: it talks about it here: http://dogpilecache.readthedocs.org/en/latest/api.html21:32
lbragstaddstanek: nice, thank you -- http://dogpilecache.readthedocs.org/en/latest/api.html#dogpile.cache.region.CacheRegion.invalidate21:34
openstackgerritMerged openstack/keystone: Test v2 tokens being deleted by v3  https://review.openstack.org/20173821:34
lbragstaddstanek: makes sense, so the test that is failing for me isn't because of not invalidating the cache because it looks like we already do that, which is a good thing21:36
lbragstadhttps://github.com/openstack/keystone/blob/master/keystone/catalog/core.py#L25021:37
dstaneklbragstad: is the cache actually caching durign the tests?21:37
lbragstaddstanek: it must be21:37
lbragstaddstanek: i could skip that test if caching it enable d21:37
lbragstaddstanek: because I think that is related to the templated backend21:37
*** sigmavirus24 is now known as sigmavirus24_awa21:38
dstaneklbragstad: hmmm... i thought during test we used a cache backend that didn't actually cache21:41
lbragstaddstanek: I'm not sure, but the failing assertion of the tests looks like its caching21:41
dstanekwell that's a bummer :-(21:41
lbragstaddstanek: http://cdn.pasteraw.com/3asqlld0xqo3rat38t9g5w5qdifh6zn21:42
*** doug-fish has quit IRC21:42
*** doug-fish has joined #openstack-keystone21:42
dstaneklbragstad: i'll have to look into this more when i get home .. .going to be leaving for the Browns game in a bit21:44
*** edmondsw has quit IRC21:44
dstaneklbragstad: i thought http://git.openstack.org/cgit/openstack/keystone/tree/keystone/tests/unit/ksfixtures/cache.py was to enable caching only for certain tests.21:44
lbragstaddstanek: no worries, it's not a really big deal, I was more or less curious21:44
dstanekmaybe that got turned on somehow21:44
*** tjcocozz__ has quit IRC21:46
lbragstadpossibly21:46
*** doug-fish has quit IRC21:46
*** alejandrito has quit IRC21:49
*** bapalm has quit IRC21:49
*** bapalm has joined #openstack-keystone21:52
*** alejandrito has joined #openstack-keystone21:53
*** sigmavirus24_awa is now known as sigmavirus2421:54
*** henrynash has quit IRC21:55
openstackgerritLance Bragstad proposed openstack/keystone: Add caching to get_catalog  https://review.openstack.org/21521221:55
*** topol has quit IRC21:56
*** piyanai has quit IRC21:56
*** bapalm has quit IRC21:57
*** geoffarnold has quit IRC22:01
*** geoffarnold has joined #openstack-keystone22:02
*** bapalm has joined #openstack-keystone22:03
*** petertr7 is now known as petertr7_away22:06
*** bapalm has quit IRC22:08
*** doug-fish has joined #openstack-keystone22:14
*** bapalm has joined #openstack-keystone22:14
*** narengan_ has quit IRC22:16
*** doug-fish has quit IRC22:18
*** bapalm has quit IRC22:19
*** ayoung has joined #openstack-keystone22:19
*** ChanServ sets mode: +v ayoung22:19
*** chlong has quit IRC22:25
*** jecarey has quit IRC22:28
*** bapalm has joined #openstack-keystone22:29
*** bapalm has quit IRC22:34
*** bapalm has joined #openstack-keystone22:35
*** flwang1 has quit IRC22:39
*** tomas_c has quit IRC22:39
roxanaghebknudson: for deprecating the admin default endpoint type is this the correct place you suggested to submit the patch? https://review.openstack.org/#/c/215261/22:40
*** bapalm has quit IRC22:43
*** bapalm has joined #openstack-keystone22:43
*** bapalm has quit IRC22:48
*** btully has quit IRC22:49
*** bapalm has joined #openstack-keystone22:51
*** hrou has quit IRC23:00
*** rm_work is now known as rm_work|away23:01
*** jasonsb has quit IRC23:02
*** jasonsb has joined #openstack-keystone23:03
*** ayoung has quit IRC23:03
*** alejandrito has quit IRC23:06
*** jasonsb has quit IRC23:07
*** tiny-hands has joined #openstack-keystone23:07
*** arunkant_ has quit IRC23:13
*** sigmavirus24 is now known as sigmavirus24_awa23:18
*** dramakri has joined #openstack-keystone23:19
*** samuel-dmq has joined #openstack-keystone23:24
*** zzzeek has quit IRC23:25
samuel-dmqgyee, you around ?23:26
*** r-daneel has quit IRC23:27
samuel-dmqdstanek ?23:27
openstackgerritTerry Howe proposed openstack/keystoneauth: Keep a consistent logger name for keystoneauth  https://review.openstack.org/21260223:27
samuel-dmqdstanek, I am tending to think a small inconsistency when policies get updated could be aceptable23:28
*** flwang1 has joined #openstack-keystone23:29
samuel-dmqdstanek, just need to figure out if this would be really acceptable, wanted to check gyee and morgan's view on that23:29
samuel-dmqdstanek, if deployers think we should reduce this inconsistency to about 0 (even when updates occur), we know how to do it already23:30
samuel-dmqdstanek, maybe it's a matter of starting simpler, if simpler is acceptable23:30
*** roxanaghe has quit IRC23:36
*** ayoung has joined #openstack-keystone23:37
*** ChanServ sets mode: +v ayoung23:37
openstackgerritLin Hua Cheng proposed openstack/keystone: Add region_id filter for List Endpoints API  https://review.openstack.org/21537823:43
*** geoffarnold has quit IRC23:43
*** samuel-dmq has quit IRC23:51
*** hrou has joined #openstack-keystone23:55
*** topol has joined #openstack-keystone23:56
*** ChanServ sets mode: +v topol23:56

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