Skip to content

Commit 981cd34

Browse files
committed
Makefile,scripts: Remove references to handling OCP helm charts/manifests
- Remove the scripts/add_release_annotation.sh bash script as it only injects OCP-related annotations. - Remove any Makefile conditional checks for the OCP platform when generating release manifests. Signed-off-by: timflannagan <[email protected]>
1 parent d8c000a commit 981cd34

File tree

4 files changed

+5
-24
lines changed

4 files changed

+5
-24
lines changed

Makefile

+4-8
Original file line numberDiff line numberDiff line change
@@ -29,12 +29,12 @@ GO := GO111MODULE=on GOFLAGS="$(MOD_FLAGS)" go
2929
GINKGO := $(GO) run github.com/onsi/ginkgo/ginkgo
3030
BINDATA := $(GO) run github.com/go-bindata/go-bindata/v3/go-bindata
3131

32-
# ART builds are performed in dist-git, with content (but not commits) copied
32+
# ART builds are performed in dist-git, with content (but not commits) copied
3333
# from the source repo. Thus at build time if your code is inspecting the local
34-
# git repo it is getting unrelated commits and tags from the dist-git repo,
34+
# git repo it is getting unrelated commits and tags from the dist-git repo,
3535
# not the source repo.
36-
# For ART image builds, SOURCE_GIT_COMMIT, SOURCE_GIT_TAG, SOURCE_DATE_EPOCH
37-
# variables are inserted in Dockerfile to enable recovering the original git
36+
# For ART image builds, SOURCE_GIT_COMMIT, SOURCE_GIT_TAG, SOURCE_DATE_EPOCH
37+
# variables are inserted in Dockerfile to enable recovering the original git
3838
# metadata at build time.
3939
GIT_COMMIT := $(if $(SOURCE_GIT_COMMIT),$(SOURCE_GIT_COMMIT),$(shell git rev-parse HEAD))
4040

@@ -223,7 +223,6 @@ release: ver=$(shell cat OLM_VERSION)
223223
release: manifests
224224
docker pull quay.io/operator-framework/olm:v$(ver)
225225
$(MAKE) target=upstream ver=$(ver) quickstart=true package
226-
$(MAKE) target=ocp ver=$(ver) package
227226

228227
verify-release: release diff
229228

@@ -240,9 +239,6 @@ endif
240239
$(YQ_INTERNAL) w -i deploy/$(target)/values.yaml package.image.ref $(olmref)
241240
./scripts/package_release.sh $(ver) deploy/$(target)/manifests/$(ver) deploy/$(target)/values.yaml
242241
ln -sfFn ./$(ver) deploy/$(target)/manifests/latest
243-
ifeq ($(target), ocp)
244-
./scripts/add_release_annotation.sh deploy/$(target)/manifests/$(ver) "$(YQ_INTERNAL)"
245-
endif
246242
ifeq ($(quickstart), true)
247243
./scripts/package_quickstart.sh deploy/$(target)/manifests/$(ver) deploy/$(target)/quickstart/olm.yaml deploy/$(target)/quickstart/crds.yaml deploy/$(target)/quickstart/install.sh
248244
endif

scripts/add_release_annotation.sh

-13
This file was deleted.

scripts/copy_crds.sh

-1
Original file line numberDiff line numberDiff line change
@@ -12,4 +12,3 @@ for f in "${CRD_PATH}"/*.yaml ; do
1212
echo "copying ${f}"
1313
cp "${f}" "${SCRIPT_ROOT}/deploy/chart/crds/0000_50_olm_00-$(basename "$f" | sed 's/^.*_\([^.]\+\)\.yaml/\1.crd.yaml/')"
1414
done
15-

scripts/package_quickstart.sh

+1-2
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
#!/usr/bin/env bash
22

3-
if [[ ${#@} < 3 ]]; then
3+
if [[ ${#@} -lt 3 ]]; then
44
echo "Usage: $0 concatenate OLM's Kubernetes manifests into a single YAML stream and writes the result to a file"
55
echo "* dir: the input directory that contains OLM's Kubernetes manifests"
66
echo "* out: the output file for the combined OLM Kubernetes manifest"
@@ -32,4 +32,3 @@ done
3232
echo "Wrote manifests to ${out} and ${outcrds}"
3333

3434
cp scripts/install.sh ${outscript}
35-

0 commit comments

Comments
 (0)