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
mixin: configurable namespace label, job label, and job prefix #3482
mixin: configurable namespace label, job label, and job prefix #3482
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👋 Hi, and thanks for your contribution! I'm open to this change and overall LGTM (good job!). I left few minor comments.
Could you also add a CHANGELOG entry under the "mixin" section, please?
Co-authored-by: Marco Pracucci <marco@pracucci.com>
Co-authored-by: Marco Pracucci <marco@pracucci.com>
docs/sources/operators-guide/monitor-grafana-mimir/requirements.md
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great job, LGTM! I will let @osg-grafana follow up with doc review. I've also marked the PR "ready for review" since it was still a draft.
docs/sources/operators-guide/monitor-grafana-mimir/requirements.md
Outdated
Show resolved
Hide resolved
docs/sources/operators-guide/monitor-grafana-mimir/requirements.md
Outdated
Show resolved
Hide resolved
docs/sources/operators-guide/monitor-grafana-mimir/requirements.md
Outdated
Show resolved
Hide resolved
docs/sources/operators-guide/monitor-grafana-mimir/requirements.md
Outdated
Show resolved
Hide resolved
docs/sources/operators-guide/monitor-grafana-mimir/requirements.md
Outdated
Show resolved
Hide resolved
docs/sources/operators-guide/monitor-grafana-mimir/requirements.md
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changes involve: fix punctuation, conform to style, and simply phrase structures.
Co-authored-by: Ursula Kallio <ursula.kallio@grafana.com>
Co-authored-by: Ursula Kallio <ursula.kallio@grafana.com>
Co-authored-by: Ursula Kallio <ursula.kallio@grafana.com>
Co-authored-by: Ursula Kallio <ursula.kallio@grafana.com>
Co-authored-by: Ursula Kallio <ursula.kallio@grafana.com>
Co-authored-by: Ursula Kallio <ursula.kallio@grafana.com>
Co-authored-by: Ursula Kallio <ursula.kallio@grafana.com>
Co-authored-by: Ursula Kallio <ursula.kallio@grafana.com>
Thanks @osg-grafana and @pracucci for your help to get this merged! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you! :)
Signed-off-by: Marco Pracucci <marco@pracucci.com>
…na#3482) * mixin: add option for custom namespace label name * mixin: add option for custom job prefix * mixin: add option for custom job label * mimir-mixin: use named placeholder Co-authored-by: Marco Pracucci <marco@pracucci.com> * mimir-mixin: use named placeholder Co-authored-by: Marco Pracucci <marco@pracucci.com> * mimir-mixin: update documentation * mixin: update CHANGELOG.md * mimir-mixin: improve documentation Co-authored-by: Ursula Kallio <ursula.kallio@grafana.com> * mimir-mixin: improve documentation Co-authored-by: Ursula Kallio <ursula.kallio@grafana.com> * mimir-mixin: improve documentation Co-authored-by: Ursula Kallio <ursula.kallio@grafana.com> * mimir-mixin: improve documentation Co-authored-by: Ursula Kallio <ursula.kallio@grafana.com> * mimir-mixin: improve documentation Co-authored-by: Ursula Kallio <ursula.kallio@grafana.com> * mimir-mixin: improve documentation Co-authored-by: Ursula Kallio <ursula.kallio@grafana.com> * mimir-mixin: improve documentation Co-authored-by: Ursula Kallio <ursula.kallio@grafana.com> * mimir-mixin: improve documentation Co-authored-by: Ursula Kallio <ursula.kallio@grafana.com> * Run 'make doc' to format markdown Signed-off-by: Marco Pracucci <marco@pracucci.com> Signed-off-by: Marco Pracucci <marco@pracucci.com> Co-authored-by: Marco Pracucci <marco@pracucci.com> Co-authored-by: Ursula Kallio <ursula.kallio@grafana.com>
What this PR does
Makes labels used for namespace and job selection, and the prefix used in the job selection configurable
Which issue(s) this PR fixes or relates to
Fixes #
Checklist
CHANGELOG.md
updated - the order of entries should be[CHANGE]
,[FEATURE]
,[ENHANCEMENT]
,[BUGFIX]