Skip to content

feat: bump wrappers version to v0.4.6 #1542

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

Merged
merged 11 commits into from
Apr 14, 2025
Merged

Conversation

burmecia
Copy link
Member

@burmecia burmecia commented Apr 12, 2025

What kind of change does this PR introduce?

  • upgrade wrappers from v0.4.5 to v0.4.6

Additional context

Add any other context or screenshots.

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 marked this pull request as ready for review April 13, 2025 02:54
@burmecia burmecia requested review from a team as code owners April 13, 2025 02:54
@burmecia
Copy link
Member Author

it seems the arm_runner disk space isn’t enough, can we increase its size?
image

@soedirgo
Copy link
Member

If that's from Testinfra Integration Tests Nix you can ignore those - it's been broken for a while. Not sure while Nix CI is failing though, and only on macOS (it builds on my machine)

@samrose
Copy link
Collaborator

samrose commented Apr 13, 2025

@burmecia you could be right. I’ll start looking into testinfra tests next week. Nix ci usually only fails if there is a real problem

@samrose
Copy link
Collaborator

samrose commented Apr 13, 2025

If that's from Testinfra Integration Tests Nix you can ignore those - it's been broken for a while. Not sure while Nix CI is failing though, and only on macOS (it builds on my machine)

@soedirgo @burmecia if those testinfra tests fail like this, you can feel free to reach out to me and I'll prioritize looking into those, as they caught issues when working correct, and we really need them to be functioning/useful tests. So please reach out to me if you are running into issues and I will look into issues where the failure is just the test not working.

@samrose
Copy link
Collaborator

samrose commented Apr 13, 2025

@burmecia when I try to build wrappers from this branch locally on macos, the nix build is failing. Looking into it now.

@soedirgo
Copy link
Member

@samrose sorry yeah, should’ve flagged these earlier. Once these incidents are resolved lmk if I can help with that as I was the one who set it up way back when 👍

* chore: fix nix build on macos and address space on gh runner

* chore: special handling for oriole

* chore: permissions

* chore: make sure permission to run is granted
@samrose
Copy link
Collaborator

samrose commented Apr 13, 2025

@burmecia @soedirgo got this all to pass in a pr to this one. Just merged and if it passes should be good to go, other than local infra testing.

@samrose
Copy link
Collaborator

samrose commented Apr 13, 2025

@burmecia @soedirgo ok every thing is passing now. If we test this in local infra, then it should be good to go

@burmecia burmecia changed the title feat: test bump wrappers version to v0.4.6 feat: bump wrappers version to v0.4.6 Apr 13, 2025
@burmecia
Copy link
Member Author

that's nice, thanks @samrose and @soedirgo !

@burmecia burmecia merged commit 67fb756 into develop Apr 14, 2025
14 checks passed
@burmecia burmecia deleted the bo/feat/bump-wrappers-v046 branch April 14, 2025 06:26
damonrand pushed a commit to cepro/postgres that referenced this pull request Jun 15, 2025
* feat: test bump wrappers version to v0.4.6

* feat: test bump wrappers version to v0.4.6

* feat: test bump wrappers version to v0.4.6

* feat: test bump wrappers version to v0.4.6

* increase runner size

* feat: test bump wrappers version to v0.4.6

* restore temp changes

* update release hash code

* chore: fix nix build on macos and address space on gh runner (supabase#1543)

* chore: fix nix build on macos and address space on gh runner

* chore: special handling for oriole

* chore: permissions

* chore: make sure permission to run is granted

* test on staging

* restore temp change for release test

---------

Co-authored-by: samrose <[email protected]>
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.

4 participants