Skip to content
This repository has been archived by the owner on Dec 15, 2021. It is now read-only.

Migrate to Prometheus 2.1.0 and Grafana 5.0.0 #748

Merged
merged 2 commits into from
Mar 6, 2018

Conversation

sliard
Copy link
Contributor

@sliard sliard commented Mar 2, 2018

Change Prometheus and Grafana version.
Will work only when infinityworks upgrade infinityworks/prom-conf with this pull request :
infinityworks/prom-conf#6

@Rucknar
Copy link
Contributor

Rucknar commented Mar 6, 2018

@sliard Looks good, it's probably worth updating the default version to match your new entry. Can be done in the main config file, you okay to push that up as an update?

@sliard
Copy link
Contributor Author

sliard commented Mar 6, 2018

No problem. I just didn't want to force this update and let you choose. :)
For the moment I set it has version 3.0.1 but don't you think 4.0.0 is better ?
Let me know what you prefer and I will update.

@Rucknar
Copy link
Contributor

Rucknar commented Mar 6, 2018

Won't force it on anyone. That version number needs adjusting in the base folder config.yaml file to update the default version. If we want to hold off on that for a little while though, that's fine.

@Rucknar
Copy link
Contributor

Rucknar commented Mar 6, 2018

@cloudnautique Looks good to merge to me, haven't had chance to test myself.

@rawmind0
Copy link
Contributor

rawmind0 commented Mar 6, 2018

Hi guys, @sliard thanks for the PR, @Rucknar thanks for the review.

Tested and working fine.

LGTM

@rawmind0 rawmind0 merged commit 1c93cde into rancher:master Mar 6, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants