Skip to content

Commit 37a2530

Browse files
committed
[OPENJDK-2408] s2i: use --archive flag for rsync
Switch from '-rl' to '--archive' which attempts to preserve as much metadata from the source files as possible, including timestamps, the motivation behind OPENJDK-2408. https://issues.redhat.com/browse/OPENJDK-2408 Signed-off-by: Jonathan Dowland <[email protected]>
1 parent 338d45b commit 37a2530

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

modules/s2i/bash/artifacts/opt/jboss/container/java/s2i/maven-s2i-overrides

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@ function maven_s2i_custom_binary_build() {
2929
binary_dir="${S2I_SOURCE_DIR}"
3030
fi
3131
log_info "Copying binaries from ${binary_dir} to ${S2I_TARGET_DEPLOYMENTS_DIR} ..."
32-
rsync -rl --out-format='%n' "${binary_dir}"/ "${S2I_TARGET_DEPLOYMENTS_DIR}"
32+
rsync --archive --out-format='%n' "${binary_dir}"/ "${S2I_TARGET_DEPLOYMENTS_DIR}"
3333
}
3434

3535
function maven_s2i_deploy_artifacts_override() {

modules/s2i/core/artifacts/opt/jboss/container/s2i/core/s2i-core

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,7 @@ function s2i_core_copy_configuration() {
5353
mkdir -pm 775 "${S2I_TARGET_CONFIGURATION_DIR}"
5454
fi
5555
log_info "Copying configuration from $(realpath --relative-to ${S2I_SOURCE_DIR} ${1}/${S2I_SOURCE_CONFIGURATION_DIR}) to ${S2I_TARGET_CONFIGURATION_DIR}..."
56-
rsync -rl --out-format='%n' "${1}/${S2I_SOURCE_CONFIGURATION_DIR}"/ "${S2I_TARGET_CONFIGURATION_DIR}"
56+
rsync --archive --out-format='%n' "${1}/${S2I_SOURCE_CONFIGURATION_DIR}"/ "${S2I_TARGET_CONFIGURATION_DIR}"
5757
fi
5858
fi
5959
}
@@ -70,7 +70,7 @@ function s2i_core_copy_data() {
7070
mkdir -pm 775 "${S2I_TARGET_DATA_DIR}"
7171
fi
7272
log_info "Copying app data from $(realpath --relative-to ${S2I_SOURCE_DIR} ${1}/${S2I_SOURCE_DATA_DIR}) to ${S2I_TARGET_DATA_DIR}..."
73-
rsync -rl --out-format='%n' "${1}/${S2I_SOURCE_DATA_DIR}"/ "${S2I_TARGET_DATA_DIR}"
73+
rsync --archive --out-format='%n' "${1}/${S2I_SOURCE_DATA_DIR}"/ "${S2I_TARGET_DATA_DIR}"
7474
# s2i used to be more forgiving, but the build will fail if this call
7575
# fails. emit a warning and allow the build to succeed
7676
chmod -R g+rwX "${S2I_TARGET_DATA_DIR}" || log_warning "Errors occurred while adding read/write permissions to S2I_TARGET_DATA_DIR ($S2I_TARGET_DATA_DIR)."

0 commit comments

Comments
 (0)