*** Guest1869 has joined #openstack-meeting-cp | 00:04 | |
*** Guest1869 has quit IRC | 00:09 | |
*** diablo_rojo has quit IRC | 00:47 | |
*** diablo_rojo has joined #openstack-meeting-cp | 02:01 | |
*** gouthamr has quit IRC | 02:39 | |
*** markvoelker has quit IRC | 03:06 | |
*** knangia has quit IRC | 03:21 | |
*** knangia has joined #openstack-meeting-cp | 03:55 | |
*** markvoelker has joined #openstack-meeting-cp | 04:07 | |
*** markvoelker has quit IRC | 04:13 | |
*** stvnoyes1 has joined #openstack-meeting-cp | 04:56 | |
*** stvnoyes has quit IRC | 04:59 | |
*** david-lyle has quit IRC | 05:04 | |
*** markvoelker has joined #openstack-meeting-cp | 06:09 | |
*** markvoelker has quit IRC | 06:13 | |
*** knangia has quit IRC | 07:31 | |
*** markvoelker has joined #openstack-meeting-cp | 08:10 | |
*** markvoelker has quit IRC | 08:14 | |
*** dmellado has quit IRC | 09:54 | |
*** MarkBaker has joined #openstack-meeting-cp | 09:59 | |
*** dmellado has joined #openstack-meeting-cp | 10:02 | |
*** markvoelker has joined #openstack-meeting-cp | 10:10 | |
*** MarkBaker has quit IRC | 10:15 | |
*** markvoelker has quit IRC | 10:15 | |
*** sdague has joined #openstack-meeting-cp | 10:27 | |
*** scottda has quit IRC | 10:44 | |
*** MarkBaker has joined #openstack-meeting-cp | 11:28 | |
*** gouthamr has joined #openstack-meeting-cp | 11:30 | |
*** markvoelker has joined #openstack-meeting-cp | 12:11 | |
*** markvoelker has quit IRC | 12:16 | |
*** MarkBaker has quit IRC | 12:18 | |
*** markvoelker has joined #openstack-meeting-cp | 12:38 | |
*** gouthamr has quit IRC | 12:54 | |
*** MarkBaker has joined #openstack-meeting-cp | 13:13 | |
*** gouthamr has joined #openstack-meeting-cp | 13:21 | |
*** markvoelker_ has joined #openstack-meeting-cp | 13:54 | |
*** markvoelker has quit IRC | 13:55 | |
*** knangia has joined #openstack-meeting-cp | 14:07 | |
*** diablo_rojo_phon has joined #openstack-meeting-cp | 14:39 | |
*** edmondsw has joined #openstack-meeting-cp | 14:55 | |
*** edmondsw has left #openstack-meeting-cp | 14:56 | |
*** antwash has joined #openstack-meeting-cp | 14:57 | |
*** antwash has left #openstack-meeting-cp | 14:57 | |
*** jaugustine has joined #openstack-meeting-cp | 15:12 | |
*** ayoung has joined #openstack-meeting-cp | 15:20 | |
*** ayoung is now known as ayoung-teeth | 15:22 | |
*** aselius has joined #openstack-meeting-cp | 15:28 | |
*** lamt has joined #openstack-meeting-cp | 15:29 | |
*** aselius has quit IRC | 15:37 | |
*** jaugustine has quit IRC | 15:40 | |
*** jaugustine has joined #openstack-meeting-cp | 15:42 | |
*** persia has quit IRC | 15:42 | |
*** persia has joined #openstack-meeting-cp | 15:45 | |
*** phelgeson has joined #openstack-meeting-cp | 15:53 | |
*** david-lyle has joined #openstack-meeting-cp | 15:56 | |
lbragstad | #startmeeting policy | 16:00 |
---|---|---|
openstack | Meeting started Wed Mar 15 16:00:55 2017 UTC and is due to finish in 60 minutes. The chair is lbragstad. Information about MeetBot at http://wiki.debian.org/MeetBot. | 16:00 |
openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 16:00 |
*** openstack changes topic to " (Meeting topic: policy)" | 16:00 | |
openstack | The meeting name has been set to 'policy' | 16:01 |
lbragstad | agenda #link https://etherpad.openstack.org/p/keystone-policy-meeting | 16:01 |
*** gagehugo has joined #openstack-meeting-cp | 16:01 | |
lbragstad | ping antwash, raildo, ktychkova, dolphm, dstanek, rderose, htruta, atrmr, gagehugo, lamt, thinrichs, edmondsw, ruan, ayoung, ravelar, morgan, raj_singh, johnthetubaguy, knikolla | 16:01 |
gagehugo | o/ | 16:01 |
lamt | o/ | 16:02 |
lamt | still getting used to DST. | 16:02 |
knikolla | o/ | 16:02 |
lbragstad | same here - it threw me for a loop earlier | 16:03 |
* johnthetubaguy hides at the back of the room | 16:03 | |
lbragstad | here is the agenda #link https://etherpad.openstack.org/p/keystone-policy-meeting | 16:04 |
lbragstad | #topic reviews | 16:04 |
*** openstack changes topic to "reviews (Meeting topic: policy)" | 16:04 | |
lbragstad | we have several things that have been in the pipe for a while | 16:04 |
lbragstad | since we have a light agenda today - as well as a low attendance (i know a bunch of people are out due to spring break) | 16:05 |
lbragstad | i figured we could use today for reviews, unless people have specific things they want to discuss | 16:05 |
johnthetubaguy | I am pushing on the nova specs side of things | 16:05 |
johnthetubaguy | but no news as such | 16:05 |
lbragstad | johnthetubaguy still pushing the specs through? | 16:05 |
johnthetubaguy | sneti and aunnam are get lots of the polcy docs patches up for review already | 16:06 |
johnthetubaguy | yeah, still pushing them | 16:06 |
lbragstad | nice | 16:06 |
johnthetubaguy | we have *a lot* of specs up for review | 16:06 |
johnthetubaguy | its a bit scary | 16:06 |
lbragstad | that brings us to our first review actually - #link https://review.openstack.org/#/c/439070/ | 16:06 |
lbragstad | ^ antwash has been spinning that over the past week and I think it's starting to look pretty good | 16:07 |
lbragstad | I'd like to see some documentation on the usage of it - but once that merges i'd like to propose a new release of oslo.policy | 16:07 |
lbragstad | that way keystone and nova can start using the same interface for the policy-docs specs | 16:07 |
johnthetubaguy | yeah, need to bump global requirements to start using that too I think | 16:08 |
*** antwash has joined #openstack-meeting-cp | 16:08 | |
lbragstad | johnthetubaguy ah - good point | 16:08 |
lbragstad | johnthetubaguy i can look into that once it merges | 16:08 |
johnthetubaguy | the other bit is in the sample generator I think aunnam was going to look at making sure the rules are commented out by default | 16:08 |
*** rderose has joined #openstack-meeting-cp | 16:08 | |
johnthetubaguy | lbragstad: cool, that be great | 16:08 |
lbragstad | johnthetubaguy i want to say antwash was looking into that, too | 16:09 |
*** phelgeson has quit IRC | 16:09 | |
johnthetubaguy | ah, thats cool if thats already happening | 16:09 |
lbragstad | #link https://review.openstack.org/#/c/443330/2 | 16:09 |
lbragstad | i think that's it ^ | 16:09 |
aunnam | then we both must talk to each other | 16:09 |
lbragstad | aunnam ++ | 16:10 |
lbragstad | aunnam meet antwash | 16:10 |
antwash | yeah I -1 it myself until the other patch gets merged | 16:10 |
johnthetubaguy | ah, there is a bit more we need there | 16:10 |
aunnam | https://review.openstack.org/#/c/443330/2 but this is for description support right? | 16:10 |
lbragstad | antwash meet aunnam | 16:10 |
lbragstad | :) | 16:10 |
*** MarkBaker has quit IRC | 16:10 | |
aunnam | not commenting the rule | 16:10 |
lbragstad | aunnam yeah - support for rendering the description i believe | 16:10 |
johnthetubaguy | aunnam +1 | 16:10 |
johnthetubaguy | lbragstad: I think thats an extra thing | 16:10 |
lbragstad | johnthetubaguy which part? | 16:11 |
johnthetubaguy | we have the operations in the same with that patch | 16:11 |
johnthetubaguy | in the sample | 16:11 |
johnthetubaguy | but currently it look like this: | 16:11 |
johnthetubaguy | https://docs.openstack.org/developer/nova/sample_policy.html | 16:11 |
johnthetubaguy | we need the rule line to start with a comment as well | 16:11 |
lbragstad | ah | 16:11 |
lbragstad | ok | 16:11 |
johnthetubaguy | I think aunnam was looking at how to re-write the tests to support that | 16:11 |
johnthetubaguy | if I remember that correctly | 16:12 |
aunnam | johnthetubaguy, yes | 16:12 |
johnthetubaguy | aunnam: did that work out ok? | 16:12 |
lbragstad | aunnam johnthetubaguy this is all stuff that needs to go into oslo.policy though, right? | 16:12 |
johnthetubaguy | want to share your plan there? | 16:12 |
johnthetubaguy | lbragstad: yes | 16:12 |
aunnam | johnthetubaguy, not yet, i started looking into code today | 16:12 |
johnthetubaguy | cool, sounds like that doesn't overlap | 16:13 |
johnthetubaguy | I am sure lbragstad would love to review that when it goes up | 16:13 |
lbragstad | ++ | 16:13 |
lbragstad | so - it sounds like i'm hearing that three different things need to happen | 16:13 |
johnthetubaguy | cool, that sounds like all the bits we need now, yeah, lets summarise | 16:13 |
antwash | We're going to have to write a set of new test to support the new policy? and leave the test to that support the old policy alone? | 16:14 |
lbragstad | 1.) we need to merge support for adding descriptions to oslo.policy objects at registration time ( #link https://review.openstack.org/#/c/439070/ ) | 16:14 |
lbragstad | 2.) add support to render descriptions in sample policy files ( #link https://review.openstack.org/#/c/443330/2 ) | 16:14 |
lbragstad | 3.) add support to comment out rule lines in sample policy files | 16:15 |
lbragstad | all three of which need to be done in oslo.policy | 16:15 |
*** spilla has joined #openstack-meeting-cp | 16:15 | |
lbragstad | and not necessarily in that order | 16:15 |
lbragstad | does that sound accurate? | 16:15 |
johnthetubaguy | lbragstad: yes, we have already merged the things that supports multi-line description I think | 16:15 |
lbragstad | johnthetubaguy in oslo.policy or nova? | 16:16 |
johnthetubaguy | antwash: I think I found a way for aunnam to fix the existing tests so the tweak is really small | 16:16 |
johnthetubaguy | aunnam: I think that multi line fix was in 1.20 right? | 16:16 |
aunnam | lbragstad, yes as these two patches has been merged already https://review.openstack.org/#/c/441342/ https://review.openstack.org/#/c/443332/ | 16:16 |
lbragstad | ah - #link https://review.openstack.org/#/c/441342/ | 16:16 |
aunnam | johnthetubaguy, yes | 16:16 |
johnthetubaguy | so step 0 is done, just need those three | 16:17 |
lbragstad | ok - so steps 1 - 3 in my list above still need to happen | 16:17 |
lbragstad | s/happen/happen? | 16:18 |
johnthetubaguy | yep | 16:18 |
lbragstad | awesome - i'm going to make some action items | 16:19 |
lbragstad | #action get approval on #link https://review.openstack.org/#/c/439070/ | 16:19 |
lbragstad | #action review and approve support for rendering descriptions in sample policy files - #link https://review.openstack.org/#/c/443330/2 | 16:20 |
lbragstad | is anyone willing to pick up #3? | 16:20 |
lbragstad | aunnam you were already working on that? | 16:20 |
aunnam | johnthetubaguy, https://github.com/openstack/oslo.policy/blob/master/oslo_policy/tests/test_generator.py#L278 as it is getting rules from sample policy file | 16:20 |
johnthetubaguy | yeah, I think aunnam and sneti were going to sort out #3 | 16:21 |
aunnam | lbragstad, yes i started looking into the failing tests if I comment out the rule | 16:21 |
lbragstad | aunnam ah - so that's test refactoring specific to nova, but we still need support for commenting out the default rule in oslo.policy? | 16:22 |
antwash | for #2, we're referring to adding the mulitiline description support..right? | 16:22 |
lbragstad | antwash yeah - #2 is an action item to make sure oslo.policy renders descriptions in sample policy files | 16:23 |
johnthetubaguy | #2 is about getting the URLs and actions into the sample file I think | 16:23 |
johnthetubaguy | the descriptions are already in the sample files | 16:23 |
lbragstad | aha | 16:23 |
johnthetubaguy | it already supports multi-line descriptions too | 16:23 |
lbragstad | #undo | 16:23 |
openstack | Removing item from minutes: #action review and approve support for rendering descriptions in sample policy files - #link https://review.openstack.org/#/c/443330/2 | 16:23 |
johnthetubaguy | yeah, my brain read what I expected, not what you wrote, I hate it when that happens | 16:24 |
antwash | this patch adds the urls and actions to the file though https://review.openstack.org/#/c/443330/ | 16:24 |
johnthetubaguy | yeah, thats (2) in my head | 16:24 |
johnthetubaguy | its doing the correct thing | 16:24 |
johnthetubaguy | roughly | 16:24 |
lbragstad | #action add support so oslo.policy to include URLs and actions in the rendered sample policy descriptions #link https://review.openstack.org/#/c/443330 | 16:24 |
lbragstad | ^ moar better? | 16:25 |
johnthetubaguy | yeah, that looks cool | 16:25 |
aunnam | lbragstad, ya i think that test is specific to nova, so without that test my change is working fine | 16:25 |
lbragstad | antwash makes sense? | 16:25 |
aunnam | lbragstad, will look into it again | 16:25 |
lbragstad | ok | 16:25 |
*** aselius has joined #openstack-meeting-cp | 16:26 | |
antwash | lbragstad: yeah I think so now, the idea is to add the URL/ACTION apart of the description when rendering the file | 16:26 |
lbragstad | antwash yep - exactly | 16:26 |
antwash | That's why the multiline support was added here https://review.openstack.org/#/c/441342/ | 16:26 |
lbragstad | #action add support to oslo.policy to comment out the default rule when rendering sample policy files | 16:27 |
lbragstad | ^ that should be #3 | 16:27 |
aunnam | lbragstad, yes | 16:27 |
lbragstad | so after those three action items are addressed, the sample policy files should just be one big commented out file (much like a generated config file) | 16:28 |
lbragstad | that has better descriptions | 16:28 |
lbragstad | and once all three are done - i'll propose a new version of oslo.policy and look at bumping requirements so that we can consume it | 16:30 |
lbragstad | #action lbragstad to roll a new version of oslo.policy and bump global requirements once the above actions are complete | 16:30 |
johnthetubaguy | yeah, that sounds good | 16:31 |
lbragstad | awesome | 16:31 |
lbragstad | I'll be available to do oslo.policy reviews all week - so if anyone has things that need to be reviewed, let me know | 16:31 |
*** kbyrne has quit IRC | 16:32 | |
aunnam | lbragstad, sure, thanks | 16:32 |
lbragstad | and with all ^ that work being done to move us closer to policy docs - i'd like to see if we can get #link https://review.openstack.org/#/c/435078/ merged soon | 16:32 |
antwash | ++ | 16:33 |
lbragstad | so if anyone wants to give that a once over - I'd appreciate any feedback on it | 16:33 |
*** david-lyle_ has joined #openstack-meeting-cp | 16:34 | |
*** david-lyle has quit IRC | 16:34 | |
lbragstad | and we can also focus on landing #link https://review.openstack.org/#/q/status:open+project:openstack/keystone+branch:master+topic:bp/policy-in-code | 16:34 |
lbragstad | ideally, once all that is done keystone should be up-to-par with nova as far as policy is concerned (i believe) | 16:35 |
lbragstad | i think that's all i had for review | 16:36 |
lbragstad | reviews* | 16:36 |
*** kbyrne has joined #openstack-meeting-cp | 16:36 | |
lbragstad | #topic open discussion | 16:36 |
*** openstack changes topic to "open discussion (Meeting topic: policy)" | 16:36 | |
lbragstad | we have a bunch of time left to discuss things if folks want | 16:37 |
lbragstad | otherwise we can end early and go do oslo.policy reviews ;) | 16:37 |
antwash | I don't have nothing on my end | 16:37 |
lbragstad | alrighty - well thanks for coming and ping me if you need anything oslo.policy-wise | 16:39 |
lbragstad | #endmeeting | 16:39 |
*** openstack changes topic to "OpenStack Meetings || https://wiki.openstack.org/wiki/Meetings" | 16:39 | |
openstack | Meeting ended Wed Mar 15 16:39:50 2017 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:39 |
openstack | Minutes: http://eavesdrop.openstack.org/meetings/policy/2017/policy.2017-03-15-16.00.html | 16:39 |
openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/policy/2017/policy.2017-03-15-16.00.txt | 16:39 |
openstack | Log: http://eavesdrop.openstack.org/meetings/policy/2017/policy.2017-03-15-16.00.log.html | 16:39 |
*** antwash has left #openstack-meeting-cp | 17:21 | |
*** ayoung-teeth is now known as ayoung | 17:45 | |
*** MarkBaker has joined #openstack-meeting-cp | 17:56 | |
*** stvnoyes1 has quit IRC | 18:00 | |
*** gagehugo has left #openstack-meeting-cp | 18:03 | |
*** stvnoyes has joined #openstack-meeting-cp | 18:04 | |
*** gagehugo has joined #openstack-meeting-cp | 18:04 | |
*** gagehugo has left #openstack-meeting-cp | 18:05 | |
*** diablo_rojo_phon has quit IRC | 18:10 | |
*** MarkBaker has quit IRC | 18:29 | |
*** lamt has quit IRC | 19:02 | |
*** lamt has joined #openstack-meeting-cp | 19:03 | |
*** stvnoyes has quit IRC | 19:08 | |
*** stvnoyes has joined #openstack-meeting-cp | 19:09 | |
*** stvnoyes has left #openstack-meeting-cp | 19:30 | |
*** edtubill has joined #openstack-meeting-cp | 19:34 | |
*** aunnam has quit IRC | 19:45 | |
*** raj_singh has quit IRC | 19:45 | |
*** sneti has quit IRC | 19:45 | |
*** david-lyle_ is now known as david-lyle | 19:57 | |
*** lamt has quit IRC | 19:58 | |
*** MarkBaker has joined #openstack-meeting-cp | 20:01 | |
*** docaedo has quit IRC | 20:12 | |
*** MarkBaker has quit IRC | 20:19 | |
*** docaedo has joined #openstack-meeting-cp | 20:25 | |
*** lamt has joined #openstack-meeting-cp | 20:32 | |
*** lamt has quit IRC | 20:33 | |
*** lamt has joined #openstack-meeting-cp | 20:34 | |
*** spilla has left #openstack-meeting-cp | 20:34 | |
*** lamt has quit IRC | 20:39 | |
*** docaedo has quit IRC | 20:39 | |
*** stream10 has joined #openstack-meeting-cp | 20:40 | |
*** docaedo has joined #openstack-meeting-cp | 20:40 | |
*** raj_singh has joined #openstack-meeting-cp | 20:41 | |
*** gouthamr has quit IRC | 20:55 | |
*** stream10 has quit IRC | 21:06 | |
*** sdague has quit IRC | 21:09 | |
*** gouthamr has joined #openstack-meeting-cp | 21:13 | |
*** rderose_ has joined #openstack-meeting-cp | 21:42 | |
*** rderose has quit IRC | 21:44 | |
*** lamt has joined #openstack-meeting-cp | 21:46 | |
*** edtubill has quit IRC | 21:52 | |
*** lamt has quit IRC | 21:56 | |
*** raj_singh has quit IRC | 22:00 | |
*** raj_singh has joined #openstack-meeting-cp | 22:00 | |
*** lamt has joined #openstack-meeting-cp | 22:09 | |
*** edtubill has joined #openstack-meeting-cp | 22:09 | |
*** sheeprine has quit IRC | 22:15 | |
*** sheeprine has joined #openstack-meeting-cp | 22:19 | |
*** lamt has quit IRC | 22:35 | |
*** lamt has joined #openstack-meeting-cp | 22:57 | |
*** lamt has quit IRC | 23:02 | |
*** jaugustine has quit IRC | 23:11 | |
*** IgorYozhikov has quit IRC | 23:17 | |
*** IgorYozhikov has joined #openstack-meeting-cp | 23:17 | |
*** aselius has quit IRC | 23:38 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!