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

Try to fix permission of circleci cache #2385

Merged
merged 1 commit into from
Aug 13, 2020
Merged

Try to fix permission of circleci cache #2385

merged 1 commit into from
Aug 13, 2020

Conversation

bjoernricks
Copy link
Contributor

We are currently using two different images for the circleci jobs. For
building gsa and gsad we are using our own images that run under the
root user. For the js jobs we are are using images from circleci which
run under a circleci user. If the yarn.lock file is changed and the
build_gsa job is run before a js job the circle ci cache contains files
where root is the owner and with a wrong directory. Afterwards the js
jons will fail during restoring the cache because the circleci user
can't access the files from the cache.

Checklist:

@bjoernricks bjoernricks requested review from a team and saberlynx August 13, 2020 09:05
We are currently using two different images for the circleci jobs. For
building gsa and gsad we are using our own images that run under the
root user. For the js jobs we are are using images from circleci which
run under a circleci user. If the yarn.lock file is changed and the
build_gsa job is run before a js job the circle ci cache contains files
where root is the owner and with a wrong directory. Afterwards the js
jons will fail during restoring the cache because the circleci user
can't access the files from the cache.
@bjoernricks bjoernricks merged commit 28eaa7c into greenbone:master Aug 13, 2020
@bjoernricks bjoernricks deleted the try-to-fix-permissions-for-circleci-cache branch August 13, 2020 09:34
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.

3 participants