manifests: Update the CRD definitions with more accurate docs#225
Conversation
This refreshes our CRD schema with the latest copies from the 4.2 streams. Docs are regenerated from tooling in cluster-config-operator and merged with the current definition.
|
/assign @abhinavdahiya Bug incoming. I figure we'll potentially want to script this like cluster-config-operator in the future, it's not quite there yet. |
|
Hrm. I need to check whether the schema is too restrictive. It looks like some of the optional fields are optional. |
|
/retest |
|
@smarterclayton |
|
No it doesn't, but I'll have to dig in why later. I think the version is too old. Suggest we merge this, then I'll dig in once we get closer to feature freeze. /retest |
/lgtm |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: abhinavdahiya, smarterclayton The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
This refreshes our CRD schema with the latest copies from the 4.2
streams. Docs are regenerated from tooling in cluster-config-operator
and merged with the current definition.
crd-schema-gen --apis-dir vendor/github.com/openshift/api/config/v1/ -output-dir manifests