Skip to content

Commit 09e558a

Browse files
committed
chore: fix permission on test_results directory
test_results contains files from failed tests from debugging, but the directory is created inside of a privileged docker container. Thus after finishing a test run, fix up the permissions so that it can easily be dealt with. Signed-off-by: Patrick Roy <[email protected]>
1 parent d78fb96 commit 09e558a

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

tools/devtool

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -110,6 +110,7 @@ CTR_FC_ROOT_DIR="/firecracker"
110110

111111
# Full path to the build dir, as bind-mounted in the container.
112112
CTR_FC_BUILD_DIR="${CTR_FC_ROOT_DIR}/build"
113+
CTR_TEST_RESULTS_DIR="${CTR_FC_ROOT_DIR}/test_results"
113114

114115
# Full path to the cargo target dir, as bind-mounted in the container.
115116
CTR_CARGO_TARGET_DIR="$CTR_FC_BUILD_DIR/cargo_target"
@@ -230,7 +231,7 @@ cmd_fix_perms() {
230231
run_devctr \
231232
--workdir "$CTR_FC_ROOT_DIR" \
232233
-- \
233-
chown -R "$(id -u):$(id -g)" "$CTR_FC_BUILD_DIR"
234+
chown -R "$(id -u):$(id -g)" "$CTR_FC_BUILD_DIR" "$CTR_TEST_RESULTS_DIR"
234235
}
235236

236237
# Builds the development container from its Dockerfile.

0 commit comments

Comments
 (0)