-
Notifications
You must be signed in to change notification settings - Fork 5.1k
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
[prometheus] Apply templating to remote_read & remote_write config #3777
Conversation
1358d39
to
bf81d3a
Compare
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.
@garryod
Thanks for the contribution. Since you bump to the major version, can you put some note that the users should know, such as upgrading, what are compatible, what are not compatible, or some reference links in README.md please?
Ah yes, will do! |
Signed-off-by: Garry O'Donnell <garry.o'[email protected]>
Signed-off-by: Garry O'Donnell <garry.o'[email protected]> Signed-off-by: Garry O'Donnell <[email protected]>
Signed-off-by: Garry O'Donnell <garry.o'[email protected]> Signed-off-by: Garry O'Donnell <[email protected]>
Co-authored-by: zeritti <[email protected]> Signed-off-by: Garry O'Donnell <[email protected]>
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.
LGTM
…rometheus-community#3777) * Apply templating to remote_read & remote_write config Signed-off-by: Garry O'Donnell <garry.o'[email protected]> * Add migration instructions for prometheus 25.0 Signed-off-by: Garry O'Donnell <garry.o'[email protected]> Signed-off-by: Garry O'Donnell <[email protected]> * Only template URLs Signed-off-by: Garry O'Donnell <garry.o'[email protected]> Signed-off-by: Garry O'Donnell <[email protected]> * Fix whitespace Co-authored-by: zeritti <[email protected]> Signed-off-by: Garry O'Donnell <[email protected]> --------- Signed-off-by: Garry O'Donnell <garry.o'[email protected]> Signed-off-by: Garry O'Donnell <[email protected]> Co-authored-by: zeritti <[email protected]>
What this PR does / why we need it
Adds template evaluation (via the
tpl
function) to theremoteRead
&remoteWrite
fields in theConfigMap
. This allows users of the chart to substitute values at templating time. This is especially useful if the services which a user wishes to write to are deployed as part of an adjacent sub-chart where their name includes the deployment name.Special notes for your reviewer
N/A
Checklist
[prometheus-couchdb-exporter]
)