Sunday, 2015-03-08

*** topol has joined #openstack-keystone00:11
*** ChanServ sets mode: +v topol00:11
*** spandhe has joined #openstack-keystone00:45
*** markvoelker has joined #openstack-keystone00:55
*** markvoelker has quit IRC00:59
*** topol has quit IRC01:05
*** lhcheng has quit IRC01:09
openstackgerritMorgan Fainberg proposed openstack/keystone: Correct utf8/innodb issues with tables  https://review.openstack.org/15980301:22
morganfainbergdstanek, tokens huh?01:24
*** richm has joined #openstack-keystone01:31
*** samueldmq_away has quit IRC01:38
dstanekmorganfainberg: i want to break apart the circular dependency that caching creates, but i took a break :-)01:41
morganfainbergdstanek, ah01:41
*** richm has quit IRC01:41
*** boris-42 has quit IRC01:42
*** navid_ has quit IRC01:44
*** navid_ has joined #openstack-keystone01:45
dstanekmorganfainberg: i factored out the shared method, but caching will take a bit more thought01:45
dstaneki think only the provider or the persistence manager should manage the caching instead of both, but i'm not sure the best way to do that yet01:46
*** ayoung has quit IRC01:59
morganfainbergdstanek, probably the other way01:59
morganfainbergsince validate is more expensive than "get from db"01:59
*** erkules_ has joined #openstack-keystone02:06
*** dimsum__ has quit IRC02:06
*** erkules has quit IRC02:08
*** jbonjean has quit IRC02:11
*** jbonjean has joined #openstack-keystone02:11
*** dimsum__ has joined #openstack-keystone02:11
*** dimsum__ has quit IRC02:16
*** kaisers1 has joined #openstack-keystone02:20
*** kaisers has quit IRC02:22
openstackgerritMerged openstack/keystone: Add Federation mixin for setting up data  https://review.openstack.org/16179302:26
*** markvoelker has joined #openstack-keystone03:12
*** markvoelker has quit IRC03:17
*** stevemar has joined #openstack-keystone03:25
*** ChanServ sets mode: +v stevemar03:25
*** spandhe has quit IRC03:30
*** lhcheng has joined #openstack-keystone03:34
*** topol has joined #openstack-keystone03:38
*** ChanServ sets mode: +v topol03:38
*** iamjarvo has joined #openstack-keystone03:41
*** iamjarvo has quit IRC03:49
openstackgerritDavid Stanek proposed openstack/keystone-specs: Adds a spec for fixing Keystone's DI  https://review.openstack.org/13593103:49
*** chlong has quit IRC03:50
*** iamjarvo has joined #openstack-keystone03:51
dstanekstevemar: wassup?03:58
*** iamjarvo has quit IRC03:58
*** iamjarvo has joined #openstack-keystone04:01
*** iamjarvo has quit IRC04:01
openstackgerritDavid Stanek proposed openstack/keystone-specs: environment setup for functional tests  https://review.openstack.org/15330004:02
*** topol has quit IRC04:03
openstackgerritDavid Stanek proposed openstack/keystone: Fixes the SQL model tests  https://review.openstack.org/15952104:11
openstackgerritDavid Stanek proposed openstack/keystone: Removes useless setup from SQL model tests  https://review.openstack.org/15952204:11
*** markvoelker has joined #openstack-keystone04:13
stevemardstanek, oh you know... the usual04:14
stevemardstanek, i was trying to let people to where they can review your patches :P04:14
dstaneki have 9 more DI related patches - that i was able to rebase on top of our current code!04:15
dstanekstevemar: for the last month i have had the fun of deep diving in the strange behaviors all over the place04:16
*** markvoelker has quit IRC04:17
stevemardstanek, oh i was guessing that you would have lined up a bunch of func. testing patches04:19
dstanekstevemar: i have a few i've been working on, but not as many. i have more stuff to fix other areas of our testing though.04:20
dstanekstevemar: i've been in rebase hell!04:23
dstaneki'm trying not to push too much into gerrit because i gets really hard to manage patches that are 5+ deep04:23
*** rwsu-afk has quit IRC04:24
stevemardstanek, yeah, 4 to 5 is the realistic limit, o/w you're in rebase hell04:27
stevemari count 8 bugs for me to address by kilo end time04:27
*** dimsum__ has joined #openstack-keystone04:56
*** dimsum__ has quit IRC05:01
*** iamjarvo has joined #openstack-keystone05:11
*** markvoelker has joined #openstack-keystone05:14
*** markvoelker has quit IRC05:19
openstackgerritDavid Stanek proposed openstack/keystone: Fixes the SQL model tests  https://review.openstack.org/15952105:48
openstackgerritDavid Stanek proposed openstack/keystone: Removes useless setup from SQL model tests  https://review.openstack.org/15952205:48
stevemarfixerizing the sql tests eh mr stanek05:54
*** r-daneel_ has quit IRC05:54
stevemardstanek, it's about to become 3am in one hour05:54
openstackgerritOpenStack Proposal Bot proposed openstack/keystone: Imported Translations from Transifex  https://review.openstack.org/16238606:10
*** markvoelker has joined #openstack-keystone06:15
*** markvoelker has quit IRC06:20
*** stevemar has quit IRC06:47
*** stevemar2 has joined #openstack-keystone06:54
*** stevemar2 has quit IRC06:56
*** stevemar2 has joined #openstack-keystone06:57
*** stevemar has joined #openstack-keystone06:59
*** ChanServ sets mode: +v stevemar06:59
*** stevemar2 has quit IRC07:01
*** stevemar has quit IRC07:02
*** stevemar2 has joined #openstack-keystone07:02
*** stevemar2 has quit IRC07:02
*** stevemar2 has joined #openstack-keystone07:04
*** iamjarvo has quit IRC07:04
*** stevemar2 has quit IRC07:04
*** stevemar has joined #openstack-keystone07:04
*** ChanServ sets mode: +v stevemar07:04
stevemartesting07:05
dstanekfailed07:06
stevemara failed test is usually a good sign07:08
*** stevemar has quit IRC07:10
*** stevemar has joined #openstack-keystone07:11
*** ChanServ sets mode: +v stevemar07:11
*** stevemar has quit IRC07:12
*** markvoelker has joined #openstack-keystone07:16
*** markvoelker has quit IRC07:21
*** markvoelker has joined #openstack-keystone08:17
*** markvoelker has quit IRC08:22
*** dimsum__ has joined #openstack-keystone08:34
*** dimsum__ has quit IRC08:38
*** henrynash has joined #openstack-keystone08:40
*** ChanServ sets mode: +v henrynash08:40
*** jistr has joined #openstack-keystone09:11
*** markvoelker has joined #openstack-keystone09:18
*** jistr has quit IRC09:23
*** markvoelker has quit IRC09:23
*** henrynash has quit IRC09:42
*** henrynash has joined #openstack-keystone09:49
*** ChanServ sets mode: +v henrynash09:49
*** _cjones_ has joined #openstack-keystone10:04
*** _cjones_ has quit IRC10:05
*** markvoelker has joined #openstack-keystone10:19
*** markvoelker has quit IRC10:24
*** chlong has joined #openstack-keystone10:40
openstackgerrithenry-nash proposed openstack/keystone-specs: Add domain-config group/option resource relationships  https://review.openstack.org/16245910:40
*** boris-42 has joined #openstack-keystone10:48
*** dimsum__ has joined #openstack-keystone10:59
*** dims_ has joined #openstack-keystone11:00
*** dimsum__ has quit IRC11:04
*** henrynash has quit IRC11:05
*** dims_ has quit IRC11:05
*** samueldmq_away has joined #openstack-keystone11:06
*** samueldmq_away is now known as samueldmq_11:06
*** samueldmq has quit IRC11:08
*** samueldmq_ is now known as samueldmq11:09
*** samueldmq has quit IRC11:18
*** samueldmq has joined #openstack-keystone11:19
*** Akshik has joined #openstack-keystone11:19
*** markvoelker has joined #openstack-keystone11:21
*** chlong has quit IRC11:21
*** samueldmq has quit IRC11:24
*** markvoelker has quit IRC11:25
*** samueldmq has joined #openstack-keystone11:28
openstackgerritDave Chen proposed openstack/keystone: Use region or region_id in a consistent way  https://review.openstack.org/16246512:05
*** Akshik has quit IRC12:05
*** dimsum__ has joined #openstack-keystone12:06
*** dimsum__ has quit IRC12:11
*** markvoelker has joined #openstack-keystone12:22
*** markvoelker has quit IRC12:27
*** chlong has joined #openstack-keystone12:39
*** _cjones_ has joined #openstack-keystone13:05
*** _cjones_ has quit IRC13:10
openstackgerritDave Chen proposed openstack/keystone: Use region or region_id in a consistent way  https://review.openstack.org/16246513:12
openstackgerritDave Chen proposed openstack/keystone: Use region or region_id in a consistent way  https://review.openstack.org/16246513:15
*** rdo has quit IRC13:15
*** rdo has joined #openstack-keystone13:17
*** markvoelker has joined #openstack-keystone13:23
*** markvoelker has quit IRC13:28
*** topol has joined #openstack-keystone13:31
*** ChanServ sets mode: +v topol13:31
*** nellysmitt has joined #openstack-keystone13:32
*** nellysmitt has quit IRC13:32
*** Akshik has joined #openstack-keystone13:34
*** lhcheng has quit IRC13:41
*** topol has quit IRC13:47
*** chlong has quit IRC14:06
*** Akshik has quit IRC14:11
*** Akshik has joined #openstack-keystone14:12
*** markvoelker has joined #openstack-keystone14:24
*** Akshik has quit IRC14:24
*** Akshik has joined #openstack-keystone14:25
*** markvoelker has quit IRC14:29
*** lhcheng has joined #openstack-keystone14:33
*** dimsum__ has joined #openstack-keystone14:45
*** dims_ has joined #openstack-keystone14:47
*** dimsum__ has quit IRC14:50
*** dims_ has quit IRC14:56
*** lhcheng has quit IRC15:12
*** markvoelker has joined #openstack-keystone15:25
*** markvoelker has quit IRC15:29
*** iamjarvo has joined #openstack-keystone15:39
openstackgerritBrant Knudson proposed openstack/keystone: Remove unused threads argument  https://review.openstack.org/16247515:47
*** rdo has quit IRC15:48
*** iamjarvo has quit IRC15:48
*** sigmavirus24_awa is now known as sigmavirus2415:58
morganfainbergDamn it. Autocorrect. Endless recursion -> a dress recursion16:00
morganfainbergWhile amusing... Wut? :P16:00
*** rdo has joined #openstack-keystone16:01
*** Akshik has quit IRC16:08
*** r-daneel_ has joined #openstack-keystone16:11
*** r-daneel__ has joined #openstack-keystone16:12
*** r-daneel_ has quit IRC16:16
*** iamjarvo has joined #openstack-keystone16:16
*** iamjarvo has joined #openstack-keystone16:16
openstackgerritBrant Knudson proposed openstack/keystone: Import fernet providers only if used in keystone-manage  https://review.openstack.org/16247616:23
openstackgerritBrant Knudson proposed openstack/keystone: Move Fernet requirements to test-requirements.txt  https://review.openstack.org/16236016:24
*** markvoelker has joined #openstack-keystone16:26
*** iamjarvo has quit IRC16:26
*** markvoelker has quit IRC16:32
*** samueldmq has quit IRC16:33
*** Tahmina has joined #openstack-keystone16:39
*** dimsum__ has joined #openstack-keystone16:41
*** dimsum__ has quit IRC16:46
openstackgerritBrant Knudson proposed openstack/keystone: Move test_core to keysteone.tests.unit.tests  https://review.openstack.org/16247816:47
openstackgerritBrant Knudson proposed openstack/keystone: Move test_core to keysteone.tests.unit.tests  https://review.openstack.org/16247816:47
*** henrynash has joined #openstack-keystone16:47
*** ChanServ sets mode: +v henrynash16:47
openstackgerritBrant Knudson proposed openstack/keystonemiddleware: Move _memcache_pool into auth_token  https://review.openstack.org/16248017:05
*** sigmavirus24 is now known as sigmavirus24_awa17:14
*** erkules_ is now known as erkules17:16
*** erkules has quit IRC17:18
*** erkules has joined #openstack-keystone17:18
*** panbalag has joined #openstack-keystone17:18
*** Tahmina has quit IRC17:23
*** markvoelker has joined #openstack-keystone17:28
openstackgerritBrant Knudson proposed openstack/keystonemiddleware: Move unit tests into tests.unit  https://review.openstack.org/16248217:28
*** markvoelker has quit IRC17:33
*** dimsum__ has joined #openstack-keystone17:43
*** g2` has quit IRC17:44
openstackgerritBrant Knudson proposed openstack/keystone: Remove unused checkout_vendor  https://review.openstack.org/16248317:51
*** panbalag has quit IRC17:53
openstackgerritMerged openstack/keystonemiddleware: Updated from global requirements  https://review.openstack.org/16224417:53
openstackgerrithenry-nash proposed openstack/keystone-specs: Mark the domain-config API as experimental  https://review.openstack.org/16248417:55
openstackgerrithenry-nash proposed openstack/keystone-specs: Mark the domain-config API as experimental  https://review.openstack.org/16248417:58
bknudsonkeystone unit tests are broken on my dev system...17:58
bknudsonI blame the timezone change.17:58
bknudsonkeystone.tests.unit.token.test_fernet_provider.TestBaseTokenFormatter.test_time_string_to_int_conversions17:59
bknudson    MismatchError: '2015-03-08T17:58:15Z' != '2015-03-08T18:58:15Z'17:59
*** cburgess has quit IRC18:00
openstackgerritMerged openstack/keystonemiddleware: Delay denial when service token is invalid  https://review.openstack.org/15324718:13
*** g2` has joined #openstack-keystone18:15
openstackgerritBrant Knudson proposed openstack/keystonemiddleware: Move unit tests into tests.unit  https://review.openstack.org/16248218:18
openstackgerrithenry-nash proposed openstack/keystone: Add API support for domain config  https://review.openstack.org/15875218:23
*** cburgess has joined #openstack-keystone18:24
*** bknudson has left #openstack-keystone18:25
openstackgerrithenry-nash proposed openstack/keystone: Enable use of database domain config  https://review.openstack.org/15967518:27
*** iamjarvo has joined #openstack-keystone18:29
*** markvoelker has joined #openstack-keystone18:29
openstackgerritMorgan Fainberg proposed openstack/keystone: Correct utf8/innodb issues with tables  https://review.openstack.org/15980318:29
openstackgerrithenry-nash proposed openstack/keystone: Mark the domain config API as experimental  https://review.openstack.org/16003218:30
openstackgerrithenry-nash proposed openstack/keystone: Support upload domain config files to database  https://review.openstack.org/16036418:31
*** markvoelker has quit IRC18:36
*** dimsum__ has quit IRC18:38
*** samueldmq has joined #openstack-keystone18:43
*** Akshik has joined #openstack-keystone18:43
openstackgerritMerged openstack/keystone: Move test_core to keysteone.tests.unit.tests  https://review.openstack.org/16247818:45
*** markvoelker has joined #openstack-keystone18:46
*** lhcheng has joined #openstack-keystone18:46
openstackgerrithenry-nash proposed openstack/keystone-specs: Mark the domain-config API as experimental  https://review.openstack.org/16248418:47
*** r-daneel__ has quit IRC18:47
*** markvoelker has quit IRC18:48
*** spandhe has joined #openstack-keystone18:53
*** grantbow has joined #openstack-keystone19:20
openstackgerrithenry-nash proposed openstack/keystone: Move backend LDAP role testing to the new backend testing module  https://review.openstack.org/15683019:32
*** dimsum__ has joined #openstack-keystone19:38
*** dimsum__ has quit IRC19:43
*** iamjarvo_ has joined #openstack-keystone19:58
*** iamjarvo has quit IRC20:00
*** samueldmq has quit IRC20:05
*** Akshik has quit IRC20:08
*** spandhe has quit IRC20:08
*** samueldmq has joined #openstack-keystone20:09
*** bknudson has joined #openstack-keystone20:30
*** ChanServ sets mode: +v bknudson20:30
*** _cjones_ has joined #openstack-keystone20:53
*** _cjones_ has quit IRC20:53
*** _cjones_ has joined #openstack-keystone20:54
*** jamielennox|away is now known as jamielennox20:58
*** dhellmann has quit IRC21:21
*** dhellmann has joined #openstack-keystone21:22
*** stevemar has joined #openstack-keystone21:22
*** ChanServ sets mode: +v stevemar21:22
*** dhellmann has quit IRC21:23
*** dhellmann has joined #openstack-keystone21:24
*** dhellmann has quit IRC21:24
*** dhellmann has joined #openstack-keystone21:25
*** jorge_munoz has joined #openstack-keystone21:26
*** dhellmann has quit IRC21:28
*** dhellmann has joined #openstack-keystone21:29
*** dimsum__ has joined #openstack-keystone21:31
*** stevemar has quit IRC21:31
*** stevemar has joined #openstack-keystone21:32
*** ChanServ sets mode: +v stevemar21:32
stevemardstanek around-ish?21:33
bknudsonstevemar: is the keystone.tests.unit.token.test_fernet_provider.TestBaseTokenFormatter.test_time_string_to_int_conversions test working for you?21:33
stevemarbknudson let me try and see21:34
stevemarbknudson it's in master branch?21:34
bknudsonstevemar: yes.21:34
stevemarit'll be a few minutes, rebuilding env21:35
stevemarbah, i don't like this new irc client, going back to my old one21:35
*** stevemar has quit IRC21:35
*** stevemar has joined #openstack-keystone21:36
*** stevemar has quit IRC21:36
*** chlong has joined #openstack-keystone21:38
*** stevemar has joined #openstack-keystone21:40
*** ChanServ sets mode: +v stevemar21:40
stevemarbknudson, looks like it's failing: with a mismatch '2015-03-08T21:37:59Z' != '2015-03-08T22:37:59Z'21:40
bknudsonstevemar: ok. I'm not the only one then.21:40
stevemari wonder why it's working in the CI21:41
bknudsonis it?21:41
stevemarif it's part of master branch, then wouldn't it have to?21:41
bknudsony, looks like it is working...21:42
stevemarbknudson, maybe the whole suite has to be run, not just the individual test? (grasping at straws here)21:43
bknudsonit also fails for me when running the whole suite.21:44
stevemarbknudson, blah21:44
*** iamjarvo_ has quit IRC21:47
bknudsonI think I have a fix.21:49
stevemarbknudson, what are you thinking?21:51
bknudsonstevemar: the code need the timestamps to stay in utc... seconds since the epoch shouldn't be different based on the timezone.21:52
stevemarbknudson, oh... thats funny21:54
openstackgerritBrant Knudson proposed openstack/keystone: Fix seconds since epoch use in fernet tokens  https://review.openstack.org/16248921:54
stevemarbknudson, i guess when dolphinator / lbragstad wrote the tests they didn't notice the skew?21:54
bknudsontry that? ^21:54
morganfainbergstevemar, bknudson, you have UTC on your machines?21:55
bknudsonstevemar: I don't know... clocks moved forward overnight but I don't know how that could have caused the test to work and then fail.21:55
morganfainberggate, my guess is utc.21:55
bknudsonthe clock on my machine is right.21:55
bknudsonmorganfainberg: does it fail for you?21:55
morganfainberghaven't run it locally21:55
morganfainbergsince timechange21:55
morganfainbergbut the only thing that comes to mind is gate machines might be UTC and yours might be local tz21:56
stevemarbknudson, looks like it works21:56
bknudsonI can't really explain it well.21:56
*** chlong has quit IRC21:56
bknudsonbut I can see that time.time() doesn't match the expected_time_int.21:56
bknudsonand time.time() is seconds since epoch, so there was something fishy going on.21:57
openstackgerritMorgan Fainberg proposed openstack/keystone: Correct utf8/innodb issues with tables  https://review.openstack.org/15980321:57
morganfainbergmy guess is time.time() is tz specific. so the correct way is datetime.utcnow() formatted to epic21:58
morganfainbergepoch*21:58
bknudsontime.time() can't be tz specific?21:58
bknudsonseconds since epoch doesn't depend on tz.21:58
morganfainbergbknudson, it's based on your local clock, right?21:58
morganfainberghmm. somehow i think time.time is doing something wierd here21:59
morganfainbergsec21:59
bknudsonthere's also a time.gmtime()21:59
morganfainbergor at least time_object.strftime('%s')) is tz specific21:59
morganfainbergbknudson, that returns a struct(time) not int22:00
morganfainbergstruct_time()22:00
bknudson>>> datetime.datetime.now().strftime('%s')22:00
bknudson'1425852006'22:00
bknudson>>> time.time()22:00
bknudson1425852010.53218922:00
bknudson>>> datetime.datetime.now().strftime('%s'), time.time()22:00
bknudson('1425852039', 1425852039.923267)22:00
bknudson>>> datetime.datetime.utcnow().strftime('%s'), time.time()22:01
bknudson('1425870056', 1425852056.74933)22:01
stevemarcookin time22:01
bknudsonthis is very confusing.22:01
jamielennoxbknudson: i replied on https://review.openstack.org/#/c/161962/ - basically i know you're right and tried to fight it and fix it 'properly' for a while - but i think it's important that it just gets done22:02
morganfainbergbknudson, weird22:03
bknudsonjamielennox: I'd prefer passing in a getter function to copy-pasting code.22:04
jamielennoxbknudson: hmm, alright - i'll see how it looks22:05
jamielennoxit's going to miss kilo :(22:05
bknudsonI'd at least like to see how it looks.22:05
bknudsonwe've got until april 9 for release candidates.22:06
bknudsoncan't get in in a month?22:06
bknudsonI know jamielennox is a faster coder than that.22:06
jamielennoxbknudson: i'm hoping to do a bump of the global requirements22:06
jamielennoxbefore freeze22:07
jamielennoxbecause ksm is still 1.022:07
bknudsonMarch 1922:07
bknudsonapplications don't depend on any specific ksm feature, so don't need to bump requirement.22:07
jamielennoxthere just hasn't been anything urgent to bump that, so i was hoping to do a ksm release early this week, and do the bump then22:07
jamielennoxeven if we do another relase prior to the named release22:08
jamielennoxbknudson: not yet - because the requirements aren't high enough22:08
jamielennoxi have to do everything conditionally22:08
bknudsonswift will probably depend on https://review.openstack.org/#/c/153247/ so would bump requirements for that.22:08
jamielennoxbknudson: i think 1.1 was where we pass an auth plugin down from middleware22:08
jamielennoxi want that one in the min requirements22:09
jamielennoxon the other hand i still haven't figured out serialization so maybe it doesn't matter yet22:10
openstackgerritBrant Knudson proposed openstack/keystone: Fix seconds since epoch use in fernet tokens  https://review.openstack.org/16248922:16
*** jorge_munoz has quit IRC22:16
*** abhirc_ has quit IRC22:28
*** diegows has joined #openstack-keystone22:33
dstanekstevemar: kinda/sorta22:41
*** abhirc has joined #openstack-keystone22:51
*** abhirc has quit IRC22:51
*** erkules_ has joined #openstack-keystone23:03
*** erkules has quit IRC23:05
*** iamjarvo has joined #openstack-keystone23:28
samueldmqwhen we apply a migration, do we always apply all previous ones?23:31
samueldmqI mean, if I have a fresh Juno installation, I don't need Juno migrations, I'll need the kilo ones, right?23:32
*** chlong has joined #openstack-keystone23:35
*** abhirc has joined #openstack-keystone23:37

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