Friday, 2017-02-03

*** lamt has quit IRC00:02
*** catintheroof has joined #openstack-keystone00:04
openstackgerritMorgan Fainberg proposed openstack/keystone master: Remove KVS code  https://review.openstack.org/42486200:10
*** Ephur has quit IRC00:10
*** adrian_otto has joined #openstack-keystone00:19
*** jperry has quit IRC00:20
*** jose-phillips has quit IRC00:22
*** agrebennikov__ has quit IRC00:24
*** catintheroof has quit IRC00:30
openstackgerritRichard Avelar proposed openstack/keystone master: WIP add db_sync check  https://review.openstack.org/41638300:40
openstackgerritRichard Avelar proposed openstack/keystone master: WIP add db_sync check  https://review.openstack.org/41638300:41
*** jaosorior has quit IRC00:46
*** lamt has joined #openstack-keystone00:50
*** dave-mccowan has joined #openstack-keystone00:56
*** tqtran has quit IRC00:59
*** thorst_ has quit IRC01:20
*** lamt has quit IRC01:21
*** thorst_ has joined #openstack-keystone01:21
*** thorst_ has quit IRC01:25
*** lamt has joined #openstack-keystone01:26
*** rdo has quit IRC01:26
*** rdo has joined #openstack-keystone01:28
*** lamt has quit IRC01:28
*** portdirect is now known as portdirect_away01:30
*** lamt has joined #openstack-keystone01:31
*** chris_hultin|AWA is now known as chris_hultin01:41
*** chris_hultin is now known as chris_hultin|AWA01:43
*** edmondsw has joined #openstack-keystone01:45
*** edmondsw has quit IRC01:51
*** browne has quit IRC01:55
*** anteaya has quit IRC01:57
*** thorst_ has joined #openstack-keystone02:00
*** dave-mccowan has quit IRC02:03
*** anteaya has joined #openstack-keystone02:04
openstackgerritSteve Martinelli proposed openstack/keystone-specs master: move not implemented ocata specs to backlog  https://review.openstack.org/42854002:04
*** adrian_otto has quit IRC02:06
openstackgerritMerged openstack/keystone-specs master: move not implemented ocata specs to backlog  https://review.openstack.org/42854002:12
*** lamt has quit IRC02:13
*** lamt has joined #openstack-keystone02:13
*** lamt has quit IRC02:18
openstackgerritSteve Martinelli proposed openstack/keystone master: Remove KVS code  https://review.openstack.org/42486202:25
openstackgerritGage Hugo proposed openstack/pycadf master: Allow multiple joined uuids to be considered valid  https://review.openstack.org/42854302:25
openstackgerritGage Hugo proposed openstack/pycadf master: Allow multiple joined uuids to be considered valid  https://review.openstack.org/42854302:28
*** thorst_ has joined #openstack-keystone02:32
*** thorst_ has quit IRC02:32
stevemargagehugo: i love that ^02:32
gagehugostevemar: yay02:33
gagehugothat fixes the sha256 issue02:33
stevemarand the 'default' issue :)02:33
gagehugoyes02:33
*** spzala has quit IRC02:34
gagehugothis pycadf thing is tricky, it only complains once per test run02:34
*** spzala has joined #openstack-keystone02:34
gagehugoso once you patch the hole, another one pops up02:35
stevemarhehe02:35
*** adu has joined #openstack-keystone02:38
*** spzala has quit IRC02:39
gagehugomight need to break that one up a bit, there is a lot there02:40
stevemargagehugo: the pycadf patch?02:40
stevemarnah02:40
stevemargagehugo: once that merges you'll have a much easier time getting the keystone bits passing and probably much fewer changes02:41
stevemargagehugo: changing pycadf should be easy02:41
gagehugoyeah the pycadf one02:42
gagehugoah ok02:42
*** adu has quit IRC02:42
stevemargagehugo: keystone folks have +2 on that project02:42
stevemaror at least most of us do02:43
gagehugothat works lol02:44
gagehugostevemar: do you want keystone test logs as an example in the message?02:46
*** MasterOfBugs has quit IRC02:47
stevemargagehugo: meh, see what others say02:49
stevemargagehugo: morgan has an idea of whats up, he can review it02:49
gagehugook02:50
morgan?02:51
gagehugomorgan: https://review.openstack.org/42854302:52
morganah02:53
morganok will review02:53
*** dave-mccowan has joined #openstack-keystone02:53
gagehugothanks!02:53
*** dave-mccowan has quit IRC03:15
*** adu has joined #openstack-keystone03:18
*** martinlopes has quit IRC03:28
*** martinlopes has joined #openstack-keystone03:31
*** edmondsw has joined #openstack-keystone03:33
*** edmondsw has quit IRC03:38
*** thorst_ has joined #openstack-keystone03:58
*** adu has quit IRC04:01
*** thorst_ has joined #openstack-keystone04:21
*** thorst_ has quit IRC04:21
*** stingaci has quit IRC04:23
*** adriant has quit IRC04:31
*** dikonoor has joined #openstack-keystone04:32
*** adu has joined #openstack-keystone04:47
*** lucas_ has joined #openstack-keystone04:49
*** nicolasbock has quit IRC04:52
*** lucas_ has quit IRC04:52
*** lucas_ has joined #openstack-keystone04:53
*** lucas_ has quit IRC05:01
*** stingaci has joined #openstack-keystone05:05
*** stingaci has quit IRC05:06
*** adrian_otto has joined #openstack-keystone05:14
*** adu has quit IRC05:14
openstackgerritMerged openstack/keystone master: Stop reading local config dirs for domain-specific SQL config driver  https://review.openstack.org/42794005:16
*** lucas_ has joined #openstack-keystone05:17
*** edmondsw has joined #openstack-keystone05:21
*** lucas_ has quit IRC05:22
*** edmondsw has quit IRC05:26
*** phalmos has quit IRC05:58
*** MasterOfBugs has joined #openstack-keystone06:01
*** pramodrj07 has joined #openstack-keystone06:04
*** MasterOfBugs has quit IRC06:05
*** thorst_ has joined #openstack-keystone06:22
*** martinlopes has quit IRC06:23
*** thorst_ has quit IRC06:27
*** mvk has quit IRC06:37
*** mvk has joined #openstack-keystone06:39
*** edmondsw has joined #openstack-keystone07:10
*** edmondsw has quit IRC07:14
*** tesseract has joined #openstack-keystone07:20
*** adrian_otto has quit IRC07:20
*** adrian_otto has joined #openstack-keystone07:26
*** rcernin has joined #openstack-keystone07:26
*** blake has joined #openstack-keystone07:28
*** blake has quit IRC07:28
*** blake has joined #openstack-keystone07:29
*** adrian_otto has quit IRC07:33
*** nkinder has joined #openstack-keystone07:56
*** aloga has quit IRC07:59
*** aloga has joined #openstack-keystone07:59
*** pramodrj07 has quit IRC08:18
*** blake has quit IRC08:21
*** thorst_ has joined #openstack-keystone08:23
*** thorst_ has quit IRC08:28
*** ravelar has quit IRC08:47
*** edmondsw has joined #openstack-keystone08:58
*** zzzeek has quit IRC09:00
*** zzzeek has joined #openstack-keystone09:00
*** edmondsw has quit IRC09:02
*** spzala has joined #openstack-keystone09:35
*** chlong has quit IRC09:38
*** spzala has quit IRC09:40
*** erlon has joined #openstack-keystone09:57
*** thorst_ has joined #openstack-keystone10:24
*** thorst_ has quit IRC10:29
*** bauruine has joined #openstack-keystone10:41
*** edmondsw has joined #openstack-keystone10:46
*** edmondsw has quit IRC10:50
*** thiagolib has joined #openstack-keystone11:09
*** nicolasbock has joined #openstack-keystone11:24
*** timss has quit IRC11:30
*** timss has joined #openstack-keystone11:30
*** bigjools has quit IRC11:36
*** bigjools has joined #openstack-keystone11:37
*** bigjools has joined #openstack-keystone11:37
*** pnavarro has quit IRC11:41
*** dave-mccowan has joined #openstack-keystone12:10
*** catintheroof has joined #openstack-keystone12:20
*** catintheroof has quit IRC12:20
*** catintheroof has joined #openstack-keystone12:21
*** jvarlamova_ has quit IRC12:28
*** rcernin has quit IRC12:31
*** edmondsw has joined #openstack-keystone12:34
*** thorst_ has joined #openstack-keystone12:37
*** edmondsw has quit IRC12:38
*** edmondsw has joined #openstack-keystone12:43
*** raildo has joined #openstack-keystone13:00
*** markvoelker_ has joined #openstack-keystone13:29
*** markvoelker has quit IRC13:32
*** hoonetorg has quit IRC13:35
*** spzala has joined #openstack-keystone13:35
*** spzala has quit IRC13:36
*** spzala has joined #openstack-keystone13:36
*** hoonetorg has joined #openstack-keystone13:48
*** lamt has joined #openstack-keystone13:56
*** spilla has joined #openstack-keystone14:09
*** lamt has quit IRC14:13
*** jperry has joined #openstack-keystone14:14
*** mvk has quit IRC14:15
*** Dinesh_Bhor has quit IRC14:20
*** spzala has quit IRC14:23
*** jperry has quit IRC14:36
*** lamt has joined #openstack-keystone14:41
*** mvk has joined #openstack-keystone14:44
*** lucas_ has joined #openstack-keystone14:45
lbragstadoffice hours!!14:52
*** dikonoor has quit IRC14:54
*** spzala has joined #openstack-keystone14:55
*** spzala_ has joined #openstack-keystone14:57
*** spzala_ has quit IRC14:57
*** spzala_ has joined #openstack-keystone14:57
*** mvk has quit IRC14:58
*** spzala has quit IRC15:00
*** chris_hultin|AWA is now known as chris_hultin15:01
*** lamt has quit IRC15:02
*** haplo37_ has quit IRC15:09
*** chris_hultin is now known as chris_hultin|AWA15:10
*** pcaruana has quit IRC15:11
*** haplo37_ has joined #openstack-keystone15:12
*** ravelar has joined #openstack-keystone15:16
*** pcaruana has joined #openstack-keystone15:17
*** adrian_otto has joined #openstack-keystone15:19
knikollao/15:20
*** stingaci has joined #openstack-keystone15:20
knikollalbragstad: reporting in for office hours.15:21
lbragstadknikolla sweet!15:21
lbragstadknikolla anything you're planning on tackling today?15:21
knikollalbragstad: open for suggestions15:22
*** chlong has joined #openstack-keystone15:24
lbragstadknikolla couple reviews that close bugs - https://review.openstack.org/#/c/420085/515:24
*** stingaci has quit IRC15:24
lbragstadknikolla and https://review.openstack.org/#/c/424862/15:24
*** adrian_otto has quit IRC15:25
*** stingaci has joined #openstack-keystone15:25
knikollalbragstad: the second on is for pike15:25
lbragstadknikolla yeah - kvs removal15:26
knikollalbragstad: did we cut stable/ocata yet?15:26
gagehugoo/15:26
lbragstadknikolla that will probably be one of the first patches to land in pike (after hearing morgan and stevemar talk about it)15:26
lbragstadknikolla we have an RC15:27
lbragstadgagehugo o/15:27
lbragstadknikolla http://lists.openstack.org/pipermail/openstack-dev/2017-February/111534.html15:27
lbragstadknikolla and https://github.com/openstack/keystone/tree/stable/ocata15:28
knikollalbragstad: so master is open for merging pike stuff now15:28
lbragstadknikolla correct15:28
knikollalbragstad: awesome!15:28
* knikolla goes back to reviewing15:29
*** chris_hultin|AWA is now known as chris_hultin15:29
*** nkinder has quit IRC15:32
*** dave-mccowan has quit IRC15:40
*** jaosorior has joined #openstack-keystone15:40
*** edtubill has joined #openstack-keystone15:42
lbragstadheads up https://review.openstack.org/#/c/420085/5 closes a bug for us in case anyone wants to review15:45
*** lamt has joined #openstack-keystone15:48
samueldmqhi all15:52
samueldmqlbragstad: done15:52
lbragstadsamueldmq o/ thanks!15:52
samueldmq:)15:52
*** dave-mccowan has joined #openstack-keystone15:54
*** mvk has joined #openstack-keystone16:07
lbragstaddstanek stevemar samueldmq do any of you remember when we removed the stable driver interfaces work?16:08
dstaneklbragstad: hmmm16:08
lbragstadI'm looking through Launchpad and can't seem to find it (go figure)16:08
samueldmqnewton ?16:08
dstaneklbragstad: jas...looking16:08
lbragstaddstanek something tells me Newton16:08
lbragstadsamueldmq ++ that's what I was thinking - but I don't think we tracked the work with a spec16:09
samueldmqlbragstad: that's what I think it is too16:09
samueldmq:)16:09
*** BlackDex has quit IRC16:09
stevemarlbragstad: i think this round16:09
stevemarlbragstad: https://blueprints.launchpad.net/keystone/+spec/removed-as-of-ocata16:09
stevemar"all stable driver interfaces"16:10
spillalbragstad: for https://review.openstack.org/#/c/420085/ would you think `region_id_required` and `region_id_optional` would be a good naming change?16:10
lbragstadstevemar ah - yes!16:10
lbragstadstevemar thanks!16:10
dstanekstevemar: nice... just found it too16:10
lbragstadspilla I'm open to that - yeah... there are other conventions already established in the parameters module, too16:11
spillaoh, there seem to be a few of the `param_1` s. I may go through a bunch of these. Also ty, I'll check those conventions16:11
lbragstadspilla i had a note on my todo list to go through through, i remember them bugging me the last time I was mucking around in the docs16:12
lbragstadthrough those*16:12
spillalbragstad if you'd like I can take it off yours and add it to mine :)16:13
* lbragstad hands spilla the Post-It note from his monitor16:14
*** BlackDex has joined #openstack-keystone16:16
*** BlackDex has quit IRC16:16
*** BlackDex has joined #openstack-keystone16:17
lbragstadantwash o/ I updated https://review.openstack.org/#/c/277707/10 with a comment regarding our discussion yesterday16:18
*** BlackDex has quit IRC16:20
openstackgerritRichard Avelar proposed openstack/keystone master: WIP add db_sync check  https://review.openstack.org/41638316:22
*** BlackDex has joined #openstack-keystone16:25
*** agrebennikov__ has joined #openstack-keystone16:27
*** adrian_otto has joined #openstack-keystone16:28
*** stingaci has quit IRC16:32
*** chlong has quit IRC16:36
openstackgerritGage Hugo proposed openstack/keystone master: WIP Fix multiple uuid warnings with pycadf  https://review.openstack.org/42641116:37
*** ravelar has quit IRC16:44
*** ravelar has joined #openstack-keystone16:45
*** adrian_otto has quit IRC16:51
*** chlong has joined #openstack-keystone16:52
openstackgerritGage Hugo proposed openstack/pycadf master: Make `is_valid` more flexible with uuid validation  https://review.openstack.org/42854316:53
*** d0ugal is now known as d0ugal|skiing16:54
*** stingaci has joined #openstack-keystone16:58
antwashlbragstad : gotcha, I just looked at it -- i'll go ahead an implement still, just in case Kalaswan doesn't respond or push by end of day17:00
*** adrian_otto has joined #openstack-keystone17:02
*** lucas_ has quit IRC17:03
*** chlong has quit IRC17:16
*** mvk has quit IRC17:17
*** lamt has quit IRC17:19
*** lamt has joined #openstack-keystone17:24
*** tqtran has joined #openstack-keystone17:25
*** browne has joined #openstack-keystone17:30
lbragstadantwash you might be able to do a rebase, too17:31
lbragstadantwash i'd personally try and shoot for that case, that way it keeps Kalaswan as the original author17:32
antwashYeah that's what I'm doing lol17:32
lbragstadantwash awesome17:32
lbragstadantwash you're always two steps ahead :)17:32
antwashlbragstad : no point in writing all that over again lol #LAzy17:32
lbragstadantwash lazy people make great developers17:33
*** tqtran_ has joined #openstack-keystone17:34
*** tqtran has quit IRC17:34
lbragstadravelar it'd be good to get odyssey4me's eyes on https://review.openstack.org/#/c/416383/ :)17:35
*** stingaci has quit IRC17:35
*** lamt has quit IRC17:35
*** lamt has joined #openstack-keystone17:36
*** dave-mccowan has quit IRC17:38
openstackgerritSamuel Pilla proposed openstack/keystone master: Renaming of api parameters  https://review.openstack.org/42883017:40
ravelarlbragstad that it would lol17:44
openstackgerritTin Lam proposed openstack/keystone master: WIP Fix multiple uuid warnings with pycadf  https://review.openstack.org/42641117:45
*** adrian_otto1 has joined #openstack-keystone17:48
*** adrian_otto has quit IRC17:50
*** EmilienM has quit IRC17:56
*** pkoraca has quit IRC17:56
*** Kimmo_ has quit IRC17:56
*** Adobeman has quit IRC17:56
*** baffle has quit IRC17:56
*** tonyb has quit IRC17:56
*** rodrigods has quit IRC17:56
*** Tahvok has quit IRC17:56
*** baffle has joined #openstack-keystone17:56
*** Nakato has quit IRC17:56
*** bknudson has quit IRC17:56
*** mkoderer___ has quit IRC17:56
*** d34dh0r53 has quit IRC17:56
*** Anticimex has quit IRC17:56
*** peterstac has quit IRC17:56
*** alex_xu has quit IRC17:56
*** Adobeman has joined #openstack-keystone17:56
*** tonyb has joined #openstack-keystone17:56
*** rodrigods has joined #openstack-keystone17:56
*** mkoderer__ has joined #openstack-keystone17:56
*** Kimmo_ has joined #openstack-keystone17:56
*** Tahvok has joined #openstack-keystone17:56
*** andreaf has quit IRC17:56
*** agarner has quit IRC17:56
*** SamYaple has quit IRC17:56
*** topol has quit IRC17:56
*** kevinbenton has quit IRC17:56
*** d34dh0r53 has joined #openstack-keystone17:56
*** EmilienM has joined #openstack-keystone17:56
*** Anticimex has joined #openstack-keystone17:56
*** sirushti has quit IRC17:56
*** ktychkova has quit IRC17:56
*** SamYaple has joined #openstack-keystone17:56
*** peterstac has joined #openstack-keystone17:56
*** topol_ has joined #openstack-keystone17:56
*** alex_xu has joined #openstack-keystone17:56
*** bknudson has joined #openstack-keystone17:56
*** ChanServ sets mode: +v bknudson17:56
*** ktychkova has joined #openstack-keystone17:56
*** pkoraca has joined #openstack-keystone17:57
*** dave-mccowan has joined #openstack-keystone17:57
*** Nakato has joined #openstack-keystone17:57
*** kevinbenton has joined #openstack-keystone17:57
*** sirushti has joined #openstack-keystone17:57
*** andreaf has joined #openstack-keystone17:58
*** jose-phillips has joined #openstack-keystone17:59
*** aleph1 has joined #openstack-keystone17:59
*** DuncanT has quit IRC17:59
*** DuncanT has joined #openstack-keystone18:01
*** lamt has quit IRC18:03
*** alex_xu has quit IRC18:06
*** alex_xu has joined #openstack-keystone18:08
openstackgerritMerged openstack/keystone master: Update endpoint api for optional region_id  https://review.openstack.org/42008518:12
*** lamt has joined #openstack-keystone18:13
*** Ephur has joined #openstack-keystone18:17
lbragstadmorgan bang up job on https://review.openstack.org/#/c/424862/718:17
openstackgerritDavid Stanek proposed openstack/keystone master: Federated mapping doc improvements  https://review.openstack.org/42285318:21
dstanek^ i hope that help clear up some of the confusion18:23
*** lucas_ has joined #openstack-keystone18:23
lbragstaddstanek nice!18:23
samueldmqdstanek: is "Federated mapping doc improvements" ready for reviews ?18:28
samueldmqor is there more to come in that patch yet ?18:28
*** stingaci has joined #openstack-keystone18:33
dstaneksamueldmq: that's it for that patch18:34
stevemardoes someone want to do the sql migration placeholder patch for ocata backports? https://github.com/openstack/keystone/commit/3a84987ce0388a7131d9f13b1ae2ced3ba083ada ?18:34
dstaneki'm working on more docs stuff, but i'll be submitting other patches18:34
stevemari was just going to do it, but meeting time18:34
stevemarand i don't want us to merge a migration before thats in :O18:34
*** stingaci has quit IRC18:36
*** tesseract has quit IRC18:38
*** chlong has joined #openstack-keystone18:38
openstackgerritTin Lam proposed openstack/pycadf master: Make `is_valid` more flexible with uuid validation  https://review.openstack.org/42854318:39
openstackgerritRichard Avelar proposed openstack/keystone master: WIP add db_sync check  https://review.openstack.org/41638318:40
openstackgerritRichard Avelar proposed openstack/keystone master: WIP add db_sync check  https://review.openstack.org/41638318:41
*** chris_hultin is now known as chris_hultin|AWA18:44
samueldmqstevemar: I can do that if you're not on it yet19:00
samueldmqstevemar: how many slots do we need ?19:00
samueldmq5 as well ?19:02
*** Mech422 has joined #openstack-keystone19:10
Mech422Hi! Anyone had problems with Newton on a percona cluster when trying to run 'keystone manage bootstrap' it fails with a FK_CONSTRAINT error19:10
Mech422for some reason it thinks RegionOne already exists, when in fact, the region table is empty19:10
Mech422Odd thing is - it only fails like HALF the time ?19:10
*** chlong has quit IRC19:11
openstackgerritSamuel de Medeiros Queiroz proposed openstack/keystone master: Add placeholder migrations for Ocata  https://review.openstack.org/42898419:12
samueldmqstevemar: ^19:12
*** lucas_ has quit IRC19:12
*** lucas_ has joined #openstack-keystone19:12
*** lucas_ has quit IRC19:14
*** jdennis has quit IRC19:15
*** lucas_ has joined #openstack-keystone19:15
*** chlong has joined #openstack-keystone19:21
openstackgerritSamuel Pilla proposed openstack/keystone master: Renaming of api parameters  https://review.openstack.org/42883019:23
openstackgerritSamuel Pilla proposed openstack/keystone master: Renaming of api parameters  https://review.openstack.org/42883019:24
*** lamt has quit IRC19:25
*** lamt has joined #openstack-keystone19:26
*** ravelar has quit IRC19:26
*** chlong has quit IRC19:27
*** lucas_ has quit IRC19:27
samueldmqspilla: hey, just a nit on ^19:27
*** lucas_ has joined #openstack-keystone19:27
samueldmqspilla: looking good, thanks for doing that19:27
spillasamuelmq: ah, good call! and no problem19:29
*** ravelar has joined #openstack-keystone19:30
Mech422Hmm - it appears I'm getting a Conflict error trying to create RegionOne during bootstrap ?19:30
Mech422thats....odd19:30
openstackgerritSamuel Pilla proposed openstack/keystone master: Renaming of api parameters  https://review.openstack.org/42883019:33
*** stingaci has joined #openstack-keystone19:35
*** nkinder has joined #openstack-keystone19:37
*** chlong has joined #openstack-keystone19:41
*** stingaci has quit IRC19:48
morganstevemar: we should convert to alembic19:52
morganstevemar: trhen we don't need placeholders19:52
*** Marcellin__ has joined #openstack-keystone19:52
*** adrian_otto1 has quit IRC19:55
*** lucas_ has quit IRC19:55
*** adrian_otto has joined #openstack-keystone19:57
*** lucas_ has joined #openstack-keystone20:04
*** lucas_ has quit IRC20:08
mtreinishmorgan: hooray for linked lists! :)20:10
morganmtreinish: right?!20:12
openstackgerritMorgan Fainberg proposed openstack/keystone master: Deprecate (and slate for removal) UUID tokens  https://review.openstack.org/42838820:18
openstackgerritMorgan Fainberg proposed openstack/keystone master: Deprecate (and slate for removal) UUID tokens  https://review.openstack.org/42838820:19
*** pramodrj07 has joined #openstack-keystone20:24
*** pramodrj07 has quit IRC20:24
*** MasterOfBugs has joined #openstack-keystone20:25
*** lucas_ has joined #openstack-keystone20:26
openstackgerritDavid Stanek proposed openstack/keystone master: Federated mapping doc improvements  https://review.openstack.org/42285320:33
openstackgerritRichard Avelar proposed openstack/keystone master: WIP add db_sync check  https://review.openstack.org/41638320:33
*** lucas_ has quit IRC20:33
*** stingaci has joined #openstack-keystone20:34
*** chris_hultin|AWA is now known as chris_hultin20:34
*** lucas_ has joined #openstack-keystone20:34
*** lucas_ has quit IRC20:39
*** adrian_otto has quit IRC20:40
*** lucas_ has joined #openstack-keystone20:44
lbragstadantwash these two are failing?20:48
lbragstadantwash https://github.com/openstack/keystone/blob/9bb57c1eb7f295d1c7d37f880f5ed99bbee766a4/keystone/tests/unit/resource/test_backends.py#L1508-L1539 ?20:48
antwashlbragstad : yeah the assert methods are complaining about the key -- i'm trying to figure out how to print the user <k,v> so I can see what all keys are there20:49
antwashprint isn't working20:50
antwashI'm going to try and use log as a work around and see if it works20:51
Mech422so it appears my problem with keystone bootstrap goes away if I disable memcached caching20:52
Mech422commenting out [cache] backend/enabled/memcache_servers allows me to (re-)install keystone several times in a row20:53
lbragstadantwash this line? https://github.com/openstack/keystone/blob/9bb57c1eb7f295d1c7d37f880f5ed99bbee766a4/keystone/tests/unit/resource/test_backends.py#L151720:54
openstackgerritDavid Stanek proposed openstack/keystone master: Federated mapping doc improvements  https://review.openstack.org/42285320:55
dstanekMech422: you're trying to run bootstrap a second time?20:55
*** lucas_ has quit IRC20:56
Mech422dstanek: I'm testing some ansible roles, and installing / purging keystone20:56
Mech422dstanek: I thought my role was broken because it failed to re-install about every other time20:57
dstanekMech422: yeah, that doesn't surprise me at all. when you purge the database you'll have to purge the cache too20:57
Mech422dstanek: instead, it appears that keystone is picking up stale cache entries from memcache20:57
Mech422dstanek: heh - yeah, only cost me 6 hours :-P *sigh* TGIF!!20:58
antwashlbragstad : yeap -- 1517, 1521, 153920:59
lbragstadantwash huh - what's in the user reference there?21:00
stevemarsamueldmq: pep8 o_O21:00
stevemarhttps://review.openstack.org/#/c/428984/121:00
antwashlbragstad : I didn't change anything it's the same, that's why I'm not understand why it's not passing21:01
lbragstadantwash well - it is a stale patch, so something could have merged into master that changes things in the user reference21:01
lbragstadand it might be failing on what ever merged into master now that you've rebased it, right?21:01
antwashlbragstad if only I could see the <k.v> :(21:02
antwashI'm going to search thru some more test and see21:02
lbragstadantwash how are you running the tests?21:03
antwashtox -e py27 <test.method>21:03
dstanekshould we be calling is 'keystone-to-keystone' instead of 'Keystone to Keystone' federation?21:04
stevemarantwash: you can add pdb breakpoints to debug21:04
lbragstaddstanek I think I'd opt for keystone-to-keystone?21:04
stevemarantwash: "import pdb; pdb.set_trace()" in the code, and run it with "tox -e debug <test>"21:05
dstanekantwash: do you have a failing test?21:05
openstackgerritDavid Stanek proposed openstack/keystone master: Federated mapping doc improvements  https://review.openstack.org/42285321:05
dstaneklbragstad: yeah, i already jumped ahead and fix it ^21:06
dstanekdoes anyone know if there are guide for using codenames at the begininng of a sentence? i was treating it like iPhone, but maybe i shouldn't?21:06
lbragstadantwash I was able to add http://cdn.pasteraw.com/20v41xhby9t2dgc2qf7b9wjcub9fnhf to the tests and get user output from `tox -e py27 -- test_delete_project_clears_default_project_id`21:06
lbragstaddstanek I usually try and find a wording that doesn't put it at the beginning of the sentence so that I can avoid making a decision about the convention21:07
dstaneklol21:07
* lbragstad is probably a bad person21:08
lbragstadgonna step away for an ultra late lunch and coffee quick21:09
lbragstadback in 1521:09
*** lucas_ has joined #openstack-keystone21:11
*** catintheroof has quit IRC21:11
*** raildo has quit IRC21:11
*** thiagolib has quit IRC21:18
lbragstadantwash you haven't pushed a patch for review yet have you?21:23
openstackgerritSamuel Pilla proposed openstack/keystone master: Renaming of api parameters  https://review.openstack.org/42883021:24
antwashlbragstad : no I haven't but I figured out why it's failing21:24
lbragstadantwash nice!21:25
lbragstadantwash what was it?21:25
antwashlbragstad : when the identity_api deletes the project, it removes the <k> from the dic as well -- so that's why it's saying KeyError -- the key doesn't exist anymore lol21:25
lbragstadantwash aha21:26
lbragstadantwash so 'default_project_id' isn't actually in the reference as opposed to it being None or something like that21:26
antwashlbragstad : exactly, it should be there but as None, but it's not soo yeah something needs to be modified with core.py21:28
dstanekantwash: is it failing on jenkins?21:28
antwashor maybe another place -- least I know why it's failing at least21:28
antwashdstanek : nope in my local env unit test21:28
lbragstadantwash can you push what you have?21:28
antwashlbragstad : sure I can21:28
dstanekantwash: publish the code and i'll take a look21:28
lbragstaddstanek antwash ++21:28
*** r-daneel has joined #openstack-keystone21:33
openstackgerritAnthony Washington proposed openstack/keystone master: Clear the project ID from user information  https://review.openstack.org/42904721:35
dstanekantwash: which tests are failing?21:36
antwashnot sure why the commit message says conflicts but *shrugs*21:36
antwashdstanek : test_delete_project_clears_default_project_id / test_delete_project_with_roles_clears_default_project_id21:37
lbragstadantwash if you cherry pick a patch from something and there is a conflict, it will automatically be amended to the commit message21:37
lbragstaddstanek https://github.com/openstack/keystone/blob/9bb57c1eb7f295d1c7d37f880f5ed99bbee766a4/keystone/tests/unit/resource/test_backends.py#L1508-L153921:37
dstanekantwash: git does that automatically when you have local conflicts; you have to remove that by hand21:37
openstackgerritAnthony Washington proposed openstack/keystone master: Clear the project ID from user information  https://review.openstack.org/42904721:38
openstackgerritGage Hugo proposed openstack/keystone master: Fix multiple uuid warnings with pycadf  https://review.openstack.org/42641121:39
gagehugostevemar: no more warnings with that ^ when using the updated pycadf patch21:39
dstanekantwash: this passes for me "tox -e py27 -- unit.resource" - which specific test is failing?21:43
*** chlong has quit IRC21:44
antwashdstanek : this the traceback I was getting http://paste.openstack.org/show/597634/  -- maybe my env is screwed up some where.21:45
lbragstadantwash pulling it down locally, too21:46
dstanekantwash: you don't have any changes you didn't push?21:46
antwashdstanek : I know when I was able to see the user <k,v> it was removing the default_project_id completely instead of setting to None -- which isn't the behavior I was expecting to get21:46
antwashthem are all the files that were modified on my end21:47
dstanekantwash: ah, ok. i know the test that is failing now :-)21:47
lbragstadantwash dstanek passes locally form e21:47
lbragstadfor me*21:47
*** chris_hultin is now known as chris_hultin|AWA21:47
dstanekok, i have it failing for me now21:53
morganlbragstad: see responses to comments: https://review.openstack.org/#/c/428388/21:54
*** ChanServ sets mode: +v topol_21:54
*** topol_ is now known as topol21:54
*** lucas_ has quit IRC21:54
* antwash about to get food <late lunch> brb 21:55
*** nkinder has quit IRC21:55
*** stingaci has quit IRC21:55
*** spzala_ has quit IRC21:55
*** adrian_otto has joined #openstack-keystone21:57
*** edtubill has quit IRC21:59
dstaneklbragstad: antwash: i think that test is just incorrect22:03
dstaneklbragstad: does it pass for you?22:03
lbragstaddstanek https://github.com/openstack/keystone/blob/9bb57c1eb7f295d1c7d37f880f5ed99bbee766a4/keystone/tests/unit/resource/test_backends.py#L1523-L1539 ?22:03
dstanekspecifically running 'tox -e py27 -- test_delete_project' - i don't think it can work as written22:03
lbragstaddstanek huh - yeah it's broken22:04
*** edmondsw has quit IRC22:04
*** stingaci has joined #openstack-keystone22:04
dstaneklbragstad: antwash: we don't return that key if it's None - http://git.openstack.org/cgit/openstack/keystone/tree/keystone/identity/backends/sql_model.py#n21922:04
*** edmondsw has joined #openstack-keystone22:05
dstaneklbragstad: antwash: and that test is looking for a None value22:05
lbragstaddstanek yeah - so the test is wrong22:05
dstaneklbragstad: antwash: it should just make sure that key doesn't exist22:05
lbragstaddstanek but it probably merged that way because it was wrapped with a wip decorator22:05
*** edmondsw has quit IRC22:05
*** edmondsw has joined #openstack-keystone22:05
stevemargagehugo: booya22:11
*** spilla has quit IRC22:11
dstaneklbragstad: yeah, most likely22:16
*** thorst_ has quit IRC22:25
*** Marcellin__ has quit IRC22:29
*** mvk has joined #openstack-keystone22:38
*** edmondsw has quit IRC22:41
*** chlong has joined #openstack-keystone22:42
openstackgerritLance Bragstad proposed openstack/keystone master: Ensure token expiration is always consistent  https://review.openstack.org/42906722:44
*** stingaci has quit IRC22:51
*** jaosorior has quit IRC22:52
*** stingaci has joined #openstack-keystone22:53
openstackgerritEric Brown proposed openstack/keystone master: Use httplib constants for http status codes  https://review.openstack.org/37985522:54
*** stingaci has quit IRC22:57
*** spzala has joined #openstack-keystone23:02
*** chris_hultin|AWA is now known as chris_hultin23:02
*** chris_hultin is now known as chris_hultin|AWA23:03
*** adrian_otto has quit IRC23:05
openstackgerritSteve Martinelli proposed openstack/keystone master: clean up newton release notes  https://review.openstack.org/42907123:06
*** spzala has quit IRC23:06
antwashdstanek : cool I see thats what I was thinking as well, when I realized the key isn't even there23:26
antwashlbragstad: dstanek : thanks for looking at it23:26
*** jdennis has joined #openstack-keystone23:27
*** adrian_otto has joined #openstack-keystone23:30
*** Ephur has quit IRC23:36
*** Ephur has joined #openstack-keystone23:43
openstackgerritRichard Avelar proposed openstack/keystone master: Add --check to keystone-manage db_sync command  https://review.openstack.org/41638323:44
*** adu has joined #openstack-keystone23:46
openstackgerritRichard Avelar proposed openstack/keystone master: Add unit test for db_sync run out of order  https://review.openstack.org/42911323:54
*** adu has quit IRC23:55
openstackgerritMerged openstack/keystone master: Renaming of api parameters  https://review.openstack.org/42883023:56

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