Skip to content

feat(pitr): add command to fetch recovery window of a database - #1249

Merged
matthieu526-scalingo merged 2 commits into
masterfrom
feat/STORY-4094/add-recovery-window-command
Aug 13, 2026
Merged

feat(pitr): add command to fetch recovery window of a database#1249
matthieu526-scalingo merged 2 commits into
masterfrom
feat/STORY-4094/add-recovery-window-command

Conversation

@matthieu526-scalingo

@matthieu526-scalingo matthieu526-scalingo commented Aug 11, 2026

Copy link
Copy Markdown
Contributor

Add a new command to get the recovery window for a given DB.

  • Add a changelog entry in the section "To Be Released" of CHANGELOG.md
  • right now depending on this PR. Before merging it should be set to a released version
  • QA not executed yet

Basic QA on staging done on the 13/08.

➜  scalingo git:(feat/STORY-4094/add-recovery-window-command) lsscalingo --app authentication-service --addon postgresql database-pitr-recovery-window
┌─────────────────────────┬──────────────────────┐
│ Earliest recoverable at │ 2026-08-02T01:02:03Z │
│ Latest recoverable at   │ 2026-08-09T01:01:23Z │
└─────────────────────────┴──────────────────────┘

I am curious about why the recovery window is not now, maybe there was no new data at all in the DB.

@matthieu526-scalingo matthieu526-scalingo self-assigned this Aug 11, 2026
@notion-workspace

Copy link
Copy Markdown

@matthieu526-scalingo
matthieu526-scalingo marked this pull request as ready for review August 13, 2026 08:40
@matthieu526-scalingo
matthieu526-scalingo requested a review from a team as a code owner August 13, 2026 08:40
@matthieu526-scalingo
matthieu526-scalingo requested review from SCedricThomas and curzolapierre and removed request for a team August 13, 2026 08:40

@SCedricThomas SCedricThomas left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

Comment thread go.mod

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

We will need a proper release before merging my bad

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I know, this is still tracked in the description 😄

@SCedricThomas SCedricThomas left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

\o/

@matthieu526-scalingo
matthieu526-scalingo merged commit 7e83bb4 into master Aug 13, 2026
7 checks passed
@matthieu526-scalingo
matthieu526-scalingo deleted the feat/STORY-4094/add-recovery-window-command branch August 13, 2026 09:25
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