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

Support confmap providers in discovery config #3874

Conversation

samiura
Copy link

@samiura samiura commented Oct 30, 2023

Adding option for arbitrary env var to authenticate for dbs on discovery mode

Description:

Link to Splunk idea:

Testing:

Documentation:

@samiura samiura requested review from a team as code owners October 30, 2023 20:53
@github-actions
Copy link
Contributor

github-actions bot commented Oct 30, 2023

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

@samiura samiura force-pushed the config-source-integration-attempt-with-discovery-mode-2 branch from 141e402 to 8195907 Compare October 30, 2023 21:18
@samiura
Copy link
Author

samiura commented Oct 30, 2023

I have read the CLA Document and I hereby sign the CLA

srv-gh-o11y-gdi-cla added a commit to splunk/cla-agreement that referenced this pull request Oct 30, 2023
@samiura samiura force-pushed the config-source-integration-attempt-with-discovery-mode-2 branch from 8195907 to cc1fca4 Compare October 30, 2023 21:59
@samiura samiura requested a review from a team as a code owner October 30, 2023 21:59
@samiura samiura force-pushed the config-source-integration-attempt-with-discovery-mode-2 branch from 9a8ea48 to ff6dc36 Compare October 30, 2023 22:42
@samiura samiura requested a review from rmfitzpatrick October 30, 2023 23:14
@rmfitzpatrick
Copy link
Contributor

@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/ ${env:LABEL_ONE_VALUE} could be adequate: https://github.com/signalfx/splunk-otel-collector/blob/main/tests/general/discoverymode/testdata/host-observer-config.d/receivers/internal-prometheus.discovery.yaml#L9

@samiura samiura force-pushed the config-source-integration-attempt-with-discovery-mode-2 branch 2 times, most recently from cc5a26d to 10fa78a Compare October 31, 2023 22:14
@samiura samiura force-pushed the config-source-integration-attempt-with-discovery-mode-2 branch from 69cadea to b6397a2 Compare November 1, 2023 19:13
@samiura samiura changed the title Adding the option to have arbitrary environment variable to authenticate in discovery mode for dbs Support confmap providers in discovery config Nov 1, 2023
@samiura samiura force-pushed the config-source-integration-attempt-with-discovery-mode-2 branch 2 times, most recently from 0d684e3 to b43dba7 Compare November 1, 2023 19:21
Copy link
Contributor

@rmfitzpatrick rmfitzpatrick left a 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

@samiura samiura force-pushed the config-source-integration-attempt-with-discovery-mode-2 branch from b43dba7 to db99643 Compare November 1, 2023 19:47
@rmfitzpatrick rmfitzpatrick merged commit 652305e into signalfx:main Nov 1, 2023
45 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Nov 1, 2023
@samiura samiura deleted the config-source-integration-attempt-with-discovery-mode-2 branch March 12, 2024 17:50
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants