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

Allow skip loading Python kubelet check based on DD_KUBELET_CORE_CHECK_ENABLED #16856

Merged

Conversation

jennchenn
Copy link
Member

@jennchenn jennchenn commented Feb 12, 2024

What does this PR do?

This change adds logic to skip loading the Python kubelet check based on the value of the configuration variable DD_KUBELET_CORE_CHECK_ENABLED.

Motivation

This is part of the efforts to migrate from the kubelet Python check to the kubelet check defined in datadog-agent. This config variable will be added in DataDog/helm-charts#1319.

Additional Notes

Review checklist (to be filled by reviewers)

  • Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
  • Changelog entries must be created for modifications to shipped code
  • Add the qa/skip-qa label if the PR doesn't need to be tested during QA.

@jennchenn jennchenn force-pushed the jenn/CONTINT-3778_disable-python-kubelet-check-via-env-var branch from 50f1aee to 00b31dc Compare February 12, 2024 22:39
Copy link

github-actions bot commented Feb 12, 2024

Test Results

  2 files    2 suites   6s ⏱️
 72 tests  72 ✅ 0 💤 0 ❌
144 runs  144 ✅ 0 💤 0 ❌

Results for commit e86263a.

♻️ This comment has been updated with latest results.

Copy link

codecov bot commented Feb 16, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 90.00%. Comparing base (bda5107) to head (e86263a).
Report is 14 commits behind head on master.

Additional details and impacted files
Flag Coverage Δ
activemq ?
aerospike ?
airflow ?
cassandra ?
cassandra_nodetool ?
ceph ?
clickhouse ?
confluent_platform ?
hive ?
hivemq ?
ignite ?
iis ?
jboss_wildfly ?
kafka ?
kubelet 90.90% <100.00%> (+0.05%) ⬆️
marklogic ?
presto ?
sap_hana ?
solr ?
sonarqube ?
tekton ?
weblogic ?

Flags with carried forward coverage won't be shown. Click here to find out more.

@jennchenn jennchenn force-pushed the jenn/CONTINT-3778_disable-python-kubelet-check-via-env-var branch from 00b31dc to 2657ed7 Compare March 4, 2024 22:46
Kyle-Neale
Kyle-Neale previously approved these changes Mar 5, 2024
Copy link
Contributor

@Kyle-Neale Kyle-Neale left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@clamoriniere clamoriniere left a comment

Choose a reason for hiding this comment

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

Everything looks good 👍

could you update the PR title to match the new envvar name 🙇

@jennchenn jennchenn changed the title Allow skip loading Python kubelet check based on DD_KUBERNETES_KUBELET_CORE_CHECK_ENABLED Allow skip loading Python kubelet check based on DD_KUBELET_CORE_CHECK_ENABLED Mar 13, 2024
@jennchenn jennchenn merged commit b4a7f72 into master Mar 13, 2024
34 checks passed
@jennchenn jennchenn deleted the jenn/CONTINT-3778_disable-python-kubelet-check-via-env-var branch March 13, 2024 22:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants