Tecnativa / doodba

Base image for making the creation of customized Odoo environments a piece of cake
Apache License 2.0
432 stars 304 forks source link

Outdated images for 11.0 and 12.0 #540

Closed Tardo closed 1 year ago

Tardo commented 1 year ago

The github workflow is not building the images for 11.0 and 12.0 this gives problems. Already version 11.0 cannot be used because it is pointing to an address that does not exist and can't install postgresql-client.

I don't know if it is preferable to include these versions or to give them as "unattended" and remove the Dockerfile from the repository.

https://github.com/Tecnativa/doodba/blob/master/.github/workflows/ci.yaml#L37

ap-wtioit commented 1 year ago

Checked with different images (this part isn't any different in ours) and i can install postgresql-client fine:

docker run --rm --entrypoint="" -it tecnativa/doodba:12.0 bash -c 'apt-get update && apt-cache search postgres && apt-get install -y postgresql-client-10 && psql --version'
docker run --rm --entrypoint="" -it tecnativa/doodba:11.0 bash -c 'apt-get update && apt-cache search postgres && apt-get install -y postgresql-client-13 && psql --version'
docker run --rm --entrypoint="" -it ...wt-io-it.../docker-odoo-base:12.0 bash -c 'apt-get update && apt-cache search postgres && apt-get install -y postgresql-client && psql --version'
docker run --rm --entrypoint="" -it ...wt-io-it.../docker-odoo-base:11.0 bash -c 'apt-get update && apt-cache search postgres && apt-get install -y postgresql-client-12 && psql --version'

Our CI builds and tests our images daily, which url do you mean, and do you have a link to the failed install or at least some logs?

ap-wtioit commented 1 year ago

And images for 12.0 and 11.0 have been built in https://github.com/Tecnativa/doodba/pull/535

Tardo commented 1 year ago

Yes, I am referring to these images https://github.com/Tecnativa/doodba/pkgs/container/doodba which are the ones used by doodba-copier-template.

As far as I know, the way you indicate, you would have to relaunch this update every time the container is "down" no?

** EDIT: link

Tardo commented 1 year ago

My bad... I didn't see this: https://github.com/Tecnativa/doodba/blob/master/.github/workflows/ci.yaml#L42

Sorry!