Pass the builder type to verify jobs

The default builder type must be passed to packer verify jobs. For
packer jobs build on aws, set the `packer-builder: aws` in downstream
CI-management.

Change-Id: I599d167a957189584ef5b796c049d82e5a85ec7a
Signed-off-by: Anil Belur <abelur@linuxfoundation.org>

diff --git a/docs/jjb/lf-macros.rst b/docs/jjb/lf-macros.rst
index 1ea2cfa..955e224 100644 (file)
--- a/docs/jjb/lf-macros.rst
+++ b/docs/jjb/lf-macros.rst
@@ -127,6 +127,7 @@ Run `packer build` to build system images.
     :packer-version: Version of packer to use.
     :platform: Build platform as found in the vars directory.
     :template: Packer template to build as found in the templates directory.
+    :packer-builder: builder type to use. ex: aws, openstack (default: openstack)
 
 :Optional parameters:
 
@@ -145,6 +146,7 @@ Run ``packer validate`` to verify packer configuration.
     :packer-cloud-settings: Cloud configuration file. Loaded on the build
         server as CLOUDENV environment variable.
     :packer-version: Version of packer to use.
+    :packer-builder: builder type to use. ex: aws, openstack (default: openstack)
 
 lf-infra-push-gerrit-patch
 --------------------------

diff --git a/jjb/lf-ci-jobs.yaml b/jjb/lf-ci-jobs.yaml
index 39ebc8f..689892b 100644 (file)
--- a/jjb/lf-ci-jobs.yaml
+++ b/jjb/lf-ci-jobs.yaml
@@ -1781,6 +1781,7 @@
           openstack: "{openstack}"
           openstack-cloud: "{openstack-cloud}"
           packer-cloud-settings: "{packer-cloud-settings}"
+          packer-builder: "{packer-builder}"
           packer-version: "{packer-version}"
 
 - job-template:

diff --git a/jjb/lf-macros.yaml b/jjb/lf-macros.yaml
index f5021d2..9f74046 100644 (file)
--- a/jjb/lf-macros.yaml
+++ b/jjb/lf-macros.yaml
@@ -158,6 +158,7 @@
               variable: CLOUDENV
       - inject:
           properties-content: |
+            PACKER_BUILDER={packer-builder}
             PACKER_VERSION={packer-version}
       - shell: !include-raw-escape:
           - ../shell/packer-install.sh

diff --git a/releasenotes/notes/pass-packer-builder-type-to-verify-jobs-396e0f450143f6c1.yaml b/releasenotes/notes/pass-packer-builder-type-to-verify-jobs-396e0f450143f6c1.yaml
new file mode 100644 (file)
index 0000000..530506e
--- /dev/null
+++ b/releasenotes/notes/pass-packer-builder-type-to-verify-jobs-396e0f450143f6c1.yaml
@@ -0,0 +1,8 @@
+---
+fixes:
+  - |
+    Pass the builder type to verify jobs
+
+    The default builder type must be passed to packer verify jobs. For
+    packer jobs to run on aws, set the `packer-builder: aws` in downstream
+    CI-management repositories.

diff --git a/shell/packer-build.sh b/shell/packer-build.sh
index 1a44e4f..4cb3df3 100644 (file)
--- a/shell/packer-build.sh
+++ b/shell/packer-build.sh
@@ -21,6 +21,7 @@ PACKER_LOGS_DIR="$WORKSPACE/archives/packer"
 PACKER_BUILD_LOG="$PACKER_LOGS_DIR/packer-build.log"
 mkdir -p "$PACKER_LOGS_DIR"
 export PATH="${WORKSPACE}/bin:$PATH"
+export PACKER_BUILDER=${PACKER_BUILDER:-openstack}
 
 cd packer
 

diff --git a/shell/packer-validate.sh b/shell/packer-validate.sh
index 61e1431..af766c3 100644 (file)
--- a/shell/packer-validate.sh
+++ b/shell/packer-validate.sh
@@ -19,6 +19,7 @@ set -eu -o pipefail
 PACKER_LOGS_DIR="$WORKSPACE/archives/packer"
 mkdir -p "$PACKER_LOGS_DIR"
 export PATH="${WORKSPACE}/bin:$PATH"
+export PACKER_BUILDER=${PACKER_BUILDER:-openstack}
 
 cd packer
 varfiles=(vars/*.json common-packer/vars/*.json)