Change in docker-playground[master]: jenkins-common.sh: fix redundant titan image build

This is merely a historical archive of years 2008-2021, before the migration to mailman3.

A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/gerrit-log@lists.osmocom.org/.

osmith gerrit-no-reply at lists.osmocom.org
Mon Dec 6 09:20:45 UTC 2021


osmith has submitted this change. ( https://gerrit.osmocom.org/c/docker-playground/+/26442 )

Change subject: jenkins-common.sh: fix redundant titan image build
......................................................................

jenkins-common.sh: fix redundant titan image build

Do not build debian-stretch-titan, when registry.osmocom.org is enabled.
The image that would be built at this point is not useful, since other
images like ttcn3-msc-test will use the debian-stretch-titan image from
the registry instead of the one that was just built locally.

Related: OS#5336
Change-Id: I7127e3ebac3a6a985c3ba50ba8c7cb8c5de978d9
---
M jenkins-common.sh
1 file changed, 10 insertions(+), 0 deletions(-)

Approvals:
  osmith: Verified
  fixeria: Looks good to me, but someone else must approve
  laforge: Looks good to me, approved



diff --git a/jenkins-common.sh b/jenkins-common.sh
index d4258d2..88d053c 100644
--- a/jenkins-common.sh
+++ b/jenkins-common.sh
@@ -120,6 +120,16 @@
 	local dir
 
 	for i in $@; do
+		# Don't build images that are available on the private
+		# registry, if using it.
+		if [ "$REGISTRY_HOST" = "registry.osmocom.org" ]; then
+			case "$i" in
+			debian-stretch-titan)
+				continue
+				;;
+			esac
+		fi
+
 		# Build dependencies first
 		depends="$(docker_depends "$i")"
 		if [ -n "$depends" ]; then

-- 
To view, visit https://gerrit.osmocom.org/c/docker-playground/+/26442
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings

Gerrit-Project: docker-playground
Gerrit-Branch: master
Gerrit-Change-Id: I7127e3ebac3a6a985c3ba50ba8c7cb8c5de978d9
Gerrit-Change-Number: 26442
Gerrit-PatchSet: 1
Gerrit-Owner: osmith <osmith at sysmocom.de>
Gerrit-Reviewer: fixeria <vyanitskiy at sysmocom.de>
Gerrit-Reviewer: laforge <laforge at osmocom.org>
Gerrit-Reviewer: osmith <osmith at sysmocom.de>
Gerrit-MessageType: merged
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20211206/89e37113/attachment.htm>


More information about the gerrit-log mailing list