Support crd.versions field

operator-framework / operator-courier

Build, verify and push operators

Apache License 2.0

41 stars 53 forks source link

Support crd.versions field #165

Closed kevinrizza closed 4 years ago

kevinrizza commented 4 years ago

Update the verify command to respect the latest crd spec with regards to the deprecation of the version field and replacement with spec.versions

awgreene commented 4 years ago

This will LGTM once @gallettilance's comments are addressed.

kevinrizza commented 4 years ago

I think we may need to change the ui validate stuff too (here) and in the const.py file where version is required - this could be breaking if the field isn't there.

@gallettilance All of that stuff is about the spec.version of the CSV, not the CRD. I believe I have updated all the references to the CRD spec.version.

gallettilance commented 4 years ago

/lgtm

awgreene commented 4 years ago

/approve