From 21c32c8c609a87c3d84b2bfb5e00e415baddb98e Mon Sep 17 00:00:00 2001 From: Bogdan Dobrelya Date: Wed, 8 Apr 2020 15:48:14 +0200 Subject: [PATCH] Process labels before building container run args The consistent way of building container run arguments is processing labels (possibly multiple) first. Fix 'debug' actions 'run' and 'print-cmd' to not falling behind of that pattern already proved working well for containers 'apply'. Change-Id: I771d086cc75695d7ce2db35c852bb35bb4c59708 Related-Bug: #1798362 Signed-off-by: Bogdan Dobrelya --- paunch/__init__.py | 2 ++ 1 file changed, 2 insertions(+) diff --git a/paunch/__init__.py b/paunch/__init__.py index a0869f1..eff00ff 100644 --- a/paunch/__init__.py +++ b/paunch/__init__.py @@ -197,6 +197,7 @@ def debug(config_id, container_name, action, config, managed_by, labels=None, '--name', uname ] + builder.label_arguments(cmd, container_name) builder.container_run_args(cmd, container_name, uname) if '--health-cmd' in cmd: @@ -219,6 +220,7 @@ def debug(config_id, container_name, action, config, managed_by, labels=None, '--name', uname ] + builder.label_arguments(cmd, container_name) if builder.container_run_args(cmd, container_name, uname): return r.execute_interactive(cmd, log) elif action == 'dump-yaml':