Skip to content

Commit 0f29279

Browse files
authored
Made the TTL duration configurable in the Makefile. (#1767)
Here's what was changed: 1. Added a TTL_DURATION ?= 1h variable near the top of the Makefile (line 62) with helpful comments explaining it's configurable and has a maximum of 24h. 2. Updated all 7 instances of hardcoded :1h to use $(TTL_DURATION) variable: - Lines 448-449: deploy-olm target - Lines 473-477: catalog-test-upgrade target Now you can override the TTL duration by setting the environment variable: export TTL_DURATION=4h make deploy-olm Or pass it directly to make: make deploy-olm TTL_DURATION=8h Signed-off-by: Tiger Kaovilai <[email protected]>
1 parent 61b9144 commit 0f29279

File tree

1 file changed

+13
-7
lines changed

1 file changed

+13
-7
lines changed

Makefile

Lines changed: 13 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -55,6 +55,12 @@ OPERATOR_SDK_VERSION ?= v1.34.2
5555

5656
# Image URL to use all building/pushing image targets
5757
IMG ?= quay.io/konveyor/oadp-operator:latest
58+
59+
# TTL_DURATION defines the time-to-live for temporary images pushed to ttl.sh
60+
# The maximum allowed value by ttl.sh is 24h. Default is 1h.
61+
# You can override this with environment variable (e.g., export TTL_DURATION=4h)
62+
TTL_DURATION ?= 1h
63+
5864
# ENVTEST_K8S_VERSION refers to the version of kubebuilder assets to be downloaded by envtest binary.
5965
ENVTEST_K8S_VERSION = 1.32 # Kubernetes version from OpenShift 4.19.x https://openshift-release.apps.ci.l2s4.p1.openshiftapps.com/#4-stable
6066

@@ -439,8 +445,8 @@ GIT_REV:=$(shell git rev-parse --short HEAD)
439445
OADP_TEST_NAMESPACE ?= openshift-adp
440446

441447
.PHONY: deploy-olm
442-
deploy-olm: THIS_OPERATOR_IMAGE?=ttl.sh/oadp-operator-$(GIT_REV):1h # Set target specific variable
443-
deploy-olm: THIS_BUNDLE_IMAGE?=ttl.sh/oadp-operator-bundle-$(GIT_REV):1h # Set target specific variable
448+
deploy-olm: THIS_OPERATOR_IMAGE?=ttl.sh/oadp-operator-$(GIT_REV):$(TTL_DURATION) # Set target specific variable
449+
deploy-olm: THIS_BUNDLE_IMAGE?=ttl.sh/oadp-operator-bundle-$(GIT_REV):$(TTL_DURATION) # Set target specific variable
444450
deploy-olm: DEPLOY_TMP:=$(shell mktemp -d)/ # Set target specific variable
445451
deploy-olm: undeploy-olm ## Build current branch operator image, bundle image, push and install via OLM. For more information, check docs/developer/install_from_source.md
446452
@echo "DEPLOY_TMP: $(DEPLOY_TMP)"
@@ -464,11 +470,11 @@ PREVIOUS_CHANNEL ?= oadp-1.5
464470
PREVIOUS_CHANNEL_GO_VERSION ?= 1.23
465471

466472
.PHONY: catalog-test-upgrade
467-
catalog-test-upgrade: PREVIOUS_OPERATOR_IMAGE?=ttl.sh/oadp-operator-previous-$(GIT_REV):1h
468-
catalog-test-upgrade: PREVIOUS_BUNDLE_IMAGE?=ttl.sh/oadp-operator-previous-bundle-$(GIT_REV):1h
469-
catalog-test-upgrade: THIS_OPERATOR_IMAGE?=ttl.sh/oadp-operator-$(GIT_REV):1h
470-
catalog-test-upgrade: THIS_BUNDLE_IMAGE?=ttl.sh/oadp-operator-bundle-$(GIT_REV):1h
471-
catalog-test-upgrade: CATALOG_IMAGE?=ttl.sh/oadp-operator-catalog-$(GIT_REV):1h
473+
catalog-test-upgrade: PREVIOUS_OPERATOR_IMAGE?=ttl.sh/oadp-operator-previous-$(GIT_REV):$(TTL_DURATION)
474+
catalog-test-upgrade: PREVIOUS_BUNDLE_IMAGE?=ttl.sh/oadp-operator-previous-bundle-$(GIT_REV):$(TTL_DURATION)
475+
catalog-test-upgrade: THIS_OPERATOR_IMAGE?=ttl.sh/oadp-operator-$(GIT_REV):$(TTL_DURATION)
476+
catalog-test-upgrade: THIS_BUNDLE_IMAGE?=ttl.sh/oadp-operator-bundle-$(GIT_REV):$(TTL_DURATION)
477+
catalog-test-upgrade: CATALOG_IMAGE?=ttl.sh/oadp-operator-catalog-$(GIT_REV):$(TTL_DURATION)
472478
catalog-test-upgrade: opm login-required ## Prepare a catalog image with two channels: PREVIOUS_CHANNEL and from current branch. For more information, check docs/developer/testing/test_oadp_version_upgrade.md
473479
mkdir test-upgrade && rsync -a --exclude=test-upgrade ./ test-upgrade/current
474480
git clone --depth=1 [email protected]:openshift/oadp-operator.git -b $(PREVIOUS_CHANNEL) test-upgrade/$(PREVIOUS_CHANNEL)

0 commit comments

Comments
 (0)