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

Old identityProvider and postgres container images not updated to latest defaults on Operator update. #13714

Closed
davidfestal opened this issue Jun 26, 2019 · 4 comments
Assignees
Labels
area/che-operator Issues and PRs related to Eclipse Che Kubernetes Operator severity/P1 Has a major impact to usage or development of the system.
Milestone

Comments

@davidfestal
Copy link
Contributor

When the Che operator is updated (for example from beta-5.0 to rc-2.0), and a Che server CR was previously started with the default Keycloak docker image (empty string), the Che server is not upgraded with the new default keycloak image after operator upgrade, but stays on the effective Keycloak version that was initially deployed.

@davidfestal davidfestal self-assigned this Jun 26, 2019
@benoitf benoitf transferred this issue from eclipse-che/che-operator Jul 5, 2019
@benoitf benoitf added area/che-operator Issues and PRs related to Eclipse Che Kubernetes Operator target/che7GA labels Jul 5, 2019
@davidfestal davidfestal added this to the 7.1.0 milestone Jul 16, 2019
@nickboldt
Copy link
Contributor

We've seen this in downstream CRW 1.2 too. And it's not just the identity provider (Keycloak/SSO) image but also the postgres db that remains on the image:tag of the initial deployment.

WORKAROUND is to patch the deployment, as we do here:

https://github.com/redhat-developer/codeready-workspaces-deprecated/blob/6.19.x/operator-installer/migrate_1.1_to_1.2.sh#L280-L306

But a better solution would be ... um, better. :)

@nickboldt nickboldt changed the title Old Keycloak docker image not updated on Operator update. Old identityProvider and postgres container images not updated to latest defaults on Operator update. Jul 22, 2019
@nickboldt nickboldt added the severity/P1 Has a major impact to usage or development of the system. label Jul 22, 2019
@ibuziuk
Copy link
Member

ibuziuk commented Aug 28, 2019

@davidfestal can we close this one ?

@davidfestal
Copy link
Contributor Author

I assume yes, even though it is not in production of course.

@ibuziuk
Copy link
Member

ibuziuk commented Aug 29, 2019

Fixed in eclipse-che/che-operator#72 and operator nightly builds

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/che-operator Issues and PRs related to Eclipse Che Kubernetes Operator severity/P1 Has a major impact to usage or development of the system.
Projects
None yet
Development

No branches or pull requests

4 participants