Skip to content

Commit

Permalink
Merge pull request #925 from BenTheElder/push-tar
Browse files Browse the repository at this point in the history
CI: only push tar instead of also container image
  • Loading branch information
k8s-ci-robot authored Jul 8, 2024
2 parents 34e60f8 + 4e0b915 commit f0308d2
Showing 1 changed file with 5 additions and 2 deletions.
7 changes: 5 additions & 2 deletions test/build.sh
Original file line number Diff line number Diff line change
Expand Up @@ -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}
}

Expand All @@ -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
Expand Down

0 comments on commit f0308d2

Please sign in to comment.