Thursday, 2017-01-12

*** spzala has joined #openstack-keystone00:02
openstackgerritGage Hugo proposed openstack/keystone: Allow user to change own expired password  https://review.openstack.org/40402200:20
*** ayoung has joined #openstack-keystone00:25
*** ChanServ sets mode: +v ayoung00:25
*** asettle has joined #openstack-keystone00:35
*** asettle has quit IRC00:38
*** asettle has joined #openstack-keystone00:38
*** asettle has quit IRC00:44
*** jamielennox is now known as jamielennox|away00:50
*** stewie925 has quit IRC00:53
*** phalmos has joined #openstack-keystone00:53
*** thorst_ has joined #openstack-keystone00:53
*** thorst_ has quit IRC00:58
*** hoangcx has joined #openstack-keystone01:00
*** liujiong has joined #openstack-keystone01:12
*** jose-phillips has quit IRC01:15
*** thorst_ has joined #openstack-keystone01:17
*** jamielennox|away is now known as jamielennox01:21
*** spzala has quit IRC01:27
*** tqtran has quit IRC01:30
*** browne has quit IRC01:48
*** adrian_otto has quit IRC01:53
*** phalmos has quit IRC02:07
*** markvoelker has quit IRC02:38
*** thorst_ has quit IRC02:38
*** kjdeepthi has joined #openstack-keystone02:39
*** thorst_ has joined #openstack-keystone02:39
*** markvoelker_ has joined #openstack-keystone02:41
*** thorst_ has quit IRC02:43
*** spzala has joined #openstack-keystone02:51
*** markvoelker_ has quit IRC02:56
*** hoangcx_ has joined #openstack-keystone03:10
*** hoangcx has quit IRC03:10
*** hoangcx_ is now known as hoangcx03:12
*** adrian_otto has joined #openstack-keystone03:15
*** adrian_otto has quit IRC03:15
*** adrian_otto has joined #openstack-keystone03:16
*** adrian_otto has quit IRC03:22
*** spzala has quit IRC03:26
*** kjdeepthi has quit IRC03:28
*** links has joined #openstack-keystone03:39
*** markvoelker has joined #openstack-keystone03:57
stevemarcrinkle_: guten morgen04:00
stevemarcrinkle_: can you review https://review.openstack.org/#/c/390948/ again when you get a minute, i think it's finally oK?04:01
stevemarcrinkle_: i thought i knew our LDAP code, turns out i didn't04:01
openstackgerritSteve Martinelli proposed openstack/keystone: Add anonymous bind to get_connection method  https://review.openstack.org/40756104:02
*** spzala has joined #openstack-keystone04:08
*** nicolasbock has quit IRC04:11
*** spzala has quit IRC04:12
stevemargerrit is slow tonight04:21
*** adrian_otto has joined #openstack-keystone04:28
*** adrian_otto has quit IRC04:28
*** adriant has quit IRC04:47
*** dikonoor has joined #openstack-keystone04:48
*** voelzmo has joined #openstack-keystone05:07
*** voelzmo has quit IRC05:11
*** sc68cal has quit IRC05:18
*** sc68cal has joined #openstack-keystone05:20
*** edtubill has joined #openstack-keystone05:30
*** thorst_ has joined #openstack-keystone05:40
*** thorst_ has quit IRC05:44
openstackgerritSteve Martinelli proposed openstack/keystone: Fix keystone-manage mapping_engine tester  https://review.openstack.org/41816505:50
*** diazjf has joined #openstack-keystone06:02
*** itisha has quit IRC06:02
*** andrewbogott has quit IRC06:04
*** andrewbogott has joined #openstack-keystone06:04
*** diazjf has quit IRC06:05
*** thorst_ has joined #openstack-keystone06:40
*** lucas__ has joined #openstack-keystone06:42
*** richm has quit IRC06:43
*** thorst_ has quit IRC06:45
*** hoangcx_ has joined #openstack-keystone06:57
*** hoangcx has quit IRC07:00
*** lucas__ has quit IRC07:08
*** tesseract has joined #openstack-keystone07:14
*** hoangcx has joined #openstack-keystone07:49
*** hoangcx_ has quit IRC07:51
*** chrome0 has quit IRC08:05
*** chrome0 has joined #openstack-keystone08:05
*** haplo37_ has quit IRC08:05
*** guoshan has joined #openstack-keystone08:08
*** haplo37_ has joined #openstack-keystone08:09
*** aloga has quit IRC08:12
*** aloga has joined #openstack-keystone08:15
*** thorst_ has joined #openstack-keystone08:41
*** thorst_ has quit IRC08:46
*** zzzeek has quit IRC09:00
*** zzzeek has joined #openstack-keystone09:00
*** david-lyle has quit IRC09:01
*** mvk has quit IRC09:18
*** david-lyle has joined #openstack-keystone09:24
*** aloga has quit IRC09:37
*** aloga has joined #openstack-keystone09:38
*** asettle has joined #openstack-keystone09:41
*** aloga has quit IRC09:50
*** aloga has joined #openstack-keystone09:51
*** mvk has joined #openstack-keystone09:51
*** liujiong has quit IRC10:13
*** AlexeyAbashkin has joined #openstack-keystone10:17
*** thiagolib has joined #openstack-keystone10:22
*** hoangcx has quit IRC10:27
*** mvk has quit IRC10:36
*** thorst_ has joined #openstack-keystone10:42
*** thorst_ has quit IRC10:47
*** mvk has joined #openstack-keystone10:48
*** stingaci has joined #openstack-keystone10:50
*** guoshan has quit IRC10:59
*** richm has joined #openstack-keystone11:13
openstackgerritXuepeng Ji proposed openstack/keystonemiddleware: Removes unnecessary utf-8 coding  https://review.openstack.org/41942011:18
*** nicolasbock has joined #openstack-keystone11:34
*** crinkle_ is now known as crinkle11:38
crinklestevemar: lgtm!11:38
*** voelzmo has joined #openstack-keystone11:41
*** voelzmo has quit IRC11:47
*** voelzmo has joined #openstack-keystone11:48
bretoncrinkle: your reviews of that patch are super appreciated11:50
*** voelzmo has quit IRC11:52
*** masber has quit IRC11:56
crinklebreton: :)11:58
*** guoshan has joined #openstack-keystone11:59
*** voelzmo has joined #openstack-keystone12:00
*** guoshan has quit IRC12:04
*** voelzmo has quit IRC12:05
*** links has quit IRC12:13
*** dave-mccowan has joined #openstack-keystone12:13
*** sheel has joined #openstack-keystone12:19
*** voelzmo has joined #openstack-keystone12:22
*** voelzmo has quit IRC12:26
*** thorst_ has joined #openstack-keystone12:46
openstackgerritBoris Bobrov proposed openstack/keystone: Drop type in filters  https://review.openstack.org/41945112:49
*** stingaci has quit IRC12:50
openstackgerritMerged openstack/keystone: Set connection timeout for LDAP configuration  https://review.openstack.org/39094812:52
stevemarcrinkle: ty!12:53
*** voelzmo has joined #openstack-keystone12:54
*** voelzmo has quit IRC12:58
*** dgonzalez has quit IRC12:58
*** dgonzalez has joined #openstack-keystone13:05
*** stingaci has joined #openstack-keystone13:10
*** edmondsw has joined #openstack-keystone13:10
*** stingaci has quit IRC13:14
*** links has joined #openstack-keystone13:25
*** jaugustine_ has quit IRC13:41
*** voelzmo has joined #openstack-keystone13:41
*** stingaci has joined #openstack-keystone13:42
*** voelzmo has quit IRC13:46
dstanekgood morning all13:52
*** lamt has joined #openstack-keystone13:58
*** AlexeyAbashkin has quit IRC13:59
stevemardstanek: howdy partner13:59
stevemardstanek: i just realized i haven't seen you and lbragstad in many months!14:00
stevemari miss you guys <314:00
*** guoshan has joined #openstack-keystone14:01
dstanekstevemar: i miss you too steve14:01
*** AlexeyAbashkin has joined #openstack-keystone14:02
openstackgerritRodrigo Duarte proposed openstack/keystone: Cascade delete federated_user fk  https://review.openstack.org/41590614:03
lbragstadstevemar o/14:05
*** guoshan has quit IRC14:05
lbragstadstevemar here's to ATL!14:05
stevemarlbragstad: here here!14:06
*** jaugustine has joined #openstack-keystone14:08
*** jaugustine has quit IRC14:09
*** jaugustine has joined #openstack-keystone14:10
*** jaugustine has quit IRC14:14
*** nishaYadav has joined #openstack-keystone14:18
*** lucas__ has joined #openstack-keystone14:18
nishaYadavo/14:18
*** jperry has joined #openstack-keystone14:29
*** phalmos has joined #openstack-keystone14:29
*** phalmos has quit IRC14:33
*** phalmos has joined #openstack-keystone14:34
*** lucas__ has quit IRC14:37
*** jaugustine has joined #openstack-keystone14:39
*** AlexeyAbashkin has quit IRC14:45
*** edtubill has quit IRC14:46
*** dikonoor has quit IRC14:48
*** stingaci has quit IRC14:54
*** stingaci has joined #openstack-keystone14:57
openstackgerritMerged openstack/keystone: Add anonymous bind to get_connection method  https://review.openstack.org/40756115:01
*** lamt has quit IRC15:02
*** chlong has joined #openstack-keystone15:03
*** lamt has joined #openstack-keystone15:03
*** sheel has quit IRC15:07
*** edtubill has joined #openstack-keystone15:09
*** adrian_otto has joined #openstack-keystone15:09
*** jdennis1 has joined #openstack-keystone15:10
*** jdennis has quit IRC15:12
*** jaugustine has quit IRC15:12
openstackgerritLance Bragstad proposed openstack/keystone: Implement shadow mapping  https://review.openstack.org/41589515:13
*** jaugustine has joined #openstack-keystone15:13
lbragstadsamueldmq rodrigods rderose dstanek thanks for the reviews ^15:14
lbragstadlatest comments have been addressed15:14
dstaneklbragstad: nice15:14
lbragstaddstanek did you have an idea of what/how you wanted to isolate that shadow_mapping method (it's not named that anymore, but...)15:15
lbragstaddstanek the convention for doing that in the plugins it to create internal methods for it15:15
lbragstadI have no idea why that is the convention though - the pattern *seems* backwards, but I lack the tribal knowledge on why they are that way15:15
*** jperry has quit IRC15:16
*** jperry has joined #openstack-keystone15:16
*** nishaYadav has quit IRC15:16
stevemarfyi lbragstad + samueldmq + dstanek + others, i queued up a bunch of stable releases for keystone server + libs: https://review.openstack.org/#/q/project:openstack/releases+owner:stevemar+status:open15:16
stevemarturns out there were no backported fixes for middleware :)15:16
lbragstadstevemar sweet - I can review those today15:17
*** jaugustine has quit IRC15:17
lbragstadstevemar ^ those are the proposed shas for the releases, they aren't waiting on stable branch changes to merge are they?15:18
stevemarlbragstad: meh, no biggie15:18
stevemarlbragstad: correct15:18
lbragstadstevemar so they are good to go whenever?15:18
stevemarlbragstad: pending the release teams approval, they may think i broke the rules :O15:18
lbragstadstevemar ?15:19
stevemarif i backported something that goes against backport policy15:19
dstanekstevemar: did you?15:21
stevemardstanek: don't think so :)15:22
*** jaosorior has joined #openstack-keystone15:22
*** jaugustine has joined #openstack-keystone15:25
*** adrian_otto has quit IRC15:27
*** adrian_otto1 has joined #openstack-keystone15:27
*** markvoelker has quit IRC15:27
*** AlexeyAbashkin has joined #openstack-keystone15:28
*** adrian_otto1 has quit IRC15:30
*** chris_hultin|AWA is now known as chris_hultin15:38
*** nishaYadav has joined #openstack-keystone15:41
*** stingaci has quit IRC15:42
*** jperry has quit IRC15:42
*** stingaci has joined #openstack-keystone15:42
*** ravelar has joined #openstack-keystone15:43
*** mvk has quit IRC15:44
*** ayoung has quit IRC15:45
*** lucas__ has joined #openstack-keystone15:46
*** lucas__ has quit IRC15:48
*** adrian_otto has joined #openstack-keystone15:50
*** lucas__ has joined #openstack-keystone15:50
openstackgerritRodrigo Duarte proposed openstack/keystone: Cascade delete federated_user fk  https://review.openstack.org/41590615:50
*** ayoung has joined #openstack-keystone15:50
*** ChanServ sets mode: +v ayoung15:50
*** adrian_otto has quit IRC15:51
rderosestevemar: around?15:52
*** adrian_otto has joined #openstack-keystone15:52
*** spzala has joined #openstack-keystone15:52
*** markvoelker has joined #openstack-keystone15:52
*** pcaruana has joined #openstack-keystone15:58
*** jaugustine has quit IRC16:09
*** jaugustine has joined #openstack-keystone16:09
*** jaugustine has quit IRC16:14
knikollao/16:18
lbragstaddstanek about your comment on shadow mapping and the exceptions not consistently have punctuation (or a lack of convention), I feel like that's something we should open a LHF bug for16:21
lbragstaddstanek since it does (ever to slightly) affect useres16:21
lbragstadusers*16:21
*** AlexeyAbashkin has quit IRC16:27
*** adrian_otto1 has joined #openstack-keystone16:32
*** adrian_otto has quit IRC16:32
dstaneklbragstad: sure. it was just commentary, I wasn't expecting a fix for it just yet.16:33
*** raildo has joined #openstack-keystone16:37
lbragstaddstanek sweet - done https://bugs.launchpad.net/keystone/+bug/165602616:37
openstackLaunchpad bug 1656026 in OpenStack Identity (keystone) "Exception don't follow a punctuation convention" [Undecided,New]16:37
*** links has quit IRC16:37
*** jaugustine has joined #openstack-keystone16:37
*** nishaYadav has quit IRC16:43
dstaneklbragstad: thanks16:51
*** chlong has quit IRC16:52
lbragstaddstanek that'd be a good one for someone to pick up this Friday ^16:53
*** jperry has joined #openstack-keystone16:55
*** adu has quit IRC16:59
*** spzala has quit IRC17:01
*** jaugustine has quit IRC17:03
*** jaugustine has joined #openstack-keystone17:04
*** ravelar1 has joined #openstack-keystone17:06
bknudsonare exception messages part of the public API contract?17:07
*** ravelar has quit IRC17:07
lbragstadbknudson i was just thinking about that17:07
*** xek has quit IRC17:07
*** xek has joined #openstack-keystone17:08
lbragstadbknudson that's a good question - I would say the response codes are, but I don't know if I would consider the actual message text to be17:08
*** jaugustine has quit IRC17:08
bknudsonresponse codes are useless since different errors / problems can lead to the same response code.17:09
lbragstadI suppose we could implement a convention across exception messages saying it's an improvement to existing information17:09
* lbragstad shrug17:09
*** medberry has quit IRC17:10
*** sheel has joined #openstack-keystone17:10
*** aloga has quit IRC17:12
*** aloga has joined #openstack-keystone17:12
*** browne has joined #openstack-keystone17:13
*** diazjf has joined #openstack-keystone17:13
*** med_ has joined #openstack-keystone17:14
stevemarbknudson: lbragstad the type of exception and the number should be part of the public API, but the message -- that probably goes too far17:14
*** med_ is now known as Guest8778317:14
bknudsontype?17:14
*** edtubill has quit IRC17:15
*** adu has joined #openstack-keystone17:16
dstaneklunch time!17:20
stevemarbknudson: UserNotFound vs GroupNotFound vs NotFound ?17:21
*** portdirect is now known as portdirect_eatin17:21
rodrigodsstevemar, how the exception is exposed via the API?17:21
openstackgerritRodrigo Duarte proposed openstack/keystone: Implement shadow mapping  https://review.openstack.org/41589517:22
rodrigodslbragstad, ^ fixed pep817:22
stevemarrodrigods: yeah, sorta17:22
*** chlong has joined #openstack-keystone17:26
*** stingaci has quit IRC17:27
*** ravelar1 has quit IRC17:43
*** lucas__ has quit IRC17:49
*** lucas__ has joined #openstack-keystone17:50
*** lucas__ has quit IRC17:54
*** lucas__ has joined #openstack-keystone17:55
*** phalmos_ has joined #openstack-keystone17:57
*** jose-phillips has joined #openstack-keystone17:58
*** diazjf has quit IRC17:58
*** AlexeyAbashkin has joined #openstack-keystone17:59
*** lucas__ has quit IRC18:00
*** phalmos has quit IRC18:00
*** jaugustine has joined #openstack-keystone18:01
lbragstadstevemar so do you think we can change the exception messages or no?18:04
*** mvk has joined #openstack-keystone18:09
*** ravelar1 has joined #openstack-keystone18:10
rodrigodslbragstad, IMHO, we can ^18:14
*** arunkant has quit IRC18:15
*** jaugustine has quit IRC18:18
*** jaugustine has joined #openstack-keystone18:19
*** chlong has quit IRC18:19
*** jaugustine has quit IRC18:23
morganlbragstad: which excreption message?18:29
morganit is fine to update exception messages unless we explicitly make them a contract18:30
ayounglbragstad, text is internationalized. It may vary depedning on the local value of the caller18:30
morganas long as we don't violate the status code api contract guidelines18:30
ayoungSamYaple, can you give me the link to that docker container again?18:31
morganif we are looking to standardize some part of the exception string we should look at using an error-code (not http status)18:31
morganand look to encode that number separate from the i18n bits18:31
morganso it's consistent18:31
ayounghttps://github.com/yaodu/docker-keystone/tree/master/dockerfiles  got it18:31
SamYapleayoung: https://hub.docker.com/r/yaodu/keystone/ https://github.com/yaodu/docker-keystone18:31
SamYaple++18:31
morganSamYaple: ooooh18:31
morgannifty18:31
*** portdirect_eatin is now known as portdirect18:32
ayoungSamYaple, ok, would you expect me to be able to build and deploy that on F25? I assume so...about to try...18:32
portdirecto/18:32
morganayoung: i need to bug you about F25 questions when you have a few minutes (can be tomorrow or later)18:32
*** chlong has joined #openstack-keystone18:32
ayoungmorgan, anytime18:32
SamYapleayoung: yea portdirect was looking into adding F2518:32
portdirectayoung: I made a f25 version - need to get it back up18:32
morganayoung: i'm running into some oddities but it's not work related (it's desktop) so when you're not doing docker/work things18:32
ayoungSamYaple, but that continaer is a debian, and I should be able to build and deploy it on F25, right?18:33
ayoungmorgan, You need Ajax18:33
portdirectno probs :)18:33
portdirectthats what I'm doing atm18:33
ayoungportdirect, that would be awesome!18:33
*** openstackgerrit has quit IRC18:33
SamYapleayoung: correct. right now we have debian, ubuntu, centos. all should work on all distros18:33
SamYaplebut sticking with the distro you have I feel will be the best for compat18:33
portdirectayoung: should have it up in about 3 hours (on dockerhub as well)18:34
morganayoung: docker container (regardless of the container's internal runtime) should work on any distro as long as the dockerfile is not crazy18:34
*** pcaruana has quit IRC18:34
SamYaplemorgan: well, then we get to libvirt and openvsiwtch and that statment doesnt hold as true18:34
morganand i trust SamYaple and portdirect to not have insane dockerfiles18:34
ayoungnow, these run without MySQL ATM, right?18:34
ayoungthey are just the containers, but do they have running keystone instances in them>18:34
SamYaplemorgan: any _unprivileged_ container should work. past that you might be in trouble18:34
ayoung?18:34
morganSamYaple: somewhat. i mean. there are tools around that18:34
SamYaplemorgan: agreed18:34
morganSamYaple: but yeah...  unpriv and in this case this is unpriv for sure.18:35
portdirectayoung: these dont have any config in them18:35
morganif it isn't.. omg18:35
morgan:P18:35
morgan^_^18:35
SamYapleayoung: the libs exist inside the contaienr. deployment tools can drop configs using mysql. i do not have an AIO entrypoint.sh script added yet18:35
SamYaplethat entrypoint script will be a basic AIO type setup, where you can point it to a DB. not meant for production18:36
ayoungSamYaple, its ok, just trying to get the state of things.  I acutally want it for development, and want to run the non-apache server way anyway18:36
portdirectayoung: but ive been running them on https://github.com/att-comdev/openstack-helm as replacements for the existing images18:36
SamYapleayoung: well i added ref patches, so you can build these contaienrs with patches now.18:36
morganayoung: uwsgi? vs apache?18:36
morganayoung: or similar?18:36
ayoungmorgan, yeah, potentially with breakpoints etc18:36
SamYaplecan you run uwsgi direct without apache/nginx?18:37
morganayoung: ah then not uwisgi18:37
morganyou want wsgiref18:37
morganSamYaple: yes.18:37
ayoungah,  yeah, just the wsgi container18:37
morganSamYaple: it has an HTTP mode18:37
SamYaplenice. was unawares18:37
morganayoung: yeah use wsgiref - it will play better with breakpoints, no cython magic18:37
morganalso will be single worker (iirc)18:38
lbragstadayoung morgan ack18:38
lbragstad(sorry was grabbing food quick)18:38
morganlbragstad: ftr, i have long advocated we should have a series of actual error codes18:38
morganindependent of http status18:38
ayoungSamYaple, http://docs.openstack.org/developer/keystone/devref/development_best_practices.html#running-keystone18:38
morganfor where we want to pass consistent information back down18:38
portdirectayoung: I could get us to support that no problem18:38
SamYaplegot it ayoung. i don't think uwsgi exists in the containers, but im going to add that now since its recommended18:39
ayoungportdirect, I'd rather figure it out myself.18:39
SamYapleand uwsgi is tiny18:39
portdirectroger :)18:39
morganuwsgi is also damn good.18:39
morgani wish we could gunicorn too, but oslo.config gets in the way :(18:39
ayoungAh...if it is not in there, they yeah, please add.18:39
SamYapleit doesnt do federation though, right?18:39
morganuwsgi cannot do federation18:39
SamYapleright. thats what i was thinking of then. tis why im still on apache218:40
morganyou need nginx+<module for federation> or apache+module18:40
morganthere has been work to support saml natively in keystone18:40
morganbut i am going to claim that is not baked/ready/inplace/coded/etc18:40
ayoungSamYaple, what protocols do you need for Federation?18:40
portdirectayoung: I'm using saml2 atm18:40
SamYapleayoung: me _personally_, saml18:41
morganbut basically the web servers provide those modules, and uswsgi runs keystone - so you can restart keystone w/o needing to bounce apache (etc).18:41
morganayoung: most folks use SAML2 since we pushed so hard for it18:41
SamYaplemorgan: in a container thats less of an issue (bounce container), but i get it18:41
morganayoung: which is a good thing.18:41
portdirectthough we wish to use OpenIDC as well moving forward18:41
morganSamYaple: well it matters a lot more if you are looking to do graceful restarts etc18:41
ayoungSamYaple, Yeah, I really want to split Keystone into multiple containers, one per IdP/protocol, so you can change them without affecting the rest of the world.18:42
morganSamYaple: even with a container, telling uwsgi to reload is better than leaning on apache to manage the wsgi and other stuff (mod_wsgi is not great and also not py3 friendly really)18:42
dstaneki've got a todo to deliver middleware that implement SAML2 (a subset) for this cycle18:42
morgandstanek: ++18:42
dstanekportdirect: i'm also looking at oidc as i do my work18:42
morgandstanek: so... a couple 2-3 cycles and it'll be really ready, but likely early UAT like stuff this cycle.18:42
morgan?18:43
ayoungit should be something like $OS_AUTH_URL=https://saml2.myidp.com/18:43
portdirectdstanek: great it would be really usefull for the k8s work im doing18:43
ayoungprolly with a keystone or something in there, too18:43
ayoungit should be something like $OS_AUTH_URL=https://saml2.myidp.keystone.mycloud.com/18:43
dstanekmorgan: basically. we'll probably never support everything18:43
morgandstanek: "everything" isn't my goal, just knowing when we have core support for <protocol discussed>18:44
morgani figure there is usually a cycle lag from when code lands to "really ready"18:44
dstanekmorgan: yup, the plan is to get the most common things working with shibboleth and then take feature requests later18:45
SamYaplemorgan: agreed on all accounts. I will switch to uwsgi with saml support for sure. not a big apache fan18:45
dstanekmorgan: yeah, we'll need early adopters for sure18:45
dstanekSamYaple: the shib module for nginx worked ok in my mini tests18:45
morganSamYaple: well you can offload only the shib work in nginx and passthrough the rest to uwsgi (more easily than in apache)18:46
morganin fact, that is what i would run keystone under at this point18:46
morganuwsgi backend(s), nginx+binary_uwsgi_protocol+federation and then probably some HAProxy or similar at the edge.18:47
*** openstackgerrit has joined #openstack-keystone18:47
openstackgerritRon De Rose proposed openstack/keystone: WIP - Add domain_id to the user table  https://review.openstack.org/40987418:47
morganand i would run it on 443/8018:48
morganrderose: ^ nice18:48
rderosemorgan: thanks :)18:48
morganrderose: going to bug you for some reviews of the auth changes for per-user mfa soonish.18:48
morganrderose: since you've been digging around in identity magic stuff18:49
morganand we're changing the sql backend to do more ORM cross table loading18:49
*** stingaci has joined #openstack-keystone18:50
rderosemorgan: sounds good18:50
*** tesseract has quit IRC18:51
stevemaro/18:51
*** tqtran has joined #openstack-keystone18:51
openstackgerritRon De Rose proposed openstack/keystone: WIP - Add domain_id to the user table  https://review.openstack.org/40987418:52
openstackgerritMorgan Fainberg proposed openstack/keystone-specs: Update per-user-MFA spec to represent new db table not column  https://review.openstack.org/41960718:54
rodrigodsstevemar, do we have the auth plugins names documented anywhere?18:55
*** stingaci has quit IRC18:55
*** lucas__ has joined #openstack-keystone18:55
stevemarrodrigods: http://docs.openstack.org/developer/keystoneauth/plugin-options.html18:55
stevemarlike that?18:55
stevemarlook for "Available Plugins"18:55
*** AlexeyAbashkin has quit IRC18:55
stevemaror you mean from the keystone side?18:55
stevemarnot keystoneauth...18:55
rodrigodsstevemar, keystoneauth... was exactly that! :)18:56
rodrigodsthis is new, right?18:56
stevemarrodrigods: merged yesterday18:56
morganin ksa it's also listed in the entrypoints (not strictly docs)18:56
rodrigodslol18:56
rodrigodsmorgan, yeah, that's how i usually did in the past18:56
morganhmm.18:58
*** AlexeyAbashkin has joined #openstack-keystone18:59
*** vern has quit IRC18:59
ayoungsince I'm in a designing mode, to get a token should be like this19:00
ayoungit should be something like $OS_AUTH_URL=https://saml2.myidp.keystone.mycloud.com/domain/<domid>/project/<projectid>19:00
ayoungto list the projects you have access to would be19:01
ayoungit should be something like $OS_AUTH_URL=https://saml2.myidp.keystone.mycloud.com/projects19:01
ayoungit should be something like $OS_AUTH_URL=https://saml2.myidp.keystone.mycloud.com/domains  would list the domains in which you have projects assigned19:01
ayoungman, I miss REST19:01
*** AlexeyAbashkin has quit IRC19:01
morganayoung, bknudson, stevemar, dstanek: I'm looking at the auth paths. Most of the time the user object is loaded by the auth plugins themselves (for obvious reasons). There isn't a big concern on timing attacks and such (leak of information) based upon the fact that the MFA data will return actual information such as "insufficient auth methods" if not enough19:03
morganmethods are supplied. I could probably invert it but it gets a bit wonky based upon the need to do work for "external" plugins19:03
morgan(this is in "keystone" server fwiw)19:03
morganI see some minor changes needed such as holding errors for authentication until after all methods are processed... but thats nbd on the change front19:03
ayoungmorgan, I still read mfa as Museum of Fine Art19:04
*** guoshan has joined #openstack-keystone19:05
morganayoung: as you should19:06
*** stingaci has joined #openstack-keystone19:06
ayoungSamYaple, no pip inside the container?19:06
morganayoung: oh wow. i just found a bug in our auth system19:07
ayoungmorgan, put that back. I was saving that.19:08
morganayoung: if something is improperly configured, and you specify say "password" and "external" -- whichever the last plugin that runs dictates the user_id in auth_context19:08
morganwe blindly overrwrite the auth_context.user_id19:08
morganeach method we validate19:08
SamYapleayoung: we remove it in cleanup. id be open to leaving it in for use with 'FROM yaodu/keystone'19:08
ayoungmorgan, yep19:08
morgan(not exploitable externally short of bad config, hence why i said it here)19:09
morganbut... ick19:09
ayoungSamYaple, nah, that is OK,  I just need to add uwsgi to my build....19:09
morganthat is not good(tm)19:09
* morgan goes and opens a bug for hardening and will fix at the same time19:09
*** guoshan has quit IRC19:10
stevemarmorgan: are you surprised we have half-baked code :)19:12
morganstevemar: no.19:12
morganstevemar: but... this is old code19:13
morgani'm surprised no one has hit this / noticed it before19:13
stevemarmorgan: we've never had reason to try 2 auth methods at once19:14
*** spzala has joined #openstack-keystone19:14
*** spzala has quit IRC19:14
openstackgerritSamuel Pilla proposed openstack/keystone: Add password expiration queries for PCI-DSS  https://review.openstack.org/40389819:15
*** spzala has joined #openstack-keystone19:15
morganstevemar: except we have always supported it :P19:16
morganstevemar: please look and confirm https://bugs.launchpad.net/keystone/+bug/165607619:17
openstackLaunchpad bug 1656076 in OpenStack Identity (keystone) "The keystone server auth pluigin methods could mismatch user_id in auth_context" [Undecided,New]19:17
morganstevemar: i've marked it security public. looks like a Class "D" to me. (bug with security implications)19:17
morganstevemar: and i think that should be fixed for sure in Ocata, so i'll submit the fix ahead of the MFA work patches19:17
morganit is also likely something we might want to consider as a low importance backport. so it doesn't bite anyone who has multiple plugins configured19:18
openstackgerritMerged openstack/keystone-specs: Update per-user-MFA spec to represent new db table not column  https://review.openstack.org/41960719:18
*** jaugustine has joined #openstack-keystone19:19
*** diazjf has joined #openstack-keystone19:21
*** jaugustine has quit IRC19:25
*** nicolasbock has quit IRC19:28
*** nicolasbock has joined #openstack-keystone19:31
*** edtubill has joined #openstack-keystone19:40
*** chlong has quit IRC19:41
openstackgerritLance Bragstad proposed openstack/keystone: Implement shadow mapping  https://review.openstack.org/41589519:45
*** gyee has joined #openstack-keystone19:45
*** chlong has joined #openstack-keystone19:57
lbragstaddo we have a keystone+horizon meeting today?20:04
-openstackstatus- NOTICE: Gerrit will be offline between now and 20:30 for scheduled maintenance: http://lists.openstack.org/pipermail/openstack-dev/2017-January/109910.html20:08
*** ChanServ changes topic to "Gerrit will be offline between now and 20:30 for scheduled maintenance: http://lists.openstack.org/pipermail/openstack-dev/2017-January/109910.html"20:08
edtubillwas the keystone+horizon meeting cancelled?20:10
david-lylenot sure20:10
lbragstadping stevemar ^20:15
lbragstaddavid-lyle edtubill I didn't see a note about a cancellation (but I could have missed it)20:15
robcresswellShouldn't be. I think richard is around this week, but not next or last20:16
dstaneki'm just waiting for it to start if it's still on20:16
*** adrian_otto1 has quit IRC20:17
*** jaugustine has joined #openstack-keystone20:22
*** ravelar1 has quit IRC20:22
ayoungSamYaple,       raise Exception("you need a C compiler to build uWSGI")20:22
ayoung  Exception: you need a C compiler to build uWSGI20:22
ayoungmight be tough to do inside the venv20:23
SamYapleayoung: images are almost dont gating20:24
SamYapleayoung: wont be a problem in a moment20:24
SamYapleyou need to install gcc, which you dont want to do20:24
ayoungSamYaple, excellent20:24
*** stingaci has quit IRC20:25
ayoungSamYaple, I'm building the image myself. Are your changes in the git repo?20:26
*** jaugustine has quit IRC20:26
SamYapleayoung: the changes are in yaodu/openstack-requirements, but that image is still building in the dockerhub cloud, which is not fast20:27
*** adu has quit IRC20:27
SamYaplebut if you build openstack-requirements, you can then build with this PR https://github.com/yaodu/docker-keystone/pull/1920:28
SamYaplethe changes should all be in the images in dockerhub in ~30m20:28
*** jaugustine has joined #openstack-keystone20:32
-openstackstatus- NOTICE: Updated: Gerrit will be offline until 20:45 for scheduled maintenance (running longer than anticipated): http://lists.openstack.org/pipermail/openstack-dev/2017-January/109910.html20:33
*** ChanServ changes topic to "Updated: Gerrit will be offline until 20:45 for scheduled maintenance (running longer than anticipated): http://lists.openstack.org/pipermail/openstack-dev/2017-January/109910.html"20:33
*** jrist has quit IRC20:33
*** itisha has joined #openstack-keystone20:35
*** stingaci has joined #openstack-keystone20:41
*** spzala has quit IRC20:41
*** chlong has quit IRC20:44
*** AlexeyAbashkin has joined #openstack-keystone20:44
*** jrist has joined #openstack-keystone20:47
*** ravelar has joined #openstack-keystone20:47
*** agrebennikov has joined #openstack-keystone20:49
*** adriant has joined #openstack-keystone20:51
*** d0ugal has quit IRC20:52
*** ChanServ changes topic to "Meeting Agenda: https://etherpad.openstack.org/p/keystone-weekly-meeting | Ocata goals: https://docs.google.com/spreadsheets/d/156q820cXcEc8Y9YWQgoc_hyOm3AZ2jtMQM3zdDhwGFU/edit?usp=sharing"20:53
*** d0ugal has joined #openstack-keystone20:54
*** d0ugal has quit IRC20:54
*** d0ugal has joined #openstack-keystone20:54
stevemaredtubill: david-lyle lbragstad bah, i forgot about the meeting20:54
*** stingaci has quit IRC20:57
*** chlong has joined #openstack-keystone20:58
*** AlexeyAbashkin has quit IRC20:59
SamYapleayoung: /win 2621:00
*** stingaci has joined #openstack-keystone21:01
*** david-lyle has quit IRC21:01
*** stingaci has quit IRC21:01
dstanekstevemar: boo21:01
dstanek:-)21:01
*** stingaci has joined #openstack-keystone21:01
*** r1chardj0n3s has joined #openstack-keystone21:02
*** david-lyle has joined #openstack-keystone21:02
stevemardstanek: o/21:03
*** d0ugal has quit IRC21:03
dstanekstevemar: i forgot too. i was busy reviewing a word doc and realized 15 mins late that i was missing it21:04
*** chlong has quit IRC21:04
*** browne has quit IRC21:05
*** diazjf has quit IRC21:05
*** stingaci has quit IRC21:05
*** thiagolib has quit IRC21:08
SamYapleayoung: yaodu/keystone patch has passed gate and is committed. images in dockerhub will take ~30m to build and push out (its really slow atm). but you can build it successfully locally21:08
ayoungSamYaple, building now anyway...cuz I live on the edge21:09
ayoungfirst I had to clean up some old messes21:09
ayoungdocker rm `docker ps  --all | awk ' $1 !~ /CONTAINER/ {print $1}' `21:09
*** adrian_otto has joined #openstack-keystone21:10
ayoungSamYaple, I still find it hard to accept that I should run a container, and then run a virtualenv inside that container.21:11
SamYapleayoung: make sure yo `docker rm -v $(images)` to remove the volumes that would be orphaned too21:11
ayoungSamYaple, ah, good point21:11
ayoungSamYaple, shouild I see a uwsgi executable in there?21:12
ayoungdocker run -i -t   yaodu/keystone:latest bash21:12
ayoung. ./virtualenv/bin/activate21:12
ayoungbash: uwsgi: command not found21:12
SamYapleyou dont need to active the venv for starters (its in your PATH already)21:13
SamYaplebut its not in dockerhub yet, the PR built and passed, now that its committed its building the image _again_21:13
SamYaplethat will be tagged21:13
SamYaplebut if you clone yaodu/keystone and build locally, it should be there21:14
SamYapleits unfortunate that the hub is slow :/ but im working on fixing that21:15
ayoungSamYaple, that is what I did.21:15
SamYaple`docker run -i -t   yaodu/keystone:latest bash` looks like youre pulling the dockerhub image maybe?21:16
SamYaplebut ive just tested it locally and it worked21:16
SamYaple(rebuilt on master)21:16
edtubillHi, can someone help me figure out what's wrong with my federated environment? I can log into horizon using federation but any action seems to result in an invalid token (in the keystone logs).21:17
SamYapleayoung: http://paste.openstack.org/show/594771/21:17
edtubillDoes anyone have any ideas where I can start debugging this?21:17
*** spzala has joined #openstack-keystone21:18
ayoungedtubill, you probably have an unscoped token, but no project assignments21:18
lbragstadedtubill do you have your federated groups and group assignments setup?21:18
*** chlong has joined #openstack-keystone21:18
*** openstackgerrit has quit IRC21:18
ayounglbragstad, wouldn't he get the "no projects for user" thing then?21:18
edtubillayoung,lbragstad: I have the group assignments and mappings setup.21:18
portdirectayoung: I've also just built from master - and pushed to my own repo - if you are still having difficulty before the image gets into docker hub then you can try: docker.io/port/keystone:centos21:18
ayoungSamYaple, I hadn't rebased21:19
lbragstadayoung oh - sure.. .yeah you're probably write21:19
ayoungjust did, saw your commit21:19
lbragstads/write/right/21:19
ayounglbragstad, no you were write the first time21:19
SamYapleayoung: ++21:19
edtubillayoung: lbragstad: I see 'user ... has no access to project ...' in the keystone logs.21:20
ayoungSamYaple, I thought they were supposed to come in via the openstack-dependences, which I have to admit now makes no sense21:20
lbragstadhmm - i would double check that the federated group you have setup actually has role assignments on the projects you want to work on21:20
ayoungedtubill, sounds like you need to map the user to a group, and give the group a role assignemnt.  You know how to do that21:20
*** phalmos has joined #openstack-keystone21:21
SamYapleayoung: ah yea. the build is interesting, because there is no build tools in keystone container ever. we pull built wheels from a layer in yaodu/openstack-requirements21:21
*** phalmos_ has quit IRC21:21
SamYapleayoung: it allows for the quick builds and small images21:21
*** diazjf has joined #openstack-keystone21:21
edtubillayoung: lbragstad: Yeah the group has a role on the project. and the user gets mapped to the group. I feel like somehow the mapping auth method doesn't get used or something.21:22
lbragstadedtubill are you able to get an unscoped token manually from keystone using cURL?21:23
ayoungedtubill, you can always hack the code and throw in an rpdb to trace what actually happens21:23
ayoungI have to admit I do that far more often than I should21:24
SamYaple+2 for rpdb. love that method of debugging21:24
ayoungedtubill, https://adam.younglogic.com/2015/02/debugging-openstack-with-rpdb/21:25
edtubilllbragstad: ayoung: so I've been using rpdb and it appears that I get a unscoped token and also a scoped token but when that token gets validated on keystone (by nova or something), keystone sees that the user has no roles.21:25
lbragstadedtubill if you are able to get an unscoped token, I would try to get a list of projects you have access to21:25
ayoungedtubill, that does not make sense.  A scoped token has to have roles.21:25
lbragstadhm21:25
ayoungcan you see the token response somewhere?21:25
lbragstadedtubill what's the response look like?21:25
* ayoung wishes we could depend on ecp21:26
lbragstadedtubill and can you validate the token manually against keystone?21:26
edtubilllbragstad: hmm, I see the invalid token id in the mysql table if that helps....21:27
*** browne has joined #openstack-keystone21:27
lbragstadedtubill in the keystone.revocation_event table?21:27
ayoungSamYaple, its there now.  THanks21:28
edtubillayoung, let me get back to you with some more responses. I think I can't do curl requests because my IDP doesn't support ECP. (so I'll have to switch to shibtest or something)21:28
edtubilllbragstad: I see the token in the token table.21:29
lbragstadedtubill ah - you're using UUID21:29
lbragstadgotcha21:29
edtubilllbragstad: yup I think so.21:29
lbragstadok - since we in the realm of tokens and revocation events, do you have anything in your revocation_event table?21:29
lbragstador is that empty21:29
lbragstad?21:29
*** jose-phillips has quit IRC21:30
edtubilllbragstad: it seems to be empty.21:31
lbragstadedtubill ok - good deal21:31
lbragstadi didn't assume there to be anything in there, but wanted to double check21:32
lbragstadedtubill is this a devstack installation?21:32
edtubilllbragstad: yeah21:32
lbragstadmaster?21:32
edtubilllbragstad: yup21:33
lbragstadwhat's acting as the IdP/21:33
lbragstadanother keystone?21:33
lbragstador something else?21:33
*** jose-phillips has joined #openstack-keystone21:33
edtubillI'm using Auth0 but I get the same results when I set it up for K2K.21:33
lbragstadhmm21:33
*** lucas__ has quit IRC21:34
edtubilllbragstad: I feel like I'm setting up something wrong with the auth methods or something. I can log in but can't do any actions.21:34
lbragstadedtubill the interesting part that throws me for a loop is that you are able to get an unscoped and eventually a scoped, token21:35
lbragstadwhich leads me to think that you've setup your authentication methods properly21:35
edtubilllbragstad: let me debug django_openstack_auth again and make sure I get a proper scoped token. brb21:36
lbragstad(by I could be wrong - I'd probably differ to dstanek or rodrigods for more help)21:36
lbragstadedtubill ok21:36
edtubillok sure thing, but thanks for the help lbragstad and ayoung.21:36
lbragstadedtubill anytime - i'm curious about what you find21:36
*** ravelar has quit IRC21:37
*** openstackgerrit has joined #openstack-keystone21:45
openstackgerritGage Hugo proposed openstack/keystone: Removed old 'Driver' reference from token init  https://review.openstack.org/41966321:45
*** phalmos has quit IRC21:46
*** sheel has quit IRC21:47
stevemardo we have a list of bugs we're looking at for the office hours?21:51
stevemaror an etherpad?21:51
*** phalmos has joined #openstack-keystone21:53
lbragstadstevemar we don't21:54
lbragstadI wasn't sure if it would be worth it to duplicate it21:54
stevemarlbragstad: s'all good, we can use my google doc21:54
lbragstadI figured we'd just use the list you've started21:54
lbragstadstevemar yes - exactly21:54
*** spzala has quit IRC21:54
openstackgerritGage Hugo proposed openstack/keystone: Changed 'Driver' reference to 'TokenDriverBase'  https://review.openstack.org/41966321:54
lbragstadstevemar dstanek also had some pretty awesome tooling setup to make it so that we could track all the activity on Friday - which would save us a lot of time21:55
stevemarlbragstad: i'd love to get the k2k one, and the ldap and upgrade bugs fixed for ocata21:55
lbragstadand make things way easier to understand21:55
stevemar++21:55
gagehugolbragstad: thanks for the quick review21:55
lbragstadgagehugo thanks for the quick turn around21:56
gagehugo:)21:56
gagehugowanted to get this bug done last friday but things keep distracting me21:56
lbragstadgagehugo that happens21:57
stevemargagehugo: being ptl is week after week of distractions21:57
lbragstadit's a good thing tomorrow is Friday :)21:57
*** diazjf has quit IRC21:57
*** thorst_ has quit IRC21:57
stevemaryou people are very distracting21:58
lbragstadstevemar ping21:58
* stevemar laughs at his own joke21:58
stevemarlbragstad: yessum21:58
gagehugolol21:58
lbragstadstevemar you in the middle of something?21:58
stevemarlbragstad: not really21:59
lbragstadstevemar oh ok - cool...21:59
lbragstadstevemar that is all21:59
gagehugoheh21:59
* stevemar shakes fist!21:59
gagehugosupposed to get ice tomorrow so it may be fixing bugs by candlelight21:59
lbragstadgagehugo stash your changes locally in case your wifi freezes22:00
gagehugolbragstad exactly!22:00
*** chris_hultin is now known as chris_hultin|AWA22:01
*** spilla has joined #openstack-keystone22:02
lbragstadalright - stepping away for a bit, i'll check back in a little later22:04
*** d0ugal has joined #openstack-keystone22:07
*** darrenc is now known as darrenc_afk22:09
*** jaugustine has quit IRC22:11
*** jaugustine has joined #openstack-keystone22:11
jamielennoxstevemar: requests for that doc already!22:12
stevemarjamielennox: who'd thunk that docs were wanted?!22:14
jamielennoxstevemar: i've never believed it22:14
*** jaugustine_ has joined #openstack-keystone22:14
*** jaugustine has quit IRC22:15
*** jaugustine_ has quit IRC22:15
*** jose-phillips has quit IRC22:19
*** spilla has quit IRC22:22
*** spzala has joined #openstack-keystone22:22
*** edmondsw has quit IRC22:24
*** edmondsw has joined #openstack-keystone22:24
*** jose-phillips has joined #openstack-keystone22:25
*** spzala has quit IRC22:27
*** spzala has joined #openstack-keystone22:28
*** edmondsw has quit IRC22:29
*** chlong has quit IRC22:29
*** diazjf has joined #openstack-keystone22:30
*** thorst_ has joined #openstack-keystone22:32
*** spzala has quit IRC22:34
*** thorst_ has quit IRC22:37
*** chlong has joined #openstack-keystone22:37
*** darrenc_afk is now known as darrenc22:41
*** chris_hultin|AWA is now known as chris_hultin22:43
openstackgerritMerged openstack/keystone: Fix keystone-manage mapping_engine tester  https://review.openstack.org/41816522:47
*** diazjf has quit IRC22:50
*** jperry has quit IRC23:03
*** chris_hultin is now known as chris_hultin|AWA23:04
openstackgerritGage Hugo proposed openstack/keystone: Changed 'Driver' reference to 'TokenDriverBase'  https://review.openstack.org/41966323:12
*** dave-mccowan has quit IRC23:17
*** jaosorior has quit IRC23:22
morganstevemar: about to push the fix for the auth methods and mutable user_ids23:24
morganjamielennox: docs?! what is this madness23:28
*** tqtran has quit IRC23:29
*** thorst_ has joined #openstack-keystone23:33
*** haplo37_ has quit IRC23:36
*** thorst_ has quit IRC23:37
*** rvba has quit IRC23:38
*** lucas__ has joined #openstack-keystone23:38
*** haplo37_ has joined #openstack-keystone23:39
*** edtubill has quit IRC23:42
openstackgerritMorgan Fainberg proposed openstack/keystone: Do not allow auth plugins to return different user_id  https://review.openstack.org/41969323:49
morganstevemar: ^23:50
*** lamt has quit IRC23:55
*** rvba has joined #openstack-keystone23:57
*** rvba has quit IRC23:57
*** rvba has joined #openstack-keystone23:57

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