Fix version metadata submitted twice

[AlexandreYang]

What does this PR do?

Related: #4818, #5144, #5252

The issues:

1. The version metadata might be submitted twice

2. We log Cannot parse the complete Apache version from %s. instead of Cannot parse the complete Apache version from <VERSION>.

Fix version metadata submitted twice when retrying with ?auto. We should return, when calling self.check recursively (note: we should avoid making recursive call if possible, it's error prone).

This PR also includes:

• Add test for no metrics case
• Add fix logging format issue for self.log.info("Cannot parse the complete Apache version from %s.", value)

Motivation

Additional Notes

Review checklist (to be filled by reviewers)

• 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]

Codecov Report

Merging #5325 into master will increase coverage by 7.29%.
The diff coverage is 100%.

Impacted Files	Coverage Δ
apache/tests/test_apache.py	89.7% <100%> (+3.16%)	⬆️
apache/datadog_checks/apache/apache.py	95.94% <100%> (+6.9%)	⬆️
apache/tests/common.py	100% <100%> (ø)	⬆️
...ts/tooling/configuration/consumers/test_example.py
clickhouse/datadog_checks/clickhouse/utils.py
...checks_dev/tests/plugin/test_environment_runner.py
riakcs/datadog_checks/riakcs/riakcs.py
oracle/tests/test_oracle.py
...r/datadog_checks/openstack_controller/__about__.py
twistlock/datadog_checks/twistlock/twistlock.py
... and 871 more

[AlexandreYang]

Note for reviewers: before this fix check._submit_metadata was called twice.