-
Notifications
You must be signed in to change notification settings - Fork 6
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
Can't run in Github Actions due to sudo
use
#8
Comments
If you have the However, postgres itself refuses to run as root so I'm not sure what you're doing will work at all. |
It works, I assume thanks to the As for using
I'm not sure that there is a solution to this issue 🤷 |
Ah, I had gone back and forth on whether to include that feature and forgot that I actually ended up keeping it. I'm not sure why it isn't working for you in github actions then. You definitely have the |
Yup, github path is set in an earlier step and PATH is echoed in the test
step to verify that it includes the postgresql dir, but 'sudo' doesn't use
the updated path.
…On Tue, 9 Jul 2024, 23:36 Harry Stern, ***@***.***> wrote:
It works, I assume thanks to the sudo call in pgtemp.
Ah, I had gone back and forth on whether to include that feature and
forgot that I actually ended up keeping it.
I'm not sure why it isn't working for you in github actions then. You
definitely have the $GITHUB_PATH step included in your action prior to
starting pgtemp?
—
Reply to this email directly, view it on GitHub
<#8 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABDEBBULSFO7BX4TD7BH7H3ZLRCWXAVCNFSM6AAAAABKSP4IOKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMJYGY4DMMRQGM>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
When I try to use this in Github Actions I get these errors:
from what I see, this happens because
sudo
can't findinitdb
in its secure path - adding the path to postgres to$PATH
doesn't solve the issue.I mitigated the issue by copying the whole
postgresql
bin &lib folders into/usr/bin
&/usr/lib
respectively, but it would be nice if the library could handle this.The text was updated successfully, but these errors were encountered: