delete non-junit file before junit merge

[bparees]

No description provided.

[openshift-merge-robot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: bparees

The full list of commands accepted by this bot can be found here.

Details

Needs approval from an approver in each of these OWNERS Files:

• test/extended/OWNERS [bparees]

You can indicate your approval by writing /approve in a comment
You can cancel your approval by writing /approve cancel in a comment

[bparees]

/test extended_image_ecosystem
/test_extended_builds

[bparees]

@stevekuznetsov fyi

[bparees]

(the right fix for this is #16068 but that is on hold pending @smarterclayton's confirmation that we don't need junit merge for anything else)

[stevekuznetsov]

@bparees the 10x simpler solutions is to pass in TEST_REPORT_DIR that is not OUTPUT_DIR or ARTIFACT_DIR but some subdir. Why not do that?

[bparees]

@bparees the 10x simpler solutions is to pass in TEST_REPORT_DIR that is not OUTPUT_DIR or ARTIFACT_DIR but some subdir. Why not do that?

because i don't know what the subdir is supposed to be. and i'd have to change everywhere it's being set. and this function is going away.

[stevekuznetsov]

TEST_REPORT_DIR is declarative -- you pass it in to set it. But yes, let's just delete the junitmerge functionality instead.

/close

[bparees]

@stevekuznetsov i need a solution to junit merge today. since @smarterclayton is blocking the other PR, this is the expedient solution in the meantime.

[stevekuznetsov]

since @smarterclayton is blocking the other PR

His worry about Gubernator is unfounded. That PR should be unblocked.

[stevekuznetsov]

Please set $TEST_REPORT_DIR to some subdir of artifacts and mkdir it:

test/extended/networking.sh:373:    export TEST_REPORT_DIR="${ARTIFACT_DIR}"
test/extended/setup.sh:53:              export TEST_REPORT_DIR="${ARTIFACT_DIR}"

[bparees]

@stevekuznetsov has convinced me this probably won't work.

[openshift-ci-robot]

@bparees: The following tests failed, say /retest to rerun them all:

Test name	Commit	Details	Rerun command
ci/openshift-jenkins/extended_image_ecosystem	0f24396	link	/test extended_image_ecosystem
ci/openshift-jenkins/extended_conformance_gce	0f24396	link	/test extended_conformance_gce

Full PR test history. Your PR dashboard. Please help us cut down on flakes by linking to an open issue when you hit one in your PR.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

[stevekuznetsov]

@openshift-ci-robot stop, it's already dead