openstackgerrit | Merged openstack/keystone master: Document and add release note for HEAD APIs https://review.openstack.org/478284 | 00:18 |
---|---|---|
*** ducttape_ has joined #openstack-keystone | 00:29 | |
openstackgerrit | Merged openstack/keystone master: Add response example in authenticate-v3.inc https://review.openstack.org/463245 | 00:40 |
openstackgerrit | Merged openstack/keystone master: Updated from global requirements https://review.openstack.org/477944 | 00:40 |
*** ducttape_ has quit IRC | 00:49 | |
*** ducttape_ has joined #openstack-keystone | 00:49 | |
openstackgerrit | Samuel de Medeiros Queiroz proposed openstack/keystone master: Remove support for direct import of drivers https://review.openstack.org/478243 | 00:59 |
*** Shunli has joined #openstack-keystone | 01:05 | |
*** ducttape_ has quit IRC | 01:07 | |
*** wlfightup has joined #openstack-keystone | 01:11 | |
*** ducttape_ has joined #openstack-keystone | 01:14 | |
*** ducttape_ has quit IRC | 01:14 | |
*** ducttape_ has joined #openstack-keystone | 01:16 | |
openstackgerrit | Samuel de Medeiros Queiroz proposed openstack/keystone master: Remove support for direct import of drivers https://review.openstack.org/478243 | 01:27 |
*** spzala has joined #openstack-keystone | 01:30 | |
*** liujiong has joined #openstack-keystone | 01:36 | |
stevemar | hrybacki: my comment was in reply to samuel | 01:49 |
*** gyee has quit IRC | 01:53 | |
*** thorst has joined #openstack-keystone | 01:53 | |
*** thorst has quit IRC | 01:57 | |
*** liujiong has quit IRC | 02:09 | |
*** liujiong has joined #openstack-keystone | 02:09 | |
*** ducttape_ has quit IRC | 02:16 | |
*** jdennis has quit IRC | 02:26 | |
*** jdennis has joined #openstack-keystone | 02:27 | |
*** spzala has quit IRC | 02:28 | |
*** spzala has joined #openstack-keystone | 02:28 | |
*** Shunli has quit IRC | 02:33 | |
*** htruta` has quit IRC | 02:34 | |
*** cmurphy has quit IRC | 02:34 | |
*** htruta has joined #openstack-keystone | 02:35 | |
*** knikolla has quit IRC | 02:35 | |
*** cmurphy has joined #openstack-keystone | 02:36 | |
*** aselius has quit IRC | 02:40 | |
*** lucasxu has joined #openstack-keystone | 02:40 | |
*** knikolla has joined #openstack-keystone | 02:43 | |
*** lucasxu has quit IRC | 02:56 | |
*** wlfightup has quit IRC | 02:56 | |
*** lucasxu has joined #openstack-keystone | 03:07 | |
*** namnh has joined #openstack-keystone | 03:17 | |
*** thorst has joined #openstack-keystone | 03:54 | |
*** wlfightup has joined #openstack-keystone | 03:59 | |
*** links has joined #openstack-keystone | 04:00 | |
*** thorst has quit IRC | 04:00 | |
*** lucasxu has quit IRC | 04:06 | |
openstackgerrit | Merged openstack/keystone master: Improve handling of database migration checks https://review.openstack.org/475472 | 04:14 |
*** tbh_ has joined #openstack-keystone | 04:29 | |
*** spzala has quit IRC | 04:41 | |
*** spzala has joined #openstack-keystone | 04:42 | |
*** spzala has quit IRC | 04:47 | |
*** liujiong has quit IRC | 05:35 | |
*** tobberydberg has joined #openstack-keystone | 05:44 | |
*** Shunli has joined #openstack-keystone | 05:45 | |
*** thorst has joined #openstack-keystone | 05:57 | |
*** liujiong has joined #openstack-keystone | 06:01 | |
*** lbragstad_ has joined #openstack-keystone | 06:02 | |
*** rcernin has joined #openstack-keystone | 06:02 | |
*** thorst has quit IRC | 06:02 | |
*** lbragstad has quit IRC | 06:02 | |
*** spzala has joined #openstack-keystone | 06:03 | |
*** spzala has quit IRC | 06:09 | |
*** med_ has quit IRC | 06:53 | |
*** med_ has joined #openstack-keystone | 06:59 | |
*** med_ is now known as Guest39045 | 06:59 | |
*** Shunli has quit IRC | 07:03 | |
*** spzala has joined #openstack-keystone | 07:06 | |
*** adriant has quit IRC | 07:07 | |
*** tesseract has joined #openstack-keystone | 07:08 | |
*** adriant has joined #openstack-keystone | 07:08 | |
*** spzala has quit IRC | 07:10 | |
*** ducttape_ has joined #openstack-keystone | 07:17 | |
*** ducttape_ has quit IRC | 07:21 | |
*** aojea has joined #openstack-keystone | 07:23 | |
openstackgerrit | zhengliuyang proposed openstack/keystone master: Remove duplicated hacking rule https://review.openstack.org/478411 | 07:29 |
*** tbh_ has quit IRC | 07:38 | |
*** pnavarro has joined #openstack-keystone | 07:45 | |
openstackgerrit | zhengliuyang proposed openstack/keystone master: Remove duplicated hacking rule https://review.openstack.org/478411 | 07:48 |
openstackgerrit | zhengliuyang proposed openstack/keystone master: Remove duplicated list conversion https://review.openstack.org/478419 | 07:52 |
*** thorst has joined #openstack-keystone | 07:58 | |
*** zzzeek has quit IRC | 08:00 | |
*** zzzeek has joined #openstack-keystone | 08:00 | |
*** thorst has quit IRC | 08:03 | |
*** aojea has quit IRC | 08:12 | |
*** aojea has joined #openstack-keystone | 08:13 | |
*** Shunli has joined #openstack-keystone | 08:18 | |
*** Shunli has quit IRC | 08:23 | |
*** Shunli has joined #openstack-keystone | 08:23 | |
*** tobberyd_ has joined #openstack-keystone | 08:29 | |
*** admcleod has joined #openstack-keystone | 08:31 | |
*** Shunli has quit IRC | 08:31 | |
*** tobberydberg has quit IRC | 08:33 | |
*** tobberydberg has joined #openstack-keystone | 08:33 | |
*** tobberyd_ has quit IRC | 08:33 | |
*** spzala has joined #openstack-keystone | 09:07 | |
*** spzala has quit IRC | 09:12 | |
*** aojea has quit IRC | 09:12 | |
*** aojea has joined #openstack-keystone | 09:12 | |
*** thorst has joined #openstack-keystone | 09:24 | |
*** tobberydberg has quit IRC | 09:27 | |
*** tobberydberg has joined #openstack-keystone | 09:27 | |
*** pcaruana has joined #openstack-keystone | 09:28 | |
*** thorst has quit IRC | 09:29 | |
*** tobberydberg has quit IRC | 09:34 | |
*** tobberydberg has joined #openstack-keystone | 09:35 | |
*** mvk has quit IRC | 09:46 | |
*** yiorgos_272 has joined #openstack-keystone | 09:51 | |
*** aojea has quit IRC | 09:51 | |
*** aojea has joined #openstack-keystone | 09:55 | |
*** rcernin has quit IRC | 09:55 | |
*** aojea_ has joined #openstack-keystone | 09:58 | |
*** aojea has quit IRC | 10:01 | |
samueldmq | morning | 10:03 |
*** rcernin has joined #openstack-keystone | 10:03 | |
*** rcernin has quit IRC | 10:09 | |
*** rcernin has joined #openstack-keystone | 10:11 | |
*** namnh has quit IRC | 10:11 | |
*** mvk has joined #openstack-keystone | 10:13 | |
*** liujiong has quit IRC | 10:13 | |
*** jdennis1 has joined #openstack-keystone | 10:41 | |
*** jdennis has quit IRC | 10:42 | |
*** tobberyd_ has joined #openstack-keystone | 10:58 | |
*** tobbery__ has joined #openstack-keystone | 11:00 | |
*** tobberyd_ has quit IRC | 11:00 | |
*** tobberydberg has quit IRC | 11:01 | |
*** tobbery__ has quit IRC | 11:01 | |
*** tobberydberg has joined #openstack-keystone | 11:02 | |
*** spzala has joined #openstack-keystone | 11:09 | |
*** spzala has quit IRC | 11:14 | |
*** jdennis1 is now known as jdennis | 11:15 | |
*** jdennis has quit IRC | 11:15 | |
*** jdennis has joined #openstack-keystone | 11:15 | |
*** lwanderley has joined #openstack-keystone | 11:22 | |
*** thorst has joined #openstack-keystone | 11:25 | |
*** thorst has quit IRC | 11:29 | |
*** tobberydberg has quit IRC | 11:33 | |
*** tobberydberg has joined #openstack-keystone | 11:34 | |
*** ducttape_ has joined #openstack-keystone | 12:00 | |
*** pcaruana has quit IRC | 12:02 | |
*** pcaruana has joined #openstack-keystone | 12:02 | |
dstanek | morning samueldmq | 12:04 |
*** raildo has joined #openstack-keystone | 12:05 | |
cmurphy | o/ | 12:06 |
*** ducttape_ has quit IRC | 12:08 | |
*** thorst has joined #openstack-keystone | 12:10 | |
*** aloga has quit IRC | 12:12 | |
*** aloga has joined #openstack-keystone | 12:12 | |
*** chlong_ has quit IRC | 12:15 | |
*** pcaruana has quit IRC | 12:16 | |
dstanek | g'morning cmurphy | 12:17 |
cmurphy | dstanek: nice to see you back | 12:21 |
*** pcaruana has joined #openstack-keystone | 12:23 | |
*** pcaruana has quit IRC | 12:23 | |
*** pcaruana has joined #openstack-keystone | 12:23 | |
*** pcaruana has quit IRC | 12:29 | |
openstackgerrit | Monty Taylor proposed openstack/keystoneauth master: Expose getting EndpointData on adapter and session https://review.openstack.org/469091 | 12:33 |
openstackgerrit | Monty Taylor proposed openstack/keystoneauth master: Add support for version ranges https://review.openstack.org/469090 | 12:34 |
openstackgerrit | Monty Taylor proposed openstack/keystoneauth master: Support a list of interface values https://review.openstack.org/477169 | 12:34 |
openstackgerrit | Monty Taylor proposed openstack/keystoneauth master: Support explicitly requesting the 'latest' version https://review.openstack.org/469089 | 12:34 |
openstackgerrit | Monty Taylor proposed openstack/keystoneauth master: Add flags to turn discovery on and off https://review.openstack.org/469088 | 12:34 |
openstackgerrit | Monty Taylor proposed openstack/keystoneauth master: Clean up a few review nits https://review.openstack.org/477657 | 12:34 |
openstackgerrit | Monty Taylor proposed openstack/keystoneauth master: Plumb endpoint_override through get_endpoint_data https://review.openstack.org/469092 | 12:34 |
openstackgerrit | Monty Taylor proposed openstack/keystoneauth master: Update docs and add a release note https://review.openstack.org/477566 | 12:34 |
openstackgerrit | Monty Taylor proposed openstack/keystoneauth master: Ensure we discover only when we should https://review.openstack.org/477242 | 12:34 |
openstackgerrit | Monty Taylor proposed openstack/keystoneauth master: Optimize matching version no microversion needed https://review.openstack.org/470274 | 12:34 |
openstackgerrit | Monty Taylor proposed openstack/keystoneauth master: Refactor volume mock urls in identity_common tests https://review.openstack.org/477246 | 12:34 |
*** ducttape_ has joined #openstack-keystone | 12:34 | |
mordred | cmurphy, samueldmq, morgan: there's your friendly morning update to that stack | 12:34 |
* mordred waves to dstanek | 12:34 | |
cmurphy | mordred: my favorite stack | 12:36 |
*** pcaruana has joined #openstack-keystone | 12:37 | |
mordred | cmurphy: I know it. I never want you to not have plenty of patches to review | 12:37 |
cmurphy | mordred: you are so kind | 12:37 |
openstackgerrit | Merged openstack/keystone master: Remove duplicated list conversion https://review.openstack.org/478419 | 12:38 |
*** ducttape_ has quit IRC | 12:38 | |
*** pcaruana has quit IRC | 12:43 | |
*** pcaruana has joined #openstack-keystone | 12:44 | |
samueldmq | mordred: cmurphy :-) | 12:44 |
*** jmlowe_ has quit IRC | 12:50 | |
*** sjain has joined #openstack-keystone | 12:50 | |
*** pcaruana has quit IRC | 12:53 | |
mordred | cmurphy: I added a test in that one patch to show the url with and without / behavior | 12:53 |
cmurphy | mordred: awesome | 12:54 |
mordred | cmurphy: once we're done with this stack, I think it would not be a terrible idea to make a Version object and a URL object that we can use internally that contain all of the matching and combining logic, rather than having joins and splits and tuple conversions floating around everywhere | 12:55 |
mordred | but I think that would be more distracting than helpful today | 12:55 |
*** bknudson has joined #openstack-keystone | 12:55 | |
cmurphy | mordred: agree on both | 12:57 |
*** ducttape_ has joined #openstack-keystone | 13:04 | |
*** pcaruana has joined #openstack-keystone | 13:06 | |
hrybacki | stevemar: which comment were you referring to? | 13:11 |
*** spzala has joined #openstack-keystone | 13:11 | |
hrybacki | working on your comments now btw. This is my first foray into openstackclient code -- I your appreciate the verbosity! | 13:11 |
hrybacki | lbragstad_: will be moving on to the implied role policies review next | 13:12 |
*** pcaruana has quit IRC | 13:13 | |
*** lucasxu has joined #openstack-keystone | 13:14 | |
*** pcaruana has joined #openstack-keystone | 13:14 | |
*** spzala has quit IRC | 13:15 | |
*** links has quit IRC | 13:16 | |
*** ducttape_ has quit IRC | 13:17 | |
*** lwanderley has quit IRC | 13:21 | |
*** lwanderley has joined #openstack-keystone | 13:32 | |
*** links has joined #openstack-keystone | 13:33 | |
*** lwanderley has quit IRC | 13:34 | |
*** tobberyd_ has joined #openstack-keystone | 13:34 | |
*** jsavak has joined #openstack-keystone | 13:34 | |
*** chlong_ has joined #openstack-keystone | 13:34 | |
*** jmlowe has joined #openstack-keystone | 13:35 | |
*** tobberydberg has quit IRC | 13:37 | |
*** zhurong_ has joined #openstack-keystone | 13:38 | |
*** tobberyd_ has quit IRC | 13:38 | |
*** sjain has quit IRC | 13:40 | |
*** pcaruana has quit IRC | 13:40 | |
*** pcaruana has joined #openstack-keystone | 13:46 | |
stevemar | hrybacki: haha, i was responding to samuel's comments about the docs moving around | 13:51 |
dstanek | hey stevemar, long time no see | 13:52 |
hrybacki | ohh haha | 13:52 |
*** ducttape_ has joined #openstack-keystone | 13:52 | |
*** ducttap__ has joined #openstack-keystone | 13:53 | |
*** ducttap__ has quit IRC | 13:53 | |
*** ducttap__ has joined #openstack-keystone | 13:54 | |
*** links has quit IRC | 13:56 | |
*** ducttape_ has quit IRC | 13:57 | |
hrybacki | stevemar: are single line doc strings permitted in openstackclient? Specifically if there is not any parameters for a given function | 13:58 |
samueldmq | mordred: hey, I am still not getting something here https://review.openstack.org/#/c/469086/ | 13:58 |
*** aojea_ has quit IRC | 14:01 | |
samueldmq | mordred: I left one more comment there | 14:01 |
*** aojea has joined #openstack-keystone | 14:05 | |
*** jaosorior has quit IRC | 14:05 | |
*** lbragstad_ is now known as lbragstad | 14:07 | |
*** ChanServ sets mode: +o lbragstad | 14:08 | |
*** jaosorior has joined #openstack-keystone | 14:08 | |
lbragstad | whoa - a dstanek?! | 14:08 |
*** tobberydberg has joined #openstack-keystone | 14:10 | |
lbragstad | dstanek: did you happen to get my note about the native saml stuff? | 14:10 |
*** tobberydberg has quit IRC | 14:14 | |
*** ducttap__ has quit IRC | 14:18 | |
mordred | samueldmq: looking | 14:19 |
mordred | samueldmq: responded - thanks for looking so deeply | 14:20 |
*** ducttape_ has joined #openstack-keystone | 14:22 | |
*** spzala has joined #openstack-keystone | 14:22 | |
*** aojea_ has joined #openstack-keystone | 14:28 | |
*** aojea has quit IRC | 14:30 | |
samueldmq | mordred: sure, thanks for clarifying | 14:31 |
samueldmq | mordred: also, I have a high trust in cmurphy's +2s. approved | 14:32 |
cmurphy | o.0 ogod they think i know what i'm doing | 14:32 |
cmurphy | keystone is doomed | 14:33 |
samueldmq | :-) | 14:33 |
samueldmq | not that I don't from other cores, but you know, good catches | 14:33 |
samueldmq | ok, when I finish reviewing one patch, mordred adds another to the chain | 14:34 |
samueldmq | so I always have 12 left | 14:34 |
samueldmq | that'd cool | 14:34 |
samueldmq | that's* | 14:34 |
cmurphy | i don't even know what i would do with my life if we made it through this stack | 14:35 |
*** phalmos has joined #openstack-keystone | 14:36 | |
samueldmq | hehe | 14:37 |
samueldmq | lbragstad: some of those tiny files get merged into 1 in the followup patch | 14:39 |
samueldmq | https://review.openstack.org/#/c/476635 | 14:39 |
samueldmq | I also had concerns about too many files (but in the previous patch), then it became better when I looked at the end of the stack | 14:40 |
stevemar | hrybacki: you betcha | 14:40 |
stevemar | dstanek: miss me? :) | 14:40 |
samueldmq | stevemar: we all do :'( | 14:42 |
dstanek | stevemar: of course | 14:44 |
*** spzala has quit IRC | 14:45 | |
stevemar | liars, the both of you | 14:45 |
stevemar | :) | 14:45 |
dstanek | i mentioned to lbragstad that i'm going to try to be around a little more to do reviews and such | 14:45 |
*** spzala has joined #openstack-keystone | 14:45 | |
stevemar | i'm trying to review when i can, its a slower week than normal at work | 14:46 |
*** yiorgos_272 has left #openstack-keystone | 14:47 | |
dstanek | stevemar: i know the feeling. i'm getting used to the new position so i'm able to spare a little time now | 14:48 |
samueldmq | dstanek: stevemar: great to have both of you back :D | 14:48 |
*** ducttape_ has quit IRC | 14:49 | |
* samueldmq heads out for lunch | 14:49 | |
dstanek | i'll be reviewing for technical accuracy and correctness... i'll leave the direction and "do we want this?" to you guys :-) | 14:49 |
lbragstad | dstanek: but you were *so* good at that part | 14:50 |
lbragstad | s/were/are/ | 14:50 |
*** spzala has quit IRC | 14:50 | |
*** ducttape_ has joined #openstack-keystone | 14:50 | |
dstanek | i don't have enough time to keep up the direction, unfortunately :-( | 14:52 |
lbragstad | dstanek: i understand, happy to have you back regardless :) | 14:58 |
dstanek | i'm wondering if i can change my affiliation from rackspace to independent since i'll be using my own time... | 14:59 |
lbragstad | dstanek: yeah - you should be able to | 15:02 |
lbragstad | dstanek: i had to do that before the forum | 15:02 |
dstanek | i just wonder if that would somehow piss rackspace off since they still pay me...just not for this | 15:02 |
*** zhurong_ has quit IRC | 15:03 | |
lbragstad | :) | 15:06 |
*** rcernin has quit IRC | 15:06 | |
*** wlfightup has quit IRC | 15:06 | |
*** links has joined #openstack-keystone | 15:07 | |
*** gyee has joined #openstack-keystone | 15:07 | |
*** spzala has joined #openstack-keystone | 15:19 | |
hrybacki | dstanek: do you think they would notice/track that sort of thing? | 15:22 |
dstanek | hrybacki: no idea | 15:23 |
*** aselius has joined #openstack-keystone | 15:29 | |
*** links has quit IRC | 15:36 | |
ayoung | dstanek! Are you back! | 15:40 |
*** rderose has joined #openstack-keystone | 15:43 | |
lbragstad | does anyone know if yangweiwei is on IRC? | 15:47 |
*** aojea_ has quit IRC | 15:49 | |
hrybacki | Will be late to policy meeting today-- conflicting meetings : / | 15:53 |
lbragstad | hrybacki: no worries - thanks for the heads up | 15:53 |
openstackgerrit | Merged openstack/keystoneauth master: Move version discovery logic to keystoneauth1.discover https://review.openstack.org/469086 | 15:54 |
cmurphy | lbragstad: I don't think I've seen yangweiwei on IRC but they are responsive to review comments if you want to invite them that way | 15:55 |
openstackgerrit | Merged openstack/keystone master: Remove the local tempest plugin https://review.openstack.org/471060 | 15:56 |
dstanek | ayoung: i'm going to try to be | 16:04 |
ayoung | dstanek, I take it you are still at RAX but in a different role? | 16:04 |
dstanek | ayoung: yeah, no more OpenStack stuff right now | 16:04 |
lbragstad | cmurphy: ok - that's what i thought.. good idea | 16:07 |
*** spilla has joined #openstack-keystone | 16:08 | |
*** sjain has joined #openstack-keystone | 16:17 | |
*** ducttap__ has joined #openstack-keystone | 16:26 | |
*** sjain has quit IRC | 16:26 | |
*** ducttape_ has quit IRC | 16:27 | |
*** ducttape_ has joined #openstack-keystone | 16:28 | |
lbragstad | dstanek: quick federation question - should a domain scoped token be able to return a users federated projects? | 16:29 |
*** ducttap__ has quit IRC | 16:30 | |
dstanek | lbragstad: can a regular domain scoped token get a list of user's projects? i would expect the answer to be the same | 16:35 |
*** ducttap__ has joined #openstack-keystone | 16:36 | |
ayoung | lbragstad, here's how I see it: | 16:38 |
ayoung | an unscoped token represenst the users relationship with Keystone | 16:38 |
ayoung | a domain scoped token is only for use with Keystone | 16:38 |
ayoung | thus they are roughly comparable | 16:38 |
*** ducttape_ has quit IRC | 16:38 | |
ayoung | but, the scoping implies that the user should only see things related to that domain | 16:38 |
lbragstad | yeah | 16:39 |
ayoung | so, yes, they should get something back, as working with projects is one of the things a domain scoped token is used to do | 16:39 |
*** ducttap__ has quit IRC | 16:39 | |
lbragstad | but a use could have access to projects in a different domain | 16:39 |
ayoung | but it should only be projects in that domain | 16:39 |
lbragstad | right - ok | 16:39 |
ayoung | and a domain scoped token should not show those | 16:39 |
lbragstad | same page | 16:39 |
ayoung | only an unscoped token should show those | 16:39 |
ayoung | coo | 16:39 |
*** sjain has joined #openstack-keystone | 16:40 | |
dstanek | lbragstad: ayoung: is that aspirational or how it works now? | 16:42 |
ayoung | dstanek, aspirational. I have not looked. | 16:43 |
lbragstad | i'm checking now | 16:43 |
lbragstad | i don't think there is anything that prevents a user from doing that actually = https://github.com/openstack/keystone/blob/fece45d75c4ba893828cbdcbcccc6b97b8c06c68/keystone/auth/controllers.py#L363-L381 | 16:45 |
*** spilla has quit IRC | 16:45 | |
hrybacki | lbragstad: that was a record time for the policy meeting! | 16:46 |
*** sjain has quit IRC | 16:47 | |
dstanek | ayoung: everythink in keystone is aspirational :-) | 16:47 |
ayoung | I find of is somewhat respiration-restricting myself | 16:48 |
*** sjain has joined #openstack-keystone | 16:50 | |
lbragstad | dstanek: ayoung updated | 16:53 |
lbragstad | https://bugs.launchpad.net/keystone/+bug/1693704 | 16:53 |
openstack | Launchpad bug 1693704 in OpenStack Identity (keystone) "Unable to list federated projects with domain-scoped token" [Undecided,Invalid] - Assigned to yangweiwei (496176919-6) | 16:53 |
ayoung | lbragstad, cool | 16:53 |
*** spzala has quit IRC | 16:57 | |
*** spzala has joined #openstack-keystone | 16:57 | |
openstackgerrit | Samuel de Medeiros Queiroz proposed openstack/keystone master: Move implied role policies to DocumentedRuleDefault https://review.openstack.org/449246 | 17:01 |
samueldmq | lbragstad: ^ | 17:01 |
samueldmq | should be ready to go | 17:01 |
lbragstad | samueldmq: sweet - i'm not sure if hrybacki had changes locally or now | 17:02 |
lbragstad | not* | 17:02 |
*** spzala has quit IRC | 17:02 | |
samueldmq | hrybacki: I am sorry if you were working on that | 17:03 |
samueldmq | I should've asked in first place | 17:03 |
hrybacki | no worries! | 17:03 |
hrybacki | work done is work done imo | 17:03 |
lbragstad | hrybacki: now you get to review it :) | 17:04 |
lbragstad | https://review.openstack.org/#/c/449278/ is the last one for that spec | 17:04 |
hrybacki | woo | 17:06 |
hrybacki | lbragstad: I'll have a new revision of ^^ up today too | 17:06 |
lbragstad | hrybacki: awesome | 17:07 |
lbragstad | breaking for lunch | 17:08 |
*** spzala has joined #openstack-keystone | 17:12 | |
*** spzala has quit IRC | 17:17 | |
samueldmq | stevemar: would be nice to get your eyes on https://review.openstack.org/#/c/476240/ and https://review.openstack.org/#/c/476243/ | 17:19 |
samueldmq | stevemar: both are openstackclient patches | 17:19 |
openstackgerrit | Merged openstack/keystone master: Remove duplicated hacking rule https://review.openstack.org/478411 | 17:21 |
*** aojea has joined #openstack-keystone | 17:21 | |
openstackgerrit | Samriddhi proposed openstack/keystone master: Added a note for API curl examples https://review.openstack.org/478574 | 17:24 |
*** aojea has quit IRC | 17:25 | |
*** jsavak has quit IRC | 17:30 | |
openstackgerrit | Samuel de Medeiros Queiroz proposed openstack/keystone master: Update info about logging in admin guide https://review.openstack.org/478583 | 17:43 |
eandersson | Hey - if a project name changes, does the uuid stay the same? | 17:43 |
samueldmq | lbragstad: ^ adding more details to the admin guide from https://review.openstack.org/#/c/477692 | 17:43 |
*** links has joined #openstack-keystone | 17:47 | |
*** jsavak has joined #openstack-keystone | 17:48 | |
lbragstad | eandersson: yes | 17:59 |
morgan | mordred: comments left. | 17:59 |
morgan | mordred: i got about 1/2 way through the patchset before I am thinking there might be bigger changes based upon my comments. | 18:00 |
lbragstad | hrybacki: knikolla gagehugo i'm going to start using an office hours tag for launchpad | 18:00 |
*** spzala has joined #openstack-keystone | 18:01 | |
lbragstad | and we can use it to tag bugs that we think would be good to work on during office hours | 18:01 |
*** spilla has joined #openstack-keystone | 18:02 | |
mordred | morgan: kk. I will go read real quick | 18:02 |
morgan | mordred: the first patch in the chain is gating btw | 18:02 |
*** jsavak has quit IRC | 18:02 | |
*** jsavak has joined #openstack-keystone | 18:04 | |
mordred | morgan: I disagree with your comment - the contract from before is alos return None if nothing matches ... am I missing something? | 18:04 |
*** spzala has quit IRC | 18:05 | |
morgan | normalize_version was raising a typeerror on None | 18:08 |
hrybacki | lbragstad: good plan! | 18:09 |
mordred | morgan: ahhhhhh | 18:09 |
morgan | https://github.com/openstack/keystoneauth/blame/master/keystoneauth1/discover.py#L104 | 18:09 |
mordred | morgan: thanks - that's what I was missing | 18:09 |
morgan | mordred: yep ;) | 18:09 |
morgan | mordred: i almost missed it | 18:09 |
morgan | mordred: i don't like side effects from "normalize" methods. | 18:09 |
hrybacki | lbragstad: would it be possible to push deprecation of config uploads via keystone-manage to Q? | 18:10 |
morgan | but alas, we have it (for ~3yrs) so we're stuck with it (at least the behavior) | 18:10 |
openstackgerrit | Samriddhi proposed openstack/keystone master: Added a note for API curl examples https://review.openstack.org/478574 | 18:10 |
morgan | hrybacki: reason for it? | 18:10 |
hrybacki | puppet-keystone lacks support right now to do it via the api | 18:10 |
lbragstad | hrybacki: yeah - it sounds like we still have people relying on it :-/ | 18:10 |
morgan | hrybacki: pushing a deprecation is easy, but we need a concrete reason for it | 18:10 |
morgan | hrybacki: fair enough | 18:10 |
morgan | hrybacki: that is reason enough to push the deprecation back | 18:10 |
morgan | or.. we can deprecate we can't remove | 18:11 |
morgan | lbragstad: ^ | 18:11 |
lbragstad | http://lists.openstack.org/pipermail/openstack-dev/2017-June/119026.html | 18:11 |
morgan | it is fine to say "hey this is going away" | 18:11 |
morgan | it isn't fine to take it away. | 18:11 |
hrybacki | ++ thanks! | 18:11 |
lbragstad | morgan: i agree | 18:11 |
lbragstad | morgan: should we also deprecate the ability to have domain in configs | 18:11 |
lbragstad | ? | 18:11 |
lbragstad | morgan: bkudson brought up a good point during yesterday's meeting | 18:11 |
lbragstad | bknudson: * rather | 18:12 |
openstackgerrit | Doug Hellmann proposed openstack/oslo.policy master: move existing documentation into new standard layout https://review.openstack.org/478595 | 18:12 |
openstackgerrit | Doug Hellmann proposed openstack/oslo.policy master: switch from oslosphinx to openstackdocstheme https://review.openstack.org/478596 | 18:12 |
openstackgerrit | Doug Hellmann proposed openstack/oslo.policy master: import configuration guide content from openstack-manuals repo https://review.openstack.org/478597 | 18:12 |
morgan | lbragstad: I would | 18:12 |
morgan | lbragstad: not just the upload bit | 18:12 |
lbragstad | morgan: ok | 18:13 |
lbragstad | so - sounds like we need to extend the deprecation for upload and also deprecate the ability to have domains in configuration files | 18:13 |
hrybacki | lbragstad: apologies if this was in an email I missed -- do we have keystone PTG planning up anywhere? | 18:19 |
lbragstad | hrybacki: planning? not yet | 18:20 |
hrybacki | ack -- how far out do you normally start planning for PTG? | 18:20 |
lbragstad | hrybacki: let me check what i did for atlanta | 18:21 |
*** mnaser has quit IRC | 18:21 | |
lbragstad | hrybacki: i sent this http://lists.openstack.org/pipermail/openstack-dev/2017-January/110987.html on 2017-01-25 | 18:22 |
mordred | morgan: ok. I'm back to disagreeing with you ... normalize_verison_number(None) == "None" -> http://paste.openstack.org/show/613967/ | 18:22 |
*** mvk has quit IRC | 18:22 | |
lbragstad | hrybacki: which was just under a month out from the PTG in atlanta | 18:23 |
morgan | you're missing a line | 18:23 |
lbragstad | hrybacki: so i would expect us to be in full planning mode at the beginning of August | 18:23 |
hrybacki | lbragstad: ack, thanks. some folks (RH) were asking already | 18:23 |
morgan | mordred: https://github.com/openstack/keystoneauth/blob/master/keystoneauth1/discover.py#L75-L104 | 18:23 |
lbragstad | hrybacki: given the PTG in Denver start september 11th | 18:24 |
lbragstad | hrybacki: cool | 18:24 |
morgan | mordred: my highligh in gerrit was missing line 104 | 18:24 |
hrybacki | lbragstad: ack. I'll be in a weird state of flux around then -- vacation in EU returning to fly to work stuff around Boston | 18:24 |
mordred | morgan: nope. my paste wasn't | 18:24 |
mordred | morgan: the Noen gets converted intoa string | 18:24 |
mordred | earlier in the function | 18:24 |
lbragstad | hrybacki: no worries - i have a reminder to get planning started in R-7 | 18:24 |
mordred | oh - my paste was | 18:24 |
mordred | crappit | 18:24 |
mordred | one sec | 18:24 |
hrybacki | lbragstad: +1 | 18:25 |
mordred | morgan: ok. NOW I completely agree with you :) | 18:25 |
lbragstad | hrybacki: which is in two weeks | 18:25 |
morgan | mordred: ^_^ | 18:25 |
morgan | mordred: hehe yeah. like i said,i almost missed it | 18:25 |
morgan | because of the spacing and the code layout | 18:25 |
openstackgerrit | Chandan Kumar proposed openstack/keystone master: Remove keystone_tempest_plugin from setup.cfg https://review.openstack.org/478601 | 18:25 |
*** mnaser has joined #openstack-keystone | 18:26 | |
morgan | mordred: ugh. wow. | 18:26 |
morgan | mordred: this method is baad | 18:26 |
mordred | morgan: I totally agree with you that we cannot break that interface. also, it's going to be a giant pain because that interface is conceptually at odds with the spec | 18:26 |
mordred | we will, obviously, figure it out | 18:26 |
mordred | but I'm going to need to medidate on it for a minute or two | 18:27 |
morgan | mordred: yeah | 18:27 |
morgan | mordred: i just had to 2x check try/else was in-fact preventing str(None) from happening in the float. | 18:28 |
morgan | check | 18:28 |
*** links has quit IRC | 18:30 | |
*** sjain has quit IRC | 18:30 | |
mordred | morgan: oh - I think I see a decent way to solve it - will sketch it up and paste it and see what you thikn | 18:30 |
morgan | okie | 18:32 |
morgan | really the only case you have to address is version=None and url=None | 18:33 |
morgan | where a typeError is raised | 18:33 |
morgan | (require a non-none in one of them() | 18:33 |
morgan | and i think we can go on our merry way. or if version=None and url=None or version != None and normalize returns nothing, raise | 18:33 |
*** jsavak has quit IRC | 18:36 | |
*** jsavak has joined #openstack-keystone | 18:37 | |
*** aojea has joined #openstack-keystone | 18:38 | |
*** spzala has joined #openstack-keystone | 18:43 | |
gagehugo | lbragstad ack | 18:45 |
*** jsavak has quit IRC | 18:49 | |
*** jsavak has joined #openstack-keystone | 18:50 | |
*** jsavak has quit IRC | 18:52 | |
*** jsavak has joined #openstack-keystone | 18:53 | |
*** jmlowe has quit IRC | 18:53 | |
*** jmlowe has joined #openstack-keystone | 18:56 | |
openstackgerrit | Lance Bragstad proposed openstack/keystone master: WIP create a version factory https://review.openstack.org/478608 | 18:57 |
lbragstad | gagehugo: thoughts ^ | 18:57 |
gagehugo | lbragstad lemme take a look | 18:58 |
edmondsw | I'm trying to stack.sh with a local.conf that doesn't have anything specifically for keystone, and hitting an issue where keystone replies with 500 for any request and I can't find any keystone logs... any ideas? | 19:01 |
lbragstad | edmondsw: what's the 500 look like? | 19:01 |
openstackgerrit | Merged openstack/keystoneauth master: Add url manipulation and microversion collection https://review.openstack.org/469087 | 19:01 |
cmurphy | edmondsw: the keystone logs are in `journalctl -u devstack@keystone` now | 19:02 |
gagehugo | lbragstad I'll ping rarora and nhelgson | 19:02 |
edmondsw | cmurphy yeah, but that's not loaded yet at the point where the failure happens | 19:02 |
cmurphy | oh | 19:02 |
edmondsw | :( | 19:02 |
edmondsw | lbragstad http://paste.openstack.org/show/613973/ | 19:02 |
*** jsavak has quit IRC | 19:02 | |
edmondsw | just a standard 500 | 19:02 |
lbragstad | edmondsw: what about the apache logs? | 19:03 |
edmondsw | my local.conf is essentially like this: https://github.com/powervm/powervm-ci/blob/master/devstack/master/outoftree/local.conf | 19:03 |
*** jsavak has joined #openstack-keystone | 19:03 | |
edmondsw | lbragstad apache error log: http://paste.openstack.org/show/613974/ | 19:04 |
edmondsw | there's a keystone.log in that directory, but 0 size | 19:04 |
gagehugo | lbragstad it looks a lot nicer, we can definitely try to incorporate it | 19:05 |
edmondsw | here's the apache access.log: http://paste.openstack.org/show/613975/ | 19:05 |
edmondsw | I wonder if this has to do with WSGI_MODE=mod_wsgi | 19:05 |
lbragstad | gagehugo: i was trying implement something that knows about the version and what to return for stability and update times based on the version - that way you just have to ask it for the version and information | 19:06 |
lbragstad | edmondsw: that could be, since it certainly doesn't look keystone specific | 19:06 |
*** rderose has quit IRC | 19:07 | |
gagehugo | lbragstad awesome | 19:10 |
lbragstad | gagehugo: might be easier to test, too? | 19:10 |
gagehugo | probably | 19:10 |
lbragstad | gagehugo: i'm not sure - but feel free to steal any of that if it's useful for you | 19:10 |
gagehugo | lbragstad sure, thanks for writing that! | 19:11 |
lbragstad | gagehugo: anytime | 19:12 |
*** tesseract has quit IRC | 19:13 | |
*** mrhillsman has joined #openstack-keystone | 19:15 | |
*** Guest28796 is now known as mgagne | 19:19 | |
*** mgagne has quit IRC | 19:19 | |
*** mgagne has joined #openstack-keystone | 19:19 | |
openstackgerrit | Merged openstack/keystone master: Remove duplicate logging documentation https://review.openstack.org/477692 | 19:23 |
openstackgerrit | Nicolas Helgeson proposed openstack/keystone master: WIP: Add project tags https://review.openstack.org/470317 | 19:23 |
edmondsw | lbragstad cmurphy think I found the problem... something had clobbered pastedeploy... pip said installed, but the files were gone | 19:24 |
edmondsw | figured this out because when I commented that WSGI_MODE line, keystone logs became available in journalctl | 19:24 |
*** sjain has joined #openstack-keystone | 19:25 | |
lbragstad | edmondsw: ah - nice | 19:27 |
lbragstad | hrybacki: updated http://lists.openstack.org/pipermail/openstack-dev/2017-June/119064.html | 19:27 |
openstackgerrit | Harry Rybacki proposed openstack/keystone master: Move trust to DocumentedRuleDefault https://review.openstack.org/449278 | 19:30 |
hrybacki | lbragstad: thanks! And I think ^^ should pass gates now. Let the last review land. | 19:32 |
lbragstad | hrybacki: couple comments inline ' | 19:34 |
lbragstad | hrybacki: running tests on it locally now | 19:34 |
hrybacki | lbragstad: ack, I don't understand that change though | 19:34 |
hrybacki | or at least how it changes what I've done | 19:37 |
lbragstad | hrybacki: https://review.openstack.org/#/c/473902/2/keystone/trust/routers.py makes it so that all GET apis for trusts also support HEAD | 19:37 |
lbragstad | before you couldn't do something like HEAD /v3/trusts/ | 19:37 |
lbragstad | only GET /v3/trusts/ | 19:37 |
hrybacki | ohhh | 19:37 |
lbragstad | since we added that API, it's also protected under the same policy | 19:38 |
lbragstad | so list_trusts() protects both GET /v3/trusts/ and HEAD /v3/trusts/ | 19:38 |
*** sbezverk has quit IRC | 19:39 | |
lbragstad | hrybacki: so the real change there is to make sure we advertise that through policy docs | 19:40 |
lbragstad | s/that/both GET and HEAD/ | 19:40 |
hrybacki | ack, updating trust.py | 19:40 |
lbragstad | also - after that change lands we should update our sample policy file https://docs.openstack.org/developer/keystone/sample_files/sample_policy.html | 19:41 |
lbragstad | actually - nevermind, that should be done automatically | 19:42 |
openstackgerrit | Harry Rybacki proposed openstack/keystone master: Move trust to DocumentedRuleDefault https://review.openstack.org/449278 | 19:43 |
openstackgerrit | Jaewoo Park proposed openstack/keystone master: WIP: Add project tags https://review.openstack.org/470317 | 19:44 |
*** ayoung has quit IRC | 19:48 | |
cmurphy | lbragstad: I'm not clear on why domain config in config files should go away | 19:48 |
cmurphy | It's really convenient for deployment tools to just be able to lay down config files | 19:49 |
lbragstad | cmurphy: and just rely on keystone-manage tooling to handle the uploads? | 19:49 |
cmurphy | lbragstad: it doesn't need to be uploaded | 19:49 |
cmurphy | it just works | 19:49 |
lbragstad | oh - the upload is only if you want to fetch values via the API, or transition to using the API from a file-backed implementation, right? | 19:50 |
cmurphy | i don't know why we should make it harder | 19:50 |
cmurphy | right it's for transitioning | 19:50 |
cmurphy | but not everyone has a use case for transitioning | 19:51 |
lbragstad | ok - i'd buy that | 19:51 |
lbragstad | it was marked as deprecated a long time ago | 19:51 |
lbragstad | s/it/keystone-manage domain config upload/ | 19:52 |
cmurphy | the keystone-manage tool or the idea of domain config files in general? | 19:52 |
cmurphy | oh ok | 19:52 |
cmurphy | i don't care that much what happens to that | 19:52 |
lbragstad | during the meeting bknudson brought up deprecating file-backed configs | 19:52 |
lbragstad | then we started talking about it with morgan | 19:52 |
lbragstad | it *seemed* like the idea was to deprecate and remove the ability to have file backed configurations and the ability to transition | 19:53 |
lbragstad | making the API the only way to do that sort of stuff | 19:53 |
*** sjain has quit IRC | 19:53 | |
lbragstad | but if we have a usecase for it and users relying on it, i'd be happy to keep it | 19:54 |
lbragstad | i more or less just want to reach consensus on the various "deprecate without a removal date" things we have | 19:54 |
cmurphy | suse happens to be using it right now and I'd be slightly sad to see it go | 19:55 |
lbragstad | cmurphy: if suse or suse's customers are using it, then i don't think we should deprecate it then | 19:56 |
*** pnavarro has quit IRC | 19:56 | |
lbragstad | cmurphy: would you want to respond with that feedback on the mailing list so that it's persisted? | 19:56 |
cmurphy | sure | 19:56 |
*** pnavarro has joined #openstack-keystone | 19:57 | |
lbragstad | cmurphy: then i can follow up and correct the action items - or we can take the discussion further | 19:57 |
*** jmlowe has quit IRC | 19:57 | |
*** pnavarro has quit IRC | 19:57 | |
lbragstad | or see if should just revert the deprecation of the keystone-manage bits | 19:57 |
*** pnavarro has joined #openstack-keystone | 19:57 | |
*** jmlowe has joined #openstack-keystone | 19:57 | |
*** ducttape_ has joined #openstack-keystone | 20:00 | |
lbragstad | stepping away for a bit - have a meeting | 20:03 |
*** ducttap__ has joined #openstack-keystone | 20:13 | |
*** ductta___ has joined #openstack-keystone | 20:14 | |
*** ducttape_ has quit IRC | 20:16 | |
*** ducttap__ has quit IRC | 20:18 | |
openstackgerrit | Merged openstack/keystone master: Move implied role policies to DocumentedRuleDefault https://review.openstack.org/449246 | 20:18 |
*** sbezverk has joined #openstack-keystone | 20:24 | |
*** pnavarro has quit IRC | 20:24 | |
*** lucasxu has quit IRC | 20:30 | |
openstackgerrit | Eric Fried proposed openstack/keystoneauth master: Minor cleanup https://review.openstack.org/478621 | 20:35 |
*** raildo has quit IRC | 20:39 | |
*** spilla has quit IRC | 20:43 | |
*** sbezverk has quit IRC | 20:46 | |
openstackgerrit | Doug Hellmann proposed openstack/oslo.policy master: move existing documentation into new standard layout https://review.openstack.org/478595 | 20:47 |
openstackgerrit | Doug Hellmann proposed openstack/oslo.policy master: switch from oslosphinx to openstackdocstheme https://review.openstack.org/478596 | 20:47 |
openstackgerrit | Doug Hellmann proposed openstack/oslo.policy master: import configuration guide content from openstack-manuals repo https://review.openstack.org/478597 | 20:47 |
*** jsavak has quit IRC | 20:52 | |
*** jsavak has joined #openstack-keystone | 20:52 | |
*** jmlowe has quit IRC | 21:01 | |
*** pcaruana has quit IRC | 21:01 | |
*** jsavak has quit IRC | 21:01 | |
*** jsavak has joined #openstack-keystone | 21:02 | |
*** ductta___ has quit IRC | 21:05 | |
*** jsavak has quit IRC | 21:06 | |
*** jsavak has joined #openstack-keystone | 21:10 | |
*** thorst has quit IRC | 21:12 | |
*** ducttape_ has joined #openstack-keystone | 21:20 | |
*** thorst has joined #openstack-keystone | 21:46 | |
*** thorst has quit IRC | 21:50 | |
*** ducttap__ has joined #openstack-keystone | 21:53 | |
*** ducttape_ has quit IRC | 21:53 | |
*** ducttape_ has joined #openstack-keystone | 22:03 | |
*** jsavak has quit IRC | 22:05 | |
*** ducttap__ has quit IRC | 22:06 | |
*** thorst has joined #openstack-keystone | 22:10 | |
*** thorst has quit IRC | 22:17 | |
*** mvk has joined #openstack-keystone | 22:17 | |
*** thorst has joined #openstack-keystone | 22:17 | |
*** chlong_ has quit IRC | 22:18 | |
*** thorst has quit IRC | 22:22 | |
*** aojea has quit IRC | 22:29 | |
*** ducttape_ has quit IRC | 22:32 | |
*** ducttape_ has joined #openstack-keystone | 22:35 | |
*** masber has quit IRC | 22:42 | |
*** masber has joined #openstack-keystone | 22:42 | |
*** bknudson has quit IRC | 22:51 | |
*** spzala has quit IRC | 22:56 | |
*** ducttape_ has quit IRC | 23:03 | |
*** sbezverk has joined #openstack-keystone | 23:03 | |
*** spzala has joined #openstack-keystone | 23:11 | |
*** spzala has quit IRC | 23:16 | |
*** edmondsw has quit IRC | 23:19 | |
*** phalmos has quit IRC | 23:23 | |
openstackgerrit | Nicolas Helgeson proposed openstack/keystone master: WIP: Add project tags https://review.openstack.org/470317 | 23:23 |
*** ducttape_ has joined #openstack-keystone | 23:25 | |
*** spzala has joined #openstack-keystone | 23:28 | |
*** ducttape_ has quit IRC | 23:30 | |
*** jmlowe has joined #openstack-keystone | 23:38 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!