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

doc: add source info into README #452

Merged
merged 2 commits into from
Apr 14, 2022
Merged

doc: add source info into README #452

merged 2 commits into from
Apr 14, 2022

Conversation

camilamacedo86
Copy link
Contributor

@camilamacedo86 camilamacedo86 commented Nov 30, 2021

Description
Update the readme with the info where the users can find the catalog

Motivation
Closes; #382

README.md Outdated Show resolved Hide resolved
@adambkaplan
Copy link
Contributor

Has the image been moved to catalog_sa? The current catalog image in fact breaks when used with opm index --add.

See ae29062#diff-1b158839f8aa339e89bb50b1603e90ec68e0fd5565fe10a7cb32729fddf7070a

Side note - I could not find a pull request where this config change was discussed. @mvalarh is there a PR or issue that documents why this change was made?

@camilamacedo86
Copy link
Contributor Author

Hi @adambkaplan,

Could you please raise an issue about the scenario of your comment #452 (comment) that does not seem to have relation with the scope of this PR.

@adambkaplan
Copy link
Contributor

@camilamacedo86 the issue regarding opm index is discussed in operator-framework/operator-registry#870 (so not directly related to this community). Will continue discussing the issue there.

Co-authored-by: J0zi <[email protected]>
@mvalarh mvalarh closed this Feb 1, 2022
@mvalarh mvalarh reopened this Feb 1, 2022
@J0zi J0zi merged commit ac3ddec into k8s-operatorhub:main Apr 14, 2022
@camilamacedo86 camilamacedo86 deleted the update-readme-source-info branch April 14, 2022 09:58
jmesnil pushed a commit to jmesnil/community-operators that referenced this pull request Jul 26, 2022
* doc: add source info into README

Signed-off-by: Camila Macedo <[email protected]>

* Update README.md

Co-authored-by: J0zi <[email protected]>

Co-authored-by: J0zi <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

4 participants