This repository has been archived by the owner on Apr 4, 2018. It is now read-only.
[#99020160] Upgrade to Telegraf 0.1.4 and InfluxDB 0.9.1 #122
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
The package for Telegraf 0.1.2 (which our playbook defaults to) installs to
the same locations as InfluxDB and prevents one of them from starting
depending on which order they have been installed:
This has been fixed in Telegraf >= 0.1.3 by installing to a separate directory:
We have to upgrade InfluxDB at the same time in order for it to rewrite its
own init script, because it's a post-install action. Both are patch upgrades
and don't look like they'll cause any problems:
Closely related to this, we can also remove our customisations that made
Telegraf run as the InfluxDB user, because they now have separate PID files.
This also has to be done at the same time as the upgrades because there
doesn't seem to be a nice abstraction to notify the service controlled by
the Ansible playbook.
Changing the version in the playbook's default variable didn't seem right
when we have a lot of other changes to make at the same time and we're
working towards pinning the versions that we use in our own repo. I may do
it independently.
Before:
After: