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

[chore] Add e2e for receiver/kubeletstats #26676

Merged
merged 10 commits into from
Sep 14, 2023

Conversation

jinja2
Copy link
Contributor

@jinja2 jinja2 commented Sep 13, 2023

Description:

Adds an E2E test for kubeletstats receiver

Updates existing e2e k8s tests to bootstrap kind cluster with valid kubelet certificates. This will make k8s clusters used in the e2e test more inline with the security practices used by real clusters and improve testing with the kubeletstats receiver which connects to kubelet server.

Link to tracking Issue: 26319

Testing:

Documentation:

@jinja2 jinja2 requested review from a team and evan-bradley September 13, 2023 18:24
@jinja2 jinja2 marked this pull request as draft September 13, 2023 18:42
@jinja2 jinja2 changed the title test kind cluster with kubelet tls fix [chore] Run e2e k8s kind clusters with TLS enabled for kubelet server Sep 13, 2023
@jinja2 jinja2 changed the title [chore] Run e2e k8s kind clusters with TLS enabled for kubelet server [chore] Add e2e for receiver/kubeletstats Sep 14, 2023
@jinja2 jinja2 marked this pull request as ready for review September 14, 2023 01:11
Copy link
Member

@TylerHelmuth TylerHelmuth left a comment

Choose a reason for hiding this comment

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

Thank you!

@atoulme atoulme added the ready to merge Code review completed; ready to merge by maintainers label Sep 14, 2023
@TylerHelmuth TylerHelmuth merged commit 2d5eb37 into open-telemetry:main Sep 14, 2023
95 checks passed
@github-actions github-actions bot added this to the next release milestone Sep 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready to merge Code review completed; ready to merge by maintainers receiver/kubeletstats
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants