Wednesday, 2015-03-25

*** kgriffs is now known as kgriffs|afk00:24
*** SheenaG has joined #openstack-barbican00:42
*** gyee has quit IRC00:48
*** zz_dimtruck is now known as dimtruck00:56
*** SheenaG has quit IRC01:34
*** dimtruck is now known as zz_dimtruck02:00
*** SheenaG has joined #openstack-barbican02:03
*** alee_afk has quit IRC02:57
*** xaeth_afk is now known as xaeth03:08
*** alee_afk has joined #openstack-barbican03:10
*** xaeth is now known as xaeth_afk03:10
*** xaeth_afk is now known as xaeth03:10
*** xaeth is now known as xaeth_afk03:11
openstackgerritArun Kant proposed openstack/barbican: For per secret ACL support, adding db layer changes (Part 1)  https://review.openstack.org/16433403:13
openstackgerritArun Kant proposed openstack/barbican: Adding Secret ACL controller layer changes (Part 2)  https://review.openstack.org/16433503:14
openstackgerritArun Kant proposed openstack/barbican: Adding Container ACL controller layer changes (Part 3)  https://review.openstack.org/16520503:14
*** alee_afk has quit IRC03:14
openstackgerritArun Kant proposed openstack/barbican: Adding policy layer changes for ACL support (Part 4)  https://review.openstack.org/16520703:14
*** alee_afk has joined #openstack-barbican03:16
*** alee_afk has quit IRC03:20
*** SheenaG1 has joined #openstack-barbican03:26
*** SheenaG has quit IRC03:28
*** kebray has joined #openstack-barbican03:34
*** SheenaG1 has quit IRC03:38
openstackgerritDave McCowan proposed openstack/barbican: Fix string formatting for a secret store exception message  https://review.openstack.org/16748403:52
*** tkelsey has joined #openstack-barbican03:53
*** tkelsey has quit IRC03:57
*** rm_you| has joined #openstack-barbican04:03
*** rm_you|wtf has quit IRC04:05
*** woodster_ has quit IRC04:10
openstackgerritDave McCowan proposed openstack/barbican: Fix string formatting for a secret store exception message  https://review.openstack.org/16748404:16
*** kebray has quit IRC04:19
*** alee_afk has joined #openstack-barbican05:32
openstackgerritDave McCowan proposed openstack/barbican: Implement validators and tests for stored key certificate orders  https://review.openstack.org/16729105:45
*** dave-mccowan has quit IRC05:46
*** jamielennox is now known as jamielennox|away05:49
*** tkelsey has joined #openstack-barbican05:54
*** tkelsey has quit IRC05:59
*** chlong has quit IRC07:52
*** tkelsey has joined #openstack-barbican08:09
*** gitorres has joined #openstack-barbican09:08
*** gitorres has left #openstack-barbican09:36
*** SheenaG has joined #openstack-barbican09:44
*** SheenaG has quit IRC09:48
*** gitorres has joined #openstack-barbican10:40
*** woodster_ has joined #openstack-barbican11:50
*** chlong has joined #openstack-barbican11:51
*** openstackgerrit has quit IRC12:06
*** openstackgerrit has joined #openstack-barbican12:06
*** jaosorior has joined #openstack-barbican12:08
jaosoriorhockeynut, rellerreller: ping12:08
openstackgerritArun Kant proposed openstack/barbican: For per secret ACL support, adding db layer changes (Part 1)  https://review.openstack.org/16433413:03
openstackgerritArun Kant proposed openstack/barbican: Adding Secret ACL controller layer changes (Part 2)  https://review.openstack.org/16433513:03
openstackgerritArun Kant proposed openstack/barbican: Adding Container ACL controller layer changes (Part 3)  https://review.openstack.org/16520513:03
openstackgerritArun Kant proposed openstack/barbican: Adding policy layer changes for ACL support (Part 4)  https://review.openstack.org/16520713:03
*** gitorres has left #openstack-barbican13:17
*** joesavak has joined #openstack-barbican13:21
*** everjeje has joined #openstack-barbican13:27
*** joesavak has quit IRC13:27
*** alee_afk is now known as alee13:34
*** openstackgerrit has quit IRC13:36
*** openstackgerrit has joined #openstack-barbican13:36
arunkantalee, redrobot, jaosorior, woodster_: For ACL stuff, I have added changes related to new user tables in first patch. There are 3 more acl related patches..will be helpful if it can be reviewed as well.13:50
openstackgerritAde Lee proposed openstack/barbican: Fix CA related exceptions, and unskip relevant tests  https://review.openstack.org/16631613:54
aleearunkant, ok13:55
aleewoodster_, jvrbanac , hockeynut , jaosorior,   https://review.openstack.org/166316  - made the small change requested.  please review and approve.13:57
*** paul_glass has joined #openstack-barbican14:03
*** atiwari has joined #openstack-barbican14:17
openstackgerritAde Lee proposed openstack/barbican: Fix some ca_id related bugs, add more functional test code  https://review.openstack.org/16683914:22
openstackgerritAde Lee proposed openstack/barbican: Changes to get Dogtag related functional tests working  https://review.openstack.org/16760414:22
*** zz_dimtruck is now known as dimtruck14:23
aleewoodster_, jvrbanac , hockeynut , jaosorior,  ^^ more reviews for cert api please.14:23
*** kfarr has joined #openstack-barbican14:48
*** atiwari has quit IRC14:51
*** xaeth_afk is now known as xaeth14:52
*** kebray has joined #openstack-barbican14:58
*** kebray has quit IRC15:07
*** openstackgerrit has quit IRC15:08
*** openstackgerrit has joined #openstack-barbican15:09
aleeredrobot, jvrbanac https://review.openstack.org/#/c/166316/5 just needs a workflow please15:31
aleejaosorior, hockeynut , jvrbanac - and after that one -- one of ya'll want to add workflow to https://review.openstack.org/#/c/166839/4  ? its got 3 +2s15:33
aleejaosorior, hockeynut , jvrbanac , kfarr thanks for reviews.15:33
hockeynutalee done15:33
jaosoriorhockeynut: you were too fast :P15:34
aleethanks ya'll -- although we might need to quickly workflow https://review.openstack.org/#/c/166316/515:34
hockeynutjaosorior <snicker>15:34
aleecoz the one that was workflowed depends on the former15:34
aleemaybe gerrit will be smart about it15:35
aleeredrobot, don't you owe me some workflows?15:35
*** kgriffs|afk is now known as kgriffs15:36
redrobotalee I do :(  ... Looking at 166316 right now15:36
jaosoriorarunkant: hopefully will start reviewing the ACL stuff today (after climbing)15:37
aleejaosorior, hockeynut , jvrbanac , redrobot , woodster_ don't forget https://review.openstack.org/#/c/167604/1 please15:38
jaosorioralee: can you add me as a reviewer there? I'll read that after climbing.15:40
aleejaosorior, will do thanks15:41
aleelooks like we have some workflows -- thanks redrobot15:41
aleeredrobot, don't stop now though - take a look at https://review.openstack.org/#/c/167604/1 :)15:41
arunkantredrobot, just confirming that changes which are getting merged are still part of kilo..correct? Earlier, I have rebased per secret patches with latest master.15:42
redrobotarunkant correct!  the master branch is still tracking Kilo, and will continue to track Kilo until RC1 is tagged.  At that point master will turn into Liberty and any release-critical bugs for Kilo will have to be fixed in a new branch off the RC1 tag.15:44
arunkantjaosorior, thanks looking forward to review comments15:45
arunkantredrobot, ok.15:45
*** dabukalam_ is now known as dabukalam15:52
*** dabukalam has quit IRC15:52
*** dabukalam has joined #openstack-barbican15:52
openstackgerritMerged openstack/barbican: Fix CA related exceptions, and unskip relevant tests  https://review.openstack.org/16631616:04
openstackgerritMerged openstack/barbican: Fix some ca_id related bugs, add more functional test code  https://review.openstack.org/16683916:04
*** xaeth is now known as xaeth_afk16:14
*** kebray has joined #openstack-barbican16:18
*** chlong has quit IRC16:22
*** kebray has quit IRC16:23
*** jkf has joined #openstack-barbican16:26
*** kebray has joined #openstack-barbican16:30
*** kebray has quit IRC16:31
*** kebray has joined #openstack-barbican16:31
*** chlong has joined #openstack-barbican16:33
*** chlong has quit IRC16:39
*** gyee has joined #openstack-barbican16:49
aleewoodster_, ping16:49
aleewoodster_, when you get a chance, lets discuss your retry/cert retry CRs.  I'm reviewing them and trying them out, but I think we need to co-ordinate and discuss.16:54
aleehey all - so if I'm looking at a series of CRs -- for example - https://review.openstack.org/#/c/167110/2  - how do I know the order in which to apply the updates?16:58
*** kebray has quit IRC17:05
*** darrenmoffat has quit IRC17:14
*** darrenmoffat has joined #openstack-barbican17:14
*** kebray has joined #openstack-barbican17:25
kfarralee, I usually look at the Dependencies section.  You can tell which one is the first because it will be listed as dependent on a patch that is (MERGED) or (None)17:28
aleekfarr, yeah - I remember siomething like that being around.17:31
aleekfarr, I dont see it anymore though17:32
aleekfarr, maybe my settings have changed17:32
kfarralee on mine, it's right below "Add Reviewer" button, above the list of all the patchsets17:34
kfarrI don't know why yours would be different :(17:35
aleekfarr, check your preferences -- you are using what is called "old screen" -- I was using "new screen"17:36
openstackgerritArun Kant proposed openstack/barbican: For per secret ACL support, adding db model changes (Part 1)  https://review.openstack.org/16433417:36
openstackgerritArun Kant proposed openstack/barbican: For per secret ACL support, adding db repository changes (Part 2)  https://review.openstack.org/16771217:37
aleekfarr, when I switch to "old screen" , I see the dependencies17:37
kfarralee there's a new screen?!?17:37
openstackgerritArun Kant proposed openstack/barbican: Adding Secret ACL controller layer changes (Part 3)  https://review.openstack.org/16433517:37
openstackgerritArun Kant proposed openstack/barbican: Adding Container ACL controller layer changes (Part 4)  https://review.openstack.org/16520517:37
openstackgerritArun Kant proposed openstack/barbican: Adding policy layer changes for ACL support (Part 5)  https://review.openstack.org/16520717:38
kfarrWell, this is different.17:38
arunkantFor ACL changes, I have to split patch 1 into 2 as it has gone bigger. So now there are total 5 dependent patches.17:39
*** kebray has quit IRC17:40
*** kebray has joined #openstack-barbican17:40
aleekfarr, looks like if you used "New screen" and set to disply changes in reverse order, then the related changes area lists them in order17:43
aleekfarr, first one in chain at the bottom17:43
kfarralee, yes, I came to that conclusion, too17:43
aleekfarr, yeah - wasn't obvious17:44
*** crc32 has joined #openstack-barbican18:03
*** xaeth_afk is now known as xaeth18:10
*** kfarr has quit IRC18:38
*** kfarr has joined #openstack-barbican18:43
*** tkelsey has quit IRC19:20
*** dimtruck is now known as zz_dimtruck19:21
*** alee is now known as alee_afk19:30
openstackgerritJohn Vrbanac proposed openstack/barbican: Switching how we handle sessions in p11_crypto  https://review.openstack.org/16774919:35
*** zz_dimtruck is now known as dimtruck19:36
woodster_alee, sorry, back for a bit between meetings19:50
*** tkelsey has joined #openstack-barbican20:09
jvrbanacreaperhulk, woodster_, redrobot, hockeynut, could y'all review and approve a fix to the p11 plugin? https://review.openstack.org/#/c/16774920:10
*** tkelsey has quit IRC20:13
*** xaeth is now known as xaeth_afk20:14
woodster_jvrbanac, I asked a question on the CR just now...20:20
*** alee has joined #openstack-barbican20:23
*** xaeth_afk is now known as xaeth20:24
jvrbanacwoodster_, replied20:27
jvrbanacwoodster_, TL;DR: not right now20:28
redrobotgreghaynes hi!  are you still going to try to land this https://review.openstack.org/#/c/140575/ for Kilo?20:30
*** gyee has quit IRC20:30
greghaynesredrobot: Hey. Sorry but I dont think I have time to mess with it (given recent HP changes)20:32
greghaynesI can WIP it or abandon or somesuch?20:33
redrobotgreghaynes WIP would work.20:33
greghaynesok, done. ty for the ping20:33
greghaynesill leave a note on the review too20:33
redrobotgreghaynes awesome, thanks!20:35
aleewoodster_, redrobot hockeynut thanks for review and workflow!20:37
redrobotalee 👍20:39
hockeynutwe aim to please :-)20:39
aleewoodster_, I just workflowed your first retry patch20:40
redrobothockeynut that saying always reminds me of a sign at this bar I used to go to in college.  It was in the restroom and it said "We aim to please, you aim too, please!"20:40
hockeynutredrobot absolutely - we had the same sign!20:41
openstackgerritMerged openstack/barbican: Changes to get Dogtag related functional tests working  https://review.openstack.org/16760420:51
openstackgerritMerged openstack/barbican: Switching how we handle sessions in p11_crypto  https://review.openstack.org/16774920:53
*** xaeth is now known as xaeth_afk20:57
openstackgerritMerged openstack/barbican: Add retry periodic task and worker-client logic  https://review.openstack.org/16494721:00
*** jaosorior has quit IRC21:02
openstackgerritCharles Neill proposed openstack/barbican: Security tests for Consumer resources  https://review.openstack.org/16701821:14
*** alee has quit IRC21:41
*** mdarby has joined #openstack-barbican21:43
*** mdarby has quit IRC21:44
*** xaeth_afk is now known as xaeth21:45
openstackgerritCharles Neill proposed openstack/barbican: Security tests for Container resources  https://review.openstack.org/16487521:46
*** everjeje has quit IRC21:56
openstackgerritChelsea Winfree proposed openstack/barbican: Adding more detail to the secrets quickstart guide  https://review.openstack.org/16702922:00
*** xaeth is now known as xaeth_afk22:01
*** jamielennox|away is now known as jamielennox22:08
*** crc32 has quit IRC22:11
arunkantkfarr, ping22:18
kfarrarunkant pong22:18
arunkantkfarr, question on review comment https://review.openstack.org/#/c/164334/1422:19
arunkantkfarr, creator_id is not something which is provided as part of the incoming request data. Its derived from token's user. So does it still need to be added in validator?22:20
kfarrOh, I guess not, then.  I misunderstood22:21
openstackgerritChelsea Winfree proposed openstack/barbican: Completing secret reference documentation  https://review.openstack.org/16782022:22
arunkantActually the related change is in other patches..so hard to know from this patch. I will clarify in commit comment about this. Thanks for your review.22:22
*** paul_glass has quit IRC22:35
*** kfarr has quit IRC22:37
*** dimtruck is now known as zz_dimtruck22:38
*** alee has joined #openstack-barbican22:41
*** jkf has quit IRC22:41
*** gyee has joined #openstack-barbican22:54
*** chlong has joined #openstack-barbican23:10
*** chlong has quit IRC23:37
*** chlong has joined #openstack-barbican23:37

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