Add a validator for the manifest version

datadog_checks_dev/datadog_checks/dev/tooling/commands/validate/manifest.py

@@ -75,10 +75,6 @@ def manifest(ctx, check, fix):
             continue
 
         version = decoded.get('manifest_version', V2_STRING)
-        if version == V1_STRING:
-            file_failures += 1
-            display_queue.append((echo_failure, 'Manifest version must be >= 2.0.0'))
-
         all_validators = get_all_validators(ctx, version, is_extras, is_marketplace)
 
         for validator in all_validators:

datadog_checks_dev/datadog_checks/dev/tooling/manifest_validator/__init__.py

@@ -1,8 +1,15 @@
 #  (C) Datadog, Inc. 2021-present
 #  All rights reserved
 #  Licensed under a 3-clause BSD style license (see LICENSE)
+from .common.validator import VersionValidator
+from .constants import V2_STRING
 from .v2.validator import get_v2_validators
 
 
 def get_all_validators(ctx, version_string, is_extras=False, is_marketplace=False):
-    return get_v2_validators(ctx, is_extras, is_marketplace)
+    validators = [VersionValidator()]
+
+    if version_string == V2_STRING:
+        validators.extend(get_v2_validators(ctx, is_extras, is_marketplace))
+
+    return validators

datadog_checks_dev/datadog_checks/dev/tooling/manifest_validator/common/validator.py

@@ -12,7 +12,7 @@
 from ...datastructures import JSONDict
 from ...git import git_show_file
 from ...utils import get_metadata_file, has_logs, is_metric_in_metadata_file, read_metadata_rows
-from ..constants import V1, V2
+from ..constants import V1, V2, V2_STRING, V1_STRING
 
 
 class ValidationResult(object):
@@ -311,3 +311,10 @@ def validate(self, check_name, decoded, fix):
                 + ' or define the logs properly'
             )
             self.fail(output)
+
+
+class VersionValidator(BaseManifestValidator):
+
+    def validate(self, check_name, decoded, fix):
+        if decoded.get('manifest_version', V2_STRING) == V1_STRING:
+            self.fail('Manifest version must be >= 2.0.0')