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.

Sign up for GitHub

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

Jump to bottom

Fix permissions on temp dirs #189

Merged
merged 3 commits into from
Aug 5, 2020
Merged

Fix permissions on temp dirs #189

merged 3 commits into from
Aug 5, 2020

Conversation

exodist
Copy link
Member

@exodist exodist commented Jul 9, 2020

Fixes #185

@exodist exodist requested a review from rjbs July 9, 2020 04:29
@rjbs
Copy link
Contributor

rjbs commented Jul 25, 2020 •
edited
Loading

Short answer: It didn't solve the problem. I ended up with

Caught exception in subtest: Failed to start login handshake: 
$VAR1 = '500 Error in tempfile() using template /tmp/yath-229591-f2a_3Q/tmp/kKgGVq/XXXXXXXXXX: 
Could not create temp file /tmp/yath-229591-f2a_3Q/tmp/kKgGVq/6Unq1DqbCZ: Permission denied

I did not dig deeper than that, yet. The key thing to know is that yath is being run as root (not ideal, but necessary for now) and the individual test changes effective user as part of testing a service that runs as that user.

@exodist
Copy link
Member Author

exodist commented Jul 25, 2020 via email

@exodist exodist merged commit fa94b94 into master Aug 5, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@atoomic atoomic atoomic approved these changes

@rjbs rjbs Awaiting requested review from rjbs

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

TMPDIR is taken over, but weird permissions are used
3 participants
@exodist @rjbs @atoomic