-
Notifications
You must be signed in to change notification settings - Fork 158
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
Support confmap providers in discovery config #3874
Support confmap providers in discovery config #3874
Conversation
CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅ |
141e402
to
8195907
Compare
I have read the CLA Document and I hereby sign the CLA |
8195907
to
cc1fca4
Compare
9a8ea48
to
ff6dc36
Compare
@samiura do you mind updating this existing test to confirm that unexpanded provider directives are used in the initial config? I think replacing this w/ |
cc5a26d
to
10fa78a
Compare
69cadea
to
b6397a2
Compare
0d684e3
to
b43dba7
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.
lgtm w/ final items
b43dba7
to
db99643
Compare
Adding option for arbitrary env var to authenticate for dbs on discovery mode
Description:
Link to Splunk idea:
Testing:
Documentation: