This is an automated email from the ASF dual-hosted git repository.
gurwls223 pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/spark.git
The following commit(s) were added to refs/heads/master by this push:
new 2e526b543655 [SPARK-52179][INFRA][FOLLOW-UP] Do not quote wildcard in
logs
2e526b543655 is described below
commit 2e526b543655e2a6e769a2bbee00955a895566e9
Author: Hyukjin Kwon <[email protected]>
AuthorDate: Thu May 22 13:15:41 2025 +0900
[SPARK-52179][INFRA][FOLLOW-UP] Do not quote wildcard in logs
### What changes were proposed in this pull request?
This PR is a followup of https://github.com/apache/spark/pull/50911 that
does not quote wildcard in logs
### Why are the changes needed?
To correctly catch log files.
### Does this PR introduce _any_ user-facing change?
No, dev-only.
### How was this patch tested?
In my forked repository.
### Was this patch authored or co-authored using generative AI tooling?
No.
Closes #50979 from HyukjinKwon/SPARK-52179-followup.
Authored-by: Hyukjin Kwon <[email protected]>
Signed-off-by: Hyukjin Kwon <[email protected]>
---
.github/workflows/release.yml | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/.github/workflows/release.yml b/.github/workflows/release.yml
index c97777b4e404..9a43ed0cdbea 100644
--- a/.github/workflows/release.yml
+++ b/.github/workflows/release.yml
@@ -190,10 +190,10 @@ jobs:
# Zip logs/output
if [ "$DRYRUN_MODE" = "1" ]; then
- zip logs.zip "$RELEASE_DIR/docker-build.log" "$OUTPUT_DIR/*.log"
+ zip logs.zip "$RELEASE_DIR/docker-build.log" "$OUTPUT_DIR/"*.log
zip -9 output.zip -r "$OUTPUT_DIR"
else
- zip -P "$ASF_PASSWORD" logs.zip "$RELEASE_DIR/docker-build.log"
"$OUTPUT_DIR/*.log"
+ zip -P "$ASF_PASSWORD" logs.zip "$RELEASE_DIR/docker-build.log"
"$OUTPUT_DIR/"*.log
zip -9 -P "$ASF_PASSWORD" output.zip -r "$OUTPUT_DIR"
fi
- name: Upload logs
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
