Skip to content

Commit 29dc0e4

Browse files
authored
Fix release script (#187)
1 parent d9f4eec commit 29dc0e4

File tree

1 file changed

+9
-8
lines changed

1 file changed

+9
-8
lines changed

scripts/dev/release_image.py

Lines changed: 9 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,10 @@
33
import sys
44
import argparse
55

6+
ALLOWED_RELEASE_OPTIONS = frozenset(
7+
["mongodb-kubernetes-operator", "version-upgrade-hook"]
8+
)
9+
610

711
def parse_args() -> argparse.Namespace:
812
parser = argparse.ArgumentParser()
@@ -35,7 +39,7 @@ def parse_args() -> argparse.Namespace:
3539
parser.add_argument(
3640
"--image_type",
3741
help="Type of image to be released",
38-
choices=["mongodb-kubernetes-operator", "version-upgrade-hook"],
42+
choices=ALLOWED_RELEASE_OPTIONS,
3943
)
4044
args = parser.parse_args()
4145

@@ -47,18 +51,15 @@ def main() -> int:
4751
with open(args.release_file) as f:
4852
release = json.load(f)
4953

50-
if args.image_type == "operator":
51-
new_tag = release["mongodb-kubernetes-operator"]
52-
elif args.image_type == "versionhook":
53-
new_tag = release["version-upgrade-hook"]
54-
else:
55-
print(f"Image type{args.image_type} is not supported by the release script!")
54+
if args.image_type not in ALLOWED_RELEASE_OPTIONS:
55+
print(f"Image type {args.image_type} is not supported by the release script!")
5656
return 1
57+
5758
dockerutil.retag_image(
5859
args.old_repo_url,
5960
args.new_repo_url,
6061
args.old_tag,
61-
new_tag,
62+
release[args.image_type],
6263
args.path,
6364
args.labels,
6465
args.username,

0 commit comments

Comments
 (0)