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

Runbook: Add extra runbook information for distroless images #8235

Merged
merged 7 commits into from
Jun 3, 2024

Conversation

jhesketh
Copy link
Contributor

@jhesketh jhesketh commented May 31, 2024

Now that distroless images are published by default this information will be of interest to operators.

What this PR does

Which issue(s) this PR fixes or relates to

Fixes #

Checklist

  • Tests updated.
  • Documentation added.
  • CHANGELOG.md updated - the order of entries should be [CHANGE], [FEATURE], [ENHANCEMENT], [BUGFIX].
  • about-versioning.md updated with experimental features.

Now that distroless images are published by default this information
will be of interest to operators.
@jhesketh jhesketh marked this pull request as ready for review May 31, 2024 11:21
@jhesketh jhesketh requested review from jdbaldry and a team as code owners May 31, 2024 11:21
Copy link
Contributor

@dimitarvdimitrov dimitarvdimitrov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

awesome, thanks for adding this!

docs/sources/mimir/manage/mimir-runbooks/_index.md Outdated Show resolved Hide resolved
docs/sources/mimir/manage/mimir-runbooks/_index.md Outdated Show resolved Hide resolved
docs/sources/mimir/manage/mimir-runbooks/_index.md Outdated Show resolved Hide resolved
docs/sources/mimir/manage/mimir-runbooks/_index.md Outdated Show resolved Hide resolved
docs/sources/mimir/manage/mimir-runbooks/_index.md Outdated Show resolved Hide resolved
docs/sources/mimir/manage/mimir-runbooks/_index.md Outdated Show resolved Hide resolved
docs/sources/mimir/manage/mimir-runbooks/_index.md Outdated Show resolved Hide resolved
docs/sources/mimir/manage/mimir-runbooks/_index.md Outdated Show resolved Hide resolved
docs/sources/mimir/manage/mimir-runbooks/_index.md Outdated Show resolved Hide resolved
docs/sources/mimir/manage/mimir-runbooks/_index.md Outdated Show resolved Hide resolved
Co-authored-by: Charles Korn <charleskorn@users.noreply.github.com>
@jhesketh jhesketh enabled auto-merge (squash) June 3, 2024 06:36
@jhesketh jhesketh merged commit e2362ec into grafana:main Jun 3, 2024
29 checks passed
@jhesketh jhesketh deleted the jhesketh/distroless-docs branch June 3, 2024 07:49
@@ -4,6 +4,7 @@

### Grafana Mimir

* [FEATURE] Alertmanager: Added `-alertmanager.max-silences-count` and `-alertmanager.max-silence-size-bytes` to set limits on per tenant silences. Disabled by default. #6898
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Did you mean to add this here?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't see this change as part of this PR 🤔

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's not. It was just in the wrong place in the changelog and it got confusing. We fixed it in #8266

narqo pushed a commit to narqo/grafana-mimir that referenced this pull request Jun 6, 2024
…#8235)

* Runbook: Add extra runbook information for distroless images

Now that distroless images are published by default this information
will be of interest to operators.

* Update changelog

* Build doc

* Apply suggestions from code review

Co-authored-by: Charles Korn <charleskorn@users.noreply.github.com>

* Apply suggestions from code review

* Update docs/sources/mimir/manage/mimir-runbooks/_index.md

---------

Co-authored-by: Charles Korn <charleskorn@users.noreply.github.com>
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.

4 participants