search

operator-framework / operator-courier

Build, verify and push operators
Apache License 2.0
41 stars 53 forks source link

Operator-courier must not require that the quay repo name match the package name in the manifest #86

Closed ralphbean closed 5 years ago

ralphbean commented 5 years ago

See a3e0cb2769b3acc049d3fa2f6bec34747aa88e0c. That requires that the quay repo name we're pushing to matches the package name in the manifests.

This will break the certified-operators partner/ISV workflow. They push to repo names in quay that have a uuid appended on the end. They do this in order to guarantee that their system only pushes content that corresponds with the correct account. Of course, with the appended uuid, the quay repo name will never match the package manifest name.

This is a blocker for OMPS to upgrade to the next version of operator courier, which includes another critical change (see #85).

ralphbean commented 5 years ago

That change should either be reverted or made optional.

kevinrizza commented 5 years ago

@ralphbean This is a requirement for the OpenShift backend. Right now OLM and Marketplace will not work if these two fields do not match, which is why the requirement is here.

ralphbean commented 5 years ago

OK - closing this after discussion. Will file tickets to address this in RHC, OMPS, and COMET.