| *** bdpayne has joined #openstack-barbican | 00:12 | |
| *** bdpayne has quit IRC | 00:23 | |
| *** xianghui has quit IRC | 00:54 | |
| *** xianghui has joined #openstack-barbican | 00:58 | |
| *** xianghui has quit IRC | 01:07 | |
| *** xianghui has joined #openstack-barbican | 01:08 | |
| *** bdpayne has joined #openstack-barbican | 01:10 | |
| *** ayoung has quit IRC | 01:17 | |
| *** bdpayne has quit IRC | 01:22 | |
| openstackgerrit | lifeless proposed a change to openstack/python-barbicanclient: Remove hardcoded version. https://review.openstack.org/115461 | 01:53 |
|---|---|---|
| *** alee_on_way_home is now known as alee | 02:03 | |
| *** xianghui has quit IRC | 02:07 | |
| *** xianghui has joined #openstack-barbican | 02:08 | |
| *** woodster_ has quit IRC | 02:25 | |
| *** juantwo_ has joined #openstack-barbican | 02:51 | |
| *** juantwo has quit IRC | 02:55 | |
| *** xianghui has quit IRC | 03:09 | |
| *** xianghui has joined #openstack-barbican | 03:10 | |
| *** xianghui has quit IRC | 03:16 | |
| *** xianghui has joined #openstack-barbican | 03:17 | |
| *** kebray has joined #openstack-barbican | 03:21 | |
| *** kebray has quit IRC | 03:21 | |
| *** kebray has joined #openstack-barbican | 03:22 | |
| *** kebray has quit IRC | 03:45 | |
| *** kebray_ has joined #openstack-barbican | 03:45 | |
| *** kebray_ has quit IRC | 03:55 | |
| *** kebray has joined #openstack-barbican | 03:56 | |
| *** bdpayne has joined #openstack-barbican | 04:09 | |
| *** xianghui has quit IRC | 04:17 | |
| *** xianghui has joined #openstack-barbican | 04:17 | |
| *** juantwo_ has quit IRC | 04:50 | |
| *** bdpayne has quit IRC | 05:18 | |
| *** rm_you| has joined #openstack-barbican | 05:25 | |
| *** rm_you has quit IRC | 05:29 | |
| *** bdpayne has joined #openstack-barbican | 05:36 | |
| *** bdpayne has quit IRC | 06:07 | |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/barbican: Imported Translations from Transifex https://review.openstack.org/115510 | 06:10 |
| *** kebray has quit IRC | 07:24 | |
| *** tkelsey has joined #openstack-barbican | 09:50 | |
| *** openstackgerrit has quit IRC | 10:10 | |
| *** tkelsey has quit IRC | 10:59 | |
| *** SheenaG1 has joined #openstack-barbican | 11:06 | |
| *** SheenaG11 has joined #openstack-barbican | 11:10 | |
| *** SheenaG1 has quit IRC | 11:12 | |
| *** tkelsey has joined #openstack-barbican | 11:15 | |
| *** juantwo has joined #openstack-barbican | 11:45 | |
| *** juantwo has quit IRC | 11:47 | |
| *** juantwo has joined #openstack-barbican | 11:47 | |
| *** nkinder has quit IRC | 13:11 | |
| *** woodster_ has joined #openstack-barbican | 13:35 | |
| *** nkinder has joined #openstack-barbican | 14:10 | |
| *** SheenaG11 has quit IRC | 14:25 | |
| *** akoneru has joined #openstack-barbican | 14:36 | |
| *** paul_glass has joined #openstack-barbican | 14:37 | |
| alee | woodster_, ping | 14:54 |
| woodster_ | alee: hello | 14:54 |
| alee | woodster_, hey - I just uploaded a new cr for the dogtag plugin | 14:54 |
| woodster_ | alee: nice! | 14:55 |
| alee | should be good to review. | 14:55 |
| alee | interestingly the jenkins jobs did not kick off | 14:55 |
| alee | and there was nothing that showed up on this irc. | 14:55 |
| woodster_ | alee: I'm looking at initial hooking into the cert stuff from the order side...seeing issues | 14:55 |
| woodster_ | alee: zuul is angry | 14:55 |
| woodster_ | ...maybe | 14:56 |
| alee | there appears to be a "toggle ci" button on the screen -- am I supposed to click that? | 14:56 |
| woodster_ | I think that just changes your view...that's how you can get the inlined jenkins status to show up | 14:56 |
| alee | well it didn;t seem to do anything | 14:57 |
| alee | yeah | 14:57 |
| woodster_ | I don't think that triggers any zuul activity, but could be wrong | 14:57 |
| alee | no it doesn't appear to be -- but zuul appears to be on holiday | 14:57 |
| alee | woodster_, anyways - we should chat some more about the orders -> certs patch | 14:59 |
| woodster_ | alee: I'm just trying to get a cert type order to flow thru to the new cert logic now...I figure the first patch could just do that. It would be enough to get your code looped in I think | 15:00 |
| alee | woodster_, yup - something basic I can take a look at | 15:01 |
| alee | woodster_, looks like zuul is back | 15:01 |
| *** SheenaG1 has joined #openstack-barbican | 15:01 | |
| alee | but for some reason devstack gte not happy -- looking | 15:02 |
| *** ayoung has joined #openstack-barbican | 15:04 | |
| *** akoneru is now known as akoneru_brb | 15:04 | |
| alee | woodster_, is the correct entry "recheck no bug" ? | 15:06 |
| alee | woodster_, one of the things jamielennox said to me yesterday resonated with me -- which is that"having to know the kind of ca that is on the backend diminishes the power of the abstraction" | 15:08 |
| alee | why not just go directly to dogtag or symantec or whatever .. | 15:08 |
| woodster_ | alee: yes to 'recheck no bug' | 15:08 |
| woodster_ | hmmm...abstract vs direct to dogtag/symantec/etc | 15:09 |
| alee | so I do think we should resurrect the idea of creating a common cert api -- for the most common tasks -- like getting a SSL server cert -- and look to supporting that in K | 15:09 |
| woodster_ | they seem at odds :) | 15:09 |
| alee | we can still keep the ability to get at more complicated cert requests etc. through vendor specific fields | 15:10 |
| alee | but I think there is value in presenting an interface where I can just ask barbican for a server cert | 15:10 |
| alee | given a csr for instance | 15:11 |
| rm_work | +1 alee | 15:12 |
| alee | woodster_, I dont think we have time to design that interface now - although I think that the one blueprint is a start. | 15:12 |
| rm_work | degrading your abstraction by expecting a bunch of vendor specific fields in any request is really… :/ | 15:12 |
| *** openstackgerrit has joined #openstack-barbican | 15:12 | |
| alee | https://review.openstack.org/108429 | 15:12 |
| rm_work | (speaking from dealing with the same issue in Neutron-LBaaS) | 15:13 |
| alee | I think that blueprint requires something like profiles for spcific use cases | 15:13 |
| alee | woodster_, anyways I think we should push forward with allowing vendor specific fields so that we can show that we can issue certs for J | 15:14 |
| alee | but that we should do so in a way that would allow us to add a common interface in K | 15:14 |
| alee | woodster_, we might be able to do something like this .. for instance | 15:15 |
| alee | in the order metadata -- if the request is vendor specific, then have a dict labeled "dogtag" or "symantec" or .. | 15:16 |
| alee | eventually we could end up adding a dict called "barbican" or "common" | 15:16 |
| alee | for common-api specific request fields | 15:17 |
| alee | does that make sense? | 15:17 |
| alee | the advantage of doing it that way also is that you might be able to make a common request and also send in any supplemental required vendor specific request params too. | 15:19 |
| alee | woodster_, still there? | 15:21 |
| woodster_ | alee: yep, just in impromptu meetings right now :) | 15:21 |
| rm_work | need more eyes on https://review.openstack.org/#/c/114677/ | 15:24 |
| rm_work | need to get the BP through before the CR :P | 15:25 |
| *** bdpayne has joined #openstack-barbican | 15:25 | |
| redrobot | rm_work that BP looks familiar | 15:25 |
| rm_work | redrobot: :P | 15:25 |
| rm_work | it just might | 15:25 |
| rm_work | redrobot: though this might be an opportunity for you to look at https://review.openstack.org/#/c/115080/ and actually adjust the BP if you decide you want to see anything else | 15:26 |
| redrobot | rm_work RE: loonch, I forgot I'm senseing a coding dojo today. Might be able to go after 1pm | 15:26 |
| woodster_ | alee: That sounds like a good discussion for K for sure. I'd personally favor common key/values, that includes an optional 'vendor' sort of key/value pair. The key/values that come after that are specific to a vendor. So keep the overall meta dict simple and flat perhaps? | 15:26 |
| rm_work | redrobot: lol k that could still work for me | 15:26 |
| rm_work | redrobot: oh which one is that? I was thinking about going to the python one | 15:27 |
| redrobot | rm_work we may still have seats for the dojo if you're interested. | 15:27 |
| rm_work | ha | 15:27 |
| rm_work | yes | 15:27 |
| rm_work | how many seats? I think some others from my team might want to go as well... | 15:27 |
| rm_work | just a hunch | 15:27 |
| redrobot | rm_work it'a TDD dojo. Probably doing the Bowling Kata today. | 15:27 |
| rm_work | hmm | 15:27 |
| rm_work | this is all foreign to me | 15:28 |
| rm_work | i mean, I know TDD | 15:28 |
| rm_work | but the Dojo thing | 15:28 |
| redrobot | no idea, I don't organize it, I just show up and help with the tech side of it | 15:28 |
| rm_work | heh | 15:28 |
| alee | woodster_, so something like dogtag-csr - for instance? | 15:28 |
| rm_work | Bowling problem? | 15:28 |
| rm_work | should I know what that is? :/ | 15:28 |
| redrobot | rm_work everyone takes turns on the keyboard. First person writes a test, next person makes it pass and writes one of their own, rinse, repeat | 15:28 |
| rm_work | ah | 15:29 |
| rm_work | lol | 15:29 |
| rm_work | like… just ANY test? :P | 15:29 |
| redrobot | no, the sensei picks a problem to solve | 15:29 |
| alee | woodster_, I'm not sure we want to be in the business of adding/stripping off vendor prefixes .. | 15:29 |
| rm_work | ah k | 15:29 |
| redrobot | today we'll be writing a class to score a bowling game | 15:29 |
| rm_work | gotcha | 15:29 |
| alee | woodster_, maybe keep the common stuff in the meta as flat and just have a subdict for vendor specific stuff. | 15:30 |
| redrobot | rm_work not sure how useful it would be to you, but people new to Python seem to really enjoy it. | 15:30 |
| alee | woodster_, so symantec would be under order_meta{ symantec { ...}} | 15:31 |
| rm_work | yeah I might spend the time getting the containers CR rebased and updated to the new model :) | 15:31 |
| woodster_ | alee: I think that's more detail type stuff....I think a blueprint that considers segregating CSR-generic fields from optional vendor specific one makes sense, how it is done can be considered later for sure | 15:31 |
| alee | woodster_, right now for J, we can only support vert generation using vendor specific fields. | 15:33 |
| woodster_ | alee: that's correct, good starting point | 15:33 |
| alee | because we dont really have the time to define and put in the common api. If we put the vendor specific fields within a subdict from the start, then there will be no migration required for clients that start to implement using J | 15:34 |
| rm_work | thanks for the +2 jvrbanac | 15:36 |
| jvrbanac | rm_work, np | 15:36 |
| rm_work | jvrbanac: oh, and i did figure out how to kinda do what you were saying on the CR with moving some of the if/else logic down to streamline it | 15:36 |
| rm_work | just took me a bit :P | 15:36 |
| alee | woodster_, I'm mentioning this now of course because you're writing the patch that links up orders to cert plugins. I think we need to support two differentiators -- a vendor_type (like dogtag, symantec) and an SKI. | 15:39 |
| *** paul_glass has quit IRC | 15:39 | |
| rm_work | BTW, how does string translation work for our project? | 15:40 |
| rm_work | I don't actually see any _() or similar like I am used to for translation handling | 15:40 |
| alee | woodster_, yay - jenkins just approved my cr .. please take a look when you get a chance. | 15:41 |
| woodster_ | alee: I will take a look | 15:43 |
| woodster_ | rm_work: we should move to the localization stuff for the client, but not done yet....future CR | 15:43 |
| rm_work | <_< k | 15:43 |
| rm_work | woodster_: surprised I got a +2 from you that quickly on the client CR :P | 15:44 |
| rm_work | I guess redrobot / reaperhulk still have time to -1/-2 me | 15:45 |
| woodster_ | rm_work: oh sorry, didn't mean to break review form...I'll take it off until it ripens for another couple of days | 15:45 |
| rm_work | heh | 15:45 |
| woodster_ | rm_work: you did help out with my git review issue yesterday though...so I'll just keep it there, thanks! | 15:45 |
| rm_work | heh thanks ;) | 15:46 |
| rm_work | though I have to assume there's going to be at least one more patchset for some minor nitpicks if nothing else <_< | 15:46 |
| *** gregsharek1 has joined #openstack-barbican | 15:48 | |
| *** akoneru_brb is now known as akoneru | 15:53 | |
| alee | woodster_, ping. question on your certificate events CR .. | 15:58 |
| *** gregsharek1 has left #openstack-barbican | 15:58 | |
| woodster_ | alee: howdy | 15:58 |
| alee | woodster_, can you tell me what get_plugin() is supposed to return for the CertificateEventPluginManager ? | 15:59 |
| woodster_ | it just gets the plugin configured for certs...by default just the logging event plugin | 15:59 |
| alee | the first plugin or all of them? | 16:00 |
| woodster_ | alee: just the first plugin. Realistically there would only be one in a deployment I'd think. So maybe a CADF/Ceilometer one for openstack deploys, or a ticketing interface one for enterprises | 16:00 |
| *** juantwo has quit IRC | 16:01 | |
| *** juantwo has joined #openstack-barbican | 16:01 | |
| alee | woodster_, well I would think you would want to invoke all event plugins .. | 16:02 |
| alee | woodster_, anyways I'll comment in the CR -- just seeking clarification as to your intent | 16:03 |
| *** kebray has joined #openstack-barbican | 16:04 | |
| jvrbanac | woodster_, alee, reaperhulk, rellerreller, would we get a workflow on: https://review.openstack.org/#/c/115510/ ? | 16:04 |
| *** juantwo has quit IRC | 16:05 | |
| *** juantwo has joined #openstack-barbican | 16:06 | |
| *** juantwo has quit IRC | 16:07 | |
| *** juantwo has joined #openstack-barbican | 16:08 | |
| *** juantwo has quit IRC | 16:09 | |
| *** juantwo has joined #openstack-barbican | 16:10 | |
| *** juantwo has quit IRC | 16:10 | |
| *** bdpayne has quit IRC | 16:10 | |
| *** juantwo has joined #openstack-barbican | 16:11 | |
| woodster_ | jvrbanac: just workflow+1-ed it | 16:15 |
| jvrbanac | woodster_, thanks! | 16:15 |
| woodster_ | alee: are you there? If so, check out a question I sent to you please.... | 16:15 |
| rm_work | so redrobot I guess I'm down for Erik's 1pm if you are :P | 16:17 |
| arunkant | woodster_ and others, looking for review comments on https://review.openstack.org/#/c/110817/ . | 16:17 |
| rm_work | chellygel: I guess I can get your tacos if you are down to wait till 1pm, lol | 16:17 |
| chellygel | yeah doug was saying 1, if i'm done, i'll go! just ping me when yall are ready to leave | 16:18 |
| openstackgerrit | A change was merged to openstack/barbican: Imported Translations from Transifex https://review.openstack.org/115510 | 16:35 |
| *** tkelsey has quit IRC | 16:45 | |
| *** paul_glass has joined #openstack-barbican | 16:46 | |
| *** bdpayne has joined #openstack-barbican | 16:47 | |
| *** paul_glass1 has joined #openstack-barbican | 16:49 | |
| *** paul_glass has quit IRC | 16:52 | |
| openstackgerrit | Paul Kehrer proposed a change to openstack/barbican: change CryptoPluginManager to be instantiated in the module scope https://review.openstack.org/114341 | 16:53 |
| *** paul_glass1 is now known as paul_glass | 16:58 | |
| *** atiwari has joined #openstack-barbican | 17:07 | |
| openstackgerrit | A change was merged to openstack/barbican-specs: blueprint for adding HP Atalla ESKM crypto plugin. https://review.openstack.org/114580 | 17:08 |
| *** akoneru is now known as akoneru_lunch | 17:10 | |
| openstackgerrit | Arvind Tiwari proposed a change to openstack/barbican: fix for bug #1359197 https://review.openstack.org/115713 | 17:16 |
| openstackgerrit | Arvind Tiwari proposed a change to openstack/barbican: fix for bug #1359197 https://review.openstack.org/115713 | 17:20 |
| openstackgerrit | John Wood proposed a change to openstack/barbican: Initial connect orders resource to certificate processing https://review.openstack.org/115715 | 17:26 |
| reaperhulk | woodster_: https://review.openstack.org/#/c/114341/ is ready for review (and passes tests). Thanks for catching that import mistake :p I must have looked at that 50 times and never noticed | 17:35 |
| *** nkinder has quit IRC | 17:35 | |
| openstackgerrit | Arvind Tiwari proposed a change to openstack/barbican: fix for bug #1359197 https://review.openstack.org/115713 | 17:41 |
| woodster_ | reaperhulk: I ended up seeing the issue with nosetests...tox hid away the root cause exception. Interestingly I didn't see an issue running the server directly. At any rate, I think that is the only issue with that CR if you want to retry it. | 17:42 |
| reaperhulk | yeah I fixed it and it passed all the tests this time | 17:42 |
| reaperhulk | that's why I said it's ready for review :) | 17:42 |
| *** nkinder has joined #openstack-barbican | 17:48 | |
| atiwari | redrobot, can someone look in to http://logs.openstack.org/45/115345/3/check/gate-barbican-python26/04633a6/console.html | 17:50 |
| atiwari | not sure what is going on | 17:50 |
| atiwari | redrobot, nevermind | 17:50 |
| SheenaG1 | Hey woodster_ if you get a chance, I just need workflow now | 17:54 |
| woodster_ | SheenaG1: done! | 17:57 |
| atiwari | hockeynut, are you there? | 17:59 |
| *** bdpayne has quit IRC | 17:59 | |
| *** bdpayne has joined #openstack-barbican | 18:02 | |
| SheenaG1 | YAY ATC ATC ATC | 18:02 |
| SheenaG1 | atiwari: looks like he's afk, I think he went to lunch with jvrbanac | 18:02 |
| rm_work | hopefully redrobot is out of his thing soon and loonching can happen | 18:02 |
| atiwari | SheenaG1, thanks I will check back | 18:02 |
| redrobot | rm_work LOOOOONCH! | 18:07 |
| rm_work | allons-y! | 18:07 |
| SheenaG1 | Hey rm_work I need to talk to you when you get back | 18:10 |
| SheenaG1 | re: timelines | 18:10 |
| openstackgerrit | A change was merged to openstack/barbican: Edited docs to improve context https://review.openstack.org/114665 | 18:12 |
| *** akoneru_lunch is now known as akoneru | 18:13 | |
| *** paul_glass1 has joined #openstack-barbican | 18:21 | |
| *** paul_glass has quit IRC | 18:24 | |
| *** samuelbercovici has joined #openstack-barbican | 18:31 | |
| hockeynut | hi atiwari | 18:37 |
| hockeynut | apologies but this week has been (and continues to be) meeting hell! | 18:39 |
| *** paul_glass1 has quit IRC | 18:40 | |
| atiwari | hockeynut, np | 18:54 |
| hockeynut | whats up? | 18:54 |
| atiwari | hockeynut, I am trying to understand you comment on https://review.openstack.org/#/c/111412/8 | 18:54 |
| atiwari | regarding failing the devstack | 18:55 |
| atiwari | https://review.openstack.org/#/c/111412/8/barbican/plugin/resources.py | 18:56 |
| hockeynut | ah yes - I looked at the output of the devstack job to see what was failing | 18:56 |
| atiwari | hockeynut, I am trying to see where to fix | 18:57 |
| hockeynut | peek at http://logs.openstack.org/12/111412/9/check/gate-barbican-devstack-dsvm/18181a9/logs/screen-barbican.txt.gz | 18:58 |
| atiwari | ok | 18:58 |
| atiwari | let me see | 18:58 |
| atiwari | thanks | 18:58 |
| hockeynut | apologies, its 2pm which means time for another meeting :-( Back in a few (if we survive) | 18:58 |
| *** paul_glass has joined #openstack-barbican | 19:10 | |
| *** lifeless has joined #openstack-barbican | 19:11 | |
| lifeless | hi | 19:11 |
| lifeless | any cores around ? | 19:12 |
| lifeless | I'd really love to get https://review.openstack.org/#/c/115461/ landed to unblock the pbr semver blueprint | 19:12 |
| reaperhulk | +2 from me | 19:15 |
| lifeless | thank you! | 19:28 |
| rm_work | SheenaG1: hey back | 19:37 |
| rm_work | SheenaG1: timelines are shot | 19:37 |
| redrobot | lifeless on the way to merge | 19:39 |
| jvrbanac | lifeless, did something change with PBR? | 19:39 |
| openstackgerrit | A change was merged to openstack/python-barbicanclient: Remove hardcoded version. https://review.openstack.org/115461 | 19:41 |
| reaperhulk | https://review.openstack.org/#/c/114341/ is ready for review now too redrobot, jvrbanac ;) | 19:42 |
| jvrbanac | reaperhulk, taking a look now | 19:42 |
| redrobot | reaperhulk is there a Bug # for that change? | 19:42 |
| reaperhulk | uuuuhhh | 19:42 |
| reaperhulk | only if woodster_ filed one | 19:42 |
| reaperhulk | launchpad scares me | 19:42 |
| redrobot | haha | 19:42 |
| reaperhulk | i think he might have | 19:43 |
| redrobot | reaperhulk yeah https://bugs.launchpad.net/barbican/+bug/1358386, | 19:43 |
| redrobot | reaperhulk would you mind adding Closes-Bug: 1358386 to commimt message? | 19:43 |
| reaperhulk | yeah one moment | 19:43 |
| openstackgerrit | Paul Kehrer proposed a change to openstack/barbican: change CryptoPluginManager to be instantiated in the module scope https://review.openstack.org/114341 | 19:44 |
| reaperhulk | done | 19:44 |
| redrobot | as lame as launchpad is, I have to admint the Gerrit->Launchpad bug integration is pretty neat | 19:44 |
| reaperhulk | I'd rather have github style :) | 19:46 |
| jvrbanac | reaperhulk, +2 and _1 | 19:49 |
| jvrbanac | ^+1 | 19:49 |
| reaperhulk | cool, closing that out will let our dev environment be a fully operational battleship too | 19:49 |
| jvrbanac | reaperhulk, awesoem | 19:50 |
| jvrbanac | ^awesome | 19:50 |
| jvrbanac | Geez! I can't type today | 19:50 |
| reaperhulk | typing is hard | 19:50 |
| jvrbanac | reaperhulk, I type much better on a beach :D | 19:50 |
| reaperhulk | well why aren't you here then | 19:50 |
| jvrbanac | reaperhulk, good question | 19:51 |
| jvrbanac | arunkant, I added a few comments to your CR | 19:52 |
| arunkant | thanks jvrbanac | 19:52 |
| *** kebray has quit IRC | 19:58 | |
| *** kebray has joined #openstack-barbican | 20:04 | |
| *** kebray has quit IRC | 20:06 | |
| redrobot | woodster_ finally got annoyed with the Jenkins failed jobs... I'm packaging all the stuff we need | 20:09 |
| *** kebray has joined #openstack-barbican | 20:09 | |
| woodster_ | you mean the rdo stuff too?? | 20:10 |
| woodster_ | redrobot: I think the only issue is that the upload script isn't sending the keystonemiddleware rpm up to yum repo | 20:11 |
| redrobot | woodster_ crap, this sucks... the keystonemiddleware lib depends on an unreleased oslo.confg -___ | 20:11 |
| redrobot | woodster_ I uploaded python-keystonemiddleware, but now its dependencies are needed | 20:12 |
| woodster_ | redrobot: arg! Ok, that sucks | 20:12 |
| redrobot | ugh, I see the version we need in PyPI, but fpm is refusing to pull it down :( | 20:14 |
| redrobot | woodster_ this is a mess... thinking about just doing this instead http://2.bp.blogspot.com/-UUBBVvfBUcM/T1t8Zi673mI/AAAAAAAAA0c/uEdZ-PA4IpI/s1600/homer-simpson-bush-gif.gif | 20:15 |
| openstackgerrit | Arvind Tiwari proposed a change to openstack/barbican: Reorganize code to use store crypto plug-in https://review.openstack.org/111412 | 20:15 |
| atiwari | hockeynut, I have fixed the issue added more unit tests, so that in future it won't happen again. | 20:16 |
| woodster_ | redrobot: ha, let me know if that works out for ya! | 20:16 |
| redrobot | woodster_ yeah.... fpm is not wanting to play nice... going to need some serious work to get that pipeline back into shape. :( | 20:25 |
| woodster_ | redrobot: I was hoping that was the easy part...shouldn't have known better! | 20:25 |
| woodster_ | redrobot: so ditch it for the docker jenkins stuff? :) | 20:26 |
| redrobot | woodster_ that's what I'm thinking... it's just such a time sink to try to get unpackaged stuff right. | 20:26 |
| woodster_ | redrobot: I agree for sure | 20:27 |
| *** paul_glass has quit IRC | 20:27 | |
| *** jamielennox is now known as jamielennox|away | 20:33 | |
| *** SheenaG1 has quit IRC | 20:34 | |
| openstackgerrit | A change was merged to openstack/barbican: change CryptoPluginManager to be instantiated in the module scope https://review.openstack.org/114341 | 20:35 |
| *** SheenaG1 has joined #openstack-barbican | 20:40 | |
| *** juantwo has quit IRC | 20:41 | |
| *** kebray has quit IRC | 20:44 | |
| *** kebray has joined #openstack-barbican | 20:45 | |
| *** gyee_ has joined #openstack-barbican | 20:50 | |
| *** denis_makogon has quit IRC | 20:52 | |
| *** dmakogon_ has joined #openstack-barbican | 20:53 | |
| SheenaG1 | rm_work: ping | 21:04 |
| *** kebray has quit IRC | 21:05 | |
| *** bdpayne has quit IRC | 21:10 | |
| *** atiwari has quit IRC | 21:15 | |
| *** bdpayne has joined #openstack-barbican | 21:15 | |
| openstackgerrit | Arvind Tiwari proposed a change to openstack/barbican: Reorganize code to use store crypto plug-in https://review.openstack.org/111412 | 21:22 |
| *** atiwari has joined #openstack-barbican | 21:23 | |
| rm_work | SheenaG1: pong | 21:43 |
| rm_work | SheenaG1: SYN | 21:43 |
| hockeynut | atiwari awesome, thanks! | 21:59 |
| rm_work | whelp, I think I found an even cleaner way of instantiating / handling get() on the Secret/Order objects <_< | 22:18 |
| *** kebray has joined #openstack-barbican | 22:18 | |
| *** SheenaG1 has quit IRC | 22:19 | |
| *** atiwari has quit IRC | 22:19 | |
| *** jamielennox|away is now known as jamielennox | 22:22 | |
| *** nkinder has quit IRC | 22:26 | |
| openstackgerrit | John Wood proposed a change to openstack/barbican: Add initial files for certificate event handling https://review.openstack.org/115301 | 22:33 |
| openstackgerrit | John Wood proposed a change to openstack/barbican: Add initial files for certificate event handling https://review.openstack.org/115301 | 22:37 |
| openstackgerrit | John Wood proposed a change to openstack/barbican: Add initial files for certificate event handling https://review.openstack.org/115301 | 22:45 |
| rm_work | redrobot / woodster_: uh hey, so... right now, if I do a GET on a container, the "secret_refs" structure it returns gives me the "secret_id" (which is a UUID), not a "secret_ref" >_> | 23:01 |
| rm_work | that might be a blocker for containers in the client, unless I want to do really dumb workarounds | 23:01 |
| rm_work | any idea why it was implemented that way? | 23:01 |
| *** kebray has quit IRC | 23:04 | |
| redrobot | rm_work someone fubarred it? we should file a bug for that | 23:14 |
| rm_work | redrobot: like this? https://bugs.launchpad.net/barbican/+bug/1359452 | 23:14 |
| rm_work | CR incoming | 23:14 |
| redrobot | rm_work yes, just like that | 23:15 |
| redrobot | rm_work what's your launchpad id? | 23:15 |
| rm_work | umm | 23:15 |
| rm_work | shiot | 23:15 |
| rm_work | i have so many freaking IDs | 23:16 |
| rm_work | and they're all different T_T | 23:16 |
| rm_work | maybe adam-harwell | 23:16 |
| redrobot | never mind I assigned you | 23:16 |
| rm_work | try that | 23:16 |
| rm_work | otherwise rm-you | 23:16 |
| *** kebray has joined #openstack-barbican | 23:23 | |
| rm_work | woodster_ / redrobot: oh I figured it out | 23:28 |
| rm_work | it's ONLY on the LIST | 23:28 |
| rm_work | it's already handled properly in the ContainerController, just not in ContainersController | 23:29 |
| rm_work | updated bug https://bugs.launchpad.net/barbican/+bug/1359452 | 23:35 |
| rm_work | CR incoming | 23:35 |
| rm_work | umm | 23:38 |
| openstackgerrit | Adam Harwell proposed a change to openstack/barbican: Fix Container list to properly format secret_refs https://review.openstack.org/115819 | 23:38 |
| rm_work | there we go | 23:38 |
| rm_work | wow, never landed https://review.openstack.org/#/c/112149/ | 23:41 |
| rm_work | fff | 23:41 |
| rm_work | woodster_: is he "tsv" on IRC? | 23:42 |
| rm_work | reaperhulk: hey are you around? | 23:43 |
| rm_work | reaperhulk: your comment on https://review.openstack.org/#/c/112149/ is... maybe not correct? >_> | 23:43 |
| rm_work | it appears to be defined immediately above the line you commented on :P | 23:44 |
| rm_work | https://review.openstack.org/#/c/112149/7/barbicanclient/client.py | 23:44 |
| *** kebray has quit IRC | 23:44 | |
| *** nkinder has joined #openstack-barbican | 23:46 | |
| rm_work | arghhh | 23:46 |
| *** juantwo has joined #openstack-barbican | 23:48 | |
| *** juantwo has quit IRC | 23:50 | |
| *** juantwo has joined #openstack-barbican | 23:50 | |
| *** openstackgerrit has quit IRC | 23:54 | |
| rm_work | alright, guess I'll look at this stuff again tomorrow :( | 23:55 |
| *** openstackgerrit has joined #openstack-barbican | 23:58 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!
