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

[receiver/kubeletstats] Support specifying context for kubeConfig auth_type (#26665) #27240

Merged

Conversation

hypnoce
Copy link
Contributor

@hypnoce hypnoce commented Sep 27, 2023

Description:
Support specifying context for kubeConfig auth_type when communicating with a Kubernetes cluster.

Link to tracking Issue: 26665

Testing: unit test added

Documentation: kubeletstatsreceiver README updated

@hypnoce
Copy link
Contributor Author

hypnoce commented Oct 4, 2023

@TylerHelmuth anything you want me to adapt ? Thanks

receiver/kubeletstatsreceiver/README.md Outdated Show resolved Hide resolved
@hypnoce hypnoce force-pushed the kubelet_kubeconfig_context branch from c186ba4 to ca17536 Compare October 6, 2023 09:05
@hypnoce
Copy link
Contributor Author

hypnoce commented Oct 6, 2023

@dmitryax I updated the PR. Thanks for the feedback.

@TylerHelmuth TylerHelmuth added the ready to merge Code review completed; ready to merge by maintainers label Oct 6, 2023
@TylerHelmuth TylerHelmuth merged commit 484704b into open-telemetry:main Oct 6, 2023
89 checks passed
@github-actions github-actions bot added this to the next release milestone Oct 6, 2023
jmsnll pushed a commit to jmsnll/opentelemetry-collector-contrib that referenced this pull request Nov 12, 2023
…auth_type` (open-telemetry#26665) (open-telemetry#27240)

**Description:**
Support specifying context for `kubeConfig` `auth_type` when
communicating with a Kubernetes cluster.

**Link to tracking Issue:** 26665

**Testing:** unit test added

**Documentation:** kubeletstatsreceiver README updated

Signed-off-by: JACQUES Francois <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants