Skip to content

Conversation

@mosabua
Copy link
Member

@mosabua mosabua commented Jun 18, 2025

Depends on Trino Gateway 16 release - see trinodb/trino-gateway#643

CI will fail for now.

@cla-bot cla-bot bot added the cla-signed label Jun 18, 2025
@mosabua mosabua marked this pull request as draft June 18, 2025 06:23
@mosabua mosabua marked this pull request as ready for review June 18, 2025 06:23
@nineinchnick
Copy link
Member

@mosabua Trino Gateway 16 has been released, can you rebase?

@mosabua
Copy link
Member Author

mosabua commented Sep 18, 2025

Rebased and pushed now .. lets see it turn green and then I will merge.

@nineinchnick
Copy link
Member

There's an issue with starting up the PostgreSQL deployment. I've seen this in another PR recently. I need to investigate. Probably not related to the Gateway upgrade.

@mosabua
Copy link
Member Author

mosabua commented Sep 18, 2025

Yeah .. I am checking what version of PostgreSQL we use in the project itself .. maybe upgrading helps .. do you think we are save to merge anyway?

@mosabua
Copy link
Member Author

mosabua commented Sep 18, 2025

also remember some message about bitnami vs bitnami legacy

@mosabua
Copy link
Member Author

mosabua commented Sep 18, 2025

Testing in the project uses postgres 17 .. so I think we should upgrade in the chart too

@nineinchnick
Copy link
Member

Tests didn't run, so it's not safe to merge. We're using 17 in other tests and that fails to start too, even the images from the renamed bitnamilegacy repo.

@mosabua
Copy link
Member Author

mosabua commented Sep 18, 2025

bitnami is stopping their whole public container and helm chart stuff it seems .. things moved to bitnamilegacy and we might need to use those for now and then also find alternative charts

@mosabua
Copy link
Member Author

mosabua commented Sep 18, 2025

SIGH .. I will try to look later today

@mosabua
Copy link
Member Author

mosabua commented Sep 18, 2025

Pretty sure we are falling victim to the bitnami deprecation - lots of details in bitnami/containers#83267 .. we might have to migrate to bitnamilegacy and archived containers for now and then find alternatives.

Feel free to push commits in here or do another PR @nineinchnick

@mosabua
Copy link
Member Author

mosabua commented Sep 18, 2025

Fixed now @nineinchnick .. WDYT?

@mosabua mosabua merged commit 7cb7812 into trinodb:main Sep 19, 2025
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

2 participants