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

Stop building che-postgres image since it's not used in the Che operator CSV #21349

Closed
Tracked by #21351
nickboldt opened this issue Apr 25, 2022 · 1 comment
Closed
Tracked by #21351
Labels
area/ci CI build and releases, PR testing, & whitelabel/productization issues kind/task Internal things, technical debt, and to-do tasks to be performed. severity/P2 Has a minor but important impact to the usage or development of the system.

Comments

@nickboldt
Copy link
Contributor

Is your task related to a problem? Please describe

In the che operator CSV...

... we no longer reference the che-postgres images:

And quay.io/eclipse/che-postgres:next includes:

$➔ podman run -it --rm --entrypoint /bin/bash quay.io/eclipse/che-postgres:next -c "/usr/bin/postgres --version"
podman run -it --rm --entrypoint /bin/bash
postgres (PostgreSQL) 9.6.10

Describe the solution you'd like

Describe alternatives you've considered

No response

Additional context

No response

@nickboldt nickboldt added the kind/task Internal things, technical debt, and to-do tasks to be performed. label Apr 25, 2022
@che-bot che-bot added the status/need-triage An issue that needs to be prioritized by the curator responsible for the triage. See https://github. label Apr 25, 2022
@nickboldt nickboldt added area/ci CI build and releases, PR testing, & whitelabel/productization issues severity/P2 Has a minor but important impact to the usage or development of the system. and removed status/need-triage An issue that needs to be prioritized by the curator responsible for the triage. See https://github. labels Apr 25, 2022
@nickboldt nickboldt mentioned this issue Apr 28, 2022
58 tasks
@nickboldt
Copy link
Contributor Author

From Anatolii, sounds like this is a non-starter, but if we decide to drop Helm entirely we could review:

Since installation with Helm is not equal to installation with operator some images are different.
Here is the Dockerfile [1] Besides PostgreSQL 9.6 image, it contains some configuration that is required only for Helm
So, simple replacement won't work. My suggestions are:

Don't touch anything related to Helm here. Basically these charts are outdated, but maybe someone from community uses them
or
Completely remove everything related to HelmCharts
[1] https://github.com/eclipse-che/che-server/blob/main/dockerfiles/postgres/Dockerfile

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/ci CI build and releases, PR testing, & whitelabel/productization issues kind/task Internal things, technical debt, and to-do tasks to be performed. severity/P2 Has a minor but important impact to the usage or development of the system.
Projects
None yet
Development

No branches or pull requests

2 participants