Merge pull request #3730 from Yelp/u/jfong/fix_spark_run_docker_reg

Use the correct registry_uri to check if we need to sudo
Yelp · Oct 24, 2023 · f4d0fcf · f4d0fcf
2 parents f9acf9b + e5d3589
commit f4d0fcf
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/paasta_tools/cli/cmds/spark_run.py b/paasta_tools/cli/cmds/spark_run.py
@@ -1080,7 +1080,7 @@ def build_and_push_docker_image(args: argparse.Namespace) -> Optional[str]:
     if retcode != 0:
         return None
 
-    if args.docker_registry != DEFAULT_SPARK_DOCKER_REGISTRY:
+    if registry_uri != DEFAULT_SPARK_DOCKER_REGISTRY:
         command = "sudo -H docker push %s" % docker_url
     else:
         command = "docker push %s" % docker_url