From 58164e0009a2539ca5f5bbc1ab9c37080fa5e0f4 Mon Sep 17 00:00:00 2001 From: Matt Butcher Date: Tue, 1 Nov 2016 13:41:29 -0600 Subject: [PATCH] fix(linter): remove spurious namespace error Long ago, Helm did not support cross-namespace installs. There was a linter rule to catch this. When we changed the way Helm worked, we did not remove the linter rule. This commit removes that linter rule. Closes #1489 --- pkg/lint/rules/template.go | 9 --------- 1 file changed, 9 deletions(-) diff --git a/pkg/lint/rules/template.go b/pkg/lint/rules/template.go index 5dc6d8424..886f0031c 100644 --- a/pkg/lint/rules/template.go +++ b/pkg/lint/rules/template.go @@ -104,8 +104,6 @@ func Templates(linter *support.Linter) { if !validYaml { continue } - - linter.RunLinterRule(support.ErrorSev, path, validateNoNamespace(yamlStruct)) } } @@ -196,13 +194,6 @@ func validateYamlContent(err error) error { return nil } -func validateNoNamespace(yamlStruct K8sYamlStruct) error { - if yamlStruct.Metadata.Namespace != "" { - return errors.New("namespace option is currently NOT supported") - } - return nil -} - // K8sYamlStruct stubs a Kubernetes YAML file. // Need to access for now to Namespace only type K8sYamlStruct struct {