Skip to content

Conversation

@renovate
Copy link
Contributor

@renovate renovate bot commented Jan 12, 2026

This PR contains the following updates:

Package Update Change Age Confidence
postgres major 17-alpine β†’ 18-alpine age confidence

Configuration

πŸ“… Schedule: Branch creation - "before 4am on Monday" in timezone America/New_York, Automerge - At any time (no schedule defined).

🚦 Automerge: Disabled because a matching PR was automerged previously.

β™» Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

πŸ”• Ignore: Close this PR and you won't be reminded about this update again.


  • If you want to rebase/retry this PR, check this box

This PR was generated by Mend Renovate. View the repository job log.

@github-actions
Copy link
Contributor

github-actions bot commented Jan 12, 2026

Dependency Review

βœ… No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

None

@sentry
Copy link

sentry bot commented Jan 12, 2026

Codecov Report

βœ… All modified and coverable lines are covered by tests.
βœ… Project coverage is 40.21%. Comparing base (6fa7ce7) to head (a29cd38).
βœ… All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1140   +/-   ##
=======================================
  Coverage   40.21%   40.21%           
=======================================
  Files         206      206           
  Lines       15772    15772           
  Branches     1956     1956           
=======================================
  Hits         6342     6342           
  Misses       9430     9430           

β˜” View full report in Codecov by Sentry.
πŸ“’ Have feedback on the report? Share it here.

| datasource | package  | from | to |
| ---------- | -------- | ---- | -- |
| docker     | postgres | 17   | 18 |
@renovate renovate bot force-pushed the renovate/major-18-docker branch from c894940 to a29cd38 Compare January 21, 2026 18:58
@coderabbitai
Copy link

coderabbitai bot commented Jan 21, 2026

Important

Review skipped

Bot user detected.

To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.


Comment @coderabbitai help to get the list of available commands and usage tips.

container_name: tux-postgres
hostname: tux-postgres
image: postgres:17-alpine@sha256:ff4ccc02b97e0ebb6b328ef9ff92522f95586f83be6801896b615088defc8ad2
image: postgres:18-alpine@sha256:b40d931bd0e7ce6eecc59a5a6ac3b3c04a01e559750e73e7086b6dbd7f8bf545
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bug: The PostgreSQL major version upgrade from 17 to 18 lacks a data migration strategy, which will cause startup failures for existing deployments with persistent data.
Severity: HIGH

Suggested Fix

A migration path for existing data must be provided. This can be achieved by adding an automated migration script that uses pg_upgrade or a dump/restore process. Alternatively, provide a detailed, documented manual migration procedure for users to follow. Reverting to PostgreSQL 17 is also an option until a proper migration strategy is implemented.

Prompt for AI Agent
Review the code at the location below. A potential bug has been identified by an AI
agent.
Verify if this is a real issue. If it is, propose a fix; if not, explain why it's not
valid.

Location: compose.yaml#L28

Potential issue: The pull request upgrades the PostgreSQL image from version 17 to 18 in
`compose.yaml`. Major PostgreSQL versions have incompatible on-disk data formats. For
existing deployments using the persistent Docker volume `tux_postgres_data`, the new
PostgreSQL 18 container will attempt to start with a PostgreSQL 17 data directory. This
will cause an immediate startup failure because the data format is incompatible. Since
the application container depends on the database, it will also fail to start, leading
to a service outage for any existing installation.

Did we get this right? πŸ‘ / πŸ‘Ž to inform future reviews.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant