Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Updated the Backup Restore Script #22

Merged
merged 4 commits into from
Apr 3, 2025
Merged

Updated the Backup Restore Script #22

merged 4 commits into from
Apr 3, 2025

Conversation

Praveenraj-K
Copy link
Contributor

@Praveenraj-K Praveenraj-K commented Mar 27, 2025 •
edited
Loading

Added

This shell script automates the PostgreSQL database restore process with the following key features:

  • Downloads database backup and checksum files from an S3 bucket
  • Decompresses the backup and checksum files
  • Validates backup integrity using SHA-256 checksum verification
  • Restores the database from the backup file
  • Verifies successful database restoration
  • Implements error handling with Sentry integration for error reporting
  • Cleans up temporary files after successful restoration

@Praveenraj-K Praveenraj-K requested review from a team, briskt, iceman198 and ethancanne and removed request for a team March 27, 2025 07:09
@mtompset mtompset changed the title Updated the Dackup Restore Script Updated the Backup Restore Script Mar 27, 2025
mtompset
mtompset approved these changes Mar 27, 2025
View reviewed changes
Copy link

@mtompset mtompset left a comment

Choose a reason for hiding this comment

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

Somehow I feel there's a do_command_and_report_error function waiting to happen in here.

briskt
briskt reviewed Mar 31, 2025
View reviewed changes
Copy link
Contributor

@briskt briskt left a comment

Choose a reason for hiding this comment

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

Good improvements, with a couple questions for you.

briskt
briskt approved these changes Apr 2, 2025
View reviewed changes
Copy link
Contributor

@briskt briskt left a comment

Choose a reason for hiding this comment

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

thanks!

@briskt
Copy link
Contributor

briskt commented Apr 2, 2025

The title is very vague. Since the "backup restore script" is basically half of the entire codebase, and "update" covers pretty much every PR, it is mostly meaningless. Makes future study more difficult.

@Praveenraj-K
Copy link
Contributor Author

The title is very vague. Since the "backup restore script" is basically half of the entire codebase, and "update" covers pretty much every PR, it is mostly meaningless. Makes future study more difficult.

Got it, i'll update in detail

@Praveenraj-K Praveenraj-K merged commit 5c0131a into main Apr 3, 2025
@Praveenraj-K Praveenraj-K deleted the backup-restore branch April 3, 2025 11:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mtompset mtompset mtompset approved these changes

@briskt briskt briskt approved these changes

@iceman198 iceman198 Awaiting requested review from iceman198 iceman198 was automatically assigned from silinternational/developers

@ethancanne ethancanne Awaiting requested review from ethancanne ethancanne was automatically assigned from silinternational/developers

Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Praveenraj-K @briskt @mtompset