Wednesday, 2017-03-15

*** Guest1869 has joined #openstack-meeting-cp00:04
*** Guest1869 has quit IRC00:09
*** diablo_rojo has quit IRC00:47
*** diablo_rojo has joined #openstack-meeting-cp02:01
*** gouthamr has quit IRC02:39
*** markvoelker has quit IRC03:06
*** knangia has quit IRC03:21
*** knangia has joined #openstack-meeting-cp03:55
*** markvoelker has joined #openstack-meeting-cp04:07
*** markvoelker has quit IRC04:13
*** stvnoyes1 has joined #openstack-meeting-cp04:56
*** stvnoyes has quit IRC04:59
*** david-lyle has quit IRC05:04
*** markvoelker has joined #openstack-meeting-cp06:09
*** markvoelker has quit IRC06:13
*** knangia has quit IRC07:31
*** markvoelker has joined #openstack-meeting-cp08:10
*** markvoelker has quit IRC08:14
*** dmellado has quit IRC09:54
*** MarkBaker has joined #openstack-meeting-cp09:59
*** dmellado has joined #openstack-meeting-cp10:02
*** markvoelker has joined #openstack-meeting-cp10:10
*** MarkBaker has quit IRC10:15
*** markvoelker has quit IRC10:15
*** sdague has joined #openstack-meeting-cp10:27
*** scottda has quit IRC10:44
*** MarkBaker has joined #openstack-meeting-cp11:28
*** gouthamr has joined #openstack-meeting-cp11:30
*** markvoelker has joined #openstack-meeting-cp12:11
*** markvoelker has quit IRC12:16
*** MarkBaker has quit IRC12:18
*** markvoelker has joined #openstack-meeting-cp12:38
*** gouthamr has quit IRC12:54
*** MarkBaker has joined #openstack-meeting-cp13:13
*** gouthamr has joined #openstack-meeting-cp13:21
*** markvoelker_ has joined #openstack-meeting-cp13:54
*** markvoelker has quit IRC13:55
*** knangia has joined #openstack-meeting-cp14:07
*** diablo_rojo_phon has joined #openstack-meeting-cp14:39
*** edmondsw has joined #openstack-meeting-cp14:55
*** edmondsw has left #openstack-meeting-cp14:56
*** antwash has joined #openstack-meeting-cp14:57
*** antwash has left #openstack-meeting-cp14:57
*** jaugustine has joined #openstack-meeting-cp15:12
*** ayoung has joined #openstack-meeting-cp15:20
*** ayoung is now known as ayoung-teeth15:22
*** aselius has joined #openstack-meeting-cp15:28
*** lamt has joined #openstack-meeting-cp15:29
*** aselius has quit IRC15:37
*** jaugustine has quit IRC15:40
*** jaugustine has joined #openstack-meeting-cp15:42
*** persia has quit IRC15:42
*** persia has joined #openstack-meeting-cp15:45
*** phelgeson has joined #openstack-meeting-cp15:53
*** david-lyle has joined #openstack-meeting-cp15:56
lbragstad#startmeeting policy16:00
openstackMeeting 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
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.16:00
*** openstack changes topic to " (Meeting topic: policy)"16:00
openstackThe meeting name has been set to 'policy'16:01
lbragstadagenda #link https://etherpad.openstack.org/p/keystone-policy-meeting16:01
*** gagehugo has joined #openstack-meeting-cp16:01
lbragstadping antwash, raildo, ktychkova, dolphm, dstanek, rderose, htruta, atrmr, gagehugo, lamt, thinrichs, edmondsw, ruan, ayoung, ravelar, morgan, raj_singh, johnthetubaguy, knikolla16:01
gagehugoo/16:01
lamto/16:02
lamtstill getting used to DST.16:02
knikollao/16:02
lbragstadsame here - it threw me for a loop earlier16:03
* johnthetubaguy hides at the back of the room16:03
lbragstadhere is the agenda #link https://etherpad.openstack.org/p/keystone-policy-meeting16:04
lbragstad#topic reviews16:04
*** openstack changes topic to "reviews (Meeting topic: policy)"16:04
lbragstadwe have several things that have been in the pipe for a while16:04
lbragstadsince 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
lbragstadi figured we could use today for reviews, unless people have specific things they want to discuss16:05
johnthetubaguyI am pushing on the nova specs side of things16:05
johnthetubaguybut no news as such16:05
lbragstadjohnthetubaguy still pushing the specs through?16:05
johnthetubaguysneti and aunnam are get lots of the polcy docs patches up for review already16:06
johnthetubaguyyeah, still pushing them16:06
lbragstadnice16:06
johnthetubaguywe have *a lot* of specs up for review16:06
johnthetubaguyits a bit scary16:06
lbragstadthat 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 good16:07
lbragstadI'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.policy16:07
lbragstadthat way keystone and nova can start using the same interface for the policy-docs specs16:07
johnthetubaguyyeah, need to bump global requirements to start using that too I think16:08
*** antwash has joined #openstack-meeting-cp16:08
lbragstadjohnthetubaguy ah - good point16:08
lbragstadjohnthetubaguy i can look into that once it merges16:08
johnthetubaguythe other bit is in the sample generator I think aunnam was going to look at making sure the rules are commented out by default16:08
*** rderose has joined #openstack-meeting-cp16:08
johnthetubaguylbragstad: cool, that be great16:08
lbragstadjohnthetubaguy i want to say antwash was looking into that, too16:09
*** phelgeson has quit IRC16:09
johnthetubaguyah, thats cool if thats already happening16:09
lbragstad#link https://review.openstack.org/#/c/443330/216:09
lbragstadi think that's it ^16:09
aunnamthen we both must talk to each other16:09
lbragstadaunnam ++16:10
lbragstadaunnam meet antwash16:10
antwashyeah I -1 it myself until the other patch gets merged16:10
johnthetubaguyah, there is a bit more we need there16:10
aunnamhttps://review.openstack.org/#/c/443330/2 but this is for description support right?16:10
lbragstadantwash meet aunnam16:10
lbragstad:)16:10
*** MarkBaker has quit IRC16:10
aunnamnot commenting the rule16:10
lbragstadaunnam yeah - support for rendering the description i believe16:10
johnthetubaguyaunnam +116:10
johnthetubaguylbragstad: I think thats an extra thing16:10
lbragstadjohnthetubaguy which part?16:11
johnthetubaguywe have the operations in the same with that patch16:11
johnthetubaguyin the sample16:11
johnthetubaguybut currently it look like this:16:11
johnthetubaguyhttps://docs.openstack.org/developer/nova/sample_policy.html16:11
johnthetubaguywe need the rule line to start with a comment as well16:11
lbragstadah16:11
lbragstadok16:11
johnthetubaguyI think aunnam was looking at how to re-write the tests to support that16:11
johnthetubaguyif I remember that correctly16:12
aunnamjohnthetubaguy, yes16:12
johnthetubaguyaunnam: did that work out ok?16:12
lbragstadaunnam johnthetubaguy this is all stuff that needs to go into oslo.policy though, right?16:12
johnthetubaguywant to share your plan there?16:12
johnthetubaguylbragstad: yes16:12
aunnamjohnthetubaguy, not yet, i started looking into code today16:12
johnthetubaguycool, sounds like that doesn't overlap16:13
johnthetubaguyI am sure lbragstad would love to review that when it goes up16:13
lbragstad++16:13
lbragstadso - it sounds like i'm hearing that three different things need to happen16:13
johnthetubaguycool, that sounds like all the bits we need now, yeah, lets summarise16:13
antwashWe'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
lbragstad1.) we need to merge support for adding descriptions to oslo.policy objects at registration time ( #link https://review.openstack.org/#/c/439070/ )16:14
lbragstad2.) add support to render descriptions in sample policy files ( #link https://review.openstack.org/#/c/443330/2 )16:14
lbragstad3.) add support to comment out rule lines in sample policy files16:15
lbragstadall three of which need to be done in oslo.policy16:15
*** spilla has joined #openstack-meeting-cp16:15
lbragstadand not necessarily in that order16:15
lbragstaddoes that sound accurate?16:15
johnthetubaguylbragstad: yes, we have already merged the things that supports multi-line description I think16:15
lbragstadjohnthetubaguy in oslo.policy or nova?16:16
johnthetubaguyantwash: I think I found a way for aunnam to fix the existing tests so the tweak is really small16:16
johnthetubaguyaunnam: I think that multi line fix was in 1.20 right?16:16
aunnamlbragstad, yes as these two patches has been merged already https://review.openstack.org/#/c/441342/ https://review.openstack.org/#/c/443332/16:16
lbragstadah - #link https://review.openstack.org/#/c/441342/16:16
aunnamjohnthetubaguy, yes16:16
johnthetubaguyso step 0 is done, just need those three16:17
lbragstadok - so steps 1 - 3 in my list above still need to happen16:17
lbragstads/happen/happen?16:18
johnthetubaguyyep16:18
lbragstadawesome - i'm going to make some action items16: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/216:20
lbragstadis anyone willing to pick up #3?16:20
lbragstadaunnam you were already working on that?16:20
aunnamjohnthetubaguy, https://github.com/openstack/oslo.policy/blob/master/oslo_policy/tests/test_generator.py#L278 as it is getting rules from sample policy file16:20
johnthetubaguyyeah, I think aunnam and sneti were going to sort out #316:21
aunnamlbragstad, yes i started looking into the failing tests if I comment out the rule16:21
lbragstadaunnam 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
antwashfor #2, we're referring to adding the mulitiline description support..right?16:22
lbragstadantwash yeah - #2 is an action item to make sure oslo.policy renders descriptions in sample policy files16:23
johnthetubaguy#2 is about getting the URLs and actions into the sample file I think16:23
johnthetubaguythe descriptions are already in the sample files16:23
lbragstadaha16:23
johnthetubaguyit already supports multi-line descriptions too16:23
lbragstad#undo16:23
openstackRemoving item from minutes: #action review and approve support for rendering descriptions in sample policy files - #link https://review.openstack.org/#/c/443330/216:23
johnthetubaguyyeah, my brain read what I expected, not what you wrote, I hate it when that happens16:24
antwashthis patch adds the urls and actions to the file though https://review.openstack.org/#/c/443330/16:24
johnthetubaguyyeah, thats (2) in my head16:24
johnthetubaguyits doing the correct thing16:24
johnthetubaguyroughly16: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/44333016:24
lbragstad^ moar better?16:25
johnthetubaguyyeah, that looks cool16:25
aunnamlbragstad, ya i think that test is specific to nova, so without that test my change is working fine16:25
lbragstadantwash makes sense?16:25
aunnamlbragstad, will look into it again16:25
lbragstadok16:25
*** aselius has joined #openstack-meeting-cp16:26
antwashlbragstad: yeah I think so now, the idea is to add the URL/ACTION apart of the description when rendering the file16:26
lbragstadantwash yep - exactly16:26
antwashThat'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 files16:27
lbragstad^ that should be #316:27
aunnamlbragstad, yes16:27
lbragstadso 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
lbragstadthat has better descriptions16:28
lbragstadand once all three are done - i'll propose a new version of oslo.policy and look at bumping requirements so that we can consume it16:30
lbragstad#action lbragstad to roll a new version of oslo.policy and bump global requirements once the above actions are complete16:30
johnthetubaguyyeah, that sounds good16:31
lbragstadawesome16:31
lbragstadI'll be available to do oslo.policy reviews all week - so if anyone has things that need to be reviewed, let me know16:31
*** kbyrne has quit IRC16:32
aunnamlbragstad, sure, thanks16:32
lbragstadand 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 soon16:32
antwash++16:33
lbragstadso if anyone wants to give that a once over - I'd appreciate any feedback on it16:33
*** david-lyle_ has joined #openstack-meeting-cp16:34
*** david-lyle has quit IRC16:34
lbragstadand we can also focus on landing #link https://review.openstack.org/#/q/status:open+project:openstack/keystone+branch:master+topic:bp/policy-in-code16:34
lbragstadideally, once all that is done keystone should be up-to-par with nova as far as policy is concerned (i believe)16:35
lbragstadi think that's all i had for review16:36
lbragstadreviews*16:36
*** kbyrne has joined #openstack-meeting-cp16:36
lbragstad#topic open discussion16:36
*** openstack changes topic to "open discussion (Meeting topic: policy)"16:36
lbragstadwe have a bunch of time left to discuss things if folks want16:37
lbragstadotherwise we can end early and go do oslo.policy reviews ;)16:37
antwashI don't have nothing on my end16:37
lbragstadalrighty - well thanks for coming and ping me if you need anything oslo.policy-wise16:39
lbragstad#endmeeting16:39
*** openstack changes topic to "OpenStack Meetings || https://wiki.openstack.org/wiki/Meetings"16:39
openstackMeeting ended Wed Mar 15 16:39:50 2017 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)16:39
openstackMinutes:        http://eavesdrop.openstack.org/meetings/policy/2017/policy.2017-03-15-16.00.html16:39
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/policy/2017/policy.2017-03-15-16.00.txt16:39
openstackLog:            http://eavesdrop.openstack.org/meetings/policy/2017/policy.2017-03-15-16.00.log.html16:39
*** antwash has left #openstack-meeting-cp17:21
*** ayoung-teeth is now known as ayoung17:45
*** MarkBaker has joined #openstack-meeting-cp17:56
*** stvnoyes1 has quit IRC18:00
*** gagehugo has left #openstack-meeting-cp18:03
*** stvnoyes has joined #openstack-meeting-cp18:04
*** gagehugo has joined #openstack-meeting-cp18:04
*** gagehugo has left #openstack-meeting-cp18:05
*** diablo_rojo_phon has quit IRC18:10
*** MarkBaker has quit IRC18:29
*** lamt has quit IRC19:02
*** lamt has joined #openstack-meeting-cp19:03
*** stvnoyes has quit IRC19:08
*** stvnoyes has joined #openstack-meeting-cp19:09
*** stvnoyes has left #openstack-meeting-cp19:30
*** edtubill has joined #openstack-meeting-cp19:34
*** aunnam has quit IRC19:45
*** raj_singh has quit IRC19:45
*** sneti has quit IRC19:45
*** david-lyle_ is now known as david-lyle19:57
*** lamt has quit IRC19:58
*** MarkBaker has joined #openstack-meeting-cp20:01
*** docaedo has quit IRC20:12
*** MarkBaker has quit IRC20:19
*** docaedo has joined #openstack-meeting-cp20:25
*** lamt has joined #openstack-meeting-cp20:32
*** lamt has quit IRC20:33
*** lamt has joined #openstack-meeting-cp20:34
*** spilla has left #openstack-meeting-cp20:34
*** lamt has quit IRC20:39
*** docaedo has quit IRC20:39
*** stream10 has joined #openstack-meeting-cp20:40
*** docaedo has joined #openstack-meeting-cp20:40
*** raj_singh has joined #openstack-meeting-cp20:41
*** gouthamr has quit IRC20:55
*** stream10 has quit IRC21:06
*** sdague has quit IRC21:09
*** gouthamr has joined #openstack-meeting-cp21:13
*** rderose_ has joined #openstack-meeting-cp21:42
*** rderose has quit IRC21:44
*** lamt has joined #openstack-meeting-cp21:46
*** edtubill has quit IRC21:52
*** lamt has quit IRC21:56
*** raj_singh has quit IRC22:00
*** raj_singh has joined #openstack-meeting-cp22:00
*** lamt has joined #openstack-meeting-cp22:09
*** edtubill has joined #openstack-meeting-cp22:09
*** sheeprine has quit IRC22:15
*** sheeprine has joined #openstack-meeting-cp22:19
*** lamt has quit IRC22:35
*** lamt has joined #openstack-meeting-cp22:57
*** lamt has quit IRC23:02
*** jaugustine has quit IRC23:11
*** IgorYozhikov has quit IRC23:17
*** IgorYozhikov has joined #openstack-meeting-cp23:17
*** aselius has quit IRC23:38

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