From 4e0b9150b9e4dd09897c885d2f006a81b9e34fb2 Mon Sep 17 00:00:00 2001 From: Benjamin Elder Date: Mon, 8 Jul 2024 13:17:04 -0700 Subject: [PATCH] CI: build container and push tar, PR: push tar We only need the tar to run CI tests, but we should also test building the container. We release the container and binaries independently of this, this script is for e2e tests. --- test/build.sh | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/test/build.sh b/test/build.sh index 1af86bf58..544d0af60 100755 --- a/test/build.sh +++ b/test/build.sh @@ -128,7 +128,7 @@ function build-pr() { export REGISTRY="${NPD_STAGING_REGISTRY}/pr/${PR}" export VERSION=$(get-version) export TAG="${VERSION}" - make push + make push-tar write-env-file ${PR_ENV_FILENAME} } @@ -138,7 +138,10 @@ function build-ci() { export REGISTRY="${NPD_STAGING_REGISTRY}/ci" export VERSION="$(get-version)-$(date +%Y%m%d.%H%M)" export TAG="${VERSION}" - make push + # e2e tests consume the tarball, not the container + # this is simpler to manage in the infra, and we still ensure the container + # build works locally + make push-tar build-container # Create the env file with and without custom flags at the same time. build-npd-custom-flags