Skip to content

Commit 54fc914

Browse files
nammnfealebenpae
authored andcommitted
don't re-create manifest and improve logging (#3995)
# Summary comments are on the changesets
1 parent 6bf3ba0 commit 54fc914

File tree

2 files changed

+1
-5
lines changed

2 files changed

+1
-5
lines changed

pipeline.py

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -305,10 +305,6 @@ def copy_into_container(client, src, dst):
305305

306306
def create_and_push_manifest(image: str, tag: str) -> None:
307307
final_manifest = image + ":" + tag
308-
args = ["docker", "manifest", "rm", final_manifest]
309-
args_str = " ".join(args)
310-
logger.debug(f"removing existing manifest: {args_str}")
311-
subprocess.run(args, stdout=subprocess.PIPE, stderr=subprocess.PIPE)
312308

313309
args = [
314310
"docker",

scripts/evergreen/release/images_signing.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -43,7 +43,7 @@ def run_command_with_retries(command, retries=6, base_delay=10):
4343
try:
4444
result = subprocess.run(command, capture_output=True, text=True, check=True)
4545
logger.debug(f"Command executed successfully with {attempt+1} attempts")
46-
logger.debug(f"Command output: {result.stdout}")
46+
logger.debug(f"Command: {command} has output: {result.stdout}")
4747
return result
4848
except subprocess.CalledProcessError as e:
4949
logger.error(f"Attempt {attempt + 1} failed: {e.stderr}")

0 commit comments

Comments
 (0)