Problem/Motivation

New docker image will pushed for pgsql 18.0 in #3540504: Add pgsql-18 image

The release 18.0 is out https://www.postgresql.org/about/news/postgresql-18-released-3142/

Steps to reproduce

https://www.postgresql.org/developer/roadmap/

Proposed resolution

add daily and manual testing using new version

Remaining tasks

- wait for release
- review/commit

User interface changes

Introduced terminology

API changes

Data model changes

Release notes snippet

Issue fork drupal-3540788

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

andypost created an issue. See original summary.

andypost opened merge request !12962

andypost’s picture

andypost
he/him
Primary language Russian
commented
Status: Active » Needs work

Looks like the failed test is valid

    Entity Query (Drupal\KernelTests\Core\Entity\EntityQuery)
     ✘ Entity query
       ┐
       ├ Failed asserting that two arrays are identical.
       ┊ ---·Expected
       ┊ +++·Actual
       ┊ @@ @@
       ┊  Array &0 [
       ┊      16 => '4',
       ┊ +····20·=>·'12',
       ┊      8 => '8',
       ┊ -····20·=>·'12',
       ┊  ]
       │
       │ /builds/issue/drupal-3540788/core/tests/Drupal/KernelTests/Core/Entity/EntityQueryTest.php:393
       ┴
quietone’s picture

quietone commented
Version: 11.0.x-dev » 11.x-dev

Just updating the version.

andypost’s picture

andypost
he/him
Primary language Russian
commented
Issue summary: View changes
Status: Needs work » Needs review

as release is out, rebased MR

smustgrave’s picture

smustgrave commented

Should this not be a draft?

smustgrave’s picture

smustgrave commented
Status: Needs review » Reviewed & tested by the community

Going on a limb that this one was ready?

poker10’s picture

poker10 commented

Thanks for working on this!

I am curious if we need to run daily jobs with all three PostgreSQL versions (16, 17, 18), given we run daily jobs only for one MySQL/MariaDB/SQLite versions? Maybe we could change the existing PostgreSQL 17 daily job to PostgreSQL 18, to have the default and latest versions tested daily and not add a third one?

andypost’s picture

andypost
he/him
Primary language Russian
commented
Status: Reviewed & tested by the community » Needs review

good idea, replaced and rebased

smustgrave’s picture

smustgrave commented
Status: Needs review » Reviewed & tested by the community

Feedback appears to be addressed

andypost’s picture

andypost
he/him
Primary language Russian
commented
Title: CI: Add postgresql 18 to testing matrix » CI: replace postgresql 17 with 18 for PHP 8.5 jobs
Status: Reviewed & tested by the community » Needs review
Issue tags: +PHP 8.5

rebased and squashed into single commit

longwave’s picture

longwave
he/him
Primary language English
Location UK
commented
Status: Needs review » Reviewed & tested by the community

Makes sense to run on the oldest and newest versions, I don't think we need the ones in between as well.

catch closed merge request !12962

catch’s picture

catch
he/him
Primary language English
commented
Version: 11.x-dev » 11.3.x-dev
Status: Reviewed & tested by the community » Fixed

Committed/pushed to 11.x and cherry-picked to 11.3.x, thanks!

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

  • catch committed e6cf2a7c on 11.3.x 
    task: #3540788 CI: replace postgresql 17 with 18 for PHP 8.5 jobs

By:...

  • catch committed 26c7051f on 11.x 
    task: #3540788 CI: replace postgresql 17 with 18 for PHP 8.5 jobs

By:...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.