better error checking and better terminology

diff --git a/vars/buildcontainer.groovy b/vars/buildcontainer.groovy
index b1a01cae3dd63c83c4f1877930292b58497b713b..0ff43fdd7334432cd8e29f2e2aa6d653d686fb5c 100644 (file)
--- 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
     """
   }