Declarative Config: remove references to olm.gvk.provided, clarify lack of plural fields

joelanford commented 3 years ago

Background The EP originally added olm.gvk.provided for two reasons:

To make it easier to distinguish provided vs required GVKs (the existing olm.gvk property is ambiguous by itself)
To keep 100% backward-compatibility for the GRPC API. The olm.gvk property lacks a plural field, but the GRPC api.Bundle's provided APIs field contains a plural field that is populated when an sqlite DB is served.

Description This EP update removes olm.gvk.provided and clarifies handling of plural fields.

By adding the olm.gvk.required property there is now at least an implicit disambiguation between olm.gvk and olm.gvk.required.
OLM strips the plural field from the bundle's provided and required APIs upon receiving them, so it is no longer necessary to send them to OLM in the first place.

Signed-off-by: Joe Lanford joe.lanford@gmail.com

anik120 commented 3 years ago

/lgtm

kevinrizza commented 3 years ago

/approve

operator-framework / enhancements