stop calling junit merge.

[bparees]

since jenkinsci/junit-plugin#54 was merged,
it should not be needed anymore.

[bparees]

@stevekuznetsov ptal

[bparees]

/test extended_builds

[bparees]

/test extended_builds

[jim-minter]

lgtm but leaving it to @stevekuznetsov

[stevekuznetsov]

/unassign @Kargakis
/assign @smarterclayton
/assign

[bparees]

/test extended_builds

[smarterclayton]

It's required for other tools.

[bparees]

It's required for other tools.

are you saying we need to leave the junitmerge tool here, or are you saying we can't stop calling junit merge at all?

[bparees]

/retest

[smarterclayton]

I'm still investigating whether we need it for gubernator, so it's not appropriate to remove it yet.

…

On Wed, Aug 30, 2017 at 10:28 PM, OpenShift CI Robot < ***@***.***> wrote: @bparees <https://github.com/bparees>: The following tests *failed*, say /retest to rerun them all: Test name Commit Details Rerun command ci/openshift-jenkins/extended_conformance_install_update 8bc17ed <8bc17ed> link <https://openshift-gce-devel.appspot.com/build/origin-ci-test/pr-logs/pull/16068/test_pull_request_origin_extended_conformance_install_update/5161/> /test extended_conformance_install_update ci/openshift-jenkins/extended_builds 3f26fdf <3f26fdf> link <https://openshift-gce-devel.appspot.com/build/origin-ci-test/pr-logs/pull/16068/test_pull_request_origin_extended_builds/165/> /test extended_builds Full PR test history <https://openshift-gce-devel.appspot.com/pr/16068>. Your PR dashboard <https://openshift-gce-devel.appspot.com/pr/bparees>. Please help us cut down on flakes by linking to <https://github.com/kubernetes/community/blob/master/contributors/devel/flaky-tests.md#filing-issues-for-flaky-tests> an open issue <https://github.com/openshift/origin/issues?q=is:issue+is:open> when you hit one in your PR. Instructions for interacting with me using PR comments are available here <https://github.com/kubernetes/community/blob/master/contributors/devel/pull-requests.md>. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra <https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:> repository. I understand the commands that are listed here <https://github.com/kubernetes/test-infra/blob/master/commands.md>. — You are receiving this because you were assigned. Reply to this email directly, view it on GitHub <#16068 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ABG_p9hqSdvYGfrYkVwnJ3QqzwXqTOqzks5sdhoxgaJpZM4PIESj> .

[stevekuznetsov]

I confirmed we're fine for Gubernator -- it expects many jUnit files

[bparees]

/retest

[bparees]

I need a decision on whether this PR can move forward, it's screwing up our extended test runs.

[stevekuznetsov]

In my opinion it should be good

[stevekuznetsov]

/lgtm

[openshift-merge-robot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: bparees, stevekuznetsov

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,stevekuznetsov]
• tools/junitmerge/OWNERS [bparees,stevekuznetsov]

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

[0xmichalis]

/hold

[bparees]

@stevekuznetsov @smarterclayton bump.

[stevekuznetsov]

So we do NOT need this for Gubernator ... @smarterclayton any objections?

[bparees]

/hold cancel

[bparees]

(i heard no objections).

[openshift-merge-robot]

Automatic merge from submit-queue.