Friday, 2018-05-04

*** gyee has quit IRC00:02
cmurphylbragstad: fyi i'll be traveling tomorrow...if you have spare time to send the update email that'd be super ;)00:02
*** markvoelker has quit IRC00:05
*** jistr has quit IRC00:07
*** jistr has joined #openstack-keystone00:13
openstackgerritMerged openstack/keystone master: Remove unused assertions from test_v3.py  https://review.openstack.org/56616300:23
*** dmellado has quit IRC00:31
*** d0ugal has quit IRC00:36
*** dmellado has joined #openstack-keystone00:38
*** edmondsw has joined #openstack-keystone00:53
*** dmellado has quit IRC00:59
*** d0ugal has joined #openstack-keystone01:03
*** edmondsw has quit IRC01:05
*** markvoelker has joined #openstack-keystone01:05
*** annp has joined #openstack-keystone01:16
*** edmondsw has joined #openstack-keystone01:18
*** dmellado has joined #openstack-keystone01:26
*** dmellado has quit IRC01:32
*** markvoelker has quit IRC01:40
*** dmellado has joined #openstack-keystone01:56
*** r-daneel has quit IRC02:06
*** dmellado has quit IRC02:09
*** dmellado has joined #openstack-keystone02:13
*** edmondsw has quit IRC02:18
*** rcernin has quit IRC02:19
*** rcernin has joined #openstack-keystone02:20
adriantcmurphy, kmalloc, lbragstad: Any objections to me doing a patch that renames https://github.com/openstack/keystone/blob/master/keystone/common/token_utils.py#L36 and all the places it is used from TokenUtils to FernetUtils ?02:31
adriantI'd be using that exact same class for receipts and it doesn't make sense to keep calling it token utils at that stage02:32
adriantI'm also looking at what parts of the token/fernet code in the keystone/tokens/ I can also potentially move to common, but it doesn't look like too much. The for receipts is likely going to be VERY similar to keystone/tokens/ but simpler, and I'm not sure how much we can extract into a common place.02:34
adriantUnless something actually 'urgent' comes on the day, up I'm dedicating the latter half of every Friday to the receipt work until it's done. So hopefully I can have a review or two up for that soon.02:36
*** markvoelker has joined #openstack-keystone02:37
adriantHAH: https://github.com/openstack/keystone/commit/03ba867327fe950d3240fdbd6d207a8c0550e2c502:37
adriantwell damn02:37
adriantI guess I'm just going to revert that patch pretty much02:37
adriantgagehugo: ^02:37
*** dmellado has quit IRC02:38
*** germs has quit IRC02:39
*** germs has joined #openstack-keystone02:40
lbragstadadriant: yeah - there was a push to make the utils more generic so that we could essentially do other non-persistent token format using different encryption/signing mechansims03:06
adriantlbragstad: I totally get that, and when that patch came through no one knew we'd be reusing the fernet model for something other than tokens :)03:07
adriantI'll create a set of patches to be reviewed together, with that revert/renamed to fernet_utils as the first.03:08
adriantlbragstad: the alternative is call it something like crypto_utils ?03:09
adriantalthough what we have there right now is pretty specific to fernet03:10
*** markvoelker has quit IRC03:10
*** dave-mccowan has quit IRC03:14
openstackgerritAdrian Turjak proposed openstack/keystone master: [WIP] Rename token_utils back to fernet_utils  https://review.openstack.org/56620803:15
*** felipemonteiro__ has joined #openstack-keystone03:36
*** dmellado has joined #openstack-keystone03:39
*** felipemonteiro_ has joined #openstack-keystone03:39
*** felipemonteiro__ has quit IRC03:43
*** nicolasbock has quit IRC03:44
*** felipemonteiro_ has quit IRC03:45
lbragstadi suppose we reuse them for credentials now too03:53
lbragstadwe encrypt credentials at rest with fernet tokens03:53
lbragstads/tokens/keys/03:53
* lbragstad is on the verge of passing out03:53
*** rcernin has quit IRC03:59
*** rcernin has joined #openstack-keystone04:00
*** edmondsw has joined #openstack-keystone04:00
*** germs has quit IRC04:02
*** germs has joined #openstack-keystone04:02
*** germs has quit IRC04:02
*** germs has joined #openstack-keystone04:02
*** gyankum has joined #openstack-keystone04:04
*** dmellado has quit IRC04:04
*** edmondsw has quit IRC04:05
*** markvoelker has joined #openstack-keystone04:08
*** r-daneel has joined #openstack-keystone04:10
*** r-daneel_ has joined #openstack-keystone04:35
*** r-daneel has quit IRC04:36
*** r-daneel_ is now known as r-daneel04:36
*** dmellado has joined #openstack-keystone04:39
*** markvoelker has quit IRC04:41
*** jaosorior has joined #openstack-keystone04:42
*** bhagyashris has quit IRC04:43
*** bhagyashris has joined #openstack-keystone04:43
*** ianw has quit IRC05:06
*** NobodyCam has quit IRC05:15
*** cz2 has quit IRC05:15
*** Chealion has quit IRC05:15
*** pas-ha has quit IRC05:15
*** hogepodge has quit IRC05:15
*** charz has quit IRC05:15
*** charz has joined #openstack-keystone05:15
*** Chealion has joined #openstack-keystone05:16
*** hogepodge has joined #openstack-keystone05:16
*** pas-ha has joined #openstack-keystone05:16
*** NobodyCam has joined #openstack-keystone05:16
*** cz2 has joined #openstack-keystone05:16
*** markvoelker has joined #openstack-keystone05:38
*** threestrands has quit IRC05:47
*** ianw has joined #openstack-keystone05:55
*** markvoelker has quit IRC06:11
*** pcichy has joined #openstack-keystone06:24
openstackgerritwangxiyuan proposed openstack/keystone master: [WIP]Enable foreign keys for unit test  https://review.openstack.org/55819306:32
*** martinus__ has joined #openstack-keystone06:41
*** pcaruana has joined #openstack-keystone06:43
*** r-daneel has quit IRC06:49
*** alex_xu has quit IRC06:50
*** alex_xu has joined #openstack-keystone06:51
*** dmellado has quit IRC07:01
*** rcernin has quit IRC07:05
*** markvoelker has joined #openstack-keystone07:08
*** tesseract has joined #openstack-keystone07:20
*** dmellado_ has joined #openstack-keystone07:24
*** jaosorior has quit IRC07:25
*** dmellado_ has quit IRC07:34
*** edmondsw has joined #openstack-keystone07:36
*** edmondsw has quit IRC07:41
*** markvoelker has quit IRC07:42
*** slunkad has quit IRC07:45
*** kukacz has quit IRC07:46
*** slunkad has joined #openstack-keystone07:49
openstackgerritMerged openstack/keystone master: Remove dead dependency injection code  https://review.openstack.org/56616807:49
*** kukacz_ has joined #openstack-keystone07:55
*** dmellado_ has joined #openstack-keystone08:01
*** kukacz_ has quit IRC08:01
*** kukacz_ has joined #openstack-keystone08:01
*** pcichy has quit IRC08:05
*** pcichy has joined #openstack-keystone08:05
*** kukacz_ has quit IRC08:07
*** kukacz_ has joined #openstack-keystone08:11
*** dmellado_ has quit IRC08:11
*** kukacz_ has quit IRC08:14
*** kukacz_ has joined #openstack-keystone08:14
*** markvoelker has joined #openstack-keystone08:39
*** germs has quit IRC08:43
openstackgerritwangxiyuan proposed openstack/keystone master: Update IdP sql model  https://review.openstack.org/55967608:43
openstackgerritwangxiyuan proposed openstack/keystone master: Fix the test for unique IdP  https://review.openstack.org/56381208:43
*** germs has joined #openstack-keystone08:44
*** germs has quit IRC08:44
*** germs has joined #openstack-keystone08:44
*** lbragstad has quit IRC08:45
openstackgerritwangxiyuan proposed openstack/keystone master: Enable Foreign keys for sql backend unit test  https://review.openstack.org/55802908:45
openstackgerritwangxiyuan proposed openstack/keystone master: Enable foreign keys for unit test  https://review.openstack.org/55819308:45
openstackgerritwangxiyuan proposed openstack/keystone master: Fix db model inconsistency for FederatedUser  https://review.openstack.org/56624208:45
*** lbragstad has joined #openstack-keystone08:45
*** ChanServ sets mode: +o lbragstad08:45
*** dmellado_ has joined #openstack-keystone08:50
*** pcichy has quit IRC09:01
*** markvoelker has quit IRC09:12
*** markvoelker has joined #openstack-keystone10:08
*** r-daneel has joined #openstack-keystone10:29
*** mvk has quit IRC10:30
*** markvoelker has quit IRC10:42
*** dave-mccowan has joined #openstack-keystone11:13
*** pcaruana has quit IRC11:27
*** raildo has joined #openstack-keystone11:37
*** markvoelker has joined #openstack-keystone11:39
*** pcaruana has joined #openstack-keystone11:39
*** jdennis has quit IRC11:39
*** nicolasbock has joined #openstack-keystone11:42
*** linkmark has quit IRC11:51
*** edmondsw has joined #openstack-keystone11:53
*** raildo has quit IRC11:54
*** annp has quit IRC11:57
*** pcaruana has quit IRC11:59
*** jistr is now known as jistr|mtg12:11
*** pcaruana has joined #openstack-keystone12:12
*** markvoelker has quit IRC12:12
*** markvoelker has joined #openstack-keystone12:24
*** mchlumsky has joined #openstack-keystone12:26
*** mchlumsky has quit IRC12:30
*** jistr|mtg is now known as jistr12:31
*** mchlumsky has joined #openstack-keystone12:32
*** slunkad has quit IRC12:45
*** raildo has joined #openstack-keystone12:51
*** felipemonteiro has joined #openstack-keystone12:55
*** jdennis has joined #openstack-keystone13:00
*** gyankum has quit IRC13:09
*** r-daneel has quit IRC13:16
*** eckesicle has joined #openstack-keystone13:26
*** slunkad has joined #openstack-keystone13:47
*** r-daneel has joined #openstack-keystone13:52
gagehugoadriant go for it13:58
*** r-daneel has quit IRC13:59
*** r-daneel has joined #openstack-keystone13:59
*** Guest46098 has quit IRC14:13
*** Guest46098 has joined #openstack-keystone14:13
*** melwitt is now known as jgwentworth14:15
*** pcichy has joined #openstack-keystone14:17
*** germs has quit IRC14:39
*** germs has joined #openstack-keystone14:40
*** germs has quit IRC14:40
*** germs has joined #openstack-keystone14:40
*** r-daneel has quit IRC14:45
*** jistr is now known as jistr|tpb14:49
*** r-daneel has joined #openstack-keystone14:52
*** dmellado_ is now known as dmellado14:57
*** gyankum has joined #openstack-keystone14:59
*** jistr|tpb is now known as jistr15:05
*** r-daneel has quit IRC15:06
*** r-daneel has joined #openstack-keystone15:06
*** pcaruana has quit IRC15:13
*** dave-mccowan has quit IRC15:20
hrybackilbragstad: so should I port our spec to keystone-specs?15:22
hrybackialso, what about the Patrole spec?15:22
hrybackiIt already exists in keystone-specs so we could simply un-abandon it. Since we are really just aiming for PoC15:22
*** fiddletwix has quit IRC15:22
*** pooja_jadhav has quit IRC15:24
*** sonuk has joined #openstack-keystone15:26
*** nicolasbock has quit IRC15:26
*** dave-mccowan has joined #openstack-keystone15:27
lbragstadhrybacki: sure15:31
lbragstadmaybe start a ml thread, too?15:31
gagehugohrybacki patrole would be fine in keystone-specs imo15:31
lbragstadbits and pieces of both would make it into a community goal if we decided to propose it for Stein15:38
lbragstadbecause they are so tightly coupled15:38
*** nicolasbock has joined #openstack-keystone15:40
lbragstadayoung: :) http://eavesdrop.openstack.org/irclogs/%23openstack-tc/%23openstack-tc.2018-05-04.log.html#t2018-05-04T15:23:1915:42
ayounglbragstad, I think I might need more coffee first...15:42
ayounglbragstad, rip it all out and replace with istio?15:43
ayoungmnaser, and it is still assigned to me.15:43
*** dansmith is now known as superdan15:45
kmallocWhat is patrole again?15:46
ayoungkmalloc, testing of policy15:46
kmallocAh15:47
ayoungkmalloc, scares the daylights out of me15:47
ayoungafraid it is going to end up locking us more in to the 968696 path15:47
kmallocI am disinclined to support patrole (for similar reasons) until we've moved the needle on the current initiative15:47
kmallocBut, that said, long term we absolutely should use it15:48
lbragstadso make the patrole spec dependent on the default roles specification?15:48
kmallocOr a similar thing.15:48
kmalloclbragstad: ++15:48
kmallocPlease.15:48
kmallocI don't want us further locked in to the current state of the world15:48
lbragstadyeah - me either15:49
kmallocWe are actively trying to fix the issues, and we can say patrole is a great idea, we want to ensure we are t adding it just to derail the work we are doing15:49
kmallocAren't*15:49
kmallocI like policy testing. I want us to do it. I don't want it to cause problems15:50
kmallocayoung: does that help alleviate concerns?15:51
* lbragstad has a hard time saying "no" to more tests15:51
kmallocBecause, if it doesn't I want to make sure we address it before we lock things into this direction.15:51
ayoungkmalloc, the fact that you see the effort the same way as I do? Yes, minimally.  I'm afraid that we will have this implemented from the outside.15:51
ayoungOnce we have scope enforced everywhere we should be OK15:52
lbragstadayoung: ++ i agree15:52
lbragstadit would be a great way to lock things down once we actually do things right15:52
lbragstadand gate on it15:53
*** gyee has joined #openstack-keystone16:00
kmalloc+++++++++16:01
*** sonuk has quit IRC16:01
*** raildo has quit IRC16:02
*** raildo has joined #openstack-keystone16:04
*** germs has quit IRC16:08
*** EmilienM is now known as EvilienM16:09
*** r-daneel has quit IRC16:13
*** r-daneel has joined #openstack-keystone16:13
*** eckesicle has quit IRC16:15
*** AlexeyAbashkin has joined #openstack-keystone16:43
* lbragstad runs to lunch16:53
*** mvk has joined #openstack-keystone16:58
*** tesseract has quit IRC17:09
*** itlinux has joined #openstack-keystone17:15
*** germs has joined #openstack-keystone17:18
*** germs has quit IRC17:18
*** germs has joined #openstack-keystone17:18
*** germs has quit IRC17:19
*** germs has joined #openstack-keystone17:19
*** pcaruana has joined #openstack-keystone17:54
*** pcichy has quit IRC18:13
openstackgerritHarry Rybacki proposed openstack/keystone-specs master: Define a set of basic default roles  https://review.openstack.org/56637718:27
*** AlexeyAbashkin has quit IRC18:30
openstackgerritHarry Rybacki proposed openstack/keystone-specs master: Define a set of basic default roles  https://review.openstack.org/56637718:34
*** gyankum has quit IRC18:49
*** pcaruana has quit IRC19:10
lbragstadwoo19:24
hrybackilbragstad: I just noted that the forum sessions for default roles points to the original spec: https://www.openstack.org/summit/vancouver-2018/summit-schedule/events/21761/default-roles19:38
hrybackiI'll add a comment to the original spec but do you know if we can update ^^ ?19:38
lbragstadi think we might be able to, but that'd be a good question for ttx19:39
hrybackilbragstad: btw, I can't abandon the original patch as you are the owner19:43
lbragstaddone19:45
hrybackilbragstad++19:48
hrybackitbh that kinda stung :P19:48
openstackgerritHarry Rybacki proposed openstack/keystone-specs master: Define a set of basic default roles  https://review.openstack.org/56637719:49
hrybackishould be good now lbagstad XD19:49
lbragstadi dig it19:49
lbragstadhrybacki: looks good - just a few comments so that we can make the specification keystone specific for now20:06
lbragstadbut we should stash some of those examples away for the time being because they will be useful when we write up a community goal.20:06
hrybackilbragstad: ack, thank you for the sanity check. I'm jumping out a bit early today but will re-scope and adjust based on those comments tomorrow or Sunday20:07
hrybackiHave a great weekend all!20:07
lbragstadhrybacki: ack - let me know if you need a hand with anything20:08
lbragstadhave a good weekend20:08
*** raildo has quit IRC20:31
*** mchlumsky has quit IRC20:42
*** martinus__ has quit IRC21:01
*** deepika08 has joined #openstack-keystone21:05
lbragstadcmurphy: i don't suppose you have the queries available for those google links do you?21:10
lbragstadour shortened google urls have stopped working :(21:11
*** dave-mccowan has quit IRC21:19
*** felipemonteiro has quit IRC21:20
*** edmondsw has quit IRC21:47
*** itlinux has quit IRC21:47
*** edmondsw has joined #openstack-keystone21:47
*** brad[] has quit IRC21:50
*** edmondsw has quit IRC21:52
*** spilla has quit IRC22:05
*** r-daneel has quit IRC22:21
*** markvoelker has quit IRC22:29
*** brad[] has joined #openstack-keystone22:33
*** linkmark has joined #openstack-keystone22:52
*** deepika08 has quit IRC23:15
kmalloclbragstad: what laptop are you using these days?23:18
lbragstad[m]kmalloc: my x1 still23:41

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