Skip to content

ci: move the container build process to bake #80

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

Open
wants to merge 8 commits into
base: main
Choose a base branch
from
Open

Conversation

NiccoloFei
Copy link
Contributor

@NiccoloFei NiccoloFei commented Apr 4, 2025

Switching the build process to bake, creating 3 targets by default: minimal, standard and postgis.
This is an initial effort needed in order to allow running PG Major Upgrade E2E tests also in this repository, which will allow us to test major upgrades to development versions of PG18. The test's are failing at the moment for many reasons, mainly because the target images are not available. See any of the latest CD runs.

In CNPG:

  • modify the E2Es to support customizing the targetImage's repositories that are part of the buildScenario, allowing us to point to postgresql-trunk

Note:

  1. barman-cloud should eventually be removed from the standard target, which adds barman-cloud and pgaudit(currently commented out), once plugin-barman-cloud becomes stable
  2. The postgis target can also eventually be removed once extensions can be loaded via ImageVolume, but at the moment it's needed

mnencia pushed a commit to cloudnative-pg/cloudnative-pg that referenced this pull request May 19, 2025
…Es (#7303)

This patch goes together with
cloudnative-pg/postgres-trunk-containers#80.
The goal is to allow configuring a custom repository (in this case the
`postgresql-trunk`) instead of the default ones (via an env variable),
which will be used to fetch the target images required to perform the
major upgrade scenarios.
This will allow us to test major upgrades to development versions of
PG18.

Closes #7302


Signed-off-by: Niccolò Fei <niccolo.fei@enterprisedb.com>
Signed-off-by: Armando Ruocco <armando.ruocco@enterprisedb.com>
Co-authored-by: Armando Ruocco <armando.ruocco@enterprisedb.com>
cnpg-bot pushed a commit to cloudnative-pg/artifacts that referenced this pull request May 20, 2025
…Es (#7303)%0A%0AThis patch goes together with%0Ahttps://github.com/cloudnative-pg/postgres-trunk-containers/pull/80.%0AThe goal is to allow configuring a custom repository (in this case the%0A`postgresql-trunk`) instead of the default ones (via an env variable),%0Awhich will be used to fetch the target images required to perform the%0Amajor upgrade scenarios.%0AThis will allow us to test major upgrades to development versions of%0APG18.%0A%0ACloses #7302%0A%0A%0ASigned-off-by: Niccolò Fei <niccolo.fei@enterprisedb.com>%0ASigned-off-by: Armando Ruocco <armando.ruocco@enterprisedb.com>%0ACo-authored-by: Armando Ruocco <armando.ruocco@enterprisedb.com>

cloudnative-pg/cloudnative-pg@refs/heads/main
cnpg-bot pushed a commit to cloudnative-pg/artifacts that referenced this pull request May 21, 2025
…Es (#7303)%0A%0AThis patch goes together with%0Ahttps://github.com/cloudnative-pg/postgres-trunk-containers/pull/80.%0AThe goal is to allow configuring a custom repository (in this case the%0A`postgresql-trunk`) instead of the default ones (via an env variable),%0Awhich will be used to fetch the target images required to perform the%0Amajor upgrade scenarios.%0AThis will allow us to test major upgrades to development versions of%0APG18.%0A%0ACloses #7302%0A%0A%0ASigned-off-by: Niccolò Fei <niccolo.fei@enterprisedb.com>%0ASigned-off-by: Armando Ruocco <armando.ruocco@enterprisedb.com>%0ACo-authored-by: Armando Ruocco <armando.ruocco@enterprisedb.com>

cloudnative-pg/cloudnative-pg@refs/heads/main
NiccoloFei and others added 7 commits May 21, 2025 17:48
Signed-off-by: Niccolò Fei <niccolo.fei@enterprisedb.com>
Signed-off-by: Niccolò Fei <niccolo.fei@enterprisedb.com>
Signed-off-by: Niccolò Fei <niccolo.fei@enterprisedb.com>
Signed-off-by: Niccolò Fei <niccolo.fei@enterprisedb.com>
Signed-off-by: Niccolò Fei <niccolo.fei@enterprisedb.com>
Signed-off-by: Niccolò Fei <niccolo.fei@enterprisedb.com>
Signed-off-by: Marco Nenciarini <marco.nenciarini@enterprisedb.com>
@gbartolini
Copy link
Contributor

Can we just remove Barman Cloud from the images and disable backup/recovery testing for postgres-trunk? @NiccoloFei

Also, can you please update the README as part of this patch? It would be good to show a simple command to run locally to build the image. Thanks.

Signed-off-by: Niccolò Fei <niccolo.fei@enterprisedb.com>
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

Successfully merging this pull request may close these issues.

4 participants