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

build(csv): add initialization-resource annotation #268

Merged
merged 2 commits into from
Sep 17, 2021

Conversation

ebaron
Copy link
Member

@ebaron ebaron commented Sep 17, 2021

This adds an annotation to our CSV that prompts the user to create a Cryostat CR after they install the operator. Unfortunately, there doesn't seem to be any Operator SDK integration to generate this for us, so we'll have to update it manually. The sample CR I included only sets minimal which is required, and enableCertManager which we want to explicitly set to true.

Visiting the Installed Operator after using deploy_bundle:
Screenshot 2021-09-17 at 15-40-08 cryostat-operator v2 1 0-dev · Details · Red Hat OpenShift Container Platform

After uninstalling the operator and reinstalling from the CatalogSource. A similar workflow to installing from OperatorHub:
Screenshot 2021-09-17 at 15-41-26 Installing Operator · Red Hat OpenShift Container Platform

Test bundle image for convenience: quay.io/ebaron/cryostat-operator-bundle:init-res

Fixes #257

@andrewazores andrewazores added the chore Refactor, rename, cleanup, etc. label Sep 17, 2021
@andrewazores andrewazores added feat New feature or request and removed chore Refactor, rename, cleanup, etc. labels Sep 17, 2021
@ebaron ebaron merged commit 9e9b0a9 into cryostatio:main Sep 17, 2021
mergify bot pushed a commit that referenced this pull request Sep 17, 2021
ebaron added a commit that referenced this pull request Sep 17, 2021
(cherry picked from commit 9e9b0a9)

Co-authored-by: Elliott Baron <ebaron@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport feat New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add operatorframework.io/initialization-resource annotation to CSV
2 participants