Sunday, 2014-03-09

*** leseb has joined #openstack-keystone00:23
*** leseb has quit IRC00:27
*** david-lyle has joined #openstack-keystone00:29
*** stevemar has joined #openstack-keystone00:50
*** stevemar has quit IRC01:00
*** stevemar has joined #openstack-keystone01:14
*** topol has joined #openstack-keystone01:22
*** leseb has joined #openstack-keystone01:24
*** leseb has quit IRC01:28
*** gokrokve has joined #openstack-keystone01:33
*** gokrokve_ has joined #openstack-keystone01:36
*** gokrokve has quit IRC01:38
topolstevemar you realy there?01:54
stevemartopol, good evening topes02:01
topolgoing to private conversation..02:02
*** mberlin has quit IRC02:11
*** leseb has joined #openstack-keystone02:25
*** mberlin has joined #openstack-keystone02:27
*** leseb has quit IRC02:29
*** leseb has joined #openstack-keystone03:26
*** leseb has quit IRC03:30
*** morganfainberg_Z is now known as morganfainberg03:57
morganfainbergtopol, what are you doing here? :P03:57
morganfainbergstevemar, good evening sir03:57
morganfainberglbragstad__, what issue specifically with migrate?03:57
stevemarmorganfainberg, ahoy hoy04:00
morganfainberghey, do you have a devstack up?04:01
morganfainbergby chance?04:01
morganfainbergi04:01
morganfainbergm at a coffee shop and this might take a long time to get running04:01
morganfainbergif not no big deal, i can do this w/o a devstack04:01
morganfainbergjust figured someone might be able to 2x check for me if one was running04:01
stevemari can fire it up04:04
morganfainbergstevemar, nah04:04
morganfainbergstevemar, you going to be here for a few? might need a pair of eyes on a review04:04
stevemarmorganfainberg, yap04:05
morganfainbergstevemar, awesome.04:05
morganfainbergstevemar, will ping ya in a momennt04:05
morganfainbergif i need it04:05
stevemarmorganfainberg, comp is acting funny04:06
stevemarbrb04:06
morganfainbergk04:07
*** stevemar has quit IRC04:07
morganfainbergtopol, i'll send you my (super limited) slide deck this week.04:11
morganfainbergtopol, hopefully it gets a laugh :)04:11
*** ayoung has joined #openstack-keystone04:12
morganfainbergayoung, evening.04:12
ayoungmorganfainberg, good evening...I'm not here for long, as I need to head to bed...but04:13
ayoungI do have an issue with python304:13
ayounghttp://logs.openstack.org/81/71181/18/check/gate-python-keystoneclient-python33/67b34b4/console.html.gz04:13
ayoung2014-03-06 22:40:04.350 | TypeError: 'str' does not support the buffer interface04:13
morganfainberguhm04:13
ayoungseems like something I should be using six to handle? morganfainberg  any pointers04:13
* morganfainberg looks04:13
ayoungmorganfainberg, it is the popen call04:14
morganfainbergthe typeerrors?04:14
ayoungnot sure why it works now in python3, but It seems to be passing unit tests.  In my code, I've changed from passing a string to binary data04:14
morganfainbergbyte_str type vs. text04:15
ayoungthe error seems to be triggered by the fact that the call is returing DER (binary) instead of PEM (text)04:15
morganfainbergin python3 all "strings" are text unless decoded to byte_str04:15
morganfainbergin python 2.7, everything is a byte_str vs text04:15
morganfainbergi'll bet the DER needs to be encoded04:15
ayoungmeaning what?04:16
morganfainbergthe equivalent of str().encode / decode in py2704:16
ayoung        output, err = process.communicate(text)04:17
morganfainbergoh ok04:17
morganfainberggot it.04:17
morganfainbergthis is the same issue hashlib has04:17
ayoungso it is reutnred from the called process04:17
morganfainbergi think04:17
morganfainbergyeah04:17
morganfainbergthis is a type issue w/ the binary str vs the text str04:18
morganfainbergit's the inverse of unicode hell w/ py2.704:18
ayoungyep04:18
ayoungbut this is deep in the bowels of the process_communicate call04:18
ayoungcan't blieve we are the first to trip over that04:18
morganfainbergwe might be04:19
ayoungpython 3.3?  Nahe04:19
morganfainberghow many people call openssl and return binary data?04:19
ayoungcan't believe no one ever tried to call a binary from python04:19
morganfainberglet me look at the code vs. just the erro04:19
morganfainbergi actually have one question for you before you duck out as well04:20
morganfainbergre: revoke_api04:20
morganfainbergneed to craft a test to exercise the syncronize_map method04:20
morganfainberghttps://bugs.launchpad.net/keystone/+bug/128993504:20
ayoungdeal04:20
morganfainbergi have a fix, just want to test it so we don't hit something like this again04:20
ayoungooops04:20
morganfainbergyeah04:20
morganfainbergso, just create a test, an event and maek sure the event is expired then syncronize?04:21
morganfainbergshould be good right?04:21
*** stevemar has joined #openstack-keystone04:21
ayoungYou just want to exercise the code, not check that sync actually , ya know, syncs?04:21
morganfainbergneed to make sure we exercise that code path04:22
morganfainbergsyncronizing across green threads ... might be a challenge04:22
ayoungthat code, IIRC without context, is supposed to clean up events that are already held04:22
stevemarmorganfainberg, backs, what is goin on04:22
morganfainbergstevemar, https://bugs.launchpad.net/keystone/+bug/1289935 going to want a pair of eyes on the review i'll post for this in a little bit04:22
ayoungyou might need to add the event, then  change the clock with the timeutils thingy that sets the current time04:23
morganfainbergstevemar, asking ayoung about adding a test first :)04:23
morganfainbergayoung, ok, that works.04:23
ayounglemm make sure I'm thinking of the right code04:23
morganfainbergayoung, i think i can make this all work. thanks. now let me look at your py33 code :)04:23
ayoungmorganfainberg, its all on that review...04:23
morganfainbergi think i can just muck with expiration time of the token(s) and events.04:24
ayounghttps://review.openstack.org/#/c/71181/04:24
ayoungmorganfainberg,  for e in self._current_events:04:24
ayoungso, the idea is that it will clean up expireed events04:24
morganfainbergk. yeah04:25
ayoung cutoff = revoked_before_cutoff_time()04:25
morganfainbergi can make them expired.04:25
morganfainbergcool04:25
ayoungand that comes from04:25
ayoung oldest = timeutils.utcnow() - expire_delta04:25
ayoungpretty sure that timeutils lets you mess with that04:26
morganfainbergayoung, any thoughts on keystone middleware not being part of keystoneclient ?04:26
morganfainbergrepo-wise04:26
morganfainbergayoung, unrelated.04:26
ayoungdef set_time_override(override_time=None):04:26
morganfainbergyep.04:26
morganfainbergcool.04:26
morganfainbergthanks dude, i'll have a review for that posted by tomorrow.04:26
morganfainbergthe revoke_api stuff04:26
ayoungkeystone middleware should be part of client, I think04:26
*** leseb has joined #openstack-keystone04:27
ayoungbut...almost thinking that middleware might break off under its own weight04:27
morganfainbergayoung, i know we've waffled on splitting it out before.04:27
*** gokrokve_ has quit IRC04:27
ayoungI could see and argument that client really is "common code the lives in all things that call keystone"  and middleware caries along the "its for webservers"04:28
morganfainbergyeah04:28
ayounganyway...six04:29
morganfainbergnot sure six can help here04:29
ayoung'str' does not support the buffer interface04:29
ayoungthe stackoverflow seems to indicate it needs to be converted to bytes04:29
ayounghttp://stackoverflow.com/questions/5471158/typeerror-str-does-not-support-the-buffer-interface04:29
morganfainbergyep04:29
morganfainbergthat was my first thought04:29
ayoungso I tried:04:29
morganfainberglol i have that page open in a tab04:29
ayoung        if isinstance(text, six.text_type):04:29
ayoung            text = text.encode('utf-8')04:29
ayoungthat makes no sense to me, though04:35
ayoungas I have no idea why it would be a str04:35
ayoungand ^^ doesn't work04:35
ayoungwell, it makes a different error04:36
ayoung  File "/usr/lib64/python3.3/subprocess.py", line 1580, in _save_input04:36
ayoung    self._input = self._input.encode(self.stdin.encoding)04:36
ayoungAttributeError: 'bytes' object has no attribute 'encode'04:36
ayoungwhich implies it converted a string to bytes04:36
ayoungif self.universal_newlines04:36
ayoungah...ok, that is probably part of the problem04:36
morganfainbergyeah.04:37
ayoungthey are using the .universal_newlines  to check if it is text04:37
morganfainbergaha04:37
morganfainbergthat should be if isinstance(thing, six.basestring) i think04:38
ayoungso I have to test both old and new code paths, some usoing der (new) and some using pem04:38
morganfainbergnot assuming .universal_newlines04:38
morganfainbergactually means anything04:38
ayoungI think that was PEM that failed04:38
ayoungso my hack was wrong04:38
morganfainbergayoung, that makes sense. this is def. a strange problem04:43
ayoungyeah04:43
ayoungits the first good argument I've seen against the popen approach04:45
morganfainbergok so formatted.encode('utf-8') might solve it04:47
morganfainbergline 124?04:48
stevemarmorganfainberg, bug 1289935 looks like an easy fix04:48
morganfainbergoh wait you tried that04:48
morganfainbergbut in the _process part04:49
morganfainbergsorry *reads up again*04:49
ayoungI think I might have it...04:59
morganfainbergso universal_newlines is stupid.05:00
morganfainberglets blindly replace \n\r and \r with \n05:01
morganfainbergbut that aside05:02
morganfainbergi think you need to encode (or decode?) only in some specific cases e.g. pem vs der.05:04
morganfainbergwithout universal_newlines i think you want to encode, else you don't.05:06
morganfainbergbecause it'll do the encode for you.05:07
morganfainbergayoung, ^ (i think)05:07
ayoungOk...that doesn't seem to be working, but let me make my logic clearer05:10
*** gokrokve has joined #openstack-keystone05:11
*** gokrokve has quit IRC05:12
*** gokrokve has joined #openstack-keystone05:13
ayoungOK...so I don't think it make much difference05:13
ayoungthe problem is that it is text going out and binary coming back05:14
morganfainbergayoung, icky.05:14
ayoungwell, the command line was written assuming it was writing to a file, I'm just making use of the lesser used : fd is stdout05:14
morganfainberghehe05:15
morganfainbergwho would ever want to do that?!05:15
morganfainberg /s05:15
ayounggnight...too late for this kind of debugging05:16
*** ayoung is now known as ayoung-ZZZZZZZzz05:16
morganfainbergstevemar, i think i have a review, just running tests to validate the fix05:19
morganfainbergstevemar, will have it posted as soon as that is done05:19
*** gokrokve has quit IRC05:21
*** leseb has quit IRC05:21
*** harlowja_away has quit IRC05:22
*** nkinder has quit IRC05:24
*** gokrokve has joined #openstack-keystone05:25
*** nkinder_ has joined #openstack-keystone05:25
*** leseb has joined #openstack-keystone05:27
*** gokrokve has quit IRC05:28
*** gokrokve has joined #openstack-keystone05:31
*** leseb has quit IRC05:31
*** gokrokve has quit IRC05:37
*** gokrokve_ has joined #openstack-keystone05:41
topolmorganfainberg, I'm here cause its another exciting saturday night in my life.05:51
topolmorganfainberg looks like you found a nasty bug!05:52
morganfainbergyeah05:52
stevemartopol, keystone is exciting05:52
morganfainbergreported in #openstack-dev05:52
morganfainbergjust jumped on it because ick05:52
topolstevemar still here too. We just need a virtual case of beer and we are ready to party05:52
morganfainbergand it's not an _easy_ test case to wrap your head around05:52
morganfainbergyou need to have a revocation event, validate a token, move your clock forward, create a new event, validate a token05:53
morganfainbergthe first event (now expired) will raise the error05:53
stevemarha, easy fix, hard test case, seen that too often05:53
topolmorganfainberg, does this bug only shows up if the experimental revocation code is enabled I hope05:54
morganfainbergnope05:54
morganfainbergwell05:54
morganfainbergyes05:54
morganfainbergbut it's still reallly icky05:54
* topol I was gonna lobby to get someone to red x that code but thought, its experimental so no biggie :-)05:55
morganfainbergit was red x for a while05:56
morganfainbergbut it really needed to get in for I if we wanted a hope of making it the default in J05:56
morganfainbergbeing a 2000+ line review is what let things like this sneak through05:57
openstackgerritJenkins proposed a change to openstack/keystone: Imported Translations from Transifex  https://review.openstack.org/7852506:02
openstackgerritMorgan Fainberg proposed a change to openstack/keystone: Call an existing method in sync cache for revoke events  https://review.openstack.org/7917406:06
morganfainbergstevemar, ^06:10
morganfainbergstevemar, i want it ready to go in on monday if possible06:11
*** gokrokve_ has quit IRC06:11
stevemarmonday shouldn't be an issue06:19
morganfainbergstevemar, there are a couple typos in the comments06:19
morganfainbergtopol pointed them out to me06:19
morganfainbergin the test cases06:20
stevemarmorganfainberg, reviewing it now06:20
morganfainbergso there will be another patch before it's ready to go06:20
morganfainbergbut yeah monday.06:20
topolI caught some nits :-)06:20
morganfainbergtopol, saves brant having to ;)06:20
topolso whats token['token']['issued_at'] = timeutils.isotime(06:23
topol149                token['token']['issued_at'], subsecond=True)  doing???06:23
morganfainbergit's remapping the "issued_at" from a datetime.datetime to an isotime string06:23
morganfainbergthe revoke code always wants isotime strings06:24
topolk06:24
stevemarmorganfainberg, so mock_utcnow in the args is generated from the decorator?06:24
morganfainbergyeah06:24
stevemarmorganfainberg, i don't recall seeing any tests like that before, neato06:25
morganfainbergstevemar, there is a pending review to use that06:25
morganfainbergstevemar, saves having to do the timeutils.set_time_override bs06:25
stevemarit always takes the form of mock_<argname> ?06:25
stevemaryeah06:25
morganfainbergno, i think it is just the positional first arg06:26
topolso model.build_token_values_v2(token, 'default') does the heavy lifting of building you a real token?06:26
morganfainbergit builds the "revoke_api" specific "values"06:26
morganfainbergthere is an expected value subset that is common between v2/v3 tokens06:26
morganfainbergso, build the token_values, and pass them in.06:27
morganfainbergit's so check_token can be dumb06:27
morganfainberglike it should be06:27
morganfainberg* for certain values of "dumb", more like not worried about reformatting a token06:27
*** leseb has joined #openstack-keystone06:28
stevemarmorganfainberg, so what line in the test actually calls synchronize_revoke_map?06:28
morganfainbergthe self.revoke_api.check_token06:28
stevemarmorganfainberg, oyee this *is* a complicated thing to test06:29
morganfainbergi am not convinced we need to sync between green threads, but *shrug* if it worksssssssss.06:29
morganfainbergyeah06:29
topolso the magic to make this work is mock_utcnow.return_value = now  correct?06:29
morganfainbergunless you have taken advanced data structures (or possibly read TAoCP) don't look at the revoke tree logic06:30
morganfainbergtopol, that is the magic that lets me move our clock forward06:30
morganfainbergtopol, so it's part of the magic06:30
morganfainbergthe other part is "checking" a token and ensureing the cache is populated06:30
morganfainbergnone of the tests populate the cache and then check a token06:31
morganfainbergwe're testing a _very_ narrow bit of the code06:31
topolclock moves fwd when you do this--> mock_utcnow.return_value = now_plus_2h correct?06:31
morganfainbergyep06:32
*** devlaps has joined #openstack-keystone06:32
stevemarmorganfainberg, point of 'default' in  return model.build_token_values_v2(token, 'default') ?06:32
morganfainbergrequired argument06:32
morganfainbergthe domain06:32
stevemarah okay06:32
*** leseb has quit IRC06:32
morganfainbergsorry default_domain_id06:32
morganfainbergit's v2-ism06:32
morganfainbergwhen v2 dies, we'll need to make this V306:33
topolpretty slick Emmett "Doc" Brown06:33
morganfainbergbut ... that can wait06:33
morganfainbergi am hoping revoke sees a good amount of polish between now and K06:33
morganfainbergi could use the mock.patch.object as a context manager instead if you don't like it as a decorator06:35
topolwhat ensures that synchornized should be exercised?  revoke_by_expiration?06:35
morganfainbergtopol, check_token06:35
topolI like the decorator.I've seen that done before06:35
morganfainbergon check_token we syncronize events between greenthreads.06:35
topolwell I had a 50 50 shot :-)06:36
morganfainbergtopol, hehe yeah06:36
* topol no one like my back to the future reference???06:36
morganfainbergtopol, eh. i'm a whiskey and a glass of wine in06:37
morganfainbergi might have forgotten to type "haha"06:37
topolNICE06:37
morganfainberglove the local coffee shop.06:37
morganfainberguppers and downers!06:37
* topol I code better with a few in me...06:37
morganfainbergand open until midnight tonight06:38
topolwhiskey and not single malt scotch?  you always seemed like a single malt scotch guy to me06:38
morganfainbergtonight was burbon06:38
morganfainbergtypically i am all about single malts06:38
topolyou went southern tonight...06:39
morganfainbergbut good whiskey is good whiskey06:39
topolgetting ready for atlanta06:39
morganfainbergdamn straight06:39
morganfainbergwe need another whiskey night06:39
morganfainberg:)06:39
stevemarmorganfainberg, does self.revoke_api.check_token(token_values) return anything?06:41
morganfainbergstevemar, hmmm. in this case we don't care06:41
morganfainbergbut let me see06:42
*** gokrokve has joined #openstack-keystone06:42
morganfainbergnope06:42
morganfainbergit raises TokenNotFound if it fails06:42
stevemarmorganfainberg, just 1 comment06:43
morganfainbergcool06:43
*** gokrokve_ has joined #openstack-keystone06:43
morganfainbergi'll get that addressed and get it reposted in a couple06:44
stevemarmorganfainberg, it's a nit06:45
*** gokrokv__ has joined #openstack-keystone06:45
stevemarmorganfainberg, it looks good though06:45
stevemarthat is not an easy test to write06:45
stevemarvery narrow06:46
*** gokrokve has quit IRC06:46
morganfainbergyeah06:46
*** gokrokve_ has quit IRC06:48
*** gokrokv__ has quit IRC06:49
stevemarmorganfainberg, reading dstaneks patch on #'s makes me cringe - so many typos in our comments06:50
stevemarIt would be unfair to point them out and ask to fix it?06:51
morganfainberglol06:55
morganfainberg>.>06:55
stevemarmorganfainberg, i'm going to approve it, i can't burden him with all those typos, it's out of scope06:56
morganfainbergheheeh06:56
morganfainbergok06:56
stevemarmorganfainberg, +A https://review.openstack.org/#/c/78352/ ?07:03
morganfainbergcool07:03
stevemarty07:03
morganfainbergok fixing that nit and topol's07:05
morganfainbergand uploading another patch07:05
topolwait let mesend up my review07:06
topolsent07:07
stevemartopol, btw - we gotta start thinking about http://summit.openstack.org/07:12
stevemartopol, might want to let our other folks know07:13
topolstevemar, yes we do07:13
stevemarI want to go to this session:07:13
stevemarSwiftHoly Pipeline Batman! creiht07:13
stevemarugh, i should sleep07:15
stevemarlater folks07:15
*** stevemar has quit IRC07:20
openstackgerritMorgan Fainberg proposed a change to openstack/keystone: Call an existing method in sync cache for revoke events  https://review.openstack.org/7917407:21
*** leseb has joined #openstack-keystone07:28
openstackgerritMorgan Fainberg proposed a change to openstack/keystone: Call an existing method in sync cache for revoke events  https://review.openstack.org/7917407:30
*** leseb has quit IRC07:33
*** topol has quit IRC07:37
*** gokrokve has joined #openstack-keystone07:44
*** gokrokve has quit IRC07:49
*** devlaps has quit IRC08:23
*** leseb has joined #openstack-keystone08:29
*** jraim has quit IRC08:33
*** leseb has quit IRC08:33
*** gokrokve has joined #openstack-keystone08:44
*** gokrokve has quit IRC08:49
*** leseb has joined #openstack-keystone09:25
*** leseb has quit IRC09:29
*** gokrokve has joined #openstack-keystone09:44
*** gokrokve has quit IRC09:49
openstackgerritA change was merged to openstack/python-keystoneclient: Remove blank space after print  https://review.openstack.org/7747810:04
*** leseb has joined #openstack-keystone10:26
*** leseb has quit IRC10:30
*** gokrokve has joined #openstack-keystone10:44
*** gokrokve has quit IRC10:49
*** jraim has joined #openstack-keystone10:59
*** zhiyan_ is now known as zhiyan10:59
*** leseb has joined #openstack-keystone11:27
*** leseb has quit IRC11:31
*** leseb has joined #openstack-keystone11:35
*** gokrokve has joined #openstack-keystone11:44
*** gokrokve has quit IRC11:49
*** leseb has quit IRC12:13
*** leseb has joined #openstack-keystone12:21
*** leseb has quit IRC12:25
*** gokrokve has joined #openstack-keystone12:44
*** gokrokve has quit IRC12:49
*** leseb has joined #openstack-keystone13:28
*** gokrokve has joined #openstack-keystone13:44
*** gokrokve has quit IRC13:48
*** gokrokve has joined #openstack-keystone14:44
*** gokrokve has quit IRC14:48
openstackgerritBrant Knudson proposed a change to openstack/python-keystoneclient: Remove reference to non-existent shell doc  https://review.openstack.org/7061814:49
openstackgerritBrant Knudson proposed a change to openstack/python-keystoneclient: Update docs for auth_token middleware config options  https://review.openstack.org/7387514:53
*** gokrokve has joined #openstack-keystone15:44
*** gokrokve has quit IRC15:49
*** prad has joined #openstack-keystone15:53
*** prad has left #openstack-keystone15:53
*** david-lyle has quit IRC15:58
*** gokrokve has joined #openstack-keystone16:05
*** gokrokve has quit IRC16:20
*** gokrokve has joined #openstack-keystone16:30
*** gokrokve has quit IRC16:35
*** leseb has quit IRC16:41
*** leseb has joined #openstack-keystone16:42
openstackgerritA change was merged to openstack/python-keystoneclient: Add link to the v3 client api doc  https://review.openstack.org/7061216:44
*** leseb_ has joined #openstack-keystone16:45
*** leseb has quit IRC16:46
*** bknudson has joined #openstack-keystone16:46
openstackgerritA change was merged to openstack/python-keystoneclient: Fix keystone command man page  https://review.openstack.org/7387716:54
openstackgerritBrant Knudson proposed a change to openstack/keystone: Fix db_version failed with wrong arguments  https://review.openstack.org/7919616:56
*** gokrokve has joined #openstack-keystone16:59
*** leseb_ has quit IRC17:10
*** zhiyan is now known as zhiyan_17:28
*** topol has joined #openstack-keystone17:31
openstackgerritBrant Knudson proposed a change to openstack/keystone: Fix db_version failed with wrong arguments  https://review.openstack.org/7919617:34
*** henrynash has quit IRC18:05
openstackgerritA change was merged to openstack/keystone: Removes use of timeutils.set_time_override  https://review.openstack.org/6785718:05
*** devlaps has joined #openstack-keystone18:22
*** henrynash has joined #openstack-keystone18:25
*** kremzer has joined #openstack-keystone18:27
*** jimbaker has quit IRC18:33
*** jimbaker has joined #openstack-keystone18:35
*** jimbaker has quit IRC18:35
*** jimbaker has joined #openstack-keystone18:35
*** devlaps has quit IRC18:43
*** gokrokve has quit IRC19:05
*** topol has quit IRC19:06
*** gokrokve has joined #openstack-keystone19:09
*** kremzer has quit IRC19:29
*** bknudson has left #openstack-keystone19:35
openstackgerritA change was merged to openstack/keystone: Update ADMIN_TOKEN description in docs  https://review.openstack.org/7835219:45
*** bknudson has joined #openstack-keystone19:45
openstackgerritA change was merged to openstack/python-keystoneclient: Make keystoneclient not log auth tokens  https://review.openstack.org/7424219:46
openstackgerritA change was merged to openstack/python-keystoneclient: Log the command output on CertificateConfigError  https://review.openstack.org/7844819:50
openstackgerritA change was merged to openstack/python-keystoneclient: Remove dependent module py3kcompat  https://review.openstack.org/7376019:50
openstackgerritA change was merged to openstack/keystone: V3 xml responses should use v3 namespace.  https://review.openstack.org/6408419:50
*** luterking has joined #openstack-keystone19:59
*** stevemar has joined #openstack-keystone20:13
*** mlemay has joined #openstack-keystone20:15
openstackgerritBrant Knudson proposed a change to openstack/keystone: Use oslo db.sqlalchemy.session.EngineFacade.from_config  https://review.openstack.org/7845920:22
openstackgerritBrant Knudson proposed a change to openstack/keystone: Sync db, db.sqlalchemy from oslo-incubator 6976cfeb  https://review.openstack.org/7842920:22
*** luterking has quit IRC20:29
openstackgerritBrant Knudson proposed a change to openstack/keystone: Use oslo db.sqlalchemy.session.EngineFacade.from_config  https://review.openstack.org/7845920:47
openstackgerritBrant Knudson proposed a change to openstack/keystone: Sync db, db.sqlalchemy from oslo-incubator d6bfcc0  https://review.openstack.org/7842920:47
*** corsir has joined #openstack-keystone20:49
*** morganfainberg is now known as morganfainberg_Z21:18
*** jamielennox|away is now known as jamielennox21:26
openstackgerritA change was merged to openstack/python-keystoneclient: Capitalize Client API title consistently  https://review.openstack.org/7728221:34
*** stevemar has quit IRC21:36
openstackgerritJamie Lennox proposed a change to openstack/python-keystoneclient: Convert auth_token to use session  https://review.openstack.org/7490822:00
openstackgerritJamie Lennox proposed a change to openstack/python-keystoneclient: Fix state modifying catalog tests  https://review.openstack.org/7840922:08
*** mlemay has quit IRC22:09
openstackgerritJamie Lennox proposed a change to openstack/python-keystoneclient: Add service name to catalog  https://review.openstack.org/7841022:10
openstackgerritJamie Lennox proposed a change to openstack/python-keystoneclient: Propagate kwargs to get_token  https://review.openstack.org/7673522:15
*** henrynash has quit IRC22:23
openstackgerritJamie Lennox proposed a change to openstack/python-keystoneclient: Provide more data to AuthMethod plugins  https://review.openstack.org/7852322:35
openstackgerritA change was merged to openstack/python-keystoneclient: Privatize auth construction parameters  https://review.openstack.org/7752523:07
*** leseb has joined #openstack-keystone23:08
*** daneyon has joined #openstack-keystone23:08
*** browne has joined #openstack-keystone23:08
*** browne has quit IRC23:09
*** leseb has quit IRC23:09
openstackgerritBrant Knudson proposed a change to openstack/keystone: Remove unused db_sync from extensions  https://review.openstack.org/7921123:15
openstackgerritA change was merged to openstack/python-keystoneclient: Remove reference to non-existent shell doc  https://review.openstack.org/7061823:15
*** amcrn has joined #openstack-keystone23:17
openstackgerritBrant Knudson proposed a change to openstack/keystone: Remove unused db_sync from extensions  https://review.openstack.org/7921123:17
*** henrynash has joined #openstack-keystone23:23
*** henrynash has quit IRC23:23
*** amcrn has quit IRC23:27
openstackgerritBrant Knudson proposed a change to openstack/keystone: Remove useless oauth.Manager constructions  https://review.openstack.org/7921323:28
*** daneyon has quit IRC23:33
*** daneyon has joined #openstack-keystone23:33
*** amcrn has joined #openstack-keystone23:34
*** dstanek_afk is now known as dstanek23:49

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