Skip to content

IBX-12043: Fixed PostgreSQL server version parameter in CI - #31

Merged
ViniTou merged 1 commit into
6.0from
ibx-12043-pgsql-server-version
Sep 4, 2026
Merged

IBX-12043: Fixed PostgreSQL server version parameter in CI#31
ViniTou merged 1 commit into
6.0from
ibx-12043-pgsql-server-version

Conversation

@ViniTou

@ViniTou ViniTou commented Sep 4, 2026

Copy link
Copy Markdown
Contributor
🎫 Issue IBX-12043

Description:

The PostgreSQL integration job passes its server version as server_version. DBAL 4 reads serverVersion, camel case — the snake_case spelling is read by nothing, so the version arrives empty and AbstractPostgreSQLDriver throws InvalidPlatformVersion during bootstrap, before a single test runs. ibexa/core is on DBAL 4 on 6.0, so this job cannot pass as written.

The value is raised to match the service image rather than kept at 10. DBAL 4 selects PostgreSQL120Platform for anything >= 12.0 and falls back to the legacy PostgreSQLPlatform below that, with Support for Postgres < 12 is deprecated and will be removed in DBAL 5. Declaring 10 therefore opted this suite into a deprecated platform that does not match the container it runs against.

For QA:

CI only. No runtime code changed.

Documentation:

N/A

DBAL 4 reads serverVersion, camel case. The snake_case server_version spelling is read by nothing, so the version stayed empty and AbstractPostgreSQLDriver rejected it with InvalidPlatformVersion before any test ran.

The value now matches the service image. DBAL 4 maps anything below 12.0 onto the legacy PostgreSQLPlatform and warns that Postgres < 12 support is removed in DBAL 5, so declaring 10 opted this suite into a deprecated platform it was not running against.
@ViniTou
ViniTou merged commit ab6dbde into 6.0 Sep 4, 2026
13 checks passed
@ViniTou
ViniTou deleted the ibx-12043-pgsql-server-version branch September 4, 2026 08:10
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.

2 participants