Update check signature #8157

coignetp · 2020-12-08T15:29:45Z

Update the check signature to only use self.instance

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-08T15:38:46Z

Codecov Report

Merging #8157 (c2f4485) into master (256c3e7) will decrease coverage by 3.22%.
The diff coverage is 100.00%.

Impacted Files	Coverage Δ
aerospike/datadog_checks/aerospike/aerospike.py	`80.16% <100.00%> (ø)`
aerospike/tests/test_aerospike.py	`93.02% <100.00%> (ø)`
aerospike/tests/test_unit.py	`96.55% <100.00%> (ø)`
mapr/datadog_checks/mapr/mapr.py
process/datadog_checks/process/process.py
php_fpm/datadog_checks/php_fpm/__about__.py
powerdns_recursor/tests/common.py
druid/tests/test_unit.py
http_check/datadog_checks/http_check/__about__.py
ambari/tests/manual_test.py
... and 819 more

Update check signature

c2f4485

coignetp added the changelog/Fixed label Dec 8, 2020

coignetp requested a review from a team as a code owner December 8, 2020 15:29

ghost added the integration/aerospike label Dec 8, 2020

ChristineTChen approved these changes Dec 8, 2020

View reviewed changes

coignetp merged commit 6fa7f77 into master Dec 9, 2020

coignetp deleted the paul/ae-signature branch December 9, 2020 08:57

github-actions bot pushed a commit that referenced this pull request Dec 9, 2020

Update check signature (#8157) 6fa7f77

92bd9bc