Skip to content
This repository has been archived by the owner on Jun 26, 2024. It is now read-only.

[WIP] Fix random e2e failures. (#534 follow-up) #539

Closed
wants to merge 3 commits into from

Conversation

pmacik
Copy link
Contributor

@pmacik pmacik commented Jul 3, 2020

Motivation

fix E2E failure

Changes

  • Seed random with time.Now to generate unique digits on every subtest call.

  • Update github.com/stretchr/testify from v1.4.0 to v1.6.1
    The 1.4.0 version we were using had a bug in the Eventually function,
    which caused flaky results.

  • Increasing timeout for Eventually function

  • Update vendored dependencies.

Testing

make test-e2e

@openshift-ci-robot
Copy link
Collaborator

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
To complete the pull request process, please assign sbose78
You can assign the PR to them by writing /assign @sbose78 in a comment when ready.

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

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@pmacik
Copy link
Contributor Author

pmacik commented Jul 3, 2020

Followup on #534

@pmacik pmacik changed the title Pjagrut fix e2e [WIP] Fix random e2e failure. #534 Jul 3, 2020
@pmacik pmacik changed the title [WIP] Fix random e2e failure. #534 [WIP] Fix random e2e failures. (#534 follow-up) Jul 3, 2020
@pmacik
Copy link
Contributor Author

pmacik commented Jul 3, 2020

/retest

pratikjagrut and others added 3 commits July 3, 2020 13:57
- Seed random with time.Now to generate unique digits on every subtest call.

- Update github.com/stretchr/testify from v1.4.0 to v1.6.1
	The 1.4.0 version we were using had a bug in the Eventually function,
	which caused flacky results.

- Update vendored dependencies.
@pmacik
Copy link
Contributor Author

pmacik commented Jul 3, 2020

/retest

@openshift-ci-robot
Copy link
Collaborator

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

Test name Commit Details Rerun command
ci/prow/4.5-e2e 3771507 link /test 4.5-e2e
ci/prow/e2e 3771507 link /test e2e

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.

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.

@openshift-ci-robot
Copy link
Collaborator

@pmacik: PR needs rebase.

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.

@pmacik pmacik deleted the pjagrut-fix_e2e branch November 27, 2020 16:16
This pull request was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants