-
Notifications
You must be signed in to change notification settings - Fork 5k
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
[alertmanager] Variable to add extra Pod Configurations #4845
[alertmanager] Variable to add extra Pod Configurations #4845
Conversation
Signed-off-by: rakesh-von <rakesh.akula@vonage.com>
25c9f3c
to
3c92c6e
Compare
@monotek - can you please review this change? We need this on an urgent basis. Sorry for this urgency. |
Hi @naseemkullah - can you please review this PR? |
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
HI @dongjiang1989 - thanks for reviewing, is it possible for you to merge this please? |
@monotek and @naseemkullah - can you please review this please? we are waiting for this change. |
What this PR does / why we need it
This PR lets users add additional Pod configs such as dnsPolicy and hostNetwork
Which issue this PR fixes
(optional, in
fixes #<issue number>(, fixes #<issue_number>, ...)
format, will close that issue when PR gets merged)Special notes for your reviewer
Checklist
[prometheus-couchdb-exporter]
)