Skip to content

Commit 2bb5647

Browse files
authored
Move registry login to script (#372)
1 parent 411250a commit 2bb5647

File tree

2 files changed

+15
-3
lines changed

2 files changed

+15
-3
lines changed

Makefile

+6-2
Original file line numberDiff line numberDiff line change
@@ -92,7 +92,8 @@ docker-publish: quay-login docker-build docker-push
9292
## This target is run by the master branch Jenkins Job
9393
.PHONY: build-push
9494
build-push: docker-publish
95-
CONTAINER_ENGINE="${CONTAINER_ENGINE}" \
95+
@echo "## Building bundle and catalog images..."
96+
@(CONTAINER_ENGINE="${CONTAINER_ENGINE}" \
9697
CONTAINER_ENGINE_CONFIG_DIR="${CONTAINER_ENGINE_CONFIG_DIR}" \
9798
CURRENT_COMMIT="${CURRENT_COMMIT}" \
9899
OLM_BUNDLE_IMAGE="${OLM_BUNDLE_IMAGE}" \
@@ -102,4 +103,7 @@ build-push: docker-publish
102103
OPERATOR_VERSION="${OPERATOR_VERSION}" \
103104
OPERATOR_IMAGE="${OPERATOR_IMAGE}" \
104105
OPERATOR_IMAGE_TAG="${OPERATOR_IMAGE_TAG}" \
105-
build/build_opm_catalog.sh
106+
IMAGE_REGISTRY=${IMAGE_REGISTRY} \
107+
REGISTRY_USER="${REGISTRY_USER}" \
108+
REGISTRY_TOKEN="${REGISTRY_TOKEN}" \
109+
build/build_opm_catalog.sh)

build/build_opm_catalog.sh

+9-1
Original file line numberDiff line numberDiff line change
@@ -40,6 +40,10 @@ function precheck_required_files() {
4040
return 0
4141
}
4242

43+
function login_image_registry() {
44+
${CONTAINER_ENGINE} login -u="${REGISTRY_USER}" -p="${REGISTRY_TOKEN}" ${IMAGE_REGISTRY}
45+
}
46+
4347
function prepare_temporary_folders() {
4448
BASE_FOLDER=$(mktemp -d --suffix "-$(basename "$0")")
4549
DIR_BUNDLE=$(mktemp -d -p "$BASE_FOLDER" bundle.XXXX)
@@ -87,6 +91,10 @@ function set_previous_operator_version() {
8791
}
8892

8993
function setup_environment() {
94+
log "Login Image registry"
95+
login_image_registry
96+
log " Successfully login to $IMAGE_REGISTRY"
97+
9098
log "Generating temporary folders to contain artifacts"
9199
prepare_temporary_folders
92100
log " base path: $BASE_FOLDER"
@@ -221,7 +229,7 @@ function tag_and_push_images() {
221229

222230
function main() {
223231
log "Building $OPERATOR_NAME version $OPERATOR_VERSION"
224-
232+
225233
precheck_required_files || return 1
226234

227235
setup_environment

0 commit comments

Comments
 (0)