Skip to content

Commit a288bee

Browse files
committed
Remove CRDs from old locations
This is a cleanup commit to allow the verify-crd-compatibility target to continue working through the config consoldiation effort. This updates the verify-crd-compatibility recipe to target the new CRD locations and removes the CRDs from the old locations. Signed-off-by: Todd Short <[email protected]>
1 parent 2a5e9b8 commit a288bee

File tree

4 files changed

+8
-1040
lines changed

4 files changed

+8
-1040
lines changed

.github/workflows/crd-diff.yaml

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,5 @@ jobs:
1616
- name: Run make verify-crd-compatibility
1717
run: |
1818
make verify-crd-compatibility \
19-
CRD_DIFF_ORIGINAL_REF=${{ github.event.pull_request.base.sha }} \
20-
CRD_DIFF_UPDATED_SOURCE="git://${{ github.event.pull_request.head.sha }}?path=config/base/crd/bases/olm.operatorframework.io_clusterextensions.yaml" \
21-
CATALOGD_CRD_DIFF_UPDATED_SOURCE="git://${{ github.event.pull_request.head.sha }}?path=catalogd/config/base/crd/bases/olm.operatorframework.io_clustercatalogs.yaml"
19+
CRD_DIFF_ORIGINAL_REF="git://${{ github.event.pull_request.base.sha }}?path=" \
20+
CRD_DIFF_UPDATED_REF="git://${{ github.event.pull_request.head.sha }}?path="

Makefile

Lines changed: 6 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -151,15 +151,14 @@ bingo-upgrade: $(BINGO) #EXHELP Upgrade tools
151151
done
152152

153153
.PHONY: verify-crd-compatibility
154-
CRD_DIFF_ORIGINAL_REF := main
155-
CRD_DIFF_UPDATED_SOURCE := file://config/base/crd/bases/olm.operatorframework.io_clusterextensions.yaml
156-
CATALOGD_CRD_DIFF_UPDATED_SOURCE := file://catalogd/config/base/crd/bases/olm.operatorframework.io_clustercatalogs.yaml
154+
CRD_DIFF_ORIGINAL_REF := git://main?path=
155+
CRD_DIFF_UPDATED_REF := file://
156+
CRD_DIFF_OPCON_SOURCE := config/base/operator-controller/crd/bases/olm.operatorframework.io_clusterextensions.yaml
157+
CRD_DIFF_CATD_SOURCE := config/base/catalogd/crd/bases/olm.operatorframework.io_clustercatalogs.yaml
157158
CRD_DIFF_CONFIG := crd-diff-config.yaml
158-
159159
verify-crd-compatibility: $(CRD_DIFF) manifests
160-
$(CRD_DIFF) --config="${CRD_DIFF_CONFIG}" "git://${CRD_DIFF_ORIGINAL_REF}?path=config/base/crd/bases/olm.operatorframework.io_clusterextensions.yaml" ${CRD_DIFF_UPDATED_SOURCE}
161-
$(CRD_DIFF) --config="${CRD_DIFF_CONFIG}" "git://${CRD_DIFF_ORIGINAL_REF}?path=catalogd/config/base/crd/bases/olm.operatorframework.io_clustercatalogs.yaml" ${CATALOGD_CRD_DIFF_UPDATED_SOURCE}
162-
160+
$(CRD_DIFF) --config="${CRD_DIFF_CONFIG}" "${CRD_DIFF_ORIGINAL_REF}${CRD_DIFF_OPCON_SOURCE}" ${CRD_DIFF_UPDATED_REF}${CRD_DIFF_OPCON_SOURCE}
161+
$(CRD_DIFF) --config="${CRD_DIFF_CONFIG}" "${CRD_DIFF_ORIGINAL_REF}${CRD_DIFF_CATD_SOURCE}" ${CRD_DIFF_UPDATED_REF}${CRD_DIFF_CATD_SOURCE}
163162

164163
.PHONY: test
165164
test: manifests generate fmt lint test-unit test-e2e #HELP Run all tests.

0 commit comments

Comments
 (0)