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

[Backport 2.19-maintenance] Fix adding symlink to the sandbox paths #10469

Merged
merged 2 commits into from
Apr 11, 2024

Conversation

github-actions[bot]
Copy link

Automatic backport to 2.19-maintenance, triggered by a label in #10456.

Théophane Hufschmitt added 2 commits April 11, 2024 12:08
Regression test for #9579

(cherry picked from commit 872d93e)
…o a symlink out of the store

Bind-mounting symlinks is apparently not possible, which is why the
thing was failing.

Fortunately, symlinks are small, so we can fallback to copy them at no cost.

Fix #9579

Co-authored-by: Artturin <[email protected]>
(cherry picked from commit 913db9f)
@thufschmitt thufschmitt reopened this Apr 11, 2024
@github-actions github-actions bot added the with-tests Issues related to testing. PRs with tests have some priority label Apr 11, 2024
@thufschmitt thufschmitt force-pushed the backport-10456-to-2.19-maintenance branch from ccc37a2 to 3461198 Compare April 11, 2024 12:31
@thufschmitt thufschmitt merged commit 7986891 into 2.19-maintenance Apr 11, 2024
12 checks passed
@thufschmitt thufschmitt deleted the backport-10456-to-2.19-maintenance branch April 11, 2024 13:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
with-tests Issues related to testing. PRs with tests have some priority
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant