Wednesday, 2014-05-28

gabriel-bezerrathe configuration files on /etc/httpd/conf/00:00
gabriel-bezerrain*00:00
gabriel-bezerraconf.d*00:01
openstackgerritBrant Knudson proposed a change to openstack/keystone-specs: Spec for V3 extension advertisement  https://review.openstack.org/9597300:01
*** gokrokve has quit IRC00:03
gabriel-bezerramorganfainberg, dtroyer ^00:03
*** gokrokve has joined #openstack-keystone00:04
gabriel-bezerramarekd|away ^00:04
morganfainberggabriel-bezerra, i think they are in conf.d00:04
morganfainberggabriel-bezerra, for RHEL and fedora that is.00:04
gabriel-bezerramorganfainberg: but are there default files in those directories as on Ubuntu?00:05
*** sbfox has quit IRC00:05
gabriel-bezerraor is the directory empty when apache is installed ?00:05
morganfainberggabriel-bezerra, honestly, not sure haven't looked recently00:05
morganfainberggabriel-bezerra, i want to say they are ... empty00:05
morganfainbergjamielennox, ayoung, ^ gabriel-bezerra's question you guys might know00:06
gabriel-bezerraI ask that because I'm moving forward with this review https://review.openstack.org/9077100:06
gabriel-bezerraAnd would like to know if using site.conf{.disabled,} would break anything00:06
gabriel-bezerrainstead of site{,.conf} as it is today00:07
jamielennoxumm, the equivalent is just files in /etc/httpd/conf.d/*.conf00:07
jamielennoxi think they have to end with .conf as there is a README in there that is ignored00:08
*** gokrokve has quit IRC00:08
*** ncoghlan has joined #openstack-keystone00:08
gabriel-bezerrajamielennox: my question is whether conf.d comes with any file for default sites00:08
gabriel-bezerraas ubuntu does00:08
jamielennoxit has a welcome.conf which is that standard splash screen00:09
jamielennoxphp.conf ends up in there00:09
gabriel-bezerraon Ubuntu, a fresh installation of apache comes with /etc/apache2/sites-available/{000-default.conf,default-ssl.conf}00:09
jamielennoxgabriel-bezerra: no, not like that, though from memory if you install mod_nss it puts some sample files in there00:10
gabriel-bezerraok00:10
gabriel-bezerrabtw, should I create a new Change on Gerrit?00:10
gabriel-bezerramy new patch will be like this:..00:10
gabriel-bezerrahttps://github.com/gabriel-bezerra/devstack/commit/0d42bd7b44e3ba5ea22e68166a92bc3e449186ae00:11
openstackgerritMorgan Fainberg proposed a change to openstack/keystone-specs: Add spec for non-persistent-tokens  https://review.openstack.org/9597600:11
gabriel-bezerraIt will touch many more files and resolve things in a different way.00:11
openstackgerritMorgan Fainberg proposed a change to openstack/keystone-specs: Add spec for non-persistent-tokens  https://review.openstack.org/9597600:13
bknudsondstanek_zzz: is https://review.openstack.org/#/c/91825/ abandoned?00:14
gabriel-bezerramorganfainberg: ^00:15
gabriel-bezerrathank you, jamielennox and morganfainberg, for the information about httpd00:16
*** shakamunyi has quit IRC00:18
morganfainbergbknudson, https://review.openstack.org/#/c/80398/ looks like it wont pass (merge conflict)00:21
bknudsonmorganfainberg: you are shitting me.00:21
morganfainbergbknudson, will need a rebase it's related to the sample tokens not being valid00:21
morganfainbergbknudson, just checked. like 2 line change :(00:21
morganfainbergbknudson, conflicted with one of your changes :P00:22
bknudsonwell, maybe in another 2 weeks it'll merge.00:22
morganfainbergwell i was about to rebase but wanted to wait for the parent to merge00:22
morganfainbergi was just going to +2/+A once the rebase was done since it was a simple rebase00:23
bknudsonok00:23
morganfainbergbknudson, i aim to get that out in the next ksc release, and we need a ksc release to land compressed tokens ASAP00:23
morganfainbergit was mostly a heads up incase you saw it before i got to rebase - either way +2/+A right away.00:24
gabriel-bezerrahttps://review.openstack.org/9597700:27
morganfainberggabriel-bezerra, awesome! I had forgotten who was working on that00:28
morganfainberggabriel-bezerra, thanks :) /me looks at now00:28
*** praneshp has quit IRC00:29
*** nkinder has joined #openstack-keystone00:32
*** gokrokve has joined #openstack-keystone00:32
*** stevemar has joined #openstack-keystone00:32
gabriel-bezerramorganfainberg: thanks :)00:33
*** dstanek_zzz is now known as dstanek00:34
*** zhiyan_ is now known as zhiyan00:34
ayoungmorganfainberg, where are we with the "split keystone middleware into its own repo?"00:35
morganfainbergayoung, i think we're at "do we want to make middleware depend on KSC or vice versa"00:36
morganfainbergayoung, and 2: name00:36
*** zhiyan is now known as zhiyan_00:36
ayoungmorganfainberg, oh, that is easy.00:36
ayoungpython-keystonemiddleware;  and pkm depends on pkc00:37
morganfainbergayoung, in that case we will have to copy the middleware changes to pkc to maintain compatibility00:37
ayoungmorganfainberg, we just need some sort of hack so that people with keystoneclient.middleware.auth_token in their config files don't get broken00:38
morganfainbergayoung, unless we plan a full refactor "no you get nothing new in old middleware" (which is probably not the right answer)00:38
morganfainbergayoung, that one should be easy-ish to do.00:38
morganfainbergayoung, as long as peiople don't try and load both middlewares (hey can't guard against "didn't RTFM")00:38
*** zhiyan_ is now known as zhiyan00:38
ayoungmorganfainberg, even that will work.  Sort of00:38
morganfainbergayoung, nah, will explode due to re-registering config opts00:39
ayoungmorganfainberg, should I write up a BP?00:39
morganfainbergayoung, yeah we should have one for this00:39
ayoungmorganfainberg, I'm on it00:39
morganfainbergayoung, awesome!00:39
ayoungmorganfainberg, we are going to pull in gordon chungs audit middleware, too00:39
morganfainbergayoung, i'd like to pull in all the middleware from ksc.00:40
ayoungKeystone will openstacks Identity, Policy, and Audit....00:40
ayoungmuahahahahahaha00:40
morganfainbergayoung, we should rename the programt to AAA00:40
ayoungOpenStackIPA00:40
morganfainberg>.>00:40
ayoung%-)00:41
morganfainberghehe00:41
*** zhiyan is now known as zhiyan_00:41
*** rodrigods_ has joined #openstack-keystone00:42
morganfainbergayoung, soo about some magic to auto register VMs with FreeIPA00:43
ayoungI have a blog post for that00:43
ayounghttp://adam.younglogic.com/2013/09/register-vm-freeipa/00:43
morganfainbergayoung, thought so.00:44
morganfainbergayoung, awesome.00:44
*** dstanek is now known as dstanek_zzz00:44
ayoungmorganfainberg, does that help?  It only works from a script00:44
ayoungnot from Horizon00:44
ayoungwhich might be OK...but I want more00:44
ayounghere's what I want:00:44
morganfainbergayoung, i think it could be made to work from horizon... but it'll be a bit bigger x-project work00:44
ayoung1.  Nova generates an OTP (Random Hash)00:44
ayoung2.  Nova sends notification to FreeIPA with OTP00:44
ayoungmust be encrypted or somehow protected00:45
morganfainbergwe should also support domain == realm00:45
morganfainbergso new domains can create a realm >.>00:45
ayoung3.  Nova adds OTP to Metadata for new VM00:45
ayoungmorganfainberg, Designate is doing something really similar with their freeIPA backend, but just for DNS00:45
ayoungso no OTP00:45
morganfainbergah yeah00:46
ayoungand the OTP generation is outside the scope of Designate, but the mechanism on that side would almost be identical,  just an ipa host-create instead of the dns call00:46
morganfainbergayoung, yeah sounds like there is def. some alignment there00:46
stevemarbknudson, ping?00:47
bknudsonstevemar: what's up?00:47
stevemarbknudson, your remark on https://review.openstack.org/#/c/95845/ - i was referring to https://review.openstack.org/#/c/92228/ (yours)00:48
stevemarbknudson, is this one of the 'do as i say, not as i do' situations?00:48
bknudsonstevemar: in my case I didn't have the previous commit, so I don't know how I'd generate the list00:50
stevemarbknudson, ah okay - does this list have a specific format?00:50
bknudsonfor you it will be easy, it's like git log --oneline 2640847..4a777e5 -- (list of files that were synced)00:50
bknudsonstevemar: git log --oneline00:51
stevemaroh00:51
stevemarneat00:51
bknudsonstevemar: --no-merges00:51
stevemarsomeone should add that here https://wiki.openstack.org/wiki/ReviewChecklist#Oslo_Syncing_Checklist00:51
openstackgerritSteve Martinelli proposed a change to openstack/python-keystoneclient: Sync with oslo-incubator 4a777e5  https://review.openstack.org/9584500:54
stevemarbknudson, i think ^ addressed it00:55
bknudsonthat was a short list.00:55
stevemaryes.. it was just grabbing the doc build errors00:55
bknudsony, it finally merged00:56
bknudsoneverything is finally going my way00:56
*** gokrokve_ has joined #openstack-keystone00:56
gabriel-bezerramorganfainberg: I improved the commit message of that change00:56
stevemarbknudson, everything is coming up brant!00:56
ayoungmorganfainberg, trying a git review of a spec00:57
ayoungI'm reabse against origin master, but get rebase issues from gerrit00:57
ayoung.gitreview  I understand why00:57
ayounghttp://paste.fedoraproject.org/105245/2386941400:58
ayoungbut after that:  http://paste.fedoraproject.org/105246/3875114000:59
stevemarayoung, that would do it00:59
morganfainbergayoung hehe00:59
ayoungmorganfainberg, I think gerrit is ahead of master01:00
*** gokrokve has quit IRC01:00
morganfainbergayoung, YOUR master or keystone-specs?01:00
ayoungmorganfainberg, keystone-specs01:00
ayounglet me confirm...thought I did a fetch01:00
*** dstanek_zzz is now known as dstanek01:01
morganfainberghttps://github.com/openstack/keystone-specs/blob/master/.gitreview01:01
morganfainbergayoung, looks ok to me, but i remember having to fix that as well when i snagged your tree01:01
ayoungnah, twas me01:01
morganfainbergayoung, eh new stuff = mistakes both pebcak and non01:02
ayoungmorganfainberg, nah, thins are still out of sync, just a different list01:02
ayoungmorganfainberg, I did fetch and rebase origin/master, then cherrypicked my commit01:03
morganfainbergayoung, which origin you on?01:04
ayoungmorganfainberg, [remote "origin"]01:04
ayoung 16     url = git://git.openstack.org/openstack/keystone-specs01:04
morganfainbergayoung, try a clean 'checkout origin/master'01:04
morganfainbergunless... is your commit have extra cruft in it?01:05
morganfainbergs/is/does/01:05
ayoungmorganfainberg, nope...is git.openstack out of sync with github?01:06
* morganfainberg checks01:06
stevemardelete it all, clone it fresh01:06
morganfainbergayoung, looks ok to me http://git.openstack.org/cgit/openstack/keystone-specs/tree/.gitreview01:07
morganfainbergayoung, i think you have some cruft that git is confused about, clean clone might be best bet (copy file afterwards and add/commit)01:07
stevemar++01:07
ayoung$ git diff origin/master  --stat01:07
ayoungwarning: refname 'origin/master' is ambiguous.01:07
ayoung specs/juno/OS-SIMPLECERT-CRL.rst | 272 +++++++++++++++++++++++++++++++++++++++01:07
ayoung 1 file changed, 272 insertions(+)01:07
morganfainbergoh. i bet i know.01:08
morganfainbergit's probably just confused because you tried a rebase01:08
morganfainbergwe didn't keep any history, was squashed01:08
ayoungnope01:08
ayoungoh, maybe...01:08
ayounglet me try just pulling the one file over...but it was a cherry pick first, should be no confusion01:09
*** mfisch has quit IRC01:09
morganfainbergayoung, i've seen it do silly things before when you have trees that are "close" in end result but have massive object differences01:09
morganfainbergask me about the time i accidently rebased keystoneclient to keystone master01:10
morganfainberg>.>01:11
stevemarmorganfainberg, lol01:11
stevemarmorganfainberg, how does one even do that01:11
ayoungnope01:12
morganfainbergstevemar, git add <keystoneclient remote>; git fetch ; git checkout ksc ; git rebase origin/master01:12
morganfainbergstevemar, it was spectacular01:12
morganfainbergayoung, i'd copy that file out and just clean clone, something's gone wonky01:13
*** mfisch has joined #openstack-keystone01:13
ayoungmorganfainberg, yeah...I hate to give up on git that way01:13
*** mfisch has quit IRC01:13
*** mfisch has joined #openstack-keystone01:13
morganfainbergayoung, ditto, but every now and again (esp. in cases like keystone-specs repo getting muddled up) it is the easiest01:14
morganfainbergayoung, it was based on your repo, triple-o, some of nova upstream, squashed, then named identity-specs, then renamed keystone-specs01:14
morganfainbergayoung, something is bound to get fouled up somewhere01:14
stevemarayoung, who cares, isn't that the beauty of remote repos, you can nuke it at anytime, and rebuild01:15
*** ncoghlan is now known as ncoghlan_afk01:15
ayoungstevemar, it matters in that I want to understand what my tools are doing.  Otherwise I can't trust them01:16
morganfainbergayoung, i'll bet it has some ref in there that is making it unhappy and you could clean the object up and it'd be fine.  this is a case though, where it doesn't seem worth it01:17
ayoungtrue...draft review just went through fine post clone01:18
* morganfainberg glares at trove patch: https://jenkins06.openstack.org/job/gate-tempest-dsvm-full/3690/console MERGE damn it01:19
morganfainbergso i can rebase the ksc change and get that in so we can get a new ksc released...01:19
morganfainbergthen... dinner time01:19
stevemarmorganfainberg, don't you hate admitting that you've been so concerned about a patch that you look at zuuls console...01:19
stevemari've done that more than i care to admit01:20
morganfainbergstevemar, nah, i watch ZuulTV on a regular basis01:20
stevemar:P01:20
morganfainbergbetter than sitcoms most of the time.  the characters have more depth and are more believable01:20
morganfainbergi feel an investment in them that i don't get from cable tv01:20
morganfainbergit may have something to do with not having TV service though...01:21
*** dstanek is now known as dstanek_zzz01:24
*** dstanek_zzz is now known as dstanek01:24
openstackgerritA change was merged to openstack/python-keystoneclient: auth_token hashes PKI token once  https://review.openstack.org/9249901:26
*** zhiyan_ is now known as zhiyan01:26
*** gokrokve_ has quit IRC01:27
*** gokrokve has joined #openstack-keystone01:28
*** gokrokve has quit IRC01:32
openstackgerritayoung proposed a change to openstack/keystone-specs: create python-keystonemiddleware repo  https://review.openstack.org/9598701:35
*** browne has joined #openstack-keystone01:35
morganfainbergayoung, i think you need to link into the docs and index.rst in that review as well01:37
ayoungmorganfainberg, probably.01:37
morganfainbergayoung, i'll 2x check and update if its needed.01:38
ayoung++01:38
ayoungmorganfainberg, I 'm on it01:39
morganfainbergayoung, ok works for me01:40
openstackgerritMorgan Fainberg proposed a change to openstack/python-keystoneclient: auth_token middleware hashes tokens with configurable algorithm  https://review.openstack.org/8039801:40
openstackgerritayoung proposed a change to openstack/keystone-specs: create python-keystonemiddleware repo  https://review.openstack.org/9598701:40
ayoungmorganfainberg, BTW, I want to put policy in there as well, although, strictly speaking, policy is not going to be a middleware, it is going to be code called by middleware01:42
morganfainbergayoung, thats fine.01:42
morganfainbergayoung, it's server specific code really01:42
ayoungyeah01:42
morganfainbergnot cli/utility01:42
morganfainbergayoung, btw, you had copy/pasta error in your change keystoneclient != middleware in index.rst01:43
ayoungah01:43
morganfainbergi also expect us to release middleware like client, independant of major releases (e.g. juno)01:43
openstackgerritayoung proposed a change to openstack/keystone-specs: create python-keystonemiddleware repo  https://review.openstack.org/9598701:44
ayoung++01:44
lbragstadwoo! another spec up01:44
morganfainberglbragstad, i think i have 4 more to propose (myself)01:44
lbragstad++01:44
morganfainberglbragstad, gonna be a busy cycle01:45
*** RockKuo_Office has joined #openstack-keystone01:45
morganfainbergand we need to figue out the combining of identity-api with keystone-specs01:45
morganfainbergnot sure how we'll handle that.01:46
*** devkulkarni has joined #openstack-keystone01:47
lbragstadmorganfainberg: how come we could just include it in the  keystone-specs tree?01:48
* lbragstad rookie question? 01:49
*** devkulkarni1 has joined #openstack-keystone01:49
morganfainberglbragstad, we could. it's a question of converting it? do we want it separate from the specs themselves... what if a spec doesn't get implemented and we chuck it next cycle -- do we revert the changes to identity-api docs?01:49
lbragstaddidn't dolphm have the idea of generating the docs from the specs?01:50
*** devkulkarni has quit IRC01:51
*** devkulkarni has joined #openstack-keystone01:51
morganfainberglbragstad, sure.. still need to get from here to there ;)01:52
stevemarmorganfainberg, lbragstad i think the two will have to live separately for now01:53
morganfainberglbragstad, totally doable, just need to decide what we want.01:53
lbragstadmorganfainberg: so for that, we would need to provide some specs to cover what already exists.01:53
lbragstador something to convert,01:53
*** devkulkarni1 has quit IRC01:54
morganfainberglbragstad, yea01:56
*** morazi has quit IRC01:57
openstackgerritayoung proposed a change to openstack/keystone: Kerberos as method name  https://review.openstack.org/9598901:58
*** browne has quit IRC01:59
openstackgerritLance Bragstad proposed a change to openstack/keystone-specs: Purpose keystone-api-validation blueprint  https://review.openstack.org/9595702:01
*** mberlin has quit IRC02:03
*** mberlin has joined #openstack-keystone02:04
*** raildo has quit IRC02:09
*** diegows has quit IRC02:09
*** htruta has quit IRC02:09
*** tellesnobrega has quit IRC02:09
*** gabriel-bezerra has quit IRC02:09
*** rodrigods has quit IRC02:09
*** xianghui has joined #openstack-keystone02:22
*** lbragstad has quit IRC02:33
*** lbragstad has joined #openstack-keystone02:33
*** BAKfr has quit IRC02:33
*** htruta has joined #openstack-keystone02:36
*** zhiyan is now known as zhiyan_02:36
*** browne has joined #openstack-keystone02:36
*** rodrigods has joined #openstack-keystone02:38
*** browne has quit IRC02:38
*** ncoghlan_afk is now known as ncoghlan02:39
*** raildo has joined #openstack-keystone02:43
*** tellesnobrega has joined #openstack-keystone02:45
*** gyee has quit IRC02:49
*** zhiyan_ is now known as zhiyan02:49
*** dims has quit IRC02:57
*** mberlin has quit IRC03:04
*** ncoghlan is now known as ncoghlan_afk03:09
*** devkulkarni has quit IRC03:10
*** gabriel-bezerra has joined #openstack-keystone03:10
*** Camisa has joined #openstack-keystone03:12
*** Camisa has joined #openstack-keystone03:12
openstackgerritA change was merged to openstack/python-keystoneclient: Add description param to v3 service create/update  https://review.openstack.org/7977403:13
openstackgerritMatt Fischer proposed a change to openstack/python-keystoneclient: Add support for extensions-list  https://review.openstack.org/9297803:16
*** harlowja_ is now known as harlowja_away03:16
*** mberlin has joined #openstack-keystone03:16
*** zhiyan is now known as zhiyan_03:16
*** shakamunyi has joined #openstack-keystone03:19
*** dstanek is now known as dstanek_zzz03:21
*** zhiyan_ is now known as zhiyan03:22
*** shakamunyi has quit IRC03:24
openstackgerritayoung proposed a change to openstack/keystone: compressed tokens  https://review.openstack.org/7132503:32
openstackgerritJamie Lennox proposed a change to openstack/python-keystoneclient: Add endpoint handling to Token/Endpoint auth  https://review.openstack.org/9121603:36
*** zhiyan is now known as zhiyan_03:41
openstackgerritA change was merged to openstack/keystone: replaced unicode() with six.text_type()  https://review.openstack.org/9546903:41
openstackgerritA change was merged to openstack/keystone: no one uses macports  https://review.openstack.org/9013703:41
openstackgerritA change was merged to openstack/keystone: indicate that sensitive messages can be disabled  https://review.openstack.org/9487103:50
openstackgerritA change was merged to openstack/python-keystoneclient: Add /role_assignments endpoint support  https://review.openstack.org/9157803:51
*** shakamunyi has joined #openstack-keystone03:52
openstackgerritJamie Lennox proposed a change to openstack/python-keystoneclient: Convert auth_token to use session  https://review.openstack.org/7490803:58
*** david-lyle has joined #openstack-keystone04:01
*** afazekas has quit IRC04:12
*** zhiyan_ is now known as zhiyan04:13
*** ukalifon has joined #openstack-keystone04:24
*** marcoemorais has joined #openstack-keystone04:25
*** shakamunyi has quit IRC04:27
*** praneshp has joined #openstack-keystone04:27
*** praneshp_ has joined #openstack-keystone04:29
*** praneshp has quit IRC04:32
*** praneshp_ is now known as praneshp04:32
*** marcoemorais1 has joined #openstack-keystone04:32
*** marcoemorais has quit IRC04:34
*** afazekas has joined #openstack-keystone04:40
*** ajayaa has joined #openstack-keystone04:48
*** bvandenh has joined #openstack-keystone04:50
*** shakamunyi has joined #openstack-keystone04:52
*** zhiyan is now known as zhiyan_04:53
openstackgerritSteve Martinelli proposed a change to openstack/keystone: Add openID Connect auth plugin for federation  https://review.openstack.org/6166204:53
*** shakamunyi has quit IRC04:54
*** david-lyle has quit IRC04:57
*** david-lyle has joined #openstack-keystone04:57
*** ukalifon has quit IRC05:01
*** david-lyle has quit IRC05:02
*** ncoghlan_afk is now known as ncoghlan05:02
openstackgerritA change was merged to openstack/python-keystoneclient: auth_token middleware hashes tokens with configurable algorithm  https://review.openstack.org/8039805:11
*** shakamunyi has joined #openstack-keystone05:12
*** bvandenh has quit IRC05:12
*** zhiyan_ is now known as zhiyan05:14
*** zhiyan is now known as zhiyan_05:25
*** askb has joined #openstack-keystone05:35
*** shakamunyi has quit IRC05:40
*** shakamunyi has joined #openstack-keystone05:40
*** askb has quit IRC05:49
*** stevemar has quit IRC05:53
openstackgerritOpenStack Proposal Bot proposed a change to openstack/keystone: Imported Translations from Transifex  https://review.openstack.org/9028806:00
*** chen has joined #openstack-keystone06:06
openstackgerritguang-yee proposed a change to openstack/keystone: Make sure scoping to the project of a disabled domain result in 401.  https://review.openstack.org/9425106:09
*** jaosorior has joined #openstack-keystone06:15
*** ukalifon has joined #openstack-keystone06:16
openstackgerritJuan Antonio Osorio Robles proposed a change to openstack/keystone: Refactor driver_hints  https://review.openstack.org/9399206:17
*** tomoiaga has joined #openstack-keystone06:25
*** boris-42 has quit IRC06:45
*** ncoghlan is now known as ncoghlan_afk06:46
*** ncoghlan_afk is now known as ncoghlan06:49
*** boris-42 has joined #openstack-keystone06:50
*** praneshp has quit IRC06:59
*** BAKfr has joined #openstack-keystone07:15
*** zhiyan_ is now known as zhiyan07:18
openstackgerritLi Ma proposed a change to openstack/keystone: Password trunction makes password insecure  https://review.openstack.org/7732507:25
*** zhiyan is now known as zhiyan_07:27
openstackgerritMarek Denis proposed a change to openstack/keystone: Suggest users to remove REMOTE_USER from shibd conf  https://review.openstack.org/9393607:34
*** bvandenh has joined #openstack-keystone07:41
*** ncoghlan has quit IRC08:10
*** andreaf has joined #openstack-keystone08:18
*** shakamunyi has quit IRC08:43
openstackgerrithenry-nash proposed a change to openstack/keystone: multi-backend support for identity  https://review.openstack.org/7421409:01
*** ByteSore_ is now known as ByteSore09:05
*** marcoemorais1 has quit IRC09:07
*** zhiyan_ is now known as zhiyan09:19
*** zhiyan is now known as zhiyan_09:28
openstackgerritRodrigo Duarte Sousa proposed a change to openstack/python-keystoneclient: Fix attributes ordering at v3/client.py  https://review.openstack.org/9611309:29
openstackgerritRodrigo Duarte Sousa proposed a change to openstack/python-keystoneclient: Fix attributes ordering at v3/client.py  https://review.openstack.org/9611309:31
*** zhiyan_ is now known as zhiyan09:37
*** shakamunyi has joined #openstack-keystone09:40
*** shakamunyi has quit IRC09:45
*** zhiyan is now known as zhiyan_09:46
*** fmarco76 has joined #openstack-keystone09:48
*** fmarco76 has quit IRC09:49
*** boris-42 has quit IRC09:52
*** rodrigods_ has quit IRC09:54
*** boris-42 has joined #openstack-keystone10:16
*** diegows has joined #openstack-keystone10:53
*** yfujioka has joined #openstack-keystone11:20
yfujiokahello11:21
yfujiokaI want to try Keystone v3 api11:22
yfujiokaI tried set IDENTITY_API_VERSION=3 in localrc, but stack.sh is failing.11:24
*** rdxc has joined #openstack-keystone11:28
*** rdxc has left #openstack-keystone11:29
*** RockKuo_Office has quit IRC11:32
*** dims has joined #openstack-keystone11:45
*** dstanek_zzz is now known as dstanek11:47
*** xianghui has quit IRC11:49
gabriel-bezerraMight you take a look at? https://bugs.launchpad.net/keystone/+bug/132014011:59
uvirtbotLaunchpad bug 1320140 in keystone "Federation documentation is not clear about mapping.rules.local.user.name" [Undecided,New]11:59
*** roby_ has joined #openstack-keystone12:12
*** dims has quit IRC12:39
*** hrybacki has joined #openstack-keystone12:40
*** andreaf has quit IRC12:41
*** erecio has joined #openstack-keystone12:41
*** mberlin has quit IRC12:42
*** gordc has joined #openstack-keystone12:44
dolphmbknudson: i see you've been on a code review rampage lol12:44
openstackgerritJuan Antonio Osorio Robles proposed a change to openstack/keystone: Refactor tests regarding required attributes  https://review.openstack.org/9253512:49
*** mberlin has joined #openstack-keystone12:54
bknudsondolphm: rampage is the right word. is there any other way?12:57
*** stevemar has joined #openstack-keystone13:06
*** dstanek is now known as dstanek_zzz13:07
*** bknudson has quit IRC13:11
*** hrybacki has quit IRC13:12
*** dstanek_zzz is now known as dstanek13:23
*** yfujioka has quit IRC13:23
rodrigodsstevemar, there?13:25
dstanekdolphm: is there any official guidance for setting the importance on bugs?13:30
dstanekdolphm: or is just experience and intuition?13:31
dolphmdstanek: i set priority mostly based on impact compared to most common use cases / code paths13:31
dolphmdstanek: and use low for bugs just looking for a refactor, since it's confusing to see them appear as wishlist in 6 months13:32
dolphmdstanek: there is some guidance in a wiki somewhere, but there's not any surprises - so yes, use your intuition :)13:33
*** bknudson has joined #openstack-keystone13:34
dstanekthere are some new/undecided bugs that haven't been commented on in months - i was going to set them to low (unless they are a security issue)13:34
dstaneki have now officially flipped through att 261 open bugs at least once! i feel accomplished13:35
dolphmdstanek: and closed 2 :D13:35
stevemarrodrigods, i am now, good morning13:36
dstanekdolphm: that was easy - the work had long been done13:36
dstanekfollowing all of the crazy discussions was the challenge13:36
stevemardstanek, the discussions can be a bit crazy13:37
dstaneki think there are actually a few more we fixed months ago, but i need to check on that13:37
dolphmdstanek: so you said you were going to go close all the stale blueprints next, right?13:37
stevemarthat shouldn't be as bad13:37
* dstanek slowly and quietly backs out of the room13:38
dstanekdolphm: how do you know if they are stale? just haven't been updated in a long time?13:38
dolphmdstanek: you know you want to click me! blueprints.launchpad.net/keystone/13:38
rodrigodsstevemar, gm =)13:38
dolphmalthough it'd be easier if it was a valid link https://blueprints.launchpad.net/keystone/13:39
rodrigodsstevemar, the v3 client ordering stuff: https://review.openstack.org/#/c/96113/13:39
dolphmdstanek: i suppose? especially if there's no assignee / no progress against it13:39
dolphmdstanek: we probably need to do something to indicate all the ones that do not have approved -specs soon13:39
*** gokrokve has joined #openstack-keystone13:40
stevemarrodrigods, ohh ty13:40
dstanekdolphm: i'll start poking a blueprints in a bit - i have a list of bugs to follow up on13:40
stevemarrodrigods, +2 / +A13:40
*** gokrokve has quit IRC13:40
*** gokrokve has joined #openstack-keystone13:41
rodrigodsstevemar, great!13:41
dstanekdolphm: i also have to finish drafting my service scoped token spec13:41
*** afaranha has joined #openstack-keystone13:41
dolphmdstanek: both of those are more important than cleaning up bp's :)13:42
dstanekdolphm: morganfainberg: ayoung: bknudson: stevemar: all: before i forget again - i'm out next week so you'll see much less of me and it may take me a while to respond to things13:43
dolphmdstanek: ack13:43
bknudsondstanek: do you have time to update https://review.openstack.org/#/c/91825/ ?13:44
bknudsonseems like oslo syncs are blocked since this is not making progress13:44
*** rodrigods_ has joined #openstack-keystone13:45
dstanekbknudson: sure, i can do it right now13:45
*** rodrigods_ has quit IRC13:48
dstanekbknudson: is it OK to update the config and sync all in one commit?13:49
bknudsondstanek: I don't have a problem with that.13:50
*** afaranha has left #openstack-keystone13:50
bknudsonit could also be done separately if you think that they're not related.13:51
openstackgerritLance Bragstad proposed a change to openstack/keystone-specs: Purpose keystone-api-validation blueprint  https://review.openstack.org/9595713:51
bknudsonfor example if the config update has no effect on what modules are used then it could be separate13:51
*** andreaf has joined #openstack-keystone13:52
*** shakamunyi has joined #openstack-keystone13:54
*** rodrigods_ has joined #openstack-keystone13:55
*** gokrokve has quit IRC13:58
dstanekbknudson: do you just manually look for the last olso sync in keystone so you can get the list of changes?13:58
*** gokrokve has joined #openstack-keystone13:58
bknudsondstanek: yes, a git log in keystone/openstack/common should show it13:58
*** bvandenh has quit IRC13:59
openstackgerritLance Bragstad proposed a change to openstack/keystone-specs: Purpose keystone-api-validation blueprint  https://review.openstack.org/9595713:59
*** gokrokve has quit IRC14:02
*** rodrigods_ has quit IRC14:03
*** hrybacki has joined #openstack-keystone14:03
dstanekbknudson: running 'git log --oneline 74ae271..HEAD | grep -v Merge' in olso give 42 changes :-)14:03
bknudsondstanek: --no-merges14:03
bknudsondstanek: also, add the files that were updated so you only get the changes for those files.14:04
dstanekbknudson: ah, that's a good tip14:04
bknudsone.g., git log --no-merges --oneline 74ae271..HEAD -- openstack/common/gettextutils.py14:04
raildodolphm: I want to resolve this bug: https://bugs.launchpad.net/keystone/+bug/129473514:07
uvirtbotLaunchpad bug 1294735 in keystone "Disable domain doesn't disable users in the domain" [Medium,Triaged]14:07
raildodolphm: the error occurs at this point https://github.com/openstack/keystone/blob/master/keystone/assignment/core.py#L316 ,  correct?14:08
*** roby_ has quit IRC14:10
*** gokrokve has joined #openstack-keystone14:12
*** dstanek is now known as dstanek_zzz14:19
htrutastevemar, dtroyer, could you review my patch? https://review.openstack.org/#/c/91634/9  It was depending on the rodrigods patch that was merged today.14:19
stevemarhtruta, will keep it open in a tab :)14:21
openstackgerritKévin Bernard-Allies proposed a change to openstack/keystone: Fixes duplicated DELETE queries on SQL backends  https://review.openstack.org/9617314:23
*** david-lyle has joined #openstack-keystone14:24
htrutahtruta that's something. thanks14:24
htrutastevemar: that's something. thanks14:24
openstackgerritKévin Bernard-Allies proposed a change to openstack/keystone: Fixes duplicated DELETE queries on SQL backends  https://review.openstack.org/9617314:26
dolphmraildo: yes; a handler in the identity driver should subscribe to that notification14:26
dolphmraildo: that method should also emit a token revocation event if it doesn't already14:27
dolphm(i believe it does, but you could ensure it's tested)14:27
*** ukalifon has quit IRC14:27
*** ajayaa has quit IRC14:29
*** dstanek_zzz is now known as dstanek14:34
raildodolphm: Ok, I will investigate this and I think I'll sign in the bug. Thank you.14:36
dolphmraildo: thanks!14:36
dstanekbknudson: git log --oneline 74ae271.. -- `(cd ../keystone; git diff-tree --no-commit-id --name-only -r HEAD | egrep '/openstack/|^tools' | sed -e 's/^keystone\///')`14:37
dstanekugg...deleted the --no-merge14:38
bknudsondstanek: nice!14:38
dstanekbknudson: if you follow your instructions verbatim the README gets deleted14:42
bknudsondstanek: I always restore it14:43
*** thedodd has joined #openstack-keystone14:48
openstackgerritDavid Stanek proposed a change to openstack/keystone: Cleanup openstack-common.conf and sync from olso  https://review.openstack.org/9182514:49
openstackgerritLance Bragstad proposed a change to openstack/keystone-specs: Purpose keystone-api-validation blueprint  https://review.openstack.org/9595714:52
*** thedodd has quit IRC14:53
*** thedodd has joined #openstack-keystone14:53
*** diegows has quit IRC14:55
*** gokrokve has quit IRC14:56
*** gokrokve has joined #openstack-keystone14:57
*** radez_g0n3 is now known as radez14:58
BAKfrsomeone know if there is a way to manage keystone clients apps ?14:58
BAKfrI would block clients if user has not explicitly allowed it.14:59
*** gokrokve has quit IRC15:02
dstanekBAKfr: why are you looking to do that?15:06
BAKfrstanek, I would let developers making new client application using my storage services.15:10
*** afazekas has quit IRC15:10
BAKfrBut i should possibly block an application (for security purpose)15:12
stevemarhtruta, fastest turn around time ever15:13
morganfainbergmornin15:14
dstanekBAKfr: i think you'll have to implement something to do that15:14
dstanekmorganfainberg: morning15:15
BAKfrdstanek, I think so.15:15
*** shakamunyi has quit IRC15:16
dstanekBAKfr: sounds like you need a way for developers to register an application and a way for you to verify that at request time15:16
morganfainbergdolphm, i'm going to do one more pass on ksc patches before we should be good for a release, but I think we got the big ones in.15:16
dolphmmorganfainberg: sounds good, let me know when you need me15:17
BAKfrI was hoping for a fast and simple solution I hadn't seen yet.15:17
BAKfrdstanek, Ideally, OAuth2 would have been perfect for me15:18
dstanekBAKfr: i know commercial companies that provide that kind of service, but no open source that i have seen15:18
*** shakamunyi has joined #openstack-keystone15:18
*** tomoiaga has quit IRC15:21
BAKfrdstanek, I am in a commercial company who wants  provide that kind of service ^^15:23
BAKfr But i don't think i'm the only one which be interested by that.15:23
dstanekBAKfr: what company?15:24
BAKfrdstanek, very small french company, Bajoo15:24
*** devkulkarni has joined #openstack-keystone15:24
BAKfrdstanek,  http://www.bajoo.fr/en/15:25
BAKfrour actual implementation was done quickly, with git, without any plan to scale.15:27
BAKfrand now we've started to migrate to Keystone + Swift15:28
morganfainbergdolphm, https://bugs.launchpad.net/keystone/+bug/1253482 comment came up earlier, this doesn't really seem "critical" (only because it's slow fix and _mostly_ mitigated, documentation part was critical). the only real fix is abandoning 35357 and/or moving to shared 80/443 via apache15:29
uvirtbotLaunchpad bug 1253482 in devstack "Keystone's IANA-assigned default port in linux local ephemeral port range" [Undecided,In progress]15:29
morganfainbergdolphm, possibly move to high or medium now?15:29
morganfainbergcc dstanek, ^15:29
dolphmmorganfainberg: *shrug* i had it as Won't Fix15:30
morganfainbergah i'm ok with that as well.15:30
dolphmi guess we could still do the doc fix approach15:31
*** gokrokve has joined #openstack-keystone15:31
dolphmmorganfainberg: https://review.openstack.org/#/c/58013/15:32
openstackgerritDolph Mathews proposed a change to openstack/keystone: recommend excluding 35357 from ephemeral ports  https://review.openstack.org/5801315:32
BAKfrdstanek, btw, I think to implement OAuth2 in Keystone myself15:32
dolphmstevemar: ^15:32
morganfainbergdolphm, ah yep i +2'd that already15:32
stevemarBAKfr, use oauth1 :P15:33
morganfainbergdolphm, works for me.15:33
stevemarBAKfr, what was the use case again? I think we spoke on monday?15:33
stevemarBAKfr, tbh, i wouldn't mind seeing oauth2 in keystone either :)15:34
BAKfrstevemar, let user to allow (or not) client apps.15:34
BAKfrstevemar, and for we (admin), be able to block a client app15:36
*** devkulkarni has left #openstack-keystone15:36
stevemarBAKfr, might be worth looking at: https://github.com/openstack/identity-api/blob/master/v3/src/markdown/identity-api-v3-os-oauth1-ext.md15:36
stevemarit's a way for an admin (or user) to delegate some role(s) on a specific project to a consumer (client)15:38
stevemarit can then delete the consumer, or revoke it's access15:38
BAKfrstevemar, I've seen it, but it seems to me that OAuth1 supports only website ?15:40
dstanekdolphm: restoring that patch?15:40
morganfainbergdolphm, ok another pass on ksc, i think we should wait for the gating changes to clear https://review.openstack.org/#/c/92978/ [currently gating]15:41
dolphmdstanek: which one?15:41
morganfainbergdolphm, but that should be ~1h or so.15:41
dolphmdstanek: 58013?15:41
dstanekdolphm: yes, why bring it back? or rather...why was it abandoned?15:43
stevemarBAKfr, i don't think so15:44
dolphmdstanek: it was abandoned because it's a less-than-ideal solution, but i still think it's the best step forward that we have15:44
dolphmdstanek: short of dropping 35357 which might break hardcoded clients15:45
dstanekdolphm: i'm totally fine with it...just having trouble with Apsu's comment15:46
stevemarBAKfr, there is code in keystoneclient to show how to create request/access tokens using a python library (oauthlib)15:46
*** gyee has joined #openstack-keystone15:46
dstanekstevemar: BAKfr: all the cool flows were added in oauth215:46
stevemardstanek, and all the cool vulnerabilities too15:47
morganfainbergstevemar, you're stealing bknudson's comments! :P15:47
stevemarmorganfainberg, haha, that was rather bknudson inspired - wasn't it? I'm learning well15:48
morganfainbergstevemar, yep15:48
dstanekstevemar: i lost interest in oauth2 when one of the main authors left and blogged about its issues15:48
stevemardstanek, that blog was glorious15:48
BAKfrdstanek, stevemar , I've read this post and looked at OAuth215:50
BAKfrthat not as dirty as he say15:51
dstanekone of my favorite quotes ever: "They say the road to hell is paved with good intentions. Well, that’s OAuth 2.0."15:51
dolphmOAuth 3.0 for Humans15:52
dolphm%s/v1.0/v3.0/15:52
stevemarBAKfr, yeah, it's not bad, openID Connect is based off of it, and that looks sweet15:54
*** afazekas has joined #openstack-keystone15:54
*** gokrokve has quit IRC15:58
*** gokrokve has joined #openstack-keystone15:58
BAKfrOAuth2 throws out some good part of OAuth1, like encryption, but covers more use cases.15:59
BAKfrBy the way, I guess I've no easy way to do what I want with OAuth1 :(16:00
*** marcoemorais has joined #openstack-keystone16:00
arunkantall, can anybody provide me the guideline on how to add ldappool dependency (not present in global requirements) for https://review.openstack.org/#/c/95300/.16:01
BAKfrSo I think  I'll try to implement OAuth216:01
*** gokrokve has quit IRC16:03
dolphmstevemar: marekd|away: https://review.openstack.org/#/c/96173/16:04
*** marcoemorais has quit IRC16:04
*** marcoemorais has joined #openstack-keystone16:04
BAKfrdolphm, thanks for the review :)16:05
*** marcoemorais has quit IRC16:05
*** marcoemorais has joined #openstack-keystone16:05
dolphmBAKfr: good catch - i bet we've all read that code before and glossed straight over it!16:06
morganfainbergdolphm, ++16:06
*** marcoemorais has quit IRC16:07
*** marcoemorais has joined #openstack-keystone16:07
morganfainbergdolphm, though (*nitpicky hat on*) isn't the query(Region).filter_by(id=region).delete() more efficient?16:08
dolphmmorganfainberg: why?16:08
dolphmmorganfainberg: they emit the same sql16:08
dolphmmorganfainberg: or should, no?16:08
morganfainbergdolphm, i belive it saves a select16:08
morganfainbergdolphm, we do the select at line 12616:08
morganfainbergwhich is ... superfluous except to validate the region exists?16:09
dolphmmorganfainberg: ah - you could eliminate the select16:09
dolphmmorganfainberg: yep16:09
morganfainbergat this point in the code, my guess is we don't care if the region really exists. the delete (obviously) doesn't raise an exception16:09
gabriel-bezerraRedHat folks, does httpd works in RHEL the same way as in Fedora? dtroyer asked me that in https://review.openstack.org/9597716:10
morganfainbergthis is pretty damn nit-picky, i think this is not a common operation16:10
morganfainbergdolphm, i'm fine with leaving it as is though.16:11
bknudsondstanek: I got different results when I synced oslo-incubator caed79d16:11
BAKfrmorganfainberg, dolphm   I've seen code to use similar select queries for checking if an entity exists, so I've kept it16:11
dolphmBAKfr: understood; the more efficient way would be to issue the delete and raise a 404 if no rows were affected16:12
morganfainbergdolphm, i wonder....16:12
morganfainbergdolphm, ah we rely on _get_region to raise the 40416:13
morganfainbergdolphm, lets not do a massive restructure for a minor gain16:13
morganfainbergBAKfr, no need to change it, this is the least amount of change - my suggestion for efficiency would be a much larger restructure i think.16:14
gabriel-bezerraayoung, jamielennox ^16:14
dolphmmorganfainberg: ++16:14
dstanekbknudson: really?16:15
bknudsondstanek: https://review.openstack.org/#/c/91825/2/keystone/openstack/common/gettextutils.py16:16
bknudson_translators = TranslatorFactory('keystone') -> _translators = TranslatorFactory('oslo')16:16
bknudsonthat's actually going to break translation16:16
bknudsondomain='keystone', *args): -> domain='oslo', *args):16:16
dstanekbknudson: hmm...i wonder what happened there...i'll try that again16:16
morganfainbergbknudson, looks like the magic {"rename oslo to {project}" code failed?16:16
bknudsonright, the update.py script is supposed to change `oslo` to `keystone`16:17
bknudsondstanek: I wasn't using the in-review oslo-incubator update.py change, just the plain one.16:17
ayounggabriel-bezerra, yes, although RHEL is always going to trail Fedora, so it won't be identical16:23
dstanekbknudson: it's odd too because there was a systemd change that i didn't have16:23
ayoungmorganfainberg, I would love to get the Kerberos patch in for client, but understand if we are not ready for it16:24
BAKfrIf someone wants another small patch to review: https://review.openstack.org/#/c/95212/  :)16:25
morganfainbergayoung, i'm not sure about the consensus on it.16:25
gabriel-bezerraayoung: so is it valid that RHEL6 enables all sites whose config file ends with .conf in /etc/httpd/conf.d? Can we use the way to configure Fedora there as well?16:25
openstackgerritA change was merged to openstack/python-keystoneclient: Fix attributes ordering at v3/client.py  https://review.openstack.org/9611316:25
openstackgerritA change was merged to openstack/python-keystoneclient: Authenticate via oauth  https://review.openstack.org/8198116:25
ayounggabriel-bezerra, "sites"  meaning entries in /etc/httpd/conf.d?16:26
gabriel-bezerrayes16:26
openstackgerritDavid Stanek proposed a change to openstack/keystone: Cleanup openstack-common.conf and sync from olso  https://review.openstack.org/9182516:26
dstanekbknudson: see if that matches ^16:26
gabriel-bezerraayoung: can you take a look at the review?16:26
ayoungwill do16:26
bknudsondstanek: no diffs this time.16:27
dstanekbknudson: i have to idea what happened...i just ran the same commands from my zsh history16:28
bknudsondstanek: switch to bash16:28
dstanekbknudson: can't downgrade now - i'm too invested16:30
ayounggabriel-bezerra, this is for Devstack, right?  Pretty sure that the CentOS code there has bit rotted.16:31
morganfainbergayoung, ++16:31
morganfainbergayoung, re: reberos, i just ran a recheck on it now that global reqs has requests-kerb16:31
morganfainbergayoung, there are a couple nice-to-have patches we could hold for the client if we could get them in, but i haven't seen commitment on that kerb patch being ready, cc jamielennox16:32
gabriel-bezerraayoung: yes, it is for DevStack...16:32
morganfainbergayoung, but i'm really eager to get compression in as well (which is ready to go)16:32
gabriel-bezerraayoung: You mean that it won't work on CentOS after the patch?16:33
morganfainberggabriel-bezerra, meaning it may not work on centos before the patch16:33
openstackgerritLance Bragstad proposed a change to openstack/keystone-specs: Purpose keystone-api-validation blueprint  https://review.openstack.org/9595716:35
*** ozialien has quit IRC16:46
morganfainbergayoung, are you ok with waiting until the next ksc release for krb plugin? or should we push on getting that reviewed today prior to making the cut?16:47
morganfainbergayoung, also https://jenkins05.openstack.org/job/gate-python-keystoneclient-python33/251/console py33 pip isn't happy (at the moment) with the kerberos 1.1 requirement16:50
ayoungmorganfainberg, yeah, I'm OK with waiting16:50
ayoungI can work out of dev16:50
morganfainbergayoung ok.16:50
*** gokrokve has joined #openstack-keystone16:51
ayoungmorganfainberg, that was the problem jamielennox saw...some sort of pip versioning issue.  Not sure how the requests-kerberos change passed local tox16:51
ayounggabriel-bezerra, is the question of whether we can make elif is_fedora;   into something more inclusive?16:52
ayounggabriel-bezerra, you should be able to test that on CentOS16:52
openstackgerritA change was merged to openstack/python-keystoneclient: Add support for extensions-list  https://review.openstack.org/9297816:57
openstackgerritA change was merged to openstack/python-keystoneclient: Mark keystoneclient as being a universal wheel  https://review.openstack.org/9405016:57
*** browne has joined #openstack-keystone16:58
*** BAKfr has quit IRC16:59
morganfainbergdolphm, ^ I think we're good for cutting a ksc release unless anyone has a solid reason not to17:00
*** Camisa has quit IRC17:02
*** praneshp has joined #openstack-keystone17:06
*** andreaf has quit IRC17:06
*** bvandenh has joined #openstack-keystone17:06
*** diegows has joined #openstack-keystone17:07
*** bvandenh has quit IRC17:08
*** andreaf has joined #openstack-keystone17:10
*** andreaf has quit IRC17:10
*** andreaf has joined #openstack-keystone17:12
*** sbfox has joined #openstack-keystone17:14
*** harlowja_away is now known as harlowja_17:16
*** marcoemorais has quit IRC17:16
*** marcoemorais has joined #openstack-keystone17:16
*** dstanek is now known as dstanek_zzz17:17
*** diegows has quit IRC17:18
htrutasetevemar: hahaha.17:23
htrutastevemar: could you review it again? https://review.openstack.org/#/c/91634/17:23
stevemarhtruta, i don't review as fast as you code :(17:24
stevemari wanted to try it out too17:24
morganfainberganyone know if https://review.openstack.org/#/c/86025 author has been around? I don't think it would be bad to get that change in, but if not around someone else should pick it up.17:24
ayoungmorganfainberg, You might be interested in this little ditty about Kerberos http://adam.younglogic.com/2014/05/tgt-forwarding-and-cleanup/17:27
morganfainbergayoung, standard GSSAPI stuff17:27
morganfainbergayoung, yep, used that before :)17:27
htrutastevemar: take your time. hahaha17:28
ayoungmorganfainberg, I like the config option to scope it to a certain set of hosts17:32
morganfainbergayoung, ++17:32
*** praneshp has quit IRC17:32
morganfainbergayoung, very similar to some other stuff (proxy command) work i've had to do in the past17:33
morganfainbergok i need to go get either an early lunch or a late breakfast.17:34
morganfainbergbbib.17:34
*** praneshp has joined #openstack-keystone17:35
*** praneshp has quit IRC17:46
stevemarhtruta, 2 nits with the commit message! I would change them myself, but then it'll change the committer value too, (not sure if that steals credit)17:55
stevemarhtruta, hopefully dtroyer or thowe can have a look at it (https://review.openstack.org/#/c/91634/), but if they don't get back any time soon, i'll +A it17:56
*** ukalifon has joined #openstack-keystone17:58
openstackgerritArun Kant proposed a change to openstack/keystone: Adding support for ldap connection pooling.(Work-in-progress)  https://review.openstack.org/9530018:07
*** dstanek_zzz is now known as dstanek18:08
*** BAKfr has joined #openstack-keystone18:08
*** sbfox1 has joined #openstack-keystone18:09
*** jaosorior has quit IRC18:11
*** sbfox has quit IRC18:13
*** diegows has joined #openstack-keystone18:14
*** jamielennox is now known as jamielennox|away18:16
*** dstanek is now known as dstanek_zzz18:17
*** praneshp has joined #openstack-keystone18:21
*** marcoemorais has quit IRC18:33
*** marcoemorais has joined #openstack-keystone18:34
*** ukalifon has quit IRC18:35
*** dstanek_zzz is now known as dstanek18:37
htrutastevemar: wouldn't it be better if I just submit another patch changing the message?18:39
stevemarhtruta, that would be awesome18:39
*** rodrigods_ has joined #openstack-keystone18:42
*** gabriel-bezerra has quit IRC18:47
*** rodrigods_ has quit IRC18:48
openstackgerritA change was merged to openstack/keystone: Fixes duplicated DELETE queries on SQL backends  https://review.openstack.org/9617318:48
*** gabriel-bezerra has joined #openstack-keystone18:50
htrutastevemar: done! your +2 remains. hehe18:51
morganfainberghtruta, you can also change the commit message from the gerrit web interface now (as well)18:52
morganfainberghtruta, if you're the patch owner18:52
openstackgerritA change was merged to openstack/keystone: recommend excluding 35357 from ephemeral ports  https://review.openstack.org/5801318:53
gabriel-bezerraayoung: is_fedora checks for CentOS: [ "$os_VENDOR" = "Fedora" ] || [ "$os_VENDOR" = "Red Hat" ] || [ "$os_VENDOR" = "CentOS" ]18:54
htrutamorganfainberg: that's what I did! thanks18:55
ayounggabriel-bezerra, cool.  did you test your script on a Centos VM?  They are cheap to come by18:55
gabriel-bezerraayoung: Not yet.18:57
ayounggabriel-bezerra, trystack.org if you need it18:59
gabriel-bezerra:D19:00
gabriel-bezerraayoung: there are only ubuntu12.10, and a couple fedora images there. no centos.19:03
*** piousbox has joined #openstack-keystone19:03
ayoungbleh19:03
piousboxalo people19:04
piousboxI'm trying to use Cyberduck to connect to a swift storage service.19:04
morganfainbergayoung, gabriel-bezerra, maybe stacklet? (might cost $ though)19:04
piousboxCyberduck is asking the following: tenand ID, access key, secret key19:04
piousboxwhat are access key and secret key and how do I associate them with a tenant?19:04
piousboxThanks in advance19:04
ayoungmorganfainberg, CentOS is free,  and he could run in a kvm instance local.  Just download time19:04
morganfainbergayoung, right. i meant if you wanted a pre-canned vm ready to go (no install / etc for any hypervisor)19:05
*** bobt has joined #openstack-keystone19:05
ayoungmorganfainberg, I have one in my glance server I could send him19:05
morganfainbergayoung, hrm. doesn't RAX or HP offer some limited free vms to OS developers?19:06
ayoungmorganfainberg, ask the people that work there.  I have internal resources I use19:06
*** dims has joined #openstack-keystone19:07
openstackgerritA change was merged to openstack/keystone: Imported Translations from Transifex  https://review.openstack.org/9028819:11
morganfainbergdolphm, ^ RAX and VMs for OS contributors? cc dstanek (I remeber something about this a while ago)19:12
openstackgerritOpenStack Proposal Bot proposed a change to openstack/keystone: Updated from global requirements  https://review.openstack.org/9122519:14
ayounggabriel-bezerra, do you have and openstack instance ?  I can post a CentOS cloud image you can import19:14
gabriel-bezerraayoung: I've got one here19:16
gabriel-bezerrahtruta: got one for me :)19:16
gabriel-bezerraayoung: thx19:16
gabriel-bezerrawill just test that19:17
dstanekmorganfainberg: i thought so, but i'm not sure19:18
morganfainbergdstanek, yeah. :P19:19
* morganfainberg shrugs19:19
morganfainbergomg... coffee sooooo gooooooood19:19
dstanekmorganfainberg: i have some dev promo passes that give you $50 (i think) a month for 6 months - are you looking for a VM?19:19
morganfainbergnot me, was a convo ^19:19
morganfainbergabout testing some centos devstack-y-ness19:20
morganfainbergok i need to go buy a USB stick.19:20
morganfainbergturns out it's really hard to install an OS w/o one these days :P19:21
* morganfainberg wonders if tftpboot from mac -> other laptop would be viable19:21
gabriel-bezerraayoung: is it enough to test on CentOS 6.5 cloud img?19:22
morganfainberggabriel-bezerra, that should probably be sufficient19:23
gabriel-bezerraok, that's the one I have19:23
openstackgerritDolph Mathews proposed a change to openstack/keystone: Add v2 & v3 API documentation  https://review.openstack.org/9624219:39
*** sbfox1 has quit IRC19:42
stevemardolphm, giving us a history lesson eh ^19:46
*** diegows has quit IRC19:47
morganfainbergayoung, dolphm, ~300s is the accepted clock-skew minimum theoretical TTL for tokens right?19:47
morganfainbergayoung, dolphm, any reason we don't enforce a minimum if that is the case?19:48
ayoungmorganfainberg, um...I went by what we discussed for Kerberos, but no idea if that is a standard or anything19:48
ayoungI was treating it as a rule of thumb19:48
morganfainbergayoung, hrm. i'd be ok with enforcing a lower limit if its' accepted fro a similar construct19:49
ayoungmorganfainberg, do we need to enforce a minimum?19:50
*** diegows has joined #openstack-keystone19:52
morganfainbergayoung, depends on how much rope we want to give deployers19:57
morganfainbergayoung, not sure if we should... was thinking out loud mostly19:57
ayoungmorganfainberg, I'm not really worried about people setting it too low19:57
ayoungif we get there, we've won19:57
morganfainberghehe19:57
bknudsonall we give deployers is rope.19:57
*** piousbox has left #openstack-keystone19:57
morganfainbergbknudson, is it magic rope?19:57
morganfainbergcause i want magic rope19:58
morganfainberg:P19:58
openstackgerritA change was merged to openstack/keystone: Suggest users to remove REMOTE_USER from shibd conf  https://review.openstack.org/9393619:59
*** hrybacki has quit IRC20:03
morganfainbergok i really have to go run some errands, be back shortly20:03
dstanekstevemar: do you understand the difference between the existing design and chadwick's proposals? it seems more like incremental refactoring, but i don't understand the diagrams20:04
*** radez is now known as radez_g0n320:12
*** erecio has quit IRC20:12
gabriel-bezerraayoung: morganfainberg I'm getting this error on CentOS. Any idea of what it might be?20:14
gabriel-bezerra[client 10.1.0.69] (13)Permission denied: mod_wsgi (pid=32151): Unable to connect to WSGI daemon process 'keystone-admin' on '/etc/httpd/logs/wsgi.32074.0.2.sock' after multiple attempts.20:14
gabriel-bezerrain /var/log/httpd/keystone20:14
ayounggabriel-bezerra, SELinux?20:15
ayoungtry  sudo setenforce permissive20:16
*** hrybacki has joined #openstack-keystone20:18
stevemardstanek, sorta? it seems like a refactoring imo20:20
stevemardstanek, i don't see how the current design limits anything20:20
gabriel-bezerrasudo getenforce returned Permissive20:21
gabriel-bezerraayoung: ^20:21
stevemardstanek, for instance, he wrote that it'll allow for openID Connect, and keystone2keystone support. But i'm already doing one of those in the framework20:21
ayounggabriel-bezerra, strange20:21
dstanekstevemar: the only thing i don't like about the current design is the 'if' statement20:22
ayounggabriel-bezerra, is the wsgi process actually up and running?20:22
dstaneki should probably add that to my review commentary20:22
*** r-daneel has joined #openstack-keystone20:22
*** browne has quit IRC20:22
gabriel-bezerraayoung: sorry, I'll have to stack.sh again20:23
stevemardstanek, please do, since i don't know what you're talking about20:23
ayoungNP20:23
*** browne has joined #openstack-keystone20:24
dstanekstevemar: ha, ha - i say *the if* statement because there is one that just jumps out at me20:24
*** hrybacki has quit IRC20:29
gabriel-bezerraayoung: I have a bunch of httpd processes, how can i know which one is the right one?20:30
ayounggabriel-bezerra, wsgi process should be owned by a different user, and should be python processes.20:30
*** andreaf has quit IRC20:32
bknudsondo the specs get built and published somewhere?20:33
gabriel-bezerraayoung: there is only one python process running: tuned20:37
ayounglook in the httpd error log gabriel-bezerra20:38
openstackgerritBrant Knudson proposed a change to openstack/keystone-specs: Spec for V3 extension advertisement  https://review.openstack.org/9597320:40
*** radez_g0n3 is now known as radez20:40
*** hrybacki has joined #openstack-keystone20:42
gabriel-bezerraayoung: I was just looking there. These are the 3 lines that look more meaningful..20:43
gabriel-bezerraSELinux policy enabled; httpd running as context unconfined_u:system_r:httpd_t:s020:43
gabriel-bezerra[notice] suEXEC mechanism enabled (wrapper: /usr/sbin/suexec)20:44
ayounggabriel-bezerra, its not selinux if Permive20:44
gabriel-bezerra[notice] Apache/2.2.15 (Unix) DAV/2 mod_wsgi/3.2 Python/2.6.6 configured -- resuming normal operations20:44
openstackgerritSteve Martinelli proposed a change to openstack/python-keystoneclient: Sync with oslo-incubator caed79d  https://review.openstack.org/9584520:45
stevemarbknudson, i think i was referring to the wrong commit hash20:46
bknudsonstevemar: using that commit hash from oslo-incubator I don't see any diffs, so that must be in.20:48
bknudsonit20:48
stevemarbknudson, sadly this is my first oslo sync, thanks for catching the mistake20:49
bknudsonthat's a long list of commits -- does that include changes from files that weren't synced?20:49
stevemarbknudson, i believe so, no good?20:51
bknudsonstevemar: we just want the changes for the files that were changed...20:51
gabriel-bezerraayoung: I tried to setenforce permissive and run stack.sh, but the same happened20:51
bknudsonstevemar: like this: git log --oneline --no-merges 2640847..caed79d -- openstack/common/apiclient/auth.py openstack/common/apiclient/base.py20:52
bknudsonexcept all the files that were changed.20:52
ayounggabriel-bezerra, I'd have to debug it....not sure what is going on20:52
bknudsonstevemar: I only get 4 changes.20:53
gabriel-bezerraayoung: tcp        0      0 :::35357                    :::*                        LISTEN      25928/httpd20:53
gabriel-bezerratcp        0      0 :::5000                     :::*                        LISTEN      25928/httpd20:53
gabriel-bezerranestat shows taht20:53
openstackgerritOpenStack Proposal Bot proposed a change to openstack/keystone: Updated from global requirements  https://review.openstack.org/9122520:53
gabriel-bezerrathat*20:53
stevemarbknudson, okay, i see, let me try and make sure i get the same20:53
bknudsonstevemar: 4a777e5 18f2bc1 90ae24b 1173e4620:53
stevemaryep20:54
gabriel-bezerrabut curl http://localhost:5000/v2.0 returns a 50320:54
stevemarbknudson, i'll update20:54
openstackgerritSteve Martinelli proposed a change to openstack/python-keystoneclient: Sync with oslo-incubator caed79d  https://review.openstack.org/9584520:54
gabriel-bezerra[error] [client ::1] (13)Permission denied: mod_wsgi (pid=25940): Unable to connect to WSGI daemon process 'keystone-public' on '/etc/httpd/logs/wsgi.25928.0.1.sock' after multiple attempts.20:55
gabriel-bezerrabut I can see some wsgi.XXXXX.X.X.sock in /var/log/httpd/20:55
gabriel-bezerrathat is a log...20:56
gabriel-bezerra/var/log/keystone is owned by root.root and /var/log/wsgi.XXXX.sock by apache.root20:57
gabriel-bezerrasorry.. /etc/httpd/logs is a link and in there: keystone belongs to root:root and wsgi.XXXX.sock, to apache:root20:57
bknudsonstevemar: looks like your sync includes https://review.openstack.org/#/c/95697/20:58
bknudsonwhich Closes-Bug: 131412920:58
bknudsonso how about add that to your sync and then the other is abandoned20:59
openstackgerritOpenStack Proposal Bot proposed a change to openstack/python-keystoneclient: Updated from global requirements  https://review.openstack.org/9626521:00
openstackgerritLance Bragstad proposed a change to openstack/keystone: Implement validation on Catalog V3 resources  https://review.openstack.org/9626621:00
*** topol has joined #openstack-keystone21:01
openstackgerritLance Bragstad proposed a change to openstack/keystone: Initial implementation of validator  https://review.openstack.org/8648321:02
openstackgerritLance Bragstad proposed a change to openstack/keystone: Implement validation on Catalog V3 resources  https://review.openstack.org/9626621:02
openstackgerritLance Bragstad proposed a change to openstack/keystone: Implement validation on Assignment V3 resources  https://review.openstack.org/8648421:02
openstackgerritLance Bragstad proposed a change to openstack/keystone: Implement validation on Catalog V3 resources  https://review.openstack.org/9626621:06
*** browne has quit IRC21:06
*** marcoemorais has quit IRC21:07
*** marcoemorais has joined #openstack-keystone21:07
gabriel-bezerraayoung, morganfainberg: at least it was not a regression that my patch introduced21:09
openstackgerritOpenStack Proposal Bot proposed a change to openstack/keystone: Updated from global requirements  https://review.openstack.org/9122521:14
*** sbfox has joined #openstack-keystone21:14
*** gyee has quit IRC21:14
*** hrybacki has quit IRC21:18
*** marcoemorais1 has joined #openstack-keystone21:19
*** marcoemorais has quit IRC21:20
*** marcoemorais has joined #openstack-keystone21:21
*** browne has joined #openstack-keystone21:22
*** marcoemorais1 has quit IRC21:25
*** gokrokve has quit IRC21:26
*** gokrokve has joined #openstack-keystone21:27
*** gokrokve has quit IRC21:30
*** david-lyle has quit IRC21:35
*** diegows has quit IRC21:38
stevemarbknudson, was afk, cool, i'll mention: Closes-Bug: 1314129 in the message21:43
openstackgerritSteve Martinelli proposed a change to openstack/python-keystoneclient: Sync with oslo-incubator caed79d  https://review.openstack.org/9584521:45
*** topol has quit IRC21:47
openstackgerritLance Bragstad proposed a change to openstack/keystone: Initial implementation of validator  https://review.openstack.org/8648321:50
openstackgerritLance Bragstad proposed a change to openstack/keystone: Implement validation on Catalog V3 resources  https://review.openstack.org/9626621:50
openstackgerritLance Bragstad proposed a change to openstack/keystone: Implement validation on Assignment V3 resources  https://review.openstack.org/8648421:50
*** browne1 has joined #openstack-keystone21:52
*** browne has quit IRC21:52
stevemarlbragstad, posting a bunch of reviews!21:54
*** dhellmann is now known as dhellmann_21:58
*** dhellmann_ is now known as dhellmann21:58
*** dhellmann is now known as dhellmann_21:58
*** ekarlso has quit IRC22:01
morganfainberglbragstad, omg SPAM! :) nice.22:03
openstackgerritSteven Hardy proposed a change to openstack/python-keystoneclient: Enable forcing re-authentication for trust-scoped clients  https://review.openstack.org/9629822:04
openstackgerritBrant Knudson proposed a change to openstack/keystone-specs: Spec for V3 extension advertisement  https://review.openstack.org/9597322:10
*** raildo has quit IRC22:11
*** tellesnobrega has quit IRC22:12
*** rodrigods has quit IRC22:12
*** htruta has quit IRC22:12
*** gabriel-bezerra has quit IRC22:13
*** htruta has joined #openstack-keystone22:13
*** gabriel-bezerra has joined #openstack-keystone22:14
*** tellesnobrega has joined #openstack-keystone22:14
*** raildo has joined #openstack-keystone22:14
*** rodrigods has joined #openstack-keystone22:14
*** rodrigods has quit IRC22:15
*** rodrigods has joined #openstack-keystone22:15
*** afaranha has joined #openstack-keystone22:15
*** bknudson has quit IRC22:16
*** dstanek is now known as dstanek_zzz22:19
*** dstanek_zzz is now known as dstanek22:20
lbragstadstevemar: morganfainberg gettin' there22:23
*** r-daneel has quit IRC22:26
*** ekarlso has joined #openstack-keystone22:26
*** rodrigods_ has joined #openstack-keystone22:30
*** gordc has left #openstack-keystone22:30
*** browne1 has quit IRC22:30
*** sbfox has quit IRC22:33
*** thedodd has quit IRC22:35
*** marcoemorais has quit IRC22:36
*** marcoemorais has joined #openstack-keystone22:36
*** browne has joined #openstack-keystone22:36
morganfainbergstevemar, dstanek, https://bugs.launchpad.net/keystone/+bug/1324260 any complaints with that? i think that makes sense.22:39
uvirtbotLaunchpad bug 1324260 in keystone "Always migrate the the db for extensions instead of conditionally" [Medium,Triaged]22:39
stevemarmorganfainberg, i think it's a good idea22:40
morganfainbergstevemar, yeah always felt odd to migrate those schemas conditionally... bad ux for deployers "oh and don't forget to migrate the new extension"22:41
stevemarmorganfainberg, yeah, its a bit weird. is there anything we can do from the keystone side?22:45
morganfainbergstevemar, yeah, just always migrate, we already know the extensions, don't play "if this then migrate" with the migrate_repos22:45
morganfainbergstevemar, we should simply ensure the tables are always there. if we remove an extension permanently, the migrate repo becomes "delete the tables" or some such.22:46
stevemarmorganfainberg, i hear ya22:46
morganfainbergstevemar, but for schema consistency across deployments, always add the tables if you're using db_migrate. same as we do for identity even if we use ldap identity22:46
morganfainbergerm db_sync22:46
morganfainbergit's probably less than 5 line change :)22:47
dstanekmorganfainberg: stevemar: agreed - it also makes testing a little easier because the schema is always predictable22:48
morganfainbergdstanek, ++22:48
*** dims has quit IRC22:51
*** dims has joined #openstack-keystone22:52
*** sbfox has joined #openstack-keystone22:52
*** sbfox has quit IRC23:15
*** ozialien has joined #openstack-keystone23:18
*** radez is now known as radez_g0n323:30
*** dstanek is now known as dstanek_zzz23:37
*** dstanek_zzz is now known as dstanek23:47
*** gyee has joined #openstack-keystone23:53
*** sbfox has joined #openstack-keystone23:57
*** bobt has quit IRC23:57

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