Thursday, 2018-05-03

*** xinran_ has joined #openstack-keystone00:01
*** lbragstad has quit IRC00:17
*** r-daneel has joined #openstack-keystone00:46
*** r-daneel_ has joined #openstack-keystone00:49
*** r-daneel has quit IRC00:50
*** r-daneel_ is now known as r-daneel00:50
*** threestrands has joined #openstack-keystone01:05
*** r-daneel has quit IRC01:06
*** panbalag has quit IRC01:14
*** panbalag has joined #openstack-keystone01:15
*** panbalag has left #openstack-keystone01:15
*** Guest10014 is now known as jlvillal01:19
*** namnh has joined #openstack-keystone01:31
*** namnh has quit IRC01:31
*** openstackgerrit has joined #openstack-keystone01:40
openstackgerritwangxiyuan proposed openstack/keystone master: [WIP]Enable foreign keys for unit test  https://review.openstack.org/55819301:40
*** r-daneel has joined #openstack-keystone01:53
*** xinran_ has quit IRC02:11
*** annp has joined #openstack-keystone02:12
openstackgerritwangxiyuan proposed openstack/keystone master: [WIP]Enable foreign keys for unit test  https://review.openstack.org/55819302:20
openstackgerritLance Bragstad proposed openstack/keystone-specs master: Add scenarios to strict hierarchy enforcement model  https://review.openstack.org/56541202:21
*** lbragstad has joined #openstack-keystone02:24
*** ChanServ sets mode: +o lbragstad02:24
*** nicolasbock has quit IRC03:21
*** namnh has joined #openstack-keystone03:38
*** germs has quit IRC04:02
*** hoonetorg has quit IRC04:21
*** chrome0_ has quit IRC04:39
*** chrome0 has joined #openstack-keystone04:41
*** gyee has quit IRC04:41
*** links has joined #openstack-keystone04:46
*** jdennis has quit IRC05:07
*** markvoelker has quit IRC05:08
*** jdennis has joined #openstack-keystone05:22
*** pcichy has joined #openstack-keystone05:51
*** hoonetorg has joined #openstack-keystone05:55
*** redrobot has quit IRC05:59
*** redrobot has joined #openstack-keystone06:03
*** markvoelker has joined #openstack-keystone06:09
*** markvoelker has quit IRC06:44
*** hoonetorg has quit IRC06:44
*** martinus__ has joined #openstack-keystone06:45
*** bhagyashris has joined #openstack-keystone06:46
*** hoonetorg has joined #openstack-keystone06:50
*** pcaruana has joined #openstack-keystone06:53
*** rcernin has quit IRC06:58
*** threestrands has quit IRC06:58
*** aloga has joined #openstack-keystone07:03
usr2033lbragstad, sorry i had to leave, user_z's default domain id is domain_a07:14
*** tesseract has joined #openstack-keystone07:18
openstackgerritwangxiyuan proposed openstack/keystone master: [WIP]Enable foreign keys for unit test  https://review.openstack.org/55819307:26
*** markvoelker has joined #openstack-keystone07:41
*** markvoelker has quit IRC08:13
*** links has quit IRC08:43
*** links has joined #openstack-keystone08:44
*** namnh has quit IRC08:48
*** namnh has joined #openstack-keystone08:48
openstackgerritwangxiyuan proposed openstack/keystone master: [WIP]Enable foreign keys for unit test  https://review.openstack.org/55819308:51
*** jrist has quit IRC08:57
*** jrist has joined #openstack-keystone09:01
wxylbragstad: sure. I'll take a look. Thanks for the update.09:05
*** markvoelker has joined #openstack-keystone09:10
*** markvoelker has quit IRC09:44
openstackgerritMerged openstack/oslo.policy master: make the sphinxpolicygen extension handle multiple input/output files  https://review.openstack.org/56462709:49
openstackgerritMerged openstack/oslo.policy master: Remove stale pip-missing-reqs tox test  https://review.openstack.org/56580309:49
*** namnh has quit IRC10:11
*** pcichy has quit IRC10:12
*** nicolasbock has joined #openstack-keystone10:31
*** edmondsw has joined #openstack-keystone10:31
*** edmondsw has quit IRC10:36
*** annp has quit IRC10:40
*** markvoelker has joined #openstack-keystone10:41
usr2033i have a problem about deleting domain. In my case: I have already user_z and it's default domain is domain_a. I create new domain_b and i gave user_z domain admin role to this domain. When i delete domain_b, openstack doesn't remove user_z's domain admin role. I tried this from openstack cli. No luck. Openstack keystone rest api document says that deleting domain will remove role assignments. Can you help me about this?10:55
*** markvoelker has quit IRC11:14
*** raildo has joined #openstack-keystone11:22
*** opetrenko has quit IRC11:26
*** opetrenko has joined #openstack-keystone11:27
*** panbalag has joined #openstack-keystone11:37
*** panbalag has left #openstack-keystone11:40
*** markvoelker has joined #openstack-keystone12:11
*** edmondsw has joined #openstack-keystone12:16
*** markvoelker has quit IRC12:20
*** markvoelker has joined #openstack-keystone12:20
*** panbalag has joined #openstack-keystone12:42
*** panbalag has left #openstack-keystone12:45
*** mvk has quit IRC12:55
*** nicolasbock has quit IRC12:57
*** mchlumsky has joined #openstack-keystone13:07
*** nicolasbock has joined #openstack-keystone13:14
*** mvk has joined #openstack-keystone13:28
*** pcichy has joined #openstack-keystone13:46
*** spilla has joined #openstack-keystone14:04
ayoungusr2033, pretty sure I wrote that code.14:05
ayounghttp://git.openstack.org/cgit/openstack/keystone/tree/keystone/resource/controllers.py#n73  is the controller code for delete_domain...14:07
*** Horrorcat has left #openstack-keystone14:07
ayoungand I am already starting to believe you14:07
ayounghttp://git.openstack.org/cgit/openstack/keystone/tree/keystone/resource/core.py#n752  is the internal code for delete domain14:09
*** spilla has quit IRC14:13
*** spilla has joined #openstack-keystone14:14
lbragstadayoung: i can't recreate it http://paste.openstack.org/raw/720230/14:14
lbragstadwe're going to be lucky at the summit because the project update is early in the week14:15
lbragstadwe can use it as an opportunity to plug other related talks14:15
ayounglbragstad, OK here is what I have so far14:15
lbragstadif you have a talk in vancouver that you'd like us to mention in keystone's project update, please add it to the list https://etherpad.openstack.org/p/YVR-keystone-forum-sessions14:16
ayounghttp://git.openstack.org/cgit/openstack/keystone/tree/keystone/assignment/core.py#n6814:16
ayounglbragstad, will do14:17
ayounglbragstad, so we don't delete assignments directly, only based on a notification14:18
ayoungif we delete a domain, then the notification should get picked up by the code I linked to above, and that should remove all domain_assignments.14:18
ayoungI can't recall discussing the need to defer the domain assignments to this level, suspect it was part of henrynash's split of the assignment api off the others14:19
openstackgerritMerged openstack/pycadf master: Trivial: Update pypi url to new url  https://review.openstack.org/55169514:19
lbragstadusr2033: do you have a paste that shows the steps you're taking and points out where you think the bug is?14:21
lbragstadbecause based on my understanding - this is how i'm recreating the scenario http://paste.openstack.org/raw/720230/14:22
lbragstadin that example, i have the "default" domain and a "new-domain"14:22
ayounglbragstad, do the same thing, but use the admin role.14:24
lbragstadok - let me try that then14:24
ayounglbragstad, I wonder if there is another step, too, that he didn't say14:24
ayounglike...do something as admin as that user14:24
ayounghttp://git.openstack.org/cgit/openstack/keystone/tree/keystone/assignment/backends/sql.py#n265  delete_domain_assignments should be pretty simple14:26
lbragstadthe admin role doesn't seem to matter http://paste.openstack.org/raw/720294/14:28
ayounglbragstad, agreed that it should not, either.14:30
*** r-daneel has quit IRC14:36
lbragstadmy pastes are getting repetitive14:38
lbragstadi'm just going to paste the assignment list14:38
lbragstadhttp://paste.openstack.org/show/720296/14:38
*** germs has joined #openstack-keystone14:38
*** germs has quit IRC14:38
*** germs has joined #openstack-keystone14:38
lbragstad^ still works14:38
*** germs has quit IRC14:39
*** germs has joined #openstack-keystone14:39
lbragstadso does - http://paste.openstack.org/show/720297/14:41
lbragstadregardless of the user being in the domain that gets deleted or another domain, their assignment are always cleaned up14:41
*** mvk has quit IRC14:42
*** _d34dh0r53_ is now known as d34dh0r5314:46
*** afazekas_ has quit IRC14:54
*** afazekas has joined #openstack-keystone14:58
*** r-daneel has joined #openstack-keystone14:59
*** AlexeyAbashkin has joined #openstack-keystone15:00
*** usr2033 has quit IRC15:03
*** AlexeyAbashkin has quit IRC15:03
*** links has quit IRC15:06
*** felipemonteiro__ has joined #openstack-keystone15:41
*** spilla has quit IRC15:43
*** spilla has joined #openstack-keystone15:43
gagehugolbragstad don't we already allow uuids with dashes?15:43
gagehugore: keystone feedback15:44
lbragstadgagehugo: it depends, i think that statement is missing specific context15:44
gagehugohmm15:44
lbragstadif you have a uuid with dashes in an external system and we put that uuid in a token, the dashes won't get reinflated15:44
lbragstadon token validation15:44
lbragstadCERN hit that i think, or i rememeber talking to jose about it15:45
gagehugoah15:45
*** felipemonteiro_ has quit IRC15:45
hrybackilbragstad: were those submission results surprising?15:45
lbragstadmmm15:46
lbragstadi'm inclined to say no15:46
hrybackiI did think people wanted policy fixed more than the federation bits15:46
lbragstadif i had to say anything was surprising it was that federation was to the right of the curve15:46
lbragstadbut - the middle three are really close to each other15:47
*** gyee has joined #openstack-keystone15:47
gagehugothere's quite a bit of asking for performance improvements too15:48
lbragstadyeah15:48
*** pcaruana has quit IRC15:50
knikollayeah, i'm also surprised that federation was above policy16:13
lbragstadto be fair16:15
lbragstadmost of the other feedback was policy related16:15
lbragstadso i would consider it tied with scaling out to multiple regions16:16
*** felipemonteiro__ has quit IRC16:17
*** felipemonteiro_ has joined #openstack-keystone16:17
*** tesseract has quit IRC16:28
hrybackigagehugo: do you know if there has been any movement on w/ Patrole? I saw https://review.openstack.org/#/c/464678/ was basically re-created here: https://review.openstack.org/#/c/559137/16:57
*** tesseract has joined #openstack-keystone16:57
hrybackiI'm not sure how closely you work with him (if at all)16:57
*** itlinux has joined #openstack-keystone17:00
*** panbalag1 has joined #openstack-keystone17:01
*** panbalag1 has left #openstack-keystone17:01
gagehugoI'm not sure17:02
gagehugofelipemonteiro_ ^17:02
mordredlbragstad: could I bug you to re-look at https://review.openstack.org/#/c/559154/7 and https://review.openstack.org/#/c/462218 and maybe change your +1 to a +2? - I think your comments have been addressed17:14
hrybackiack, thanks gagehugo :)17:25
*** tesseract has quit IRC17:25
*** mvk has joined #openstack-keystone17:28
lbragstadmordred: done17:54
felipemonteiro_hrybacki: i only moved the spec because i noticed nothing was being committed to keystone-specs so i wasn't sure if openstack-specs was more appropriate. in any case, i can consolidate them into keystone-specs if need be18:00
hrybackilbragstad: ^^ is that the case?18:01
lbragstadif it affects multiple projects, then openstack specs is probably more appropriate18:02
mordredlbragstad: \o/ thanks!18:04
hrybackifelipemonteiro_: you may want to take a look at https://review.openstack.org/#/c/523973 again now that seems to have become stable18:12
hrybackiI'm going to be digging into your spec/approach bc we'll need a way to test ^^18:12
felipemonteiro_ok, thanks will take a look18:15
*** felipemonteiro__ has joined #openstack-keystone18:39
*** felipemonteiro_ has quit IRC18:42
lbragstadwoot - we got a +1 from tim on our hierarchical limit enforcement model https://review.openstack.org/#/c/565412/418:45
lbragstad:8418:53
lbragstadbah18:53
* lbragstad goes to terminal18:53
hrybackicomments added to your newer spec felipemonteiro__ :)19:00
felipemonteiro__Thanks19:01
*** openstackgerrit has quit IRC19:05
*** sapd_ has joined #openstack-keystone19:19
*** felipemonteiro_ has joined #openstack-keystone19:21
*** felipemonteiro__ has quit IRC19:21
*** sapd has quit IRC19:22
*** openstackgerrit has joined #openstack-keystone19:24
openstackgerritLance Bragstad proposed openstack/keystone-specs master: Add scenarios to strict hierarchy enforcement model  https://review.openstack.org/56541219:25
lbragstadwxy: thanks for the review - addressed your comments in ^19:35
*** zzzeek_ has quit IRC19:35
lbragstadwxy: feel free to roll that into your proposal if you'd like19:37
*** zzzeek has joined #openstack-keystone19:37
*** pcichy has quit IRC19:49
*** chrome0 has quit IRC20:10
*** chrome0 has joined #openstack-keystone20:12
*** chrome0 has quit IRC20:15
*** chrome0 has joined #openstack-keystone20:16
*** mchlumsky has quit IRC20:17
openstackgerritLance Bragstad proposed openstack/keystone master: Add policy for limit model protection  https://review.openstack.org/56271420:21
openstackgerritLance Bragstad proposed openstack/keystone master: Implement enforcement model logic in Manager  https://review.openstack.org/56271520:21
openstackgerritLance Bragstad proposed openstack/keystone master: Expose endpoint to return enforcement model  https://review.openstack.org/56271620:21
*** spilla has quit IRC20:26
openstackgerritMonty Taylor proposed openstack/keystoneauth master: Implement service_type alias lookups  https://review.openstack.org/46221820:28
openstackgerritMonty Taylor proposed openstack/keystoneauth master: Add methods to get all of the version data  https://review.openstack.org/55915420:29
openstackgerritMonty Taylor proposed openstack/keystoneauth master: Make VersionData class  https://review.openstack.org/56446920:29
openstackgerritMonty Taylor proposed openstack/keystoneauth master: Turn normalize_status into a class  https://review.openstack.org/56411020:29
openstackgerritMonty Taylor proposed openstack/keystoneauth master: Infer version from old versioned service type aliases  https://review.openstack.org/56429920:29
mordredlbragstad, kmalloc: I had to rebase the whole stack because the first patch didn't add os-service-types to lower-constraints.txt (merge race condition)20:29
lbragstadah20:29
openstackgerritMonty Taylor proposed openstack/keystoneauth master: Use Status variables in tests  https://review.openstack.org/56425820:29
openstackgerritMonty Taylor proposed openstack/keystoneauth master: Reference class variable in Status  https://review.openstack.org/56426220:30
kmallocDamn20:30
mordredright?20:30
kmallocNbd, it still can land easy...just..ugh20:30
mordredyah.20:31
mordredthose moments when you see the red x and you just get sad20:31
*** jdennis has quit IRC20:31
openstackgerritLance Bragstad proposed openstack/keystone master: Remove unused exception  https://review.openstack.org/56615620:33
*** jdennis has joined #openstack-keystone20:33
kmallocAyup20:34
*** raildo has quit IRC20:42
*** martinus__ has quit IRC20:47
*** spilla has joined #openstack-keystone20:47
openstackgerritLance Bragstad proposed openstack/keystone master: Remove dead code in token provider  https://review.openstack.org/56616020:57
*** dtruong_ has quit IRC21:13
openstackgerritLance Bragstad proposed openstack/keystone master: Remove unused assertions from test_v3.py  https://review.openstack.org/56616321:13
*** nkinder has quit IRC21:14
*** edmondsw has quit IRC21:15
*** edmondsw has joined #openstack-keystone21:16
*** edmondsw has quit IRC21:21
openstackgerritLance Bragstad proposed openstack/keystone master: Remove dead dependency injection code  https://review.openstack.org/56616821:25
*** itlinux_ has joined #openstack-keystone21:53
*** itlinux has quit IRC21:56
*** itlinux_ has quit IRC21:58
*** rcernin has joined #openstack-keystone22:07
*** felipemonteiro__ has joined #openstack-keystone22:43
*** felipemonteiro__ has quit IRC22:44
*** felipemonteiro__ has joined #openstack-keystone22:44
*** felipemonteiro_ has quit IRC22:47
*** threestrands has joined #openstack-keystone22:59
*** felipemonteiro__ has quit IRC23:04
*** dmellado has quit IRC23:19
*** masber has joined #openstack-keystone23:23
*** dmellado has joined #openstack-keystone23:51

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