search

operator-framework / enhancements

Apache License 2.0
9 stars 40 forks source link

CRD Conversion Webhook Enhancement #30

Closed simrandhaliw closed 4 years ago

simrandhaliw commented 4 years ago

Enhancement to Add CRD Conversion Webhook Support for OLM

ecordell commented 4 years ago

Is there anything we need to be concerned about when updating a conversion webhook?

simrandhaliw commented 4 years ago

Is there anything we need to be concerned about when updating a conversion webhook?

No concerns as such, just that when an updated CSV is applied which doesn't have conversion webhook, we would remove its support.

simrandhaliw commented 4 years ago

@ecordell @awgreene @mhrivnak Any further comments before I merge this?

mhrivnak commented 4 years ago

@ecordell @awgreene @mhrivnak Any further comments before I merge this?

I have no additional feedback beyond my PDB advocacy.