sts-sponsors team mailing list archive

[Adam Collard <mp+434515@xxxxxxxxxxxxxxxxxx>]

Adam Collard has proposed merging ~adam-collard/maas-ci/+git/system-tests:output-file-filter-envs into ~maas-committers/maas-ci/+git/system-tests:master.

Commit message:
Add --output-file option to filter_envs



Requested reviews:
  MAAS Committers (maas-committers)

For more details, see:
https://code.launchpad.net/~adam-collard/maas-ci/+git/system-tests/+merge/434515
-- 
Your team MAAS Committers is requested to review the proposed merge of ~adam-collard/maas-ci/+git/system-tests:output-file-filter-envs into ~maas-committers/maas-ci/+git/system-tests:master.

diff --git a/utils/filter_envs.py b/utils/filter_envs.py
index df3976a..05d2d0f 100644
--- a/utils/filter_envs.py
+++ b/utils/filter_envs.py
@@ -4,7 +4,6 @@ import sys
 
 
 def main(argv: list[str]) -> int:
-
     parser = argparse.ArgumentParser()
     parser.add_argument(
         "desired_envs",
@@ -12,6 +11,10 @@ def main(argv: list[str]) -> int:
         help="list of desired envs separated by ',' (ie: vm1,vm2,opelt)",
     )
 
+    parser.add_argument(
+        "--output-file", type=argparse.FileType("w"), default=sys.stdout
+    )
+
     args = parser.parse_args(argv)
 
     valid_envs_raw = subprocess.check_output(["tox", "-a"], encoding="utf8")
@@ -19,7 +22,7 @@ def main(argv: list[str]) -> int:
 
     desired_envs = set(args.desired_envs.split(","))
 
-    print(",".join(desired_envs & valid_envs))
+    print(",".join(desired_envs & valid_envs), file=args.output_file)
     return 0