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
unit tests: Skip flaky tests on Windows (part 2) #116659
unit tests: Skip flaky tests on Windows (part 2) #116659
Conversation
This issue is currently awaiting triage. If a SIG or subproject determines this is a relevant issue, they will accept it by applying the The 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. |
/assign @ffromani |
25acf91
to
7547dd1
Compare
}, | ||
} | ||
for _, tt := range tests { | ||
t.Run(tt.name, func(t *testing.T) { | ||
// TODO: Remove skip once flakyness is resolved. |
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.
Is there some issue/PR that we could refer here to better understand when to enable?
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.
big +1 to this. IMO Skip is fine, TODO is fine, just need a reference to learn when we can re-evaluate (it was done for other TODO in this PR if I'm not mistaken?)
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.
/assign @pacoxu |
7547dd1
to
1cde591
Compare
/lgtm |
LGTM label has been added. Git tree hash: 93e4cec276d7de7f2d9bea0b2d4d7cb9b1b5680e
|
moving test-only PRs out of the milestone since we're past test freeze |
Some of the unit tests are currently flaky on Windows. This commit skips them until they are resolved.
1cde591
to
0979d55
Compare
/milestone v1.28 |
1 similar comment
/milestone v1.28 |
/lgtm |
LGTM label has been added. Git tree hash: 4bd9165ecf6b982fe7de0298f8435a41b33462c5
|
/cc @msau42 |
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: claudiubelu, mrunalp, msau42 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 |
// TODO: Remove skip once https://github.com/kubernetes/kubernetes/issues/116693 is fixed. | ||
if goruntime.GOOS == "windows" { | ||
t.Skip("Skipping test on Windows.") | ||
} |
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.
@@ -197,6 +197,10 @@ func TestDevicePluginReRegistration(t *testing.T) { | |||
// While testing above scenario, plugin discovery and registration will be done using | |||
// Kubelet probe based mechanism | |||
func TestDevicePluginReRegistrationProbeMode(t *testing.T) { | |||
// TODO: Remove skip once https://github.com/kubernetes/kubernetes/pull/115269 merges. |
This comment was marked as abuse.
This comment was marked as abuse.
Sorry, something went wrong.
What type of PR is this?
/kind failing-test
/kind flake
/sig windows
/sig testing
/priority important-soon
/milestone v1.27
What this PR does / why we need it:
Some of the unit tests are currently flaky on Windows [0]. This commit skips them until they are resolved.
Fixes for the flakes have been proposed, but not merged yet [1][2].
[0] https://testgrid.k8s.io/sig-windows-signal#windows-unit-master
[1] #114607
[2] #115269
Which issue(s) this PR fixes:
Related: #51540
Special notes for your reviewer:
Does this PR introduce a user-facing change?
Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.: