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

docs: add promtool version in rule migration cmd #6989

Merged

Conversation

hoffie
Copy link
Contributor

@hoffie hoffie commented Mar 15, 2020

promtool update rules is no longer available after Prometheus 2.5.
Document this to avoid confusion such as in #6988.

@roidelapluie
Copy link
Member

Thanks! Can we be shorter and more pragmatic?

@hoffie hoffie force-pushed the docs-migration-promtool-rules branch 2 times, most recently from 1e402d7 to d87e909 Compare March 16, 2020 20:50
@hoffie
Copy link
Contributor Author

hoffie commented Mar 16, 2020

Tried to condense it into a single sentence. If you think it should be shortened even further, we would need to drop details, I think.

@hoffie hoffie force-pushed the docs-migration-promtool-rules branch from d87e909 to f5485e5 Compare March 28, 2020 21:36
`promtool update rules` is no longer available after Prometheus 2.5.
Document this to avoid confusion such as in prometheus#6988.

Signed-off-by: Christian Hoffmann <mail@hoffmann-christian.info>
@hoffie hoffie force-pushed the docs-migration-promtool-rules branch from f5485e5 to ab2488d Compare March 30, 2020 13:02
@stale stale bot added the stale label May 29, 2020
@roidelapluie roidelapluie merged commit 51306cb into prometheus:master Sep 7, 2020
@roidelapluie
Copy link
Member

Thanks!

damnever pushed a commit to damnever/prometheus that referenced this pull request Sep 9, 2020
`promtool update rules` is no longer available after Prometheus 2.5.
Document this to avoid confusion such as in prometheus#6988.

Signed-off-by: Christian Hoffmann <mail@hoffmann-christian.info>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants