Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 0 additions & 2 deletions test/extended/conformance.sh
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,4 @@ TEST_PARALLEL="${PARALLEL_NODES:-5}" TEST_REPORT_FILE_NAME=conformance_parallel
os::log::info "Running serial tests"
TEST_REPORT_FILE_NAME=conformance_serial os::test::extended::run -- -suite "serial.conformance.openshift.io" -test.timeout 2h ${TEST_EXTENDED_ARGS-} || exitstatus=$?

os::test::extended::merge_junit

exit $exitstatus
2 changes: 0 additions & 2 deletions test/extended/core.sh
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,4 @@ os::log::info ""
os::log::info "Running serial tests"
TEST_REPORT_FILE_NAME=core_serial os::test::extended::run -- -suite "serial.conformance.openshift.io" -test.timeout 2h ${TEST_EXTENDED_ARGS-} || exitstatus=$?

os::test::extended::merge_junit

exit $exitstatus
18 changes: 0 additions & 18 deletions test/extended/setup.sh
Original file line number Diff line number Diff line change
Expand Up @@ -23,8 +23,6 @@ function os::test::extended::focus () {
os::log::info "Running serial tests with focus ${FOCUS}"
TEST_REPORT_FILE_NAME=core_parallel os::test::extended::run -- -suite "serial.conformance.openshift.io" -test.timeout 6h ${TEST_EXTENDED_ARGS-} || exitstatus=$?

os::test::extended::merge_junit

exit $exitstatus
fi
}
Expand All @@ -39,7 +37,6 @@ function os::test::extended::setup () {
os::util::ensure::built_binary_exists 'extended.test' 'test/extended/extended.test'
os::util::ensure::built_binary_exists 'oadm'
os::util::ensure::built_binary_exists 'oc'
os::util::ensure::built_binary_exists 'junitmerge'

# ensure proper relative directories are set
export KUBE_REPO_ROOT="${OS_ROOT}/vendor/k8s.io/kubernetes"
Expand Down Expand Up @@ -238,18 +235,3 @@ function os::test::extended::test_list () {
export TEST_COUNT=${#selected_tests[@]}
}
readonly -f os::test::extended::test_list

# Merge all of the JUnit output files in the TEST_REPORT_DIR into a single file.
# This works around a gap in Jenkins JUnit reporter output that double counts skipped
# files until https://github.com/jenkinsci/junit-plugin/pull/54 is merged.
function os::test::extended::merge_junit () {
if [[ -z "${JUNIT_REPORT:-}" ]]; then
return
fi
local output
output="$( mktemp )"
"$( os::util::find::built_binary junitmerge )" "${TEST_REPORT_DIR}"/*.xml > "${output}"
rm "${TEST_REPORT_DIR}"/*.xml
mv "${output}" "${TEST_REPORT_DIR}/junit.xml"
}
readonly -f os::test::extended::merge_junit
4 changes: 0 additions & 4 deletions tools/junitmerge/OWNERS

This file was deleted.

155 changes: 0 additions & 155 deletions tools/junitmerge/junitmerge.go

This file was deleted.