Skip to content

Commit 1881187

Browse files
committed
fix: devtool fix_perms command
PR #4856 extended the `fix_perms` devtool command to also fix permissions of the directory in which we put CI artifacts from `build_ci_artifacts` command. Since, we are now calling `fix_perms` from two different "workflows", i.e. build/test and CI artifacts building, it might be the case that some of the directories `fix_perms` is trying to fix permissions for, don't exist. Add `-f` flag to `chown` command of `fix_perms`, to avoid issues. Signed-off-by: Babis Chalios <[email protected]>
1 parent d84d76c commit 1881187

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

tools/devtool

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -233,7 +233,7 @@ cmd_fix_perms() {
233233
run_devctr \
234234
--workdir "$CTR_FC_ROOT_DIR" \
235235
-- \
236-
chown -R "$(id -u):$(id -g)" "$CTR_FC_BUILD_DIR" "$CTR_TEST_RESULTS_DIR" "$CTR_CI_ARTIFACTS_PATH"
236+
chown -f -R "$(id -u):$(id -g)" "$CTR_FC_BUILD_DIR" "$CTR_TEST_RESULTS_DIR" "$CTR_CI_ARTIFACTS_PATH"
237237
}
238238

239239
# Builds the development container from its Dockerfile.

0 commit comments

Comments
 (0)