Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Split E2E into default and full #148

Merged
merged 2 commits into from
Jun 22, 2021
Merged

Conversation

dfarrell07
Copy link
Member

Following the pattern from other repos, split the E2E test job into a
default job that runs a single default-only test always on PRs and a
full job that runs the full matrix when the ready-to-test label is
added.

Signed-off-by: Daniel Farrell dfarrell@redhat.com

@dfarrell07 dfarrell07 added this to the 0.10-m3 milestone Jun 11, 2021
@dfarrell07 dfarrell07 self-assigned this Jun 11, 2021
@dfarrell07 dfarrell07 marked this pull request as ready for review June 11, 2021 19:21
Following the pattern from other repos, split the E2E test job into a
default job that runs a single default-only test always on PRs and a
full job that runs the full matrix when the ready-to-test label is
added.

Signed-off-by: Daniel Farrell <dfarrell@redhat.com>
@dfarrell07
Copy link
Member Author

There's a lot of flakiness showing up in the Flake Finder here and in the submariner-operator repo. I think these failures are the same, not something new caused by this PR.

https://github.com/submariner-io/submariner-operator/actions?query=workflow%3A%22Flake+Finder%22
https://github.com/submariner-io/submariner-charts/actions?query=workflow%3A%22Flake+Finder%22

@dfarrell07 dfarrell07 requested a review from nyechiel June 17, 2021 16:19
@submariner-bot
Copy link
Contributor

🤖 Created branch: z_pr148/dfarrell07/split_e2e

@skitt skitt merged commit 97782b4 into submariner-io:devel Jun 22, 2021
@submariner-bot
Copy link
Contributor

🤖 Closed branches: [z_pr148/dfarrell07/split_e2e]

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants