-
Notifications
You must be signed in to change notification settings - Fork 3.9k
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
chore: tests don't run on Linux system #19989
Conversation
The first argument to `mkdtemp` is a *prefix*, not a *parent directory*. In other words, on a regular Linux system to following code: ``` mkdtemp(os.tmpdir()) ``` Tries to make a *sibling* of the temp directory instead of a child (for example, `/tmpABC123` instead of `/tmp/ABC123`), which then fails with "access denied". This doesn't fail on Mac because the `tmp` directory is in another directory with write permissions so we can create a sibling directory. And in CodeBuild we run as root so we have permissions to create a sibling dir in the root. Fix it for other systems.
Thank you for contributing! Your pull request will be updated from master and then merged automatically (do not update manually, and be sure to allow changes to be pushed to your fork). |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
Thank you for contributing! Your pull request will be updated from master and then merged automatically (do not update manually, and be sure to allow changes to be pushed to your fork). |
The first argument to `mkdtemp` is a *prefix*, not a *parent directory*. In other words, on a regular Linux system the following code: ``` mkdtemp(os.tmpdir()) ``` Tries to make a *sibling* of the temp directory instead of a child (for example, `/tmpABC123` instead of `/tmp/ABC123`), which then fails with "access denied". This doesn't fail on Mac because the `tmp` directory is in another directory with write permissions so we can create a sibling directory. And in CodeBuild we run as root so we have permissions to create a sibling dir in the root. Fix it for other systems. ---- *By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
The first argument to
mkdtemp
is a prefix, not a parent directory.In other words, on a regular Linux system the following code:
Tries to make a sibling of the temp directory instead of a child (for
example,
/tmpABC123
instead of/tmp/ABC123
), which then fails with"access denied".
This doesn't fail on Mac because the
tmp
directory is in anotherdirectory with write permissions so we can create a sibling directory.
And in CodeBuild we run as root so we have permissions to create a
sibling dir in the root.
Fix it for other systems.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license