Closed joelanford closed 4 months ago
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: joelanford
The full list of commands accepted by this bot can be found here.
The pull request process is described here
@joelanford: PR needs rebase.
closing due to inactivity - please re-open if needed ^^
Description: Updates
kubectl operator install
to honor CSVs that provide anoperatorframework.io/suggested-namespace
annotation.Motivation: Closes #50
/hold I can't find any upstream documentation for
operatorframework.io/suggested-namespace
, so I'm somewhat concerned that we're adding support for a downstream feature. There's nothing actually OpenShift-specific with this annotation other than that OpenShift Console is the only current client that supports it. I think this would be as simple as a doc change to upstream OLM docs that specifies this annotation and describes what clients should do when it's present.Downstream enhancement describes this annotation here: https://github.com/openshift/enhancements/blob/master/enhancements/olm/olm-managed-operator-metrics.md#fulfilling-namespace-and-rbac-requirements