| *** gyee has quit IRC | 00:10 | |
| *** marzif_ has quit IRC | 00:13 | |
| *** zzzeek has quit IRC | 00:24 | |
| openstackgerrit | A change was merged to openstack/keystone: Add oslo.utils requirement https://review.openstack.org/112156 | 00:34 |
|---|---|---|
| *** stevemar has joined #openstack-keystone | 01:01 | |
| *** marcoemorais has quit IRC | 01:36 | |
| openstackgerrit | A change was merged to openstack/keystone: Remove S3 middleware tests from tox.ini https://review.openstack.org/112387 | 01:43 |
| *** stevemar has quit IRC | 02:33 | |
| *** diegows has quit IRC | 02:33 | |
| openstackgerrit | A change was merged to openstack/keystone: Add an OS-FEDERATION section to scoped federation tokens https://review.openstack.org/111070 | 03:00 |
| *** _elmiko has quit IRC | 03:13 | |
| *** stevemar has joined #openstack-keystone | 03:13 | |
| *** hrybacki has joined #openstack-keystone | 03:24 | |
| stevemar | morganfainberg, ping? | 03:27 |
| morganfainberg | stevemar, pong | 03:27 |
| stevemar | morganfainberg, in here: https://review.openstack.org/#/c/109041/10/keystone/token/provider.py line 190 vs 218 | 03:28 |
| stevemar | why catch exception for v3 and not v2? | 03:28 |
| morganfainberg | stevemar, because the original code that was moved did it | 03:31 |
| morganfainberg | stevemar, to be perfectly honest | 03:31 |
| morganfainberg | stevemar, was trying to limit change + moving code in that patch | 03:31 |
| stevemar | morganfainberg, gotcha, i do the same sometimes | 03:32 |
| stevemar | morganfainberg, i have a few other questions, if you don't mind | 03:32 |
| morganfainberg | stevemar, ask away | 03:32 |
| morganfainberg | stevemar, i need to go get food soon and make a phone call, but i have a few minutes | 03:32 |
| stevemar | morganfainberg, nvm i see the change between _validate_v3_token() taking id's and now token_refs is now in token/provider/common | 03:33 |
| stevemar | hadn't gotten to that file yet | 03:34 |
| morganfainberg | hehe no worries | 03:34 |
| stevemar | morganfainberg, so why move a bunch of things from common to provider? | 03:34 |
| morganfainberg | stevemar, those things call token_api | 03:34 |
| stevemar | i know it's more than just a move | 03:35 |
| stevemar | ah | 03:35 |
| morganfainberg | since i'm trying to deprecate token_api (provider.persistence) so we have better control over how tokens are interacted with... | 03:35 |
| morganfainberg | move the things that specifically call token_api to the manager | 03:35 |
| morganfainberg | also, the provider driver shouldn't call XXX_api imo | 03:35 |
| morganfainberg | the manager should talk to XXX_api and pass the information that is relevant to the driver. | 03:36 |
| stevemar | morganfainberg, that makes sense when you write it out like that | 03:38 |
| morganfainberg | :) | 03:38 |
| morganfainberg | there *is* a method to my madness | 03:38 |
| stevemar | and validate_token and revoke_token both required an id | 03:38 |
| stevemar | which is why they are the manager level and not the driver level, yes? | 03:39 |
| morganfainberg | yep | 03:39 |
| morganfainberg | anything that needs to take an id and make it into a token_ref is manager level | 03:39 |
| stevemar | nice | 03:39 |
| stevemar | ok, cool, i see what you did here | 03:40 |
| stevemar | few more minutes away from +2+A | 03:40 |
| stevemar | just needed to wrap my head around the changes | 03:40 |
| morganfainberg | and you can look at one of the follow-on patches to see why it's happening | 03:40 |
| morganfainberg | and use of the tokenmodel object ;) | 03:41 |
| stevemar | morganfainberg, done! | 03:45 |
| morganfainberg | stevemar, cool. | 03:46 |
| morganfainberg | stevemar, tyvm sir | 03:46 |
| stevemar | morganfainberg, abt that... did we decide on user domain for federation tokens? | 03:46 |
| morganfainberg | stevemar, i dunno :( | 03:46 |
| stevemar | i was half attending the meeting cause of holiday | 03:46 |
| stevemar | IIRC we decided that a dummy id was as good as putting idpId | 03:46 |
| morganfainberg | stevemar, don't really have an answer - i *think* we need the domain section because we document it elsewhere as "this is a token" | 03:46 |
| stevemar | since querying /v3/domains/<blah> won't work | 03:46 |
| morganfainberg | stevemar, and idpid vs "reserved" doesn't matter | 03:47 |
| stevemar | i think dolph wanted the model changes or the clients smarter? | 03:47 |
| morganfainberg | basically... as long as it doesn't end up being brittle - i don't care which way we go | 03:47 |
| morganfainberg | stevemar, we nixed that concept only because we documented what a token ends up looking like in the identity-api somewhere | 03:47 |
| stevemar | ah | 03:47 |
| morganfainberg | stevemar, and it *always* included a "domain" section | 03:48 |
| morganfainberg | for the user | 03:48 |
| stevemar | so we should provide one | 03:48 |
| stevemar | even if it is bogus | 03:48 |
| *** gabriel-bezerra has quit IRC | 04:00 | |
| *** ukalifon has joined #openstack-keystone | 04:37 | |
| *** hrybacki has quit IRC | 04:55 | |
| *** ukalifon has quit IRC | 04:58 | |
| *** xianghui has quit IRC | 05:01 | |
| *** xianghui has joined #openstack-keystone | 05:02 | |
| *** amirosh has joined #openstack-keystone | 05:12 | |
| *** amerine has quit IRC | 05:17 | |
| *** amirosh has quit IRC | 05:17 | |
| *** amirosh has joined #openstack-keystone | 05:18 | |
| *** amirosh has quit IRC | 05:23 | |
| openstackgerrit | A change was merged to openstack/keystone: Make token_provider_api contain token persistence https://review.openstack.org/109041 | 05:26 |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/keystone: Imported Translations from Transifex https://review.openstack.org/111920 | 06:04 |
| *** henrynash has joined #openstack-keystone | 06:50 | |
| *** henrynash has quit IRC | 06:56 | |
| *** henrynash has joined #openstack-keystone | 06:57 | |
| openstackgerrit | A change was merged to openstack/keystone: swap import order of lxml https://review.openstack.org/112915 | 06:57 |
| *** henrynash has quit IRC | 07:34 | |
| *** henrynash has joined #openstack-keystone | 08:00 | |
| *** bvandenh has joined #openstack-keystone | 08:28 | |
| *** henrynash has quit IRC | 08:33 | |
| *** designated has quit IRC | 08:44 | |
| openstackgerrit | Steve Martinelli proposed a change to openstack/keystone: Transform a Keystone token to a SAML assertion https://review.openstack.org/110542 | 08:47 |
| *** ukalifon1 has joined #openstack-keystone | 08:51 | |
| *** stevemar has quit IRC | 08:55 | |
| openstackgerrit | A change was merged to openstack/keystone: Class for V3 router packages https://review.openstack.org/111567 | 09:04 |
| *** chandankumar has joined #openstack-keystone | 09:24 | |
| *** chandankumar has quit IRC | 09:29 | |
| *** chandankumar has joined #openstack-keystone | 09:32 | |
| *** chandankumar has quit IRC | 09:38 | |
| *** chandankumar has joined #openstack-keystone | 09:39 | |
| *** afaranha has quit IRC | 09:47 | |
| *** rodrigods has quit IRC | 09:47 | |
| *** tellesnobrega1 has quit IRC | 09:47 | |
| *** samuelmz has quit IRC | 09:47 | |
| *** chandankumar has quit IRC | 10:02 | |
| *** chandankumar has joined #openstack-keystone | 10:03 | |
| *** chandankumar has quit IRC | 10:39 | |
| *** ukalifon1 has quit IRC | 11:25 | |
| *** ukalifon1 has joined #openstack-keystone | 11:31 | |
| *** chandankumar has joined #openstack-keystone | 11:36 | |
| *** ukalifon1 has quit IRC | 11:36 | |
| *** chandankumar has quit IRC | 11:47 | |
| openstackgerrit | Marek Denis proposed a change to openstack/python-keystoneclient: SAML2 federated authentication for ADFS. https://review.openstack.org/111771 | 11:56 |
| *** jaosorior has joined #openstack-keystone | 11:57 | |
| *** ukalifon1 has joined #openstack-keystone | 12:20 | |
| *** ukalifon1 has left #openstack-keystone | 12:20 | |
| *** diegows has joined #openstack-keystone | 12:32 | |
| openstackgerrit | David Stanek proposed a change to openstack/keystone: Remove strutils and timeutils from openstack-common.conf https://review.openstack.org/112158 | 12:49 |
| openstackgerrit | David Stanek proposed a change to openstack/keystone: Use functions in oslo.utils https://review.openstack.org/112157 | 12:49 |
| dstanek | ^ those are just rebased so they should be very fast reviews | 12:50 |
| *** marzif_ has joined #openstack-keystone | 13:16 | |
| *** jgriffit1 has quit IRC | 13:20 | |
| *** tomoiaga has joined #openstack-keystone | 13:30 | |
| *** nkinder_away has quit IRC | 13:49 | |
| *** rwsu has quit IRC | 13:52 | |
| *** rwsu has joined #openstack-keystone | 14:06 | |
| *** tomoiaga has quit IRC | 14:23 | |
| *** amirosh has joined #openstack-keystone | 14:24 | |
| *** tomoiaga has joined #openstack-keystone | 14:49 | |
| *** chandankumar has joined #openstack-keystone | 14:54 | |
| *** chandankumar has quit IRC | 14:55 | |
| *** chandankumar has joined #openstack-keystone | 14:55 | |
| openstackgerrit | Alexey Miroshkin proposed a change to openstack/keystone: Support the hints mechanism in list_credentials() https://review.openstack.org/113091 | 14:59 |
| *** amirosh has quit IRC | 15:00 | |
| *** hrybacki has joined #openstack-keystone | 15:12 | |
| *** amirosh has joined #openstack-keystone | 15:13 | |
| *** jaosorior has quit IRC | 15:22 | |
| *** amirosh has quit IRC | 15:30 | |
| *** amirosh has joined #openstack-keystone | 15:30 | |
| *** amirosh has quit IRC | 15:36 | |
| *** tomoiaga has quit IRC | 15:59 | |
| *** amirosh has joined #openstack-keystone | 16:07 | |
| *** amirosh has quit IRC | 16:12 | |
| *** amirosh has joined #openstack-keystone | 17:02 | |
| *** ajayaa has joined #openstack-keystone | 17:04 | |
| *** amirosh has quit IRC | 17:07 | |
| ajayaa | morganfainberg, ayoung, dolphm, https://review.openstack.org/#/c/110575 | 17:07 |
| *** ajayaa has quit IRC | 17:26 | |
| *** amirosh has joined #openstack-keystone | 17:36 | |
| *** amirosh has quit IRC | 17:41 | |
| *** amirosh has joined #openstack-keystone | 18:57 | |
| *** amerine has joined #openstack-keystone | 18:58 | |
| *** amirosh has quit IRC | 19:38 | |
| *** amirosh has joined #openstack-keystone | 19:39 | |
| *** amirosh has quit IRC | 19:44 | |
| *** chandankumar_ has joined #openstack-keystone | 19:57 | |
| *** andreaf has joined #openstack-keystone | 19:58 | |
| *** andreaf has quit IRC | 19:58 | |
| *** chandankumar has quit IRC | 19:58 | |
| *** chandankumar_ has quit IRC | 20:03 | |
| *** andreaf has joined #openstack-keystone | 20:19 | |
| *** andreaf_ has joined #openstack-keystone | 20:19 | |
| *** andreaf has quit IRC | 20:19 | |
| *** andreaf_ has quit IRC | 20:24 | |
| *** stevemar has joined #openstack-keystone | 20:56 | |
| openstackgerrit | A change was merged to openstack/python-keystoneclient: Fixes import grouping https://review.openstack.org/112299 | 20:59 |
| *** stevemar has quit IRC | 21:40 | |
| *** henrynash has joined #openstack-keystone | 22:10 | |
| *** hrybacki has quit IRC | 23:19 | |
| *** stevemar has joined #openstack-keystone | 23:27 | |
| *** hrybacki has joined #openstack-keystone | 23:30 | |
| *** henrynash has quit IRC | 23:40 | |
| *** diegows has quit IRC | 23:57 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!