-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
disable parallel test runs to prevent test skips #20593
Conversation
@@ -196,7 +196,7 @@ periodics: | |||
# Skipping "Should recreate evicted statefulset" because of an issue in dockershim for dualstack | |||
# Suggested fix - https://github.com/kubernetes/kubernetes/pull/94382 | |||
- --test_args=--ginkgo.focus=\[Feature:IPv6DualStackAlphaFeature\]|\[Conformance\] --ginkgo.skip=\[Slow\]|\[Serial\]|\[Flaky\]|Should.recreate.evicted.statefulset | |||
- --ginkgo-parallel=20 | |||
- --ginkgo-parallel=1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
IIRC you just need to remove the ginkgo-parallel line entirely.
In that case you can run the serial tests too, that are being skipped now, FYI
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
So I'm following the same format for all the jobs defined in the yaml: https://github.com/kubernetes/test-infra/blob/master/config/jobs/kubernetes/sig-network/dualstack-e2e.yaml#L152.
We can enable Serial
tests in a follow up PR after few test runs. I'm planning to do it incrementally 😄
btw, 1.20
branch tests are running all the dual-stack tests as expected.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fair enough
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: aojea, aramase The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
@aramase: Updated the
In response to this:
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. |
Running in parallel causes some of the tests to be skipped because of unavailable nodes. So setting
--ginkgo-parallel=1
to ensure all the dual-stack tests are run.We can optimize the # in a follow up PR to reduce the time taken to run the tests.
/assign @aojea
cc @lachie83 @bridgetkromhout