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.

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

chore: bump wrappers version to 0.2.0 #821

Merged
merged 1 commit into from
Dec 13, 2023
Merged

Conversation

burmecia
Copy link
Member

@burmecia burmecia commented Dec 5, 2023

What kind of change does this PR introduce?

  • upgrade wrappers from v0.1.19 to v0.2.0

Additional context

This release will be announced in LWX.

Action Items

  • New extension releases were Checked for any breaking changes
  • Extensions compatibility Checked
  • Backup and Restore Checked
    • Proceed to backup testing while extensions are enabled
      • After every restore, re-run the tests specified at point 3.1

Extensions compatibility testing

  1. Enable every extension
    1. Check Postgres’ log output for any error messages while doing so
      1. This might unearth incompatibilities due to unsupported internal functions, missing libraries, or missing permissions
  2. Disable every extension
    1. Check Postgres’ log output for any cleanup-related error messages
  3. Re-enable each extension
    1. Run basic tests against the features they offer, e.g.:
      1. pg_net - execute HTTP requests
      2. pg_graphql - execute queries and mutations
      3. …to be filled in

Backup Testing

Follow the testing steps steps for all the following cases:

  • Pause on new Postgres version, restore on new Postgres version
  • Pause on older Postgres version, restore on new Postgres version
  • Run a single-file backup backup, restore the backup

Testing steps

  1. Generate dummy data
    • the ‘Countries’ or ‘Slack clone’ SQL editor snippets are decent datasets to work with, albeit limited
  2. Save a db stats snapshot file
    • Do this by running supa db-stats gather -p <project_ref>
  3. Backup the database, through pausing the project, or otherwise
  4. Restore the backup, through unpausing the project or cli
  5. Check the data has been recovered successfully
    1. Visual checks/navigating through the tables works
    2. Run supa db-stats verify against the project and the previously saved file

@burmecia burmecia requested a review from a team as a code owner December 5, 2023 12:38
Copy link
Contributor

@darora darora left a comment

Choose a reason for hiding this comment

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

Can you confirm that you've tested pg_upgrades and pause/restores from the older versions?

common.vars.pkr.hcl Outdated Show resolved Hide resolved
@burmecia
Copy link
Member Author

burmecia commented Dec 6, 2023

Can you confirm that you've tested pg_upgrades and pause/restores from the older versions?

Yes, I've tested pause/restore on staging and it works fine.

@darora
Copy link
Contributor

darora commented Dec 7, 2023

@pcnc is testing pg_upgrades on staging possible with a custom AMI? Or does it need to be done with a local dev env?

darora
darora previously approved these changes Dec 8, 2023
Copy link
Contributor

@darora darora left a comment

Choose a reason for hiding this comment

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

@burmecia has confirmed off-channel that pg_upgrade is fine

@darora darora dismissed their stale review December 9, 2023 11:57

Looks like pg_upgrade has not been tested

@burmecia
Copy link
Member Author

@darora I've tested pg_upgrade and it is working fine.

@burmecia burmecia requested a review from darora December 12, 2023 07:31
@burmecia burmecia requested a review from a team as a code owner December 13, 2023 23:37
@darora darora merged commit 5651b60 into develop Dec 13, 2023
5 of 8 checks passed
@darora darora deleted the chore/bump_wrappers branch December 13, 2023 23:38
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