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

feat: alter lo_export & lo_import owner #863

Merged
merged 5 commits into from
Jan 26, 2024

Conversation

soedirgo
Copy link
Member

@soedirgo soedirgo commented Jan 24, 2024

TODO:

  • tested on stg

Also fixes the CI for building .debs in Docker

@soedirgo soedirgo requested review from a team as code owners January 24, 2024 08:09
@soedirgo soedirgo marked this pull request as draft January 24, 2024 08:09
@soedirgo soedirgo force-pushed the feat/lo-export-lo-import-owner branch 5 times, most recently from 90e031b to 72f26aa Compare January 25, 2024 05:26
`docker save` was exporting the image in OCI format - we can't easily
extract the `.deb`s from it.
@soedirgo soedirgo force-pushed the feat/lo-export-lo-import-owner branch from 72f26aa to 8a0467a Compare January 25, 2024 06:56
@soedirgo soedirgo marked this pull request as ready for review January 25, 2024 06:57
@soedirgo soedirgo requested a review from a team as a code owner January 25, 2024 06:57
cache-from: type=gha,scope=${{ github.ref_name }}-latest-${{ matrix.arch }}
# No need to export extensions cache because latest depends on it
- name: Extract built packages
run: |
mkdir -p /tmp/extensions ansible/files/extensions
docker save supabase/postgres:extensions | tar xv -C /tmp/extensions
Copy link
Member Author

@soedirgo soedirgo Jan 25, 2024

Choose a reason for hiding this comment

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

docker save started exporting the .tar in a different format for some reason: https://github.com/supabase/postgres/actions/runs/7637264535

@soedirgo soedirgo merged commit a32c49c into develop Jan 26, 2024
8 checks passed
@soedirgo soedirgo deleted the feat/lo-export-lo-import-owner branch January 26, 2024 02:32
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