Add output option to helm-values command (#133)

aladdin/commands/helm_values.py

@@ -45,6 +45,13 @@ def parse_args(sub_parser):
         dest="all_values",
         action="store_true",
     )
+    parser.add_argument(
+        "-o",
+        "--output",
+        help="output values to a file",
+        dest="output",
+        default=None,
+    )
 
 
 @container_command
@@ -55,6 +62,7 @@ def helm_values(
     git_ref: str = None,
     chart: str = None,
     all_values: bool = False,
+    output: str = None,
 ):
     uri = urlparse(uri)
     params = dict(parse_qsl(uri.query))
@@ -92,11 +100,16 @@ def helm_values(
             )
 
             logging.info("Executing: %s", " ".join(command))
-            subprocess.run(
+            result = subprocess.run(
                 command,
-                capture_output=False,
+                capture_output=output is not None,
                 check=True,
+                encoding="utf-8",
             )
+            if output:
+                with open(output, "w") as outputfile:
+                    outputfile.write(result.stdout)
+                logging.info("Values saved in: %s", output)
 
 
 @contextmanager

aladdin/commands/publish.py

@@ -1,4 +1,5 @@
 import logging
+import os
 import subprocess
 import tempfile
 
@@ -124,7 +125,7 @@ def publish_clean(
                 f"Could not checkout to ref {ref} in repo {git_url}. Have you pushed it to remote?"
             )
             return
-        if init_submodules:
+        if init_submodules or os.path.exists(f"{tmpdirname}/.gitmodules"):
             try:
                 g.init_submodules(tmpdirname)
             except subprocess.CalledProcessError:

pyproject.toml

@@ -1,6 +1,6 @@
 [tool.poetry]
 name = "aladdin"
-version = "1.19.7.29"
+version = "1.19.7.30"
 description = ""
 authors = ["Fivestars <dev@fivestars.com>"]
 include = [