Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Follow-Up for #411: Should unavailable Catalogs be ignored during resolution? #428

Open
everettraven opened this issue Sep 21, 2023 · 2 comments

Comments

@everettraven
Copy link
Contributor

Mentioned in #411 (comment) by @joelanford:

Follow-up: we probably should not ignore unavailable catalogs. If we do, we introduce non-determinism to resolution.

If a Catalog that is needed for resolution is unavailable (that's all Catalogs right now because there's no way to disable catalogs globally or for a particular Operator), I think we should fail resolution, report the error, and then requeue when the Catalog changes state.

@everettraven
Copy link
Contributor Author

Including my response from the comment thread:

That means that all Operator installs have a dependency on all Catalog resources being available. If someone creates a Catalog resource with an improper image (i.e not an actual catalog image) that would result in being unable to install any Operators until that is remedied which feels like a bad UX IMO. The way I am viewing it is that at resolution time we only care about resolving based on the Catalogs that are available and have been successfully unpacked. If none of them provide what we need, fail resolution. The exception here is if a user specifies that an Operator should only be installed via a specific Catalog (which I'm not sure if that is currently possible).

@joelanford
Copy link
Member

Yeah, we need to think through implications. We likely need some way for an Operator to specify which catalogs should be used to resolve it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants