michaelsembwever commented on code in PR #4211:
URL: https://github.com/apache/cassandra/pull/4211#discussion_r2557830296
##########
.jenkins/Jenkinsfile:
##########
@@ -412,12 +416,15 @@ def test(command, cell) {
dir("build") {
sh """
mkdir -p test/output/${cell.step}
- find test/output -type f -name TEST*.xml -execdir mkdir -p
jdk_${cell.jdk}/${cell.arch} ';' -execdir mv {} jdk_${cell.jdk}/${cell.arch}/{}
';'
+ find test/output -type f -name "TEST*.xml" -execdir mkdir -p
jdk_${cell.jdk}/${cell.arch} ';' -execdir mv {} jdk_${cell.jdk}/${cell.arch}/{}
';'
find test/output -name cqlshlib.xml -execdir mv cqlshlib.xml
${cell.step}/cqlshlib${cell_suffix}.xml ';'
find test/output -name nosetests.xml -execdir mv nosetests.xml
${cell.step}/nosetests${cell_suffix}.xml ';'
"""
junit testResults:
"test/**/TEST-*.xml,test/**/cqlshlib*.xml,test/**/nosetests*.xml",
testDataPublishers: [[$class: 'StabilityTestDataPublisher']]
- sh "find test/output -type f -name *.xml -exec sh -c 'xz -f {} &'
';' ; wait ; find test/output -type f -name *.xml.xz | wc -l"
+ sh """
+ find test/output -type f -name "*.xml" -print0 | xargs -0 -r
-n1 -P"\$(nproc)" xz -f
+ echo "test result files compressed"; find test/output -type f
-name "*.xml.xz" | wc -l
+ """
Review Comment:
this is the bugfix for missing test results in the summary file.
`wait` wasn't waiting for backgrounded child tasks inside the `sh -c …`
it was simpler to replace it with `xargs -P …`
ref: https://the-asf.slack.com/archives/CK23JSY2K/p1763714160975359
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
