Merge pull request #516 from deepmind/release-v0.27.4

Fix renamed flag from secrets_path to refs_path in the linter and start release-v0.27.4
kapicorp · May 14, 2020 · e106bec · e106bec
2 parents 18edad8 + a916dcd
commit e106bec
Show file tree

Hide file tree

Showing 4 changed files with 7 additions and 4 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,3 +1,6 @@
+## 0.27.4:
+- Fix linter argument flag to refs-path instead of deprecated secrets-path (#516)
+
 ## 0.27.3:
 - Fix Vaultkv Error Handling (#512)
 - Fix Running init with kapitan binary doesn't work (#514)

diff --git a/kapitan/lint.py b/kapitan/lint.py
@@ -82,7 +82,7 @@ def start_lint(args):
 
     if args.search_secrets:
         logger.info("\nChecking for orphan secrets files...\n")
-        status_secrets = lint_orphan_secrets(args.compiled_path, args.secrets_path)
+        status_secrets = lint_orphan_secrets(args.compiled_path, args.refs_path)
 
     checks_sum = status_secrets + status_class_checks + status_yamllint
     if args.fail_on_warning and checks_sum > 0:

diff --git a/scripts/kap_5_migrate.py b/scripts/kap_5_migrate.py
@@ -123,9 +123,9 @@ def post_warning():
 
     pre_warning()
 
-    print("searching for refs in secrets-path:", args.secrets_path)
+    print("searching for refs in secrets-path:", args.refs_path)
 
-    for file_path in find_files(args.secrets_path):
+    for file_path in find_files(args.refs_path):
         update_secrets(file_path)
 
     print()

diff --git a/tests/test_linter.py b/tests/test_linter.py
@@ -31,7 +31,7 @@ def test_lint(self):
         args.skip_yamllint = False
         args.inventory_path = "./tests/test_resources/inventory"
         args.search_secrets = True
-        args.secrets_path = "./tests/test_resources/secrets"
+        args.refs_path = "./tests/test_resources/secrets"
         args.compiled_path = "./tests/test_resources/compiled"
 
         num_issues_found = start_lint(args)