osmith submitted this change.

View Change

Approvals: Jenkins Builder: Verified pespin: Looks good to me, but someone else must approve fixeria: Looks good to me, approved
obs: lib.docker: fix redundant get_image_name()

Related: OS#2385
Change-Id: Ifa4e316db63c1a3bb9e0d845e7f755d04cf1297e
---
M scripts/obs/lib/docker.py
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/scripts/obs/lib/docker.py b/scripts/obs/lib/docker.py
index 0c5ec0f..2944cb3 100644
--- a/scripts/obs/lib/docker.py
+++ b/scripts/obs/lib/docker.py
@@ -20,7 +20,7 @@
lib.run_cmd(["docker", "build",
"--build-arg", f"DISTRO={distro}",
"--build-arg", f"UID={os.getuid()}",
- "-t", get_image_name(distro, image_type),
+ "-t", image_name,
"-f", f"{lib.config.path_top}/data/{image_type}.Dockerfile",
f"{lib.config.path_top}/data"])


1 is the latest approved patch-set. No files were changed between the latest approved patch-set and the submitted one.

To view, visit change 29600. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: osmo-ci
Gerrit-Branch: master
Gerrit-Change-Id: Ifa4e316db63c1a3bb9e0d845e7f755d04cf1297e
Gerrit-Change-Number: 29600
Gerrit-PatchSet: 2
Gerrit-Owner: osmith <osmith@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: osmith <osmith@sysmocom.de>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>
Gerrit-MessageType: merged