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

While setup project on local - permissions error in Postgres provisioning script #7468

Closed
Karankhatik opened this issue Oct 7, 2024 · 6 comments
Labels
scope: backend Issues that are affecting the backend side only type: chore

Comments

@Karankhatik
Copy link
Contributor

Karankhatik commented Oct 7, 2024

While running the make postgres-on-linux command, which executes the script at /twenty-postgres/linux/provision-postgres-linux.sh, I encountered a permissions error during the following line:

cp ./init.sql /tmp/init.sql

image

Also a minor code improvement on this file that we don't need su there, its unneccessary:

image

Machine description:
Distributor ID: Ubuntu
Description: Ubuntu 24.04.1 LTS

@Karankhatik
Copy link
Contributor Author

Hello @Bonapara , I have made changes to solve this issue Please let me know when I can push.

@Bonapara Bonapara added the scope: backend Issues that are affecting the backend side only label Oct 7, 2024
@Bonapara
Copy link
Member

Bonapara commented Oct 7, 2024

@Weiko what do you think?

@BOHEUS
Copy link
Contributor

BOHEUS commented Oct 7, 2024

@Karankhatik are you sure you're running newest version of Twenty? I've made changes to this file a month ago and I deleted this unnecessary su command #6819

@Karankhatik
Copy link
Contributor Author

@BOHEUS Thanks I noticed that my forked repository is not in sync with the main repository. now its sync.

I deleted the database, remove everything and did a fresh installation, but I encountered a "permission denied" error because the command cp ./init.sql /tmp/init.sql didn't include sudo. After adding sudo, the error was resolved.

Should I push this change, or is it likely that only I encountered this error?

@BOHEUS
Copy link
Contributor

BOHEUS commented Oct 7, 2024

It's like that only you have encountered this error as there were no issues related to error while copying to tmp/ directory, I don't think you should push this change as tmp/ by default should allow all users write to it (despite root being the owner of the folder https://unix.stackexchange.com/questions/71622/what-are-correct-permissions-for-tmp-i-unintentionally-set-it-all-public-recu)

@Karankhatik
Copy link
Contributor Author

Thanks @BOHEUS for letting me know. I m closing this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
scope: backend Issues that are affecting the backend side only type: chore
Projects
None yet
Development

No branches or pull requests

3 participants