New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
csi: No longer install the VolumeReplication CRDs from Rook #8845
Conversation
b81e7e8
to
6649f75
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You need to remove build/crds/crds.go
too, which should remove the module dep too.
2d60357
to
aa9ecac
Compare
images/ceph/Makefile
Outdated
@@ -104,6 +110,11 @@ generate-csv-templates: $(OPERATOR_SDK) $(YQ) ## Generate CSV templates for OLM | |||
@OLM_CATALOG_DIR=$(CSV_TEMPLATE_DIR)/cluster/olm/ceph ../../cluster/olm/ceph/generate-rook-csv-templates.sh | |||
@echo " === Generated CSV templates can be found at $(CSV_TEMPLATE_DIR)/cluster/olm/ceph/templates" | |||
|
|||
get-volume-replication-crds: | |||
@if [[ -z "$(CURL_TEMPLATE_DIR)" ]]; then echo "CURL_TEMPLATE_DIR is not set"; exit 1; fi | |||
curl $(VOL_REPL_URL)/$(VOLUME_REPLICATION_CRD) -o $(CURL_TEMPLATE_DIR)/$(VOLUME_REPLICATION_CRD) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
curl $(VOL_REPL_URL)/$(VOLUME_REPLICATION_CRD) -o $(CURL_TEMPLATE_DIR)/$(VOLUME_REPLICATION_CRD) | |
curl -L $(VOL_REPL_URL)/$(VOLUME_REPLICATION_CRD) -o $(CURL_TEMPLATE_DIR)/$(VOLUME_REPLICATION_CRD) |
Typically safer if we get a redirect.
1a0aa7e
to
5cc80bb
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As discussed, waiting for #8862
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Waiting for #8862 is a nice to have but shouldn't block the release.
Ok, will go ahead with this and get it included in v1.7.5 |
@Mergifyio rebase |
The volume replication CRDs are an external component, not owned by Rook. Therefore, they should be installed as any other independent component in case the admin will install other consumers of the volumereplication CRDs in the future in addition to Rook and the CSI driver. Signed-off-by: Travis Nielsen <tnielsen@redhat.com>
Command
|
5cc80bb
to
c420f23
Compare
csi: No longer install the VolumeReplication CRDs from Rook (backport #8845)
Description of your changes:
The volume replication CRDs are an external component, not owned by Rook. Therefore, they should be installed as any other independent component in case the admin will install other consumers of the volumereplication CRDs in the future in addition to Rook and the CSI driver.
The volume replication controller in the Ceph-CSI driver was already optional and disabled by default. To enable the volume replication controller there is just now an additional step to install the CRDs if the feature is to be enabled.
Since the CRDs are no longer included with Rook, downstream may need to find a new integration path for the CRDs.
Which issue is resolved by this Pull Request:
Resolves #8573
Checklist:
make codegen
) has been run to update object specifications, if necessary.