Logstash Repo not really supported

[davedash]

This repo:

https://github.com/brandentimm/logstash_forwarder-formula/blob/master/logstash_forwarder/repo.sls#L10

Is not supported, as noted by: elastic/logstash-forwarder#184 (comment)

There should be a more recommended way of obtaining this package.

[brandentimm]

Thanks for the report, I hadn't noticed it. I'll look into building from github's master for the project when I have time, otherwise pull requests are welcome as always!

[brandentimm]

Well, from looking at their build instructions, this will be annoying.

Building the software requires golang, which I'm not opposed to, but since so many people have carefully managed Go installations, I'm loathe to install it just to build the software and possibly add a conflicting Go runtime to somebody's system.

It's even worse if we wanted to package it on-system, as in addition to golang we'd need ruby, gem, and a specific gem 'fpm'. Again, many people have carefully managed ruby/gem setups, that I would be loathe to trample

This formula may just need to sit idle until the logstash-forwarder folks figure out a reasonable packaging strategy.

[davedash]

Yeah - or let people supply their own repo that has the package as a pillar?

I'm signing up for packagelcoud.io just so I can build this package locally
and publish it there - kind of involved for just one package, so it might
take me a while, if I have something decent I can submit a PR.

On Mon Feb 09 2015 at 1:06:22 PM Branden Timm [email protected]
wrote:

Well, from looking at their build instructions, this will be annoying.

Building the software requires golang, which I'm not opposed to, but since
so many people have carefully managed Go installations, I'm loathe to
install it just to build the software and possibly add a conflicting Go
runtime to somebody's system.

It's even worse if we wanted to package it on-system, as in addition to
golang we'd need ruby, gem, and a specific gem 'fpm'. Again, many people
have carefully managed ruby/gem setups, that I would be loathe to trample

This formula may just need to sit idle until the logstash-forwarder folks
figure out a reasonable packaging strategy.

—
Reply to this email directly or view it on GitHub
#3 (comment)
.

[brandentimm]

I like your idea of leaving the package repository url as a bit of pillar info. We'd have to remove the GPG key step too, and update the README, but those are pretty trivial changes.

I'll see what I can work up this evening. Thanks for the second set of eyes.