Skip to content

[Backport 2.21-maintenance] Fix adding symlink to the sandbox paths#10470

Merged
thufschmitt merged 2 commits into2.21-maintenancefrom
backport-10456-to-2.21-maintenance
Apr 11, 2024
Merged

[Backport 2.21-maintenance] Fix adding symlink to the sandbox paths#10470
thufschmitt merged 2 commits into2.21-maintenancefrom
backport-10456-to-2.21-maintenance

Conversation

@github-actions
Copy link
Copy Markdown

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

Théophane Hufschmitt and others added 2 commits April 11, 2024 12:17
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 <Artturin@artturin.com>
(cherry picked from commit 913db9f)
@thufschmitt thufschmitt reopened this Apr 11, 2024
@thufschmitt thufschmitt enabled auto-merge April 11, 2024 12:18
@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.21-maintenance branch from b4790c1 to 4c7f69f Compare April 11, 2024 12:35
@thufschmitt thufschmitt merged commit 0c1fcc2 into 2.21-maintenance Apr 11, 2024
@thufschmitt thufschmitt deleted the backport-10456-to-2.21-maintenance branch April 11, 2024 13:14
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