Monday, 2014-09-08

« Sunday, 2014-09-07 Index Tuesday, 2014-09-09 »
*** juantwo_ has quit IRC00:24
*** juantwo has joined #openstack-barbican00:25
*** nkinder has joined #openstack-barbican02:32
*** nkinder has quit IRC02:59
*** bdpayne has joined #openstack-barbican03:13
*** bdpayne has quit IRC03:33
*** juantwo has quit IRC04:33
openstackgerritJamie Lennox proposed a change to openstack/kite: Explicitly import _ translation function  https://review.openstack.org/11969204:44
openstackgerritJamie Lennox proposed a change to openstack/kite: Update requirements from global  https://review.openstack.org/11969304:44
openstackgerritJamie Lennox proposed a change to openstack/kite: Update requirements from global  https://review.openstack.org/11969304:54
openstackgerritJamie Lennox proposed a change to openstack/kite: Explicitly import _ translation function  https://review.openstack.org/11969204:54
*** jenkins-keep has quit IRC05:10
*** jenkins-keep has joined #openstack-barbican05:11
openstackgerritJamie Lennox proposed a change to openstack/kite: Update requirements from global  https://review.openstack.org/11969305:18
jenkins-keepProject openstack-barbican-cloudcafe build #23: ABORTED in 50 min: http://jenkins.cloudkeep.io/job/openstack-barbican-cloudcafe/23/05:57
jenkins-keepProject openstack-barbican-cloudcafe build #24: STILL FAILING in 11 min: http://jenkins.cloudkeep.io/job/openstack-barbican-cloudcafe/24/06:09
*** Nirupama has joined #openstack-barbican06:13
*** jaosorior has joined #openstack-barbican07:09
*** rm_work has quit IRC10:15
*** rm_work|away has joined #openstack-barbican10:22
*** rm_work|away is now known as rm_work10:22
*** rm_work has joined #openstack-barbican10:22
*** Guest22704 has quit IRC11:12
*** juantwo has joined #openstack-barbican12:00
*** juantwo has quit IRC12:01
*** juantwo has joined #openstack-barbican12:02
*** ayoung has joined #openstack-barbican12:40
openstackgerritTim Kelsey proposed a change to openstack/barbican: Adding a plugin to interact with HP Atalla ESKM  https://review.openstack.org/11687812:44
*** Nirupama has quit IRC12:51
*** alee_out is now known as alee13:48
*** nkinder has joined #openstack-barbican14:00
*** woodster_ has joined #openstack-barbican14:19
*** atiwari has joined #openstack-barbican14:25
openstackgerritA change was merged to openstack/python-barbicanclient: Work toward Python 3.4 support and testing  https://review.openstack.org/11879814:27
openstackgerritA change was merged to openstack/python-kiteclient: Work toward Python 3.4 support and testing  https://review.openstack.org/11880314:31
*** paul_glass has joined #openstack-barbican14:33
*** Guest22704 has joined #openstack-barbican14:35
openstackgerritA change was merged to openstack/barbican: Move to oslotest  https://review.openstack.org/11670014:36
*** paul_glass has quit IRC14:36
*** paul_glass has joined #openstack-barbican14:37
*** paul_glass1 has joined #openstack-barbican15:06
*** paul_glass has quit IRC15:09
rm_workredrobot: https://review.openstack.org/#/c/113393/ ;P15:10
*** SheenaG1 has joined #openstack-barbican15:11
*** jorge_munoz has joined #openstack-barbican15:13
*** lisaclark has joined #openstack-barbican15:29
*** mikedillion has joined #openstack-barbican15:40
*** jaosorior has quit IRC15:52
*** bdpayne has joined #openstack-barbican15:57
*** mikedillion has quit IRC16:04
*** mikedillion has joined #openstack-barbican16:09
*** bdpayne has quit IRC16:11
*** bdpayne has joined #openstack-barbican16:11
*** bdpayne has quit IRC16:16
*** lisaclark has quit IRC16:21
*** lisaclark has joined #openstack-barbican16:25
*** kebray has joined #openstack-barbican16:26
*** mikedillion has quit IRC16:28
*** mikedillion has joined #openstack-barbican16:30
*** lisaclark has quit IRC16:49
*** lisaclark has joined #openstack-barbican16:56
*** atiwari has quit IRC17:00
*** Guest22704 has quit IRC17:04
*** jaosorior has joined #openstack-barbican17:18
*** Guest22704 has joined #openstack-barbican17:18
*** atiwari has joined #openstack-barbican17:19
*** kebray has quit IRC17:20
*** kebray has joined #openstack-barbican17:21
*** bdpayne has joined #openstack-barbican17:23
openstackgerritArvind Tiwari proposed a change to openstack/barbican: Add asymmtric order validator  https://review.openstack.org/11869717:28
atiwarihockeynut, yt?17:28
*** akoneru has joined #openstack-barbican17:39
*** ayoung has quit IRC17:41
*** Guest22704 has quit IRC17:49
*** bdpayne has quit IRC17:50
*** bdpayne has joined #openstack-barbican17:51
*** ayoung has joined #openstack-barbican17:52
*** kebray has joined #openstack-barbican18:06
*** bdpayne has quit IRC18:10
*** bdpayne has joined #openstack-barbican18:11
*** Guest22704 has joined #openstack-barbican18:13
*** Guest22704 has quit IRC18:18
*** lisaclark has quit IRC18:18
*** lisaclark has joined #openstack-barbican18:19
*** Guest22704 has joined #openstack-barbican18:34
*** akoneru is now known as akoneru_lunch18:43
*** mikedillion has quit IRC18:49
*** openstackgerrit has quit IRC18:51
*** jorge_munoz has quit IRC18:55
*** jorge_munoz has joined #openstack-barbican18:56
*** mikedillion has joined #openstack-barbican19:04
*** Guest22704 has quit IRC19:06
*** openstackgerrit has joined #openstack-barbican19:07
*** lisaclark has quit IRC19:14
openstackgerritArun Kant proposed a change to openstack/barbican: Adding keystone notification listener support  https://review.openstack.org/11081719:14
*** lisaclark has joined #openstack-barbican19:22
hockeynutatiwari I am now19:32
atiwarithanks hockeynut, I did not get you second comment in https://review.openstack.org/#/c/118697/6/barbican/plugin/interface/secret_store.py19:32
hockeynutoh - alg.lower() is called both on lines 205 and 20719:33
hockeynutbetter to just set a variable once and use it in both places19:33
atiwariok19:34
atiwarilet me fix that19:34
openstackgerritArvind Tiwari proposed a change to openstack/barbican: Add asymmtric order validator  https://review.openstack.org/11869719:38
atiwarihockeynut, ^19:39
hockeynut<click>19:39
rm_workhockeynut / jvrbanac / reaperhulk: https://review.openstack.org/#/c/113393/ :P19:40
rm_workuhh BTW how do I multi-line "from barbicanclient.barbican_cli.containers import CertificateContainerFormatter"19:42
*** lisaclark has quit IRC19:42
*** lisaclark1 has joined #openstack-barbican19:43
rm_workjaosorior: hey19:44
*** rellerreller has joined #openstack-barbican19:44
jaosoriorYo19:45
rm_worksee my comment like two lines up, or else on that __str__ CR19:45
rm_workthe reason I did the imports in ( ) format was because one of them NEEDS IT anyway19:45
rm_workso I thought I may as well just combine them while I was at it19:45
rm_workAlso, I think that is actually valid hacking/pep819:45
hockeynutredrobot I may not be in mtg this afternoon - working from orthodontist office and they may call me in to empty my wallet :-)19:46
*** Guest22704 has joined #openstack-barbican19:48
*** lisaclark1 has quit IRC19:48
*** Guest22704 has quit IRC19:51
rm_workjaosorior: i wouldn't have grouped the imports that way otherwise19:52
reaperhulkfrom barbicanclient.barbican_cli.containers import (\n\tCertificateContainerFormatter\n) basically rm_work19:57
redrobotweekly meeting starts in 2 minutes in #openstack-meeting-alt19:58
jaosoriorrm_work: I'll check it out19:58
jaosoriorEnvironment to run hacking and check what's up, not so easy on my phone :P19:59
rm_workreaperhulk: right, but at that point why not just do what I did and combine all three imports since it's going to be in parens anyway <_<19:59
rm_workjaosorior: heh well I mean, you can tell it is fine for hacking since jenkins +1'd it :)\19:59
rm_workreaperhulk: https://review.openstack.org/#/c/118494/3/barbicanclient/containers.py20:00
rm_work(could use more reviews there too :P)20:00
redrobotrm_work not hacking compliant20:00
redrobotrm_work hacking says to import modules, not classes20:00
jaosoriorStill gotta search for some openstack python style guide that I saw a while ago20:00
rm_workredrobot: then why does it pass hacking tox tests? >_>20:00
redrobotrm_work because hacking sucks20:00
rm_workI guess hacking just doesn't test it? lol20:00
reaperhulkrm_work: I think that's just because hacking is stupid.20:01
rm_workheh20:01
reaperhulkoh I see others have beaten me to the punch there20:01
*** akoneru_lunch is now known as akoneru20:01
rm_workalright so, I'm not allowed to import classes... ok20:01
jaosoriorredrobot: rm_work I remember reading that somewhere, but couldn't recall which wiki page20:01
redrobotrm_work http://docs.openstack.org/developer/hacking/#imports H30220:01
reaperhulkhacking doesn't actually look at the AST much, so it's pretty stupid about certain things20:01
rm_workso I just have to import from barbicanclient.barbican_cli.containers20:01
rm_workand then use containers.ContainerFormatter20:01
rm_work?20:01
jaosoriorYup20:01
rm_workfine by me20:02
rm_workimport barbicanclient.barbican_cli.containers as cli_containers20:02
rm_workseem ok?20:02
*** Guest22704 has joined #openstack-barbican20:02
jaosoriorFine by me20:02
rm_workkk20:03
rm_workfixing the same issue in the other classes and submitting a patch20:03
*** gyee has joined #openstack-barbican20:03
*** bubbva has joined #openstack-barbican20:04
*** kebray has quit IRC20:04
openstackgerritAdam Harwell proposed a change to openstack/python-barbicanclient: Change object __str__() to use pretty formatting  https://review.openstack.org/11849420:08
*** arunkant has quit IRC20:17
*** arunkant has joined #openstack-barbican20:18
*** bdpayne_ has joined #openstack-barbican20:33
*** bdpayne has quit IRC20:36
redrobotatiwari ping21:01
woodster_atiwari: this etherpad was an attempt to identify Juno work items: https://etherpad.openstack.org/p/barbican-juno-final-roadmap21:01
redrobotatiwari is this what you're looking for? https://launchpad.net/barbican/+milestone/juno-rc121:01
woodster_folks, please put you name in the right if you make mods to that etherpad21:01
atiwariredrobot, do you have list of CR/bug fix items for RC121:01
atiwariredrobot, yes21:02
atiwariredrobot, thanks21:02
*** kaitlin-farr has joined #openstack-barbican21:03
redrobotatiwari you're welcome21:03
arunkantwoodster_ : Can this be reviewed as well https://review.openstack.org/#/c/110817/ ..have been going through review for quite some time.21:04
atiwariredrobot, woodster_ https://review.openstack.org/#/c/118697/8 is also waiting for more blessing :)21:04
*** SheenaG1 has quit IRC21:05
atiwariredrobot, we need some direction on https://review.openstack.org/#/c/117032/ too. this is regarding tempest test coverage .21:08
*** juantwo has quit IRC21:11
openstackgerritConstanze Kratel proposed a change to openstack/barbican: Updated dev guide to include feedback from previous tech review  https://review.openstack.org/11788921:13
openstackgerritConstanze Kratel proposed a change to openstack/barbican: removing cq-devguide.wadl which was added by mistake  https://review.openstack.org/11993121:13
redrobotatiwari sorry, I'm not so familiar with Tempest.  IIRC tempest tests should live in the barbican repo?  At least that's where our current tempest tests are21:15
openstackgerritConstanze Kratel proposed a change to openstack/barbican: removed image files as they referred to internal architecure  https://review.openstack.org/11993221:16
*** Guest22704 has quit IRC21:16
atiwariredrobot, can some body add note on that CR, that will be helpful21:16
redrobotatiwari what is the CR for?  It's almost 900 LOC, so I'd have to spend some time to read it21:21
rm_workyeah, right now our tempest tests live in the functionaltests directory inside barbican21:26
atiwariredrobot, no rush, our QA has added barbican tests for tempest suite and blocked for now. This CR has enough context.21:26
atiwarirm_work, is that mean https://review.openstack.org/#/c/117032/ has to go under barbican review?21:27
rm_workatiwari: yes I believe so21:28
rm_workthough from what they are saying, we may need to change to having our tempest tests live *in tempest*??21:29
rm_workwhich I guess is how other projects do it?21:29
atiwariyes21:30
redrobotwhere's Chad when you need him?21:30
redrobotas far as I know we tried to add tests to Tempest21:30
redrobotbut they told us that we should keep them in our own repo21:30
rm_workyeah, so in our current model, that would be a Barbican review and the tests would live in barbican/functionaltests/api/v121:30
rm_workbut that's not actually valid long-term21:30
rm_workredrobot: which Chad?21:31
redrobotrm_work Chad Lung... no longer a Racker, he was the one who set up our tempest/devstack stuff21:31
atiwariredrobot, I think we eventually have to move to tempest if we want to go for Graduation to integrated21:31
atiwariredrobot, https://github.com/openstack/governance/blob/master/reference/incubation-integration-requirements.rst21:31
atiwariredrobot, btw what is the plan around Graduation to integrated?21:32
redrobotatiwari we were up for review a couple of weeks ago for the TC meeting21:32
redrobotatiwari TC decided we were not ready to graduate for this cycle21:33
atiwari:(21:33
redrobotthey want to see us integrate with at least one project next cycle21:33
redrobotas far as Tempest is concerned, we need to talk to their team further.  My understanding is that hosting your own tempest tests was a new feature of Tempest, and so the test we have currently in the barbican source meet the requirement of having tempest tests.21:35
*** kebray has joined #openstack-barbican21:35
redrobotwhy is Meera trying to add them to Tempest?21:35
*** bdpayne_ has quit IRC21:35
*** bdpayne has joined #openstack-barbican21:37
rm_workredrobot: well, LBaaS will be integrating with YOU, does that count?21:37
rm_work:P21:37
redrobotrm_work possibly, but since that code hadn't landed when we went up for Graduation review, it didn't count21:37
rm_workright21:37
redrobotSwift is talking about integrating with us for encrypted-swift :)21:38
rm_worknice :)21:38
atiwariredrobot, we thought tempest is the place for all tempest tests.21:38
atiwariI will check with her and ask to add them in barbican.21:39
redrobotatiwari thanks!21:39
atiwariredrobot, rm_work sounds OK?21:39
*** SheenaG1 has joined #openstack-barbican21:39
rm_workatiwari: i'd wait for us to get some clarification...21:39
rm_worksince apparently there are two different messages circulating21:39
redrobotyeah, we definitely need to get some clarification from the Tempest team21:40
atiwarirm_work, redrobot than we will wait for a signal from you guys :)21:40
redrobotatiwari you're more than welcome to engage the Tempes team yourself21:41
atiwariredrobot, as per Matthew's comment in CR, he is looking something from barbican team21:42
redrobotatiwari where do you see that?21:44
redrobotatiwari I'm reading both of Matthew's comments21:44
redrobothis points are that21:44
atiwariredrobot,  "So I'm -2 here because we need to have a devstack gate job added that actually runs these tests before it can land."21:44
atiwariSo the requirement for tempest tests is a graduation requirement. If barbican plans to become an integrated project it needs to have tempest tests before it can do that21:45
atiwariI am not sure who is going to add tempest21:45
atiwaritests21:45
*** mikedillion has quit IRC21:46
atiwarihttps://review.openstack.org/#/c/117032/ cr is an attempt to add it21:46
redrobotok, it sounds to me like you're confused, so let me see if I can clear up the confusion21:48
redrobotMeera wrote tests for the Tempest tree that test barbican21:48
redrobotMatthew rejected the patch because Tempest does not have a dsvm gate for Barbican21:48
redrobotMeera did not understand Matthew's rejection, and told Matthew that we already have a gate in Barbican that runs Tempest tests from our own repo21:49
redrobotMatthew apparently does not know that projects are running tempests tests out of their own repos.21:50
redrobotMatthew apparently thinks that having Tempest tests in the project's repo would not work.  He's wrong, since our devstack gate does run the Tempest tests21:50
redrobotdoes that seem like an accuarate summary of what's happening?21:51
redrobotthe integration requirement does specify that we need some Tempest tets, but it does not say whether they need to be in Tempest proper, or be part of Barbican as they are now21:52
redrobotIF the TC does indeed mean that they have to be in Tempest proper, then someone (anyone) could add the extra gate they want to unblock the patch.21:52
redrobotthis sounds like something either you atiwari  or Meera can do21:52
atiwariredrobot, thanks. let me see it one more time21:56
redrobotatiwari Matthew says that he does not think that Tempest tests can be run from our own repo, but he's clearly wrong because we're doing exactly that. https://github.com/openstack/barbican/blob/master/functionaltests/pre_test_hook.sh21:57
*** SheenaG1 has quit IRC21:57
atiwariredrobot, ok21:57
redrobotnow that I think about it, I think the best course of action is to just have Meera send a patch with all those tests to the Barbican repo21:58
redrobotonce we start talking about Graduation again next cycle we can ask the TC if those tests need to be moved to Tempes proper.21:59
atiwariredrobot, make sense21:59
atiwariI will ask her to do so21:59
atiwarithanks for your time on this21:59
*** rellerreller has quit IRC22:00
*** juantwo has joined #openstack-barbican22:01
rm_workyep that makes sense22:06
rm_workalso, woodster_ / redrobot: http://i.imgur.com/zVq3Iut.png22:06
rm_workThat is our proposed workflow22:06
redrobotrm_work what happened to the shadow-copy?22:07
redrobotrm_work and step 5 better include that consumer registration, or I'm going to shank somebody22:07
rm_workredrobot: shadow copy happens after the "driver for processing"22:08
rm_workthe continuation is: 7. Driver sends data to the LoadBalancer Appliance that was chosen by the system (in our case, Octavia); 8. Octavia receives the data and stores the certificate info in Barbican using the Octavia Service-Account22:09
rm_workbut anything after step #6 is up to the specific implementation, for instance F5 won't store the cert in Barbican, it'll store it internally on the F5 Hardware22:09
*** jorge_munoz has quit IRC22:24
rm_workredrobot: so basically, the result of that meeting is that I should not worry about getting reviews on the client CRs I have up for the next week or two? >_>22:30
rm_workredrobot: I guess that's fine, but it does mess with my sprint a ton, wish I'd known that a little earlier :P22:31
*** paul_glass1 has quit IRC22:32
redrobotrm_work my bad...  I22:43
redrobotrm_work I'll start reviewing that stuff at home today22:43
redrobotbasically gonna do a bunch of after-hours work just for you. :)22:43
woodster_well, I'd like to land the Containers one ASAP, since that refactors other things in there too, and folks trying to use the client now will not be happy campers :(22:44
woodster_...at is is just a workflow+1 away22:44
*** kebray has quit IRC22:48
*** kebray has joined #openstack-barbican22:48
*** jaosorior has quit IRC22:52
rm_workredrobot: you'd do that? for me?! <322:54
redrobotrm_work I wouldn't say I'm doing it FOR you.... more like doing it so you'll stop pestering me. :-P22:55
rm_workeh, tomato tomato (that expression doesn't work well in text chat...)22:59
redrobottomato tomáto?23:00
*** atiwari has quit IRC23:05
rm_work:P23:06
*** nkinder has quit IRC23:42
*** bdpayne has quit IRC23:44
*** bdpayne has joined #openstack-barbican23:45
« Sunday, 2014-09-07 Index Tuesday, 2014-09-09 »

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