-
Notifications
You must be signed in to change notification settings - Fork 38.7k
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
tests: Enable some [LinuxOnly] tests for Windows Containerd #97045
tests: Enable some [LinuxOnly] tests for Windows Containerd #97045
Conversation
/retest |
/test pull-kubernetes-bazel-test |
1 similar comment
/test pull-kubernetes-bazel-test |
@claudiubelu It looks like this PR needs some updates to the build files. |
1a53681
to
02f9bc6
Compare
/test pull-kubernetes-e2e-gce-ubuntu-containerd |
/kind cleanup John, Aaron, the changes here looks good to me. However my question is ... was this already discussed / ok-ed by you all? (adding a new tag) |
a38908a
to
45e3059
Compare
Windows Containerd has more features than Windows Docker. One of them is single file mappings, allowing us to also map individual files into containers, not just folders. This will set the tag [Excluded:WindowsDocker] for those tests instead of [LinuxOnly]. Co-authored-by: Mark Rossetti <marosset@microsoft.com>
45e3059
to
91c095a
Compare
/retest |
5 similar comments
/retest |
/retest |
/retest |
/retest |
/retest |
/milestone v1.23 |
/test pull-kubernetes-node-kubelet-serial |
1 similar comment
/test pull-kubernetes-node-kubelet-serial |
@claudiubelu: The following test failed, say
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. |
/skip serial is irrelevant here |
I'd like to see an open issue to remove this tag in v1.24, and the description for the tag updated to link to said issue: kubernetes/community#5355 (review) Once the kubernetes/community PR merges I'm fine letting this merge |
Created issue: kubernetes/community#6123 |
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.
/approve
/lgtm
Just approved the community PR
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: claudiubelu, pacoxu, spiffxp 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 |
What type of PR is this?
/sig windows
/sig testing
What this PR does / why we need it:
Windows Containerd has more features than Windows Docker. One of them is single file mappings, allowing us to also map individual files into containers, not just folders.
This will set the tag
[Excluded:WindowsDocker]
for those tests instead of[LinuxOnly]
.Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:
Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.: