tcdi / plrust

A Rust procedural language handler for PostgreSQL
PostgreSQL License
1.12k stars 32 forks source link

Carry forward exit code of `id` command #365

Closed olirice closed 1 year ago

olirice commented 1 year ago

resolves #364

olirice commented 1 year ago

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 commented 1 year ago

@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 commented 1 year ago

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 :)