Merge pull request #27 from coreydaley/jira_ship_0004_build_env_vars

SHIP-0004: Build Environment Variables

Author: openshift-merge-robot

.github/workflows/e2e.yaml

@@ -24,15 +24,12 @@ jobs:
 
       - name: Check out code into the Go module directory
         uses: actions/checkout@v2
+        with:
+          submodules: recursive
 
       - name: Install Tools
         run: sudo apt-get update && sudo apt-get install -y make gcc
 
-      - name: Setup BATS
-        uses: mig4/setup-bats@v1
-        with:
-          bats-version: 1.2.1
-
       - name: Install kubectl
         uses: azure/setup-kubectl@v1
         with:

.gitmodules

@@ -0,0 +1,12 @@
+[submodule "test/e2e/bats/core"]
+	path = test/e2e/bats/core
+	url = https://github.com/bats-core/bats-core
+[submodule "test/e2e/bats/support"]
+	path = test/e2e/bats/support
+	url = https://github.com/bats-core/bats-support
+[submodule "test/e2e/bats/assert"]
+	path = test/e2e/bats/assert
+	url = https://github.com/bats-core/bats-assert
+[submodule "test/e2e/bats/file"]
+	path = test/e2e/bats/file
+	url = https://github.com/bats-core/bats-file

Makefile

@@ -49,9 +49,9 @@ test: test-unit test-e2e
 test-unit:
 	go test $(GO_FLAGS) $(GO_TEST_FLAGS) $(CMD) $(PKG) $(ARGS)
 
-# runs all end-to-end tests using bats, it assumes bats is installed
+# looks for *.bats files in the test/e2e directory and runs them
 test-e2e:
-	./test/e2e/e2e.bats
+	./test/e2e/bats/core/bin/bats --recursive test/e2e/*.bats
 
 # wait for KinD cluster to be on ready state, so tests can be performed
 verify-kind:

docs/testing.md

@@ -54,7 +54,22 @@ The install script waits for the Controller instance to be running.
 [BATS][batsCore] is a testing framework for Bash. It's structured as a regular script with enhanced
 syntax to define test cases, collect results, and more.
 
-To run BATS based tests, make sure you have `bats` installed, and then execute:
+The BATS core tool and it's supporting libraries are linked in this project as `git submodules`,
+alleviating the need to install BATS yourself.
+
+To run BATS based tests, make sure that you have cloned the project (including all submodules):
+
+```sh
+# Option 1: Clone the repository and populate submodules right away
+git clone --recurse-submodules --depth 1 https://github.com/shipwright-io/cli.git
+
+# Option 2: Populate submodules after cloning
+cd /your/project/directory/cli
+git submodule init
+git submodule update
+```
+
+then run:
 
 ```sh
 make test-e2e

go.mod

@@ -5,14 +5,14 @@ go 1.15
 require (
 	github.com/mailru/easyjson v0.7.1 // indirect
 	github.com/onsi/gomega v1.10.3
-	github.com/shipwright-io/build v0.5.2-0.20210715083206-5d8fb411a1eb
-	github.com/spf13/cobra v1.1.3
+	github.com/shipwright-io/build v0.6.0
+	github.com/spf13/cobra v1.2.1
 	github.com/spf13/pflag v1.0.5
 	github.com/texttheater/golang-levenshtein/levenshtein v0.0.0-20200805054039-cae8b0eaed6c
-	k8s.io/api v0.20.2
-	k8s.io/apimachinery v0.20.2
+	k8s.io/api v0.20.6
+	k8s.io/apimachinery v0.20.6
 	k8s.io/cli-runtime v0.20.2
-	k8s.io/client-go v0.20.2
+	k8s.io/client-go v0.20.6
 	k8s.io/utils v0.0.0-20210629042839-4a2b36d8d73f
 )
 

go.sum

@@ -4,13 +4,15 @@ import (
 	"fmt"
 
 	buildv1alpha1 "github.com/shipwright-io/build/pkg/apis/build/v1alpha1"
-	"github.com/shipwright-io/cli/pkg/shp/cmd/runner"
-	"github.com/shipwright-io/cli/pkg/shp/flags"
-	"github.com/shipwright-io/cli/pkg/shp/params"
 	"github.com/spf13/cobra"
 
 	metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
 	"k8s.io/cli-runtime/pkg/genericclioptions"
+
+	"github.com/shipwright-io/cli/pkg/shp/cmd/runner"
+	"github.com/shipwright-io/cli/pkg/shp/flags"
+	"github.com/shipwright-io/cli/pkg/shp/params"
+	"github.com/shipwright-io/cli/pkg/shp/util"
 )
 
 // CreateCommand contains data input from user
@@ -59,6 +61,13 @@ func (c *CreateCommand) Run(params *params.Params, io *genericclioptions.IOStrea
 		},
 		Spec: *c.buildSpec,
 	}
+
+	envs, err := c.cmd.Flags().GetStringArray("env")
+	if err != nil {
+		return err
+	}
+	b.Spec.Env = append(b.Spec.Env, util.StringSliceToEnvVarSlice(envs)...)
+
 	flags.SanitizeBuildSpec(&b.Spec)
 
 	clientset, err := params.ShipwrightClientSet()

pkg/shp/cmd/build/create.go

@@ -4,13 +4,15 @@ import (
 	"fmt"
 
 	buildv1alpha1 "github.com/shipwright-io/build/pkg/apis/build/v1alpha1"
-	"github.com/shipwright-io/cli/pkg/shp/cmd/runner"
-	"github.com/shipwright-io/cli/pkg/shp/flags"
-	"github.com/shipwright-io/cli/pkg/shp/params"
 	"github.com/spf13/cobra"
 
 	metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
 	"k8s.io/cli-runtime/pkg/genericclioptions"
+
+	"github.com/shipwright-io/cli/pkg/shp/cmd/runner"
+	"github.com/shipwright-io/cli/pkg/shp/flags"
+	"github.com/shipwright-io/cli/pkg/shp/params"
+	"github.com/shipwright-io/cli/pkg/shp/util"
 )
 
 // CreateCommand reprents the build's create subcommand.
@@ -60,6 +62,13 @@ func (c *CreateCommand) Run(params *params.Params, ioStreams *genericclioptions.
 		},
 		Spec: *c.buildRunSpec,
 	}
+
+	envs, err := c.cmd.Flags().GetStringArray("env")
+	if err != nil {
+		return err
+	}
+	br.Spec.Env = append(br.Spec.Env, util.StringSliceToEnvVarSlice(envs)...)
+
 	flags.SanitizeBuildRunSpec(&br.Spec)
 
 	clientset, err := params.ShipwrightClientSet()

pkg/shp/cmd/buildrun/create.go

@@ -3,6 +3,7 @@ package flags
 import (
 	buildv1alpha1 "github.com/shipwright-io/build/pkg/apis/build/v1alpha1"
 	"github.com/spf13/pflag"
+
 	corev1 "k8s.io/api/core/v1"
 	metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
 	"k8s.io/utils/pointer"
@@ -37,6 +38,7 @@ func BuildSpecFromFlags(flags *pflag.FlagSet) *buildv1alpha1.BuildSpec {
 	imageFlags(flags, "builder", spec.Builder)
 	imageFlags(flags, "output", &spec.Output)
 	timeoutFlags(flags, spec.Timeout)
+	envFlags(flags, spec.Env)
 
 	return spec
 }
@@ -56,6 +58,9 @@ func SanitizeBuildSpec(b *buildv1alpha1.BuildSpec) {
 		if b.Builder.Image == "" && b.Builder.Credentials == nil {
 			b.Builder = nil
 		}
+		if len(b.Env) == 0 {
+			b.Env = nil
+		}
 	}
 	if b.Timeout != nil && b.Timeout.Duration == 0 {
 		b.Timeout = nil

pkg/shp/flags/build.go

@@ -20,12 +20,14 @@ func BuildRunSpecFromFlags(flags *pflag.FlagSet) *buildv1alpha1.BuildRunSpec {
 		Output: &buildv1alpha1.Image{
 			Credentials: &corev1.LocalObjectReference{},
 		},
+		Env: []corev1.EnvVar{},
 	}
 
 	buildRefFlags(flags, spec.BuildRef)
 	serviceAccountFlags(flags, spec.ServiceAccount)
 	timeoutFlags(flags, spec.Timeout)
 	imageFlags(flags, "output", spec.Output)
+	envFlags(flags, spec.Env)
 
 	return spec
 }
@@ -52,4 +54,8 @@ func SanitizeBuildRunSpec(br *buildv1alpha1.BuildRunSpec) {
 	if br.Timeout != nil && br.Timeout.Duration == 0 {
 		br.Timeout = nil
 	}
+
+	if len(br.Env) == 0 {
+		br.Env = nil
+	}
 }