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

Fix ssl-cert-snakeoil.key permission denied #4609

Closed
wants to merge 1 commit into from
Closed

Fix ssl-cert-snakeoil.key permission denied #4609

wants to merge 1 commit into from

Conversation

liweijian
Copy link

Changelog:

Area of change
- Add some `chmod` and `chown` in the **scripts/devcontainer/_start-background-services**

alternatively:

No changelog

What is the problem/goal being addressed?

Not sure if it's always happen, but I got this postgresql related error when trying to run scripts/builder --compile --watch --test

UTC FATAL: could not access private key file "/etc/ssl/private/ssl-cert-snakeoil.key": Permission denied

What is the solution to this problem?

It sounds a lot like that's a really tricky issue, inspired by this crazy solution, finally I made it.

What are the changes here? How do they solve the problem and what other product impacts do they cause?

I just added some chmod and chown in the scripts/devcontainer/_start-background-services, I believe it won't have any other product impacts.

How are you sure this works/how was this tested?

I didn't test it, but after adding this shell commands, I got -- Initial compile succeeded finally.

What is the reversion plan if this fails after shipping?

Just rollback this script.

Has this information been included in the comments?

No, since it's an one-off script.

@CLAassistant
Copy link

CLAassistant commented Nov 23, 2022

CLA assistant check
All committers have signed the CLA.

@pbiggar
Copy link
Member

pbiggar commented Nov 23, 2022

Thanks! Have copied this into #4118

@pbiggar pbiggar closed this Nov 23, 2022
@pbiggar pbiggar mentioned this pull request Nov 23, 2022
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.

None yet

3 participants