Skip to content

chore: Add Timezone and restart policy for postgres backup docker#3083

Open
BinaryMuse wants to merge 1 commit intomainfrom
docs-pr-migration/selfhosted-docker-backup
Open

chore: Add Timezone and restart policy for postgres backup docker#3083
BinaryMuse wants to merge 1 commit intomainfrom
docs-pr-migration/selfhosted-docker-backup

Conversation

@BinaryMuse
Copy link
Member

@BinaryMuse BinaryMuse commented Jan 16, 2026

Add TZ/PGTZ environment variables and restart policy for better reliability of the docker backup setup.


Migrated from atuinsh/docs#62
Original author: @mateuscomh

Add TZ/PGTZ environment variables and restart policy for better reliability of the docker backup setup.

Co-Authored-By: Claude Opus 4.5 <[email protected]>
@greptile-apps
Copy link
Contributor

greptile-apps bot commented Jan 16, 2026

Greptile Summary

Adds timezone environment variables (TZ and PGTZ) to both postgres and backup containers, plus restart policy for the backup service.

  • Good addition of timezone config for consistent backup timestamps
  • Added restart: unless-stopped to backup service for reliability
  • Critical: Backup service references non-existent postgresql service - should be db
  • Issue: HEALTHCHECK_PORT: 5432 is incorrect for the backup container

Important Files Changed

Filename Overview
docs/docs/self-hosting/docker.md Added timezone config and restart policy to backup service, but has incorrect service names and healthcheck port

Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

Additional Comments (2)

  1. docs/docs/self-hosting/docker.md, line 114 (link)

    logic: The postgres service is named db (line 48), not postgresql.

  2. docs/docs/self-hosting/docker.md, line 125 (link)

    logic: The postgres service is named db (line 48), not postgresql.

1 file reviewed, 3 comments

Edit Code Review Agent Settings | Greptile

SCHEDULE: "@daily"
BACKUP_DIR: /db_dumps
TZ: Europe/London
HEALTHCHECK_PORT: 5432
Copy link
Contributor

Choose a reason for hiding this comment

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

logic: HEALTHCHECK_PORT should be the backup container's healthcheck port, not the postgres port. Remove this or use a different port (default is 8080 for this image).

@mateuscomh
Copy link
Contributor

Add TZ/PGTZ environment variables and restart policy for better reliability of the docker backup setup.

Migrated from atuinsh/docs#62 Original author: @matheuscmpm

Hello, if you can fix the original author is @mateuscomh

@BinaryMuse
Copy link
Member Author

Oops, not sure how that happened. Fixed.

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