Make 'Context limit reached' message a warning #8254

FlorianVeaux · 2020-12-23T16:47:29Z

Update the debug message to a real warning. Helpful to understand why the kafka_consumer check doesn't emit metrics anymore.

Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
PR title must be written as a CHANGELOG entry (see why)
Files changes must correspond to the primary purpose of the PR as described in the title (small unrelated changes should have their own PR)
PR must have changelog/ and integration/ labels attached

codecov · 2020-12-23T17:08:49Z

Codecov Report

Merging #8254 (b5f2ed3) into master (966da07) will decrease coverage by 6.91%.
The diff coverage is 60.00%.

Impacted Files	Coverage Δ
...mer/datadog_checks/kafka_consumer/legacy_0_10_2.py	`70.38% <50.00%> (ø)`
...er/datadog_checks/kafka_consumer/kafka_consumer.py	`73.93% <100.00%> (ø)`
consul/datadog_checks/consul/common.py
hazelcast/tests/metrics.py
network/tests/test_integration.py
...tadog_checks/nginx_ingress_controller/__about__.py
mcache/tests/test_check.py
ambari/datadog_checks/ambari/common.py
cisco_aci/datadog_checks/cisco_aci/__about__.py
btrfs/tests/test_btrfs.py
... and 863 more

Make 'Context limit reached' message a warning

b5f2ed3

FlorianVeaux requested a review from a team as a code owner December 23, 2020 16:47

ghost added the integration/kafka_consumer label Dec 23, 2020

yzhan289 approved these changes Dec 23, 2020

View reviewed changes

FlorianVeaux added the changelog/Fixed label Dec 28, 2020

FlorianVeaux merged commit 86a6c62 into master Dec 28, 2020

FlorianVeaux deleted the florian/update_kafka_c_context_limit_log branch December 28, 2020 09:10