Skip to content

Commit 131ec63

Browse files
authored
Fix environment variable names to avoid clashing with environment variables in Jenkins (#527)
1 parent 0f698a7 commit 131ec63

File tree

2 files changed

+19
-18
lines changed

2 files changed

+19
-18
lines changed

Makefile.shared

Lines changed: 17 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -27,10 +27,11 @@ BASE_OTE_V2_IMAGE_TAG ?= 12.8.1.1.30
2727
###
2828
# component
2929
###
30-
BUILD_DIR := ./.build
31-
CHART_DIR := ./chart
32-
EXTRA_CHART_DIRS ?=
33-
CHART_BUILD_DIR := ${BUILD_DIR}/chart
30+
TARGET_DIR := ./.build
31+
CHART_TARGET_DIR := ${TARGET_DIR}/chart
32+
33+
CHART_SRC_DIR := ./chart
34+
EXTRA_CHART_SRC_DIR ?=
3435

3536
###
3637
# docker extra arguments
@@ -85,27 +86,27 @@ publish-image:
8586
# helm charts
8687
###
8788
build-chart-default:
88-
mkdir -p ${CHART_BUILD_DIR}
89-
cp -r ${CHART_DIR}/* ${CHART_BUILD_DIR}/
90-
@if [ -n "$(EXTRA_CHART_DIRS)" ]; then \
91-
cp $(EXTRA_CHART_DIRS) ${CHART_BUILD_DIR}/; \
89+
mkdir -p ${CHART_TARGET_DIR}
90+
cp -r ${CHART_SRC_DIR}/* ${CHART_TARGET_DIR}/
91+
@if [ -n "$(EXTRA_CHART_SRC_DIR)" ]; then \
92+
cp $(EXTRA_CHART_SRC_DIR) ${CHART_TARGET_DIR}/; \
9293
fi
93-
TAG=$(TAG) COMPONENT_NAME=$(COMPONENT_NAME) envsubst < ${CHART_BUILD_DIR}/Chart.yaml.template > ${CHART_BUILD_DIR}/Chart.yaml
94-
rm "${CHART_BUILD_DIR}/Chart.yaml.template"
95-
helm dependency update ${CHART_BUILD_DIR}
96-
helm package ${CHART_BUILD_DIR} --version ${TAG} --destination ${BUILD_DIR}
94+
TAG=$(TAG) COMPONENT_NAME=$(COMPONENT_NAME) envsubst < ${CHART_TARGET_DIR}/Chart.yaml.template > ${CHART_TARGET_DIR}/Chart.yaml
95+
rm "${CHART_TARGET_DIR}/Chart.yaml.template"
96+
helm dependency update ${CHART_TARGET_DIR}
97+
helm package ${CHART_TARGET_DIR} --version ${TAG} --destination ${TARGET_DIR}
9798

9899
lint-chart-default: build-chart
99100
@echo "Running chart linter for component: ${COMPONENT_NAME}"
100-
helm lint --with-subcharts ${CHART_BUILD_DIR}
101+
helm lint --with-subcharts ${CHART_TARGET_DIR}
101102

102103
publish-chart-default: build-chart
103104
@echo "Publishing chart for component: ${COMPONENT_NAME}"
104-
helm push ${BUILD_DIR}/${COMPONENT_NAME}-${TAG}.tgz oci://${CHARTS_REGISTRY}
105+
helm push ${TARGET_DIR}/${COMPONENT_NAME}-${TAG}.tgz oci://${CHARTS_REGISTRY}
105106

106107
clean:
107-
@echo "Removing: $(BUILD_DIR)"
108-
rm -rf $(BUILD_DIR)
108+
@echo "Removing: $(TARGET_DIR)"
109+
rm -rf $(TARGET_DIR)
109110
@if [ -n "$(CLEAN_DIRS)" ]; then \
110111
echo "Removing: $(CLEAN_DIRS)"; \
111112
rm -rf $(CLEAN_DIRS); \

platform/services/opa_bundle/Makefile

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,8 +6,8 @@ include ../../../Makefile.shared
66
COMPONENT_NAME=opa-bundle
77

88
OPA_BUILDER_IMAGE := openpolicyagent/opa:0.66.0
9-
BUNDLE_BUILD_DIR := ${BUILD_DIR}/bundle
10-
EXTRA_CHART_DIRS := ${BUNDLE_BUILD_DIR}/all.tar.gz
9+
BUNDLE_BUILD_DIR := ${TARGET_DIR}/bundle
10+
EXTRA_CHART_SRC_DIR := ${BUNDLE_BUILD_DIR}/all.tar.gz
1111

1212
prepare-policies:
1313
@echo "Preparing OPA policies"

0 commit comments

Comments
 (0)