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

Carry forward exit code of id command #365

Merged
merged 1 commit into from
Jul 27, 2023
Merged

Conversation

olirice
Copy link
Contributor

@olirice olirice commented Jul 27, 2023

resolves #364

Copy link
Contributor

@eeeebbbbrrrr eeeebbbbrrrr left a comment

Choose a reason for hiding this comment

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

Thanks @olirice!

@eeeebbbbrrrr eeeebbbbrrrr merged commit 29b7643 into tcdi:main Jul 27, 2023
@olirice
Copy link
Contributor Author

olirice commented Jul 27, 2023

Any chance I could bug you to run a (non-latest release) tag like v1.2.3-preinit through the build artifact CI 🤞?

I tried it on our arm-runner via a fork but there are some incompatibilities

@BradyBonnette
Copy link
Contributor

@olirice

Is it possible that you're able to give any information about how your aarch64/arm64 runners are set up? I ask only because the artifacts are built using docker, as to try and keep the inconsistencies as low as possible. I'd love to help if I can!

At first glance, it looks like your runners are having difficulty launching anything via Docker: https://github.com/supabase/plrust/actions/runs/5682987830/job/15402603128#step:5:28

@olirice
Copy link
Contributor Author

olirice commented Jul 27, 2023

it looks like your runners are having difficulty launching anything via Docker
yeah, I just don't have the ability to debug it without jumping through hoops

from the prior step docker should be running:

Processing triggers for libc-bin (2.31-0ubuntu9.9) ...
[255](https://github.com/supabase/plrust/actions/runs/5682987830/job/15402603128#step:4:256)
 * Docker already stopped - file /var/run/docker-ssd.pid not found.
[256](https://github.com/supabase/plrust/actions/runs/5682987830/job/15402603128#step:4:257)
 * Starting Docker: docker
[257](https://github.com/supabase/plrust/actions/runs/5682987830/job/15402603128#step:4:258)
   ...done.

No worries if not possible, can spin up a server on AWS and it manually

You don't get what you don't ask for :)

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.

User 'postgres' does not exist error when postgres user does exist
3 participants