Skip to content

Commit f0308d2

Browse files
authored
Merge pull request #925 from BenTheElder/push-tar
CI: only push tar instead of also container image
2 parents 34e60f8 + 4e0b915 commit f0308d2

File tree

1 file changed

+5
-2
lines changed

1 file changed

+5
-2
lines changed

test/build.sh

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -128,7 +128,7 @@ function build-pr() {
128128
export REGISTRY="${NPD_STAGING_REGISTRY}/pr/${PR}"
129129
export VERSION=$(get-version)
130130
export TAG="${VERSION}"
131-
make push
131+
make push-tar
132132
write-env-file ${PR_ENV_FILENAME}
133133
}
134134

@@ -138,7 +138,10 @@ function build-ci() {
138138
export REGISTRY="${NPD_STAGING_REGISTRY}/ci"
139139
export VERSION="$(get-version)-$(date +%Y%m%d.%H%M)"
140140
export TAG="${VERSION}"
141-
make push
141+
# e2e tests consume the tarball, not the container
142+
# this is simpler to manage in the infra, and we still ensure the container
143+
# build works locally
144+
make push-tar build-container
142145

143146
# Create the env file with and without custom flags at the same time.
144147
build-npd-custom-flags

0 commit comments

Comments
 (0)