Store the bundle manifests in the bundle dir instead of deploy dir

Bundle directory is the standard location for storing the bundle manifests, metadata. Signed-off-by: Malay Kumar Parida <[email protected]>
red-hat-storage · Sep 27, 2023 · e0a156c · e0a156c
1 parent 8a75f6e
commit e0a156c
Show file tree

Hide file tree

Showing 42 changed files with 7 additions and 7 deletions.
diff --git a/Dockerfile.bundle b/Dockerfile.bundle
@@ -11,5 +11,5 @@ LABEL operators.operatorframework.io.metrics.mediatype.v1=metrics+v1
 LABEL operators.operatorframework.io.metrics.project_layout=go.kubebuilder.io/v3
 
 # Copy files to locations specified by labels.
-COPY deploy/ocs-operator/manifests /manifests/
-COPY deploy/ocs-operator/metadata /metadata/
+COPY bundle/manifests /manifests/
+COPY bundle/metadata /metadata/
diff --git a/deploy/ocs-operator/manifests/ceph-conf.yaml → bundle/manifests/ceph-conf.yaml b/deploy/ocs-operator/manifests/ceph-conf.yaml → bundle/manifests/ceph-conf.yaml
diff --git a/...operator/manifests/cephblockpool.crd.yaml → bundle/manifests/cephblockpool.crd.yaml b/...operator/manifests/cephblockpool.crd.yaml → bundle/manifests/cephblockpool.crd.yaml
diff --git a/...ests/cephblockpoolradosnamespace.crd.yaml → ...ests/cephblockpoolradosnamespace.crd.yaml b/...ests/cephblockpoolradosnamespace.crd.yaml → ...ests/cephblockpoolradosnamespace.crd.yaml
diff --git a/...manifests/cephbucketnotification.crd.yaml → ...manifests/cephbucketnotification.crd.yaml b/...manifests/cephbucketnotification.crd.yaml → ...manifests/cephbucketnotification.crd.yaml
diff --git a/...erator/manifests/cephbuckettopic.crd.yaml → bundle/manifests/cephbuckettopic.crd.yaml b/...erator/manifests/cephbuckettopic.crd.yaml → bundle/manifests/cephbuckettopic.crd.yaml
diff --git a/...cs-operator/manifests/cephclient.crd.yaml → bundle/manifests/cephclient.crd.yaml b/...cs-operator/manifests/cephclient.crd.yaml → bundle/manifests/cephclient.crd.yaml
diff --git a/...s-operator/manifests/cephcluster.crd.yaml → bundle/manifests/cephcluster.crd.yaml b/...s-operator/manifests/cephcluster.crd.yaml → bundle/manifests/cephcluster.crd.yaml
diff --git a/...perator/manifests/cephcosidriver.crd.yaml → bundle/manifests/cephcosidriver.crd.yaml b/...perator/manifests/cephcosidriver.crd.yaml → bundle/manifests/cephcosidriver.crd.yaml
diff --git a/...perator/manifests/cephfilesystem.crd.yaml → bundle/manifests/cephfilesystem.crd.yaml b/...perator/manifests/cephfilesystem.crd.yaml → bundle/manifests/cephfilesystem.crd.yaml
diff --git a/...r/manifests/cephfilesystemmirror.crd.yaml → ...e/manifests/cephfilesystemmirror.crd.yaml b/...r/manifests/cephfilesystemmirror.crd.yaml → ...e/manifests/cephfilesystemmirror.crd.yaml
diff --git a/...sts/cephfilesystemsubvolumegroup.crd.yaml → ...sts/cephfilesystemsubvolumegroup.crd.yaml b/...sts/cephfilesystemsubvolumegroup.crd.yaml → ...sts/cephfilesystemsubvolumegroup.crd.yaml
diff --git a/...y/ocs-operator/manifests/cephnfs.crd.yaml → bundle/manifests/cephnfs.crd.yaml b/...y/ocs-operator/manifests/cephnfs.crd.yaml → bundle/manifests/cephnfs.crd.yaml
diff --git a/...erator/manifests/cephobjectrealm.crd.yaml → bundle/manifests/cephobjectrealm.crd.yaml b/...erator/manifests/cephobjectrealm.crd.yaml → bundle/manifests/cephobjectrealm.crd.yaml
diff --git a/...erator/manifests/cephobjectstore.crd.yaml → bundle/manifests/cephobjectstore.crd.yaml b/...erator/manifests/cephobjectstore.crd.yaml → bundle/manifests/cephobjectstore.crd.yaml
diff --git a/...or/manifests/cephobjectstoreuser.crd.yaml → ...le/manifests/cephobjectstoreuser.crd.yaml b/...or/manifests/cephobjectstoreuser.crd.yaml → ...le/manifests/cephobjectstoreuser.crd.yaml
diff --git a/...perator/manifests/cephobjectzone.crd.yaml → bundle/manifests/cephobjectzone.crd.yaml b/...perator/manifests/cephobjectzone.crd.yaml → bundle/manifests/cephobjectzone.crd.yaml
diff --git a/...or/manifests/cephobjectzonegroup.crd.yaml → ...le/manifests/cephobjectzonegroup.crd.yaml b/...or/manifests/cephobjectzonegroup.crd.yaml → ...le/manifests/cephobjectzonegroup.crd.yaml
diff --git a/...operator/manifests/cephrbdmirror.crd.yaml → bundle/manifests/cephrbdmirror.crd.yaml b/...operator/manifests/cephrbdmirror.crd.yaml → bundle/manifests/cephrbdmirror.crd.yaml
diff --git a/...ocs-operator/manifests/consumer-role.yaml → bundle/manifests/consumer-role.yaml b/...ocs-operator/manifests/consumer-role.yaml → bundle/manifests/consumer-role.yaml
diff --git a/...ator/manifests/consumer-role_binding.yaml → bundle/manifests/consumer-role_binding.yaml b/...ator/manifests/consumer-role_binding.yaml → bundle/manifests/consumer-role_binding.yaml
diff --git a/...y/ocs-operator/manifests/consumer-sa.yaml → bundle/manifests/consumer-sa.yaml b/...y/ocs-operator/manifests/consumer-sa.yaml → bundle/manifests/consumer-sa.yaml
diff --git a/...ator/manifests/exporter-service-role.yaml → bundle/manifests/exporter-service-role.yaml b/...ator/manifests/exporter-service-role.yaml → bundle/manifests/exporter-service-role.yaml
diff --git a/...ifests/exporter-service-role_binding.yaml → ...ifests/exporter-service-role_binding.yaml b/...ifests/exporter-service-role_binding.yaml → ...ifests/exporter-service-role_binding.yaml
diff --git a/...s/ocs-operator.clusterserviceversion.yaml → ...s/ocs-operator.clusterserviceversion.yaml b/...s/ocs-operator.clusterserviceversion.yaml → ...s/ocs-operator.clusterserviceversion.yaml
diff --git a/...ator/manifests/ocsinitialization.crd.yaml → bundle/manifests/ocsinitialization.crd.yaml b/...ator/manifests/ocsinitialization.crd.yaml → bundle/manifests/ocsinitialization.crd.yaml
diff --git a/...ocs-operator/manifests/provider-role.yaml → bundle/manifests/provider-role.yaml b/...ocs-operator/manifests/provider-role.yaml → bundle/manifests/provider-role.yaml
diff --git a/...ator/manifests/provider-role_binding.yaml → bundle/manifests/provider-role_binding.yaml b/...ator/manifests/provider-role_binding.yaml → bundle/manifests/provider-role_binding.yaml
diff --git a/...y/ocs-operator/manifests/provider-sa.yaml → bundle/manifests/provider-sa.yaml b/...y/ocs-operator/manifests/provider-sa.yaml → bundle/manifests/provider-sa.yaml
diff --git a/...operator/manifests/rook-metrics-role.yaml → bundle/manifests/rook-metrics-role.yaml b/...operator/manifests/rook-metrics-role.yaml → bundle/manifests/rook-metrics-role.yaml
diff --git a/.../manifests/rook-metrics-role_binding.yaml → .../manifests/rook-metrics-role_binding.yaml b/.../manifests/rook-metrics-role_binding.yaml → .../manifests/rook-metrics-role_binding.yaml
diff --git a/...ator/manifests/rook-monitor-mgr-role.yaml → bundle/manifests/rook-monitor-mgr-role.yaml b/...ator/manifests/rook-monitor-mgr-role.yaml → bundle/manifests/rook-monitor-mgr-role.yaml
diff --git a/...ifests/rook-monitor-mgr-role_binding.yaml → ...ifests/rook-monitor-mgr-role_binding.yaml b/...ifests/rook-monitor-mgr-role_binding.yaml → ...ifests/rook-monitor-mgr-role_binding.yaml
diff --git a/...operator/manifests/rook-monitor-role.yaml → bundle/manifests/rook-monitor-role.yaml b/...operator/manifests/rook-monitor-role.yaml → bundle/manifests/rook-monitor-role.yaml
diff --git a/.../manifests/rook-monitor-role_binding.yaml → .../manifests/rook-monitor-role_binding.yaml b/.../manifests/rook-monitor-role_binding.yaml → .../manifests/rook-monitor-role_binding.yaml
diff --git a/...or/manifests/storageclassrequest.crd.yaml → ...le/manifests/storageclassrequest.crd.yaml b/...or/manifests/storageclassrequest.crd.yaml → ...le/manifests/storageclassrequest.crd.yaml
diff --git a/...perator/manifests/storagecluster.crd.yaml → bundle/manifests/storagecluster.crd.yaml b/...perator/manifests/storagecluster.crd.yaml → bundle/manifests/storagecluster.crd.yaml
diff --git a/...erator/manifests/storageconsumer.crd.yaml → bundle/manifests/storageconsumer.crd.yaml b/...erator/manifests/storageconsumer.crd.yaml → bundle/manifests/storageconsumer.crd.yaml
diff --git a/...oy/ocs-operator/metadata/annotations.yaml → bundle/metadata/annotations.yaml b/...oy/ocs-operator/metadata/annotations.yaml → bundle/metadata/annotations.yaml
diff --git a/hack/common.sh b/hack/common.sh
@@ -41,7 +41,7 @@ GINKGO_TEST_SUITE="${GINKGO_TEST_SUITE:-ocs}"
 # This env var allows developers to point to a custom oc tool that isn't in $PATH
 # defaults to just using the 'oc' binary provided in $PATH
 OCS_OC_PATH="${OCS_OC_PATH:-oc}"
-OCS_FINAL_DIR="deploy/ocs-operator/manifests"
+BUNDLE_MANIFESTS_DIR="bundle/manifests"
 EXTRA_MANIFESTS_DIR="deploy/extra-manifests"
 
 ROOK_CSV="$OUTDIR_TEMPLATES/rook-csv.yaml.in"

diff --git a/hack/generate-unified-csv.sh b/hack/generate-unified-csv.sh
@@ -64,6 +64,6 @@ $CSV_MERGER \
 	--ocs-must-gather-image="$OCS_MUST_GATHER_IMAGE" \
 	--crds-directory="$OUTDIR_CRDS" \
 	--manifests-directory=$EXTRA_MANIFESTS_DIR \
-	--olm-bundle-directory="$OCS_FINAL_DIR" \
+	--olm-bundle-directory="$BUNDLE_MANIFESTS_DIR" \
 	--timestamp="$TIMESTAMP" \
 	--rook-csiaddons-image="$ROOK_CSIADDONS_IMAGE"
diff --git a/hack/source-manifests.sh b/hack/source-manifests.sh
@@ -56,7 +56,7 @@ function dump_rook_csv() {
 # This is the base CSV everything else gets merged into later on.
 function gen_ocs_csv() {
 	echo "Generating OpenShift Container Storage CSV"
-	rm -rf "$(dirname $OCS_FINAL_DIR)"
+	rm -rf "$(dirname $BUNDLE_MANIFESTS_DIR)"
 	ocs_crds_outdir="$OUTDIR_CRDS/ocs"
 	rm -rf $OUTDIR_TEMPLATES/manifests/ocs-operator.clusterserviceversion.yaml
 	rm -rf $OCS_CSV
@@ -66,8 +66,8 @@ function gen_ocs_csv() {
 	pushd config/manager
 	$KUSTOMIZE edit set image ocs-dev/ocs-operator="$OCS_IMAGE"
 	popd
-	$KUSTOMIZE build config/manifests | $OPERATOR_SDK generate bundle -q --overwrite=false --output-dir deploy/ocs-operator--package ocs-operator --version "$CSV_VERSION" --extra-service-accounts=ocs-metrics-exporter
-	mv deploy/ocs-operator/manifests/*clusterserviceversion.yaml $OCS_CSV
+	$KUSTOMIZE build config/manifests | $OPERATOR_SDK generate bundle -q --package ocs-operator --version "$CSV_VERSION" --extra-service-accounts=ocs-metrics-exporter
+	mv $BUNDLE_MANIFESTS_DIR/*clusterserviceversion.yaml $OCS_CSV
 	cp config/crd/bases/* $ocs_crds_outdir
 }