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

Rename CatalogSource to OperatorCatalog #41

Closed
anik120 opened this issue Apr 13, 2023 · 3 comments · Fixed by #60
Closed

Rename CatalogSource to OperatorCatalog #41

anik120 opened this issue Apr 13, 2023 · 3 comments · Fixed by #60
Assignees

Comments

@anik120
Copy link
Collaborator

anik120 commented Apr 13, 2023

Motivation:

CatalogSource is a hangover from OLM v0.

"Here's how you can add this operator catalog to your cluster" sounds less confusing than "Here's how you can add this catalog source to your cluster", possibly since catalogSource is trying to encompass engineering implementation details, that our customers don't need to be exposed to.

Goal:

This is proposal to change the name CatalogSource to OperatorCatalog for catalogd.

@everettraven
Copy link
Collaborator

Is there a reason why we can't just call it Catalog?

@anik120
Copy link
Collaborator Author

anik120 commented Apr 24, 2023

Is there a reason why we can't just call it Catalog?

In my head it's beneficial to keep words in our products's names that are brand recognizable.
Catalog also feels very generic, it could be a catalog of furnitures that someone is talking about when they're using that in a conversation. Instead if you're using OperatorCatalog in the conversation, the context is always explicit, and reinforces the brand every time it's mentioned.

cc: @dmesser in case you want to chime in

@joelanford
Copy link
Member

I'm +1 on Catalog. The API group will include operatorframework.io

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

Successfully merging a pull request may close this issue.

3 participants