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

reset kubeconfig to empty string when using in-cluster config #2273

Merged
merged 3 commits into from
Oct 25, 2023

Conversation

changexd
Copy link
Contributor

@changexd changexd commented Oct 25, 2023

Description:
reset kubeconfig to empty string when using in-cluster config

Link to tracking Issue:
#2262

Testing:
according to the discussion in #2262, it seems like we haven't added integration tests currently to check if prometheusCR and target allocator setup actually checks metrics. So I tested on my own cluster making sure the bug is fixed.

As this is my very first pull request and with lack of experience, I simply made changes with the solution provided in the issue, but if a helping hand is needed in adding integration tests, I would love to be part of it too!

@changexd changexd requested review from a team October 25, 2023 07:31
@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Oct 25, 2023

CLA Signed

The committers listed above are authorized under a signed CLA.

Copy link
Contributor

@swiatekm swiatekm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thank you for the contribution!

I'd like to hold off on merging this until we have tests demonstrating it fixes #2262. I should be done with those today.

@swiatekm
Copy link
Contributor

We're good to merge, see #2275

@jaronoff97 jaronoff97 merged commit 1ae2fe6 into open-telemetry:main Oct 25, 2023
24 checks passed
@jaronoff97
Copy link
Contributor

Thank you for your contribution!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants