Monday, 2018-04-23

*** germs has quit IRC00:00
*** spzala has quit IRC00:02
*** spzala has joined #openstack-keystone00:11
*** spzala has quit IRC00:15
*** spzala has joined #openstack-keystone00:22
*** spzala has quit IRC00:22
*** spzala has joined #openstack-keystone00:22
*** spzala has quit IRC00:23
*** spzala has joined #openstack-keystone00:26
*** spzala has quit IRC00:31
*** dave-mcc_ has joined #openstack-keystone01:02
*** namnh has joined #openstack-keystone01:17
*** spzala has joined #openstack-keystone01:42
*** sapd has joined #openstack-keystone01:43
*** spzala has quit IRC01:54
*** masuberu has quit IRC02:06
*** masuberu has joined #openstack-keystone02:06
*** spzala has joined #openstack-keystone02:10
*** spzala has quit IRC02:14
*** spzala has joined #openstack-keystone02:15
*** annp has joined #openstack-keystone02:16
*** spzala has quit IRC02:19
*** dklyle has joined #openstack-keystone02:41
*** dklyle has quit IRC03:00
*** masuberu has quit IRC03:06
*** masuberu has joined #openstack-keystone03:08
*** dangtrinhnt has quit IRC03:09
*** masber has joined #openstack-keystone03:15
*** masuberu has quit IRC03:18
*** rcernin has quit IRC03:31
*** rcernin has joined #openstack-keystone03:35
*** edmondsw has joined #openstack-keystone03:55
*** dave-mcc_ has quit IRC04:00
*** sonuk has quit IRC04:03
*** bhagyashris has joined #openstack-keystone04:10
*** jdennis has quit IRC04:33
*** jdennis has joined #openstack-keystone04:34
*** pooja_jadhav has joined #openstack-keystone04:47
*** spzala has joined #openstack-keystone04:58
*** spzala has quit IRC05:02
*** edmondsw has quit IRC05:03
*** edmondsw has joined #openstack-keystone05:03
*** edmondsw has quit IRC05:08
*** links has joined #openstack-keystone06:13
*** spzala has joined #openstack-keystone06:51
*** spzala has quit IRC06:55
*** tesseract has joined #openstack-keystone07:04
*** sapd has quit IRC07:13
*** rcernin has quit IRC07:19
*** pcaruana has joined #openstack-keystone07:25
*** AlexeyAbashkin has joined #openstack-keystone07:36
*** pcichy has joined #openstack-keystone08:00
*** jaosorior has joined #openstack-keystone08:15
*** threestrands has quit IRC08:17
*** jaosorior has quit IRC08:32
*** Alexey_Abashkin has joined #openstack-keystone08:48
*** Alexey_Abashkin has quit IRC08:50
*** Alexey_Abashkin has joined #openstack-keystone08:50
*** AlexeyAbashkin has quit IRC08:51
*** sapd has joined #openstack-keystone08:53
*** AlexeyAbashkin has joined #openstack-keystone08:54
*** Alexey_Abashkin has quit IRC08:54
*** Alexey_Abashkin has joined #openstack-keystone09:02
*** AlexeyAbashkin has quit IRC09:05
*** Alexey_Abashkin is now known as AlexeyAbashkin09:05
*** jaosorior has joined #openstack-keystone09:07
*** namnh has quit IRC09:22
*** namnh has joined #openstack-keystone09:23
*** annp has quit IRC09:26
*** annp has joined #openstack-keystone09:27
*** spzala has joined #openstack-keystone10:07
*** nicolasbock has joined #openstack-keystone10:09
*** spzala has quit IRC10:11
*** namnh has quit IRC10:17
*** pcichy has quit IRC10:22
*** pcaruana has quit IRC10:26
*** pcaruana has joined #openstack-keystone10:33
*** nicolasbock has quit IRC10:39
*** AlexeyAbashkin has quit IRC10:47
*** nicolasbock has joined #openstack-keystone10:50
*** AlexeyAbashkin has joined #openstack-keystone10:51
*** hoonetorg has quit IRC11:03
*** hoonetorg has joined #openstack-keystone11:16
*** hoonetorg has quit IRC11:21
*** hoonetorg has joined #openstack-keystone11:39
*** edmondsw has joined #openstack-keystone11:47
*** pcaruana has quit IRC11:54
*** pcaruana has joined #openstack-keystone12:10
*** doxa has quit IRC12:10
*** raildo has joined #openstack-keystone12:17
*** martinus__ has joined #openstack-keystone12:22
*** jdennis has quit IRC12:27
*** jdennis has joined #openstack-keystone12:27
*** edmondsw has quit IRC12:37
*** rmascena has joined #openstack-keystone12:37
*** raildo has quit IRC12:40
*** spzala has joined #openstack-keystone12:42
*** panbalag has joined #openstack-keystone12:42
*** panbalag has left #openstack-keystone12:46
*** spzala has quit IRC12:46
*** dave-mccowan has joined #openstack-keystone12:49
*** spzala has joined #openstack-keystone12:52
*** spzala has quit IRC12:57
*** spzala has joined #openstack-keystone13:00
*** spzala has quit IRC13:05
*** spzala has joined #openstack-keystone13:05
*** spilla has joined #openstack-keystone13:07
*** spzala has quit IRC13:25
*** spzala has joined #openstack-keystone13:25
*** spzala has quit IRC13:25
*** spzala has joined #openstack-keystone13:26
*** spzala has quit IRC13:26
*** r-daneel has joined #openstack-keystone13:37
*** felipemonteiro_ has joined #openstack-keystone13:38
*** felipemonteiro__ has joined #openstack-keystone13:44
lbragstadkmalloc: yeah - that's a good question13:46
*** felipemonteiro_ has quit IRC13:47
*** r-daneel has quit IRC13:51
lbragstadi'm not sure why Tripleo happen to hit that index, i haven't heard of that being reported anywhere else13:52
lbragstadunless it was how it was backported?13:52
*** edmondsw has joined #openstack-keystone13:58
*** felipemonteiro_ has joined #openstack-keystone14:00
*** felipemonteiro__ has quit IRC14:00
*** spzala has joined #openstack-keystone14:04
*** felipemonteiro__ has joined #openstack-keystone14:20
*** felipemonteiro_ has quit IRC14:24
*** links has quit IRC14:30
*** jroll has quit IRC14:31
*** jroll has joined #openstack-keystone14:32
hrybackikmalloc: lbragstad so WRT https://review.openstack.org/#/c/563266/14:35
hrybackithe reason why we hit it is because we backported it downstream to OSP10 (IIRC) w/o making it idempotent14:35
hrybackiOSP10 and 11 that is. In one of the placeholders for each release respectively14:36
hrybackiI'm not sure if this is a fix we would make upstream tbh. It may be something we need to carry downstream alone14:37
lbragstadi suppose we could do it either way14:37
lbragstadthat's really why i wanted to make sure kmalloc looked at it14:37
hrybackiupstream would be nice but I'm also not trying to force our issues up here14:37
hrybackiyeah14:37
hrybackiIF any other folks carried it into their products in previous versions they may well run into the same issue so \_0_/14:38
lbragstadsure14:38
lbragstadthat's true14:38
hrybackikmalloc's comment on the review confuses me. But I think that may be because he didn't have context when reviewing14:48
*** Alexey_Abashkin has joined #openstack-keystone14:50
*** felipemonteiro__ has quit IRC14:51
*** felipemonteiro__ has joined #openstack-keystone14:51
*** r-daneel has joined #openstack-keystone14:53
*** AlexeyAbashkin has quit IRC14:54
*** Alexey_Abashkin is now known as AlexeyAbashkin14:54
*** felipemonteiro_ has joined #openstack-keystone15:09
*** felipemonteiro__ has quit IRC15:13
knikollao/15:16
hrybackio/15:16
gagehugoo/15:16
*** pcaruana has quit IRC15:19
*** felipemonteiro_ has quit IRC15:20
*** felipemonteiro_ has joined #openstack-keystone15:20
*** dklyle has joined #openstack-keystone15:32
lbragstadcmurphy: kmalloc i know we discussed this a bit - but wondering if either of you would want to leave comments on the review about the usecases - https://review.openstack.org/#/c/541903/615:45
*** dklyle has quit IRC15:49
kmalloclbragstad: sure15:50
kmallochrybacki: oh, DONT BACKPORT MIGRATIONS15:51
hrybackikmalloc: yeah, lesson learned ;)15:51
kmallochrybacki: if you do, you're liable to run into this.15:51
kmallochonestly, that makes me want to -1 this change.15:51
kmallocand say no, this is not meant to be idempotent, i dont want it to become "standard" to fix migrations because a vendor backported it poorly15:52
kmalloceven though I totally understand an am sympathetic to the situation ;)15:52
hrybackikmalloc: either way we'll have to carry the fix downstream15:52
kmalloci wouoldn't honestly mind all migrations being idempotent15:52
kmallocso... let me start here:15:53
hrybackithat seems sensical15:53
kmalloc1) i'll accept the fix if you're willing to propose and drive an initiative to make all migrations idempotent15:53
kmalloc(and get the rest of the cores to accept it+write the tests to prevent regressing15:53
kmalloc2) i'll -1 it and it's carried downstream15:54
hrybackiso I'm happy to raise the idempotent migrations to the group15:54
kmalloci don't expect you to fix all migrations in the history, just furture migrations and tests that act without manual changes (all future migrations would be caught in the new test)15:54
hrybackibut I'm not well versed enough speak with confidence as to what all that will entail15:54
kmallocbasically: you need to propose a spec / bring it up in the meeting. And you'll need to write a test that says "any migration after X we run multiple times to show it is idempotent"15:55
hrybackiregardless we'll have to backport this (downstream) further than it's maintained upstream. And I think that <-- is the larger issue we should try to think about. I'm sure other vendors run into this -- and not just with migration related changes15:55
kmallocbackporting migrations is hard.15:56
kmallocdoing it right that is15:56
kmalloci would generally encourage the "just carry the fix downstream"15:56
kmallocbecause the use case for idempotent migrations really is "supporting backports"15:56
kmallocoutside of upstream15:56
* hrybacki nods15:56
kmalloctotally unrelated... I have a picture for everyone to see.15:57
* hrybacki nods15:57
* lbragstad bets it's of a dog15:57
kmallochttps://usercontent.irccloud-cdn.com/file/kcVMJqSv/IMG_20180423_082244.jpg15:58
hrybackiI'm about to have our weekly mtg downstream. Will discuss and come back around. Thanks for your input kmalloc !15:58
lbragstadnailed it15:58
hrybackialso omfg toocute15:58
cmurphykmalloc: ^.^15:58
lbragstadkmalloc: was that your weekend project?15:59
kmallochttps://photos.app.goo.gl/KgkInMCoiLR7cONn115:59
kmalloclbragstad: hehe16:01
gagehugothe arf! blanket is very nice16:01
kmallocgagehugo: yeah. it's her fav travel pillow (blanket)16:01
kmallocin that car photo, she's so tired. she could barely keep her head up [not that you could see it from the photo]16:02
lbragstadlol - coffee tables make great hiding places16:02
*** gyee has joined #openstack-keystone16:03
jrollkmalloc: zomg, that car photo16:03
* jroll feels obligated to also post puppy pics: https://i.imgur.com/Yswxtpy.jpg16:04
lbragstadjroll: it that one yours?!16:05
jrolllbragstad: yep, brought him home a week-ish ago :D16:05
*** lbragstad changes topic to "Talking dogs at the water cooler"16:05
lbragstadjroll: whoa - what breed?16:06
jrollaustralian shepherd, half mini half standard. should end up around 30-40 lbs16:06
lbragstadthat's like the perfect size16:06
jrollyeah, it will be16:07
jrollour other dog is about 50 pounds: https://imgur.com/W9prSCL.jpg16:08
lbragstadlol - i like the hanker chief16:08
kmallocjroll: damn cute aussie16:08
jrollheh, thanks :)16:08
kmallocjroll: aussies are too high energy (lol, so we got a shiba... wait wut!?) for our space.16:09
jrolllol16:09
kmalloci just had to step away, she just peed in the house =/ 9wks, still learning the WHOLE house is her home and shouldn't be pee-central16:09
jrollyeah, I expect to spend lots of time running around outside16:09
kmallocthis weekend i did ~5mi with her16:09
jrollnice16:10
kmallocshe just doesn't stop (a lot of back-and-forth in front of the house, but my phones says it was 5mi)16:10
kmallocshe is learning what a vacuum cleaner is today (we have cleaners in for the first time since we got her)16:10
kmallocshe seems unconcerned16:10
openstackgerritMerged openstack/keystone master: Update auth_uri option to www_authenticate_uri  https://review.openstack.org/56227916:10
lbragstadlol16:10
openstackgerritGage Hugo proposed openstack/keystone master: Add functional testing gate  https://review.openstack.org/53101416:11
lbragstadi will say i'm excited to get back into the spring/summer running routine with our dogs16:11
*** pcaruana has joined #openstack-keystone16:13
openstackgerritGage Hugo proposed openstack/keystone master: Add LDAP user-backed functional testing gate  https://review.openstack.org/55894016:15
kmallocgagehugo: check with infra on the docker bit, they may want to mirror the repo directly16:16
kmallocgagehugo: typically it is a nogo to rely on external sources not mirrored16:16
*** lbragstad changes topic to "Rocky release schedule: https://releases.openstack.org/rocky/schedule.html | Meeting agenda: https://etherpad.openstack.org/p/keystone-weekly-meeting | Bugs that need triaging: http://bit.ly/2iJuN1h | Trello: https://trello.com/b/wmyzbFq5/keystone-rocky-roadmap"16:17
gagehugokmalloc ok16:17
kmallocgagehugo: notably because i also can't endorse that as a voting change if we can't guarantee docker sources wont be down (we need pip, we neet apt, we don't need much else)16:18
gagehugoyeah that's a good point16:18
gagehugodon't wanna rely on docker being always up16:18
kmallocexactly16:19
kmallocthere are some docker packages in ubuntu, if they aren't too old, they may be sufficient16:19
kmallocbut in the case we need more recent, we should talk with infra to mirror16:19
gagehugoyeah loci requires a more recent version, hopefully the ubuntu ones are good enough16:22
* lbragstad breaks for lunch quick16:25
kmallocholy crap, man, gagehugo thanks for pointing me at loci16:28
kmallocthat is going to make my current project WAY easier16:28
gagehugokmalloc :)16:28
kmalloctrying to setup an openstack control plane on my Nuc16:29
kmalloc(so, keystone, nova, neutron, cinder, glance)16:29
*** edmondsw has quit IRC16:29
gagehugoit's super useful16:31
kmallocalso... waiting for 18.04 to land (need to do some sizable upgrades)16:31
kmallochonestly, i'd use RHEL 7.5 (or Cent 7.5 when it comes out) but its a but more work.16:31
kmalloci plan to run my FreeIPA server on Cent though16:31
gagehugohmm16:31
kmalloconce i have OpenStack running.16:31
*** Alexey_Abashkin has joined #openstack-keystone16:31
kmallocand since 18.04 drops stable in like 4 days16:32
gagehugooh yeah it does16:32
gagehugoforgot about that16:32
kmalloclooking forward to upgrading my FW server16:32
*** AlexeyAbashkin has quit IRC16:34
*** Alexey_Abashkin is now known as AlexeyAbashkin16:34
gagehugokmalloc there's an openstack-loci channel too if you have any questions16:35
kmallocnice16:35
*** ayoung has joined #openstack-keystone16:38
ayoungDo we have any regular Keystoners that are fluent in Mandarin Chinese?  I am trying to figure out who to give props to for this site, and I can't deduce the author:  http://www.cnblogs.com/multi-task/p/5553830.html16:42
ayoungGoogle Translate renders the Author's line as "Catcher in a Field of Wheat"  which I assume is a play on "Catcher in the Rhye"  And I see the name translated as Wheat Field Malt...which I am wondiner is a bad translation, or a pun.16:43
*** AlexeyAbashkin has quit IRC16:59
*** jdennis has quit IRC17:02
*** jaosorior has quit IRC17:05
*** jdennis has joined #openstack-keystone17:08
*** pcichy has joined #openstack-keystone17:10
*** jdennis has quit IRC17:13
*** jdennis has joined #openstack-keystone17:17
*** mvk has quit IRC17:22
*** tesseract has quit IRC17:23
*** edmondsw has joined #openstack-keystone17:34
*** pcichy has quit IRC17:34
*** edmondsw has quit IRC17:37
*** edmondsw has joined #openstack-keystone17:37
*** jaosorior has joined #openstack-keystone17:49
kmallocayoung: hm.. gyee know Cantonese right? i'm trying to think of the other folks who might know17:51
kmallocayoung: maybe gyee knows someone to help out17:52
gyeeI can speak Cantonese, not so good with Mandarin though17:59
*** r-daneel has quit IRC18:00
gyeeI can hear Mardarin and understand most of it, but just can't speak it very well :-)18:00
*** r-daneel has joined #openstack-keystone18:01
ayounggyee, it is a reading problem in this case18:02
ayoungcan you read the name off that blog?18:02
ayoung http://www.cnblogs.com/multi-task/p/5553830.html18:02
ayoungThe Author did a good job specifying how to do things, and I'd like to properly cite18:02
gyeeayoung, author is 麦田18:06
gyeetranslate to "wheat field" :-)18:06
ayoungIs that a common name?  Is this "catcher in a a wheat field" a cross-language pun?18:07
gyeeis the blogger's handle18:08
ayoung@gyee, yeah, Google translated that as Wheat Field Malt  which I thought might be a beer reference18:08
*** felipemonteiro__ has joined #openstack-keystone18:08
gyeeit's common for Chinese blogger to use a handle than the real name18:08
ayoungOf course.18:09
ayoung@gyee, I can find out whois info on the site, not sure if that maps close to the blogger here18:10
* ayoung trying to walk that line of showing respect and yet not stalking18:11
*** felipemonteiro_ has quit IRC18:12
gyeeayoung, you'll probably need to join the site, then leave a comment on the blog to ask the author to contact you18:13
ayoung@gyee, ah cool.  I can try to do that.18:13
kmallocayoung: https://usercontent.irccloud-cdn.com/file/hcFYff0T/IMG_20180422_135911.jpg yah, she needs to be a mascot of some project18:18
ayoung+2A18:18
kmallocayoung: you'll meet her when you get out this way for the summit18:19
kmalloclet me know your travel booking so i can confirm my hotel dates.18:19
ayoungkmalloc, flying out on May 19 Saturday, staying with a friend that night.  Drive up together on Sunday.  Drive Back on Thrusday, fly out Friday morning.18:33
ayoungLet me see what hotel I booked...18:34
ayoungkmalloc, I'm at the Pan PAcific18:35
*** harlowja has joined #openstack-keystone18:44
*** jaosorior has quit IRC18:55
hugokuomorning guys.18:55
hugokuoQuick question about the keystonemiddleware.   Does the latest one still support v2 auth API for Admin to fetching token before validating user's incoming token?18:56
ayounghugokuo, um...lets see19:01
ayounghugokuo, IIRC V2 is exposed as a token format, but the middleware should be using V319:02
hugokuoin `keystonemiddleware/auth_token/_identity.py` has keystoneclient.v2_0 lib imported.19:03
hugokuoI'm thinking about if there's any case the keystone server doesn't support v3 API or not fully functional v3 API version.19:04
*** felipemonteiro__ has quit IRC19:06
*** felipemonteiro__ has joined #openstack-keystone19:07
hugokuoayoung: Do you mean if user send token that's fetched from Keystone v2.0 API, it should still working in latest keystonemiddleware? The keystonemiddlewaare's admin fetches admin token and validate user token via v3.0 API?19:07
*** harlowja has quit IRC19:07
ayounghugokuo, so...we had to sustain doing a token validation via V2.0, but what I am not sure is if the service user can usethat...but why would you want to?19:08
ayounga V2 tokebn can be validated by the V3 API...I know this is confusing19:08
*** dklyle has joined #openstack-keystone19:11
ayounghugokuo, looking here https://github.com/openstack/keystonemiddleware/blob/master/keystonemiddleware/auth_token/_auth.py#L4019:11
ayounglooks like it can be done...V2.0 is still referenced.19:12
hrybackilbragstad et. all -- I've created the rocky m1 retro board: https://trello.com/b/PiJecAs4/keystone-rocky-m1-retrospective feel free to add cards anytime :)19:56
*** spzala has quit IRC19:57
*** spzala has joined #openstack-keystone19:57
lbragstadhrybacki: awesome - thanks19:58
lbragstadi'll be sure to take a crack at it sometime before tomorrow19:58
hrybackilbragstad++ thanks!19:59
*** spzala has quit IRC20:02
*** dave-mccowan has quit IRC20:14
*** spzala has joined #openstack-keystone20:17
lbragstadhrybacki: kmalloc given the discussion this morning - do we want to mark this as invalid?20:18
lbragstadhttps://bugs.launchpad.net/keystone/+bug/176583020:18
openstackLaunchpad bug 1765830 in OpenStack Identity (keystone) "migration from osp 10 -> 11 -> 12 fails because migration script 22 is not idempotent" [Undecided,In progress] - Assigned to Ade Lee (alee-3)20:18
kmalloci think so.20:18
kmallocbut i'm willing to wait for hrybacki to come back20:18
hrybackio/20:19
hrybackiRight so, downstream aside I think kmalloc's spec proposal idea for all migrations moving forward being idempotent is a good one that should be followed through on20:19
kmallocit's a LOT of extra work20:20
kmallocto support exactly 1 case: poorly backported migrations20:20
kmallocand data migrations is even harder to make idempotent20:20
kmallocbut i am happy to entertain the idea20:21
hrybackiI'm thinking about myself (and other vendors) that have SLAs that go back well past Upstream's master-320:22
kmallocit could also have been solved with a different index20:22
kmallocfwiw20:22
kmallocuse a different name, you'd double up the index but it would be fine in general and your new index could be dropped once the real migration lands20:23
hrybackiSure. This incident in particular was my fault. I didn't test the upgrade path across multiple versions bc reasons20:25
hrybackiDoes anyone remember why this particular bug wasn't backported upstream?20:26
hrybacki(back across the stable branches that is)20:26
*** felipemonteiro__ has quit IRC20:29
*** mvk has joined #openstack-keystone20:45
lbragstadhrybacki: that's a good question20:52
lbragstadit could have been that it wasn't perceived as being urgent https://bugs.launchpad.net/keystone/+bug/1523369/comments/820:53
openstackLaunchpad bug 1523369 in OpenStack Identity (keystone) "clean a user's default project if the project has been deleted" [Wishlist,Fix released] - Assigned to Anthony Washington (anthony-washington)20:53
*** idlemind has quit IRC20:59
hugokuoayoung: the only concern was someone still running a very old keystone server which has no v3 API. But if that's the case, they should upgrade the keystone instance anyway.21:01
hugokuothanks for the confirmation. I'll perform some tests here.21:01
*** spilla has quit IRC21:01
ayounghugokuo, yeah, that does not exist21:02
ayoungthere is no Keystone out there that does not support V321:02
*** spilla has joined #openstack-keystone21:02
ayoungV3 went in Grizzly?21:03
lbragstadi believe so21:04
ayounghugokuo, I think that is older than some of the Keystone devs21:05
*** spilla has quit IRC21:07
*** spzala has quit IRC21:17
*** spzala has joined #openstack-keystone21:17
*** martinus__ has quit IRC21:17
*** spzala has quit IRC21:23
*** jdennis has quit IRC21:27
ayounghttps://wiki.openstack.org/wiki/ReleaseNotes/Grizzly#OpenStack_Identity_.28Keystone.2921:28
ayoung'The term "tenant" is now known as "project." '21:28
*** dklyle has quit IRC21:31
*** r-daneel_ has joined #openstack-keystone21:35
*** r-daneel has quit IRC21:36
*** r-daneel_ is now known as r-daneel21:36
*** rmascena__ has joined #openstack-keystone21:38
*** spzala has joined #openstack-keystone21:38
*** rmascena has quit IRC21:40
lbragstada lot of stuff happened in grizzly21:42
*** spzala has quit IRC21:43
lbragstadkmalloc: where does keystoneauth set all the headers? or where do we define those headers?21:48
lbragstadi'm shuffling through ksm but i'm not seeing where that happens21:49
*** jdennis has joined #openstack-keystone21:51
kmalloclbragstad: uhm....21:58
kmallocI.am.not sure.withoit looking21:58
lbragstadno worries - i can dig into it22:00
lbragstadjay had a question here and i thought that was handled by ksm https://review.openstack.org/#/c/530509/3/oslo_context/context.py22:00
*** jdennis has quit IRC22:08
*** jrist has quit IRC22:08
*** jdennis has joined #openstack-keystone22:09
*** jrist has joined #openstack-keystone22:12
*** jrist has quit IRC22:12
*** jrist has joined #openstack-keystone22:12
*** lbragstad has quit IRC22:18
*** dklyle has joined #openstack-keystone22:26
*** rcernin has joined #openstack-keystone22:31
kmallocoh lance disconnected22:35
ayounghrybacki, alee__ I just commented on the "Idempotency" fix.  Let me know if my comment makes sense. https://review.openstack.org/#/c/563266/122:38
*** dklyle has quit IRC22:39
ayoungBut in general, lets not spend cycles making a fix idempotent until we need it.  If we backport a fix to one of the reserved numbers in stable, as part of that backport, make the follow on migration Idempotent or a no-op.22:39
kmalloclol hey ayoung you just responded as I was doing so22:40
kmalloc:P22:40
kmallocayoung: your stance is much the same as mine22:40
*** threestrands has joined #openstack-keystone22:41
*** threestrands has quit IRC22:41
*** threestrands has joined #openstack-keystone22:41
kmallocftr, and my general guidance was "unless we change how we do mirgations in general to support idempotency (ask the keystone team at the weekly meeting), we should not accept this fix22:41
*** felipemonteiro__ has joined #openstack-keystone22:42
*** edmondsw has quit IRC22:43
*** felipemonteiro__ has quit IRC23:07
*** spzala has joined #openstack-keystone23:10
*** r-daneel has quit IRC23:21
*** spzala has quit IRC23:21

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