[make][pre-commit]Check CRD schema to avoid update issues#414
[make][pre-commit]Check CRD schema to avoid update issues#414mrkisaolamb wants to merge 1 commit intoopenstack-k8s-operators:mainfrom
Conversation
The new crd-schema-check make target compares the CRD schema of the patch with the schema on the tip of main and report errors on non backward compatible changes. This make target now also run in pre-commit both locally and in CI. This make target uses https://github.com/openshift/crd-schema-checker to do the actual checking. Related: OSPRH-11833
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: mrkisaolamb The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
|
this operator is no longer used. I think we can close it |
|
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/bf2afadb6b6e4d72996081a8038f0e46 ❌ openstack-k8s-operators-content-provider FAILURE in 6m 44s |
|
@mrkisaolamb: The following test failed, say
Full PR test history. Your PR dashboard. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here. |
jpodivin
left a comment
There was a problem hiding this comment.
Yes. We are no longer maintaining this operator and all functionality has been moved to openstack operator instead.
The new crd-schema-check make target compares the CRD schema of the patch with the schema on the tip of main and report errors on non backward compatible changes.
This make target now also run in pre-commit both locally and in CI.
This make target uses https://github.com/openshift/crd-schema-checker to do the actual checking.
Related: OSPRH-11833