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

ROSA scheduled run has an issue with Cryostat certificates during deploy #894

Open
ahus1 opened this issue Jul 15, 2024 · 7 comments
Open

Comments

@ahus1
Copy link
Contributor

ahus1 commented Jul 15, 2024

[keycloak:install-keycloak] Error: UPGRADE FAILED: failed to create resource: Internal error occurred: Internal error occurred: conversion webhook for operator.cryostat.io/v1beta1, Kind=Cryostat failed: Post "https://cryostat-operator-controller-manager-service.openshift-operators.svc:443/convert?timeout=30s": tls: failed to verify certificate: x509: certificate is valid for infinispan-operator-controller-manager-service.openshift-operators, infinispan-operator-controller-manager-service.openshift-operators.svc, not cryostat-operator-controller-manager-service.openshift-operators.svc

@ahus1
Copy link
Contributor Author

ahus1 commented Jul 15, 2024

@pruivo / @kami619 / @mhajas / @andyuk1986 / @mhajas - do we want to keep Cryostat? It seems we didn't use it so much recently. We might decide to remove it completely?

@kami619
Copy link
Contributor

kami619 commented Jul 18, 2024

I vote to deprecate it, we didn't use it for JVM analysis in the last 6 months.

@ahus1
Copy link
Contributor Author

ahus1 commented Jul 18, 2024

@ryanemerson - pinging you as well. Unfortunately I forgot to mention you above. WDYT?

@ryanemerson
Copy link
Contributor

ryanemerson commented Jul 18, 2024

I think the root cause of the issue is the Cryostat 3.0.0 release. To solve the issue with this version I think we need to migrate to a new CR version.

I would propose that we just pin the Cryostat version to v2.4.0 in our Subscription. That way we still have Cryostat if we need it at some point, but we don't have the pain of figuring out the migration. Then in the future if v2.4.0 is no longer available, we can revisit the decision to migrate/drop depending on usage.

That being said, I'm also happy to drop it as I haven't had a need for it so far.

@ahus1
Copy link
Contributor Author

ahus1 commented Jul 31, 2024

Created an upstream issue, should be fixed soon: cryostatio/cryostat-operator#924

@ebaron
Copy link

ebaron commented Aug 28, 2024

Hi all, we've backported a fix for this issue to our 3.0.x branch. We'll prioritize getting a Cryostat 3.0.1 release out soon for you to use.

@ebaron
Copy link

ebaron commented Sep 23, 2024

Hi all, we've backported a fix for this issue to our 3.0.x branch. We'll prioritize getting a Cryostat 3.0.1 release out soon for you to use.

Hi, just wanted to let you know that the 3.0.1 release is out now and should fix this bug you've been experiencing. Feel free to reach out if you run into any other issues.

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

4 participants