From e9a914d650be9f82089b23c24acb223fe8e6b786 Mon Sep 17 00:00:00 2001 From: Julien Pivotto Date: Fri, 3 Jun 2022 11:13:47 +0200 Subject: [PATCH] Do not lint duplicate rules when checking Prometheus config. Linting rules by default is a breaking change that broke our users' CI pipeline with false positive. Let's exclude this by default. Signed-off-by: Julien Pivotto --- cmd/promtool/main.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/cmd/promtool/main.go b/cmd/promtool/main.go index 01d953d2049..f70d018f895 100644 --- a/cmd/promtool/main.go +++ b/cmd/promtool/main.go @@ -94,7 +94,7 @@ func main() { checkConfigLint := checkConfigCmd.Flag( "lint", "Linting checks to apply to the rules specified in the config. Available options are: "+strings.Join(lintOptions, ", ")+". Use --lint=none to disable linting", - ).Default(lintOptionDuplicateRules).String() + ).Default(lintOptionNone).String() checkWebConfigCmd := checkCmd.Command("web-config", "Check if the web config files are valid or not.") webConfigFiles := checkWebConfigCmd.Arg(