Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

package.sh: upload raw binaries to S3 #166

Merged
merged 1 commit into from
Sep 2, 2015
Merged

package.sh: upload raw binaries to S3 #166

merged 1 commit into from
Sep 2, 2015

Conversation

sparrc
Copy link
Contributor

@sparrc sparrc commented Sep 2, 2015

/cc @beckettsean @gunnaraasen how do we feel about distributing linux binaries of telegraf? I was thinking that this might be good for people who want to try out telegraf without installing a .deb or .rpm which also installs an init service on the server.

Also nice for people who just want to manage the service themselves

@gunnaraasen
Copy link
Member

Sounds good to me. I think there was a request for something similar for InfluxDB but I can't find the issue.

@beckettsean
Copy link
Contributor

Let's do it. We get requests for binaries for chronograf and influxdb fairly frequently, it's just a matter of time for telegraf.

@sparrc
Copy link
Contributor Author

sparrc commented Sep 2, 2015

great! will also need an influxdb.com update for the downloads page (I'll do it), they'll start getting uploaded with 0.1.8

@beckettsean
Copy link
Contributor

I added a note to https://github.com/influxdb/influxdb.com/issues/267 to document the telegraf tarball, too.

@sparrc
Copy link
Contributor Author

sparrc commented Sep 2, 2015

Cool, I'll update this PR to actually tar -zcf the binary rather than zip, just for consistency with the DB

@beckettsean
Copy link
Contributor

@sparrc agreed. I think gzipped tarballs are more standard than just gzipped files.

@sparrc sparrc merged commit 13061d1 into master Sep 2, 2015
@sparrc sparrc deleted the bin-upload branch September 2, 2015 19:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants