From: Erik Brakkee <erik@brakkee.org>
Date: Sun, 17 Nov 2024 19:32:16 +0000 (+0100)
Subject: better error checking and better terminology
X-Git-Url: http://wamblee.org/gitweb/?a=commitdiff_plain;h=29ecaa9a100e3a835302242ff59966ac3acedaa8;p=pipelinelib

better error checking and better terminology
---

diff --git a/vars/buildcontainer.groovy b/vars/buildcontainer.groovy
index b1a01ca..0ff43fd 100644
--- a/vars/buildcontainer.groovy
+++ b/vars/buildcontainer.groovy
@@ -10,7 +10,7 @@ import java.rmi.dgc.VMID
 // - cache: Whether to use the cache for building containers. Defaults to true
 // - dockerfile: Name of the docker file in the context directory. Defaults to Dockerfile
 // - repo: Repository to publish container to. Defaults to the CONTAINER_REGISTRY environment variable
-// - version: Container version to build. Default to the value of the BRANCH_NAME variable 
+// - tag: Container tag to build. Default to $BRANCH_NAME-latest
 //
 def call(Map args) { 
   def defaults = [
@@ -20,7 +20,7 @@ def call(Map args) {
     dockerfile: 'Dockerfile',
     mirror: env.REGISTRY_MIRROR,
     repo: env.CONTAINER_REGISTRY,
-    version: env.BRANCH_NAME + "-latest"
+    tag: env.BRANCH_NAME + "-latest"
   ]
   args = defaults << args
   if (!args.image) {
@@ -39,7 +39,7 @@ def call(Map args) {
          --dockerfile ${args.dockerfile} \\
          --cache=${args.cache} --cache-ttl=${args.cachettl} \\
          --context \$(  pwd )/${args.context} \\
-         --destination ${args.repo}/${args.image}:${args.version}
+         --destination ${args.repo}/${args.image}:${args.tag}
       rm -rf /kaniko/*[0-9]* && rm -rf /kaniko/Dockerfile && mkdir -p /workspace
     """
   }