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
Update storage test suits for Windows #96479
Conversation
@jingxu97: 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. |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: jingxu97 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 |
/assign @Jiawei0227 |
c03cdeb
to
133dc72
Compare
/assign @msau42 |
NodeSelection: node, | ||
PVCsReadOnly: readOnly, | ||
ImageID: e2evolume.GetTestImageId(imageutils.DebianIptables), |
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.
why does this need to use iptables image?
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.
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.
I think dd command has some special options that busybox cannot support
@@ -141,7 +141,7 @@ func (t *volumeIOTestSuite) DefineTests(driver TestDriver, pattern testpatterns. | |||
l.migrationCheck.validateMigrationVolumeOpCounts() | |||
} | |||
|
|||
ginkgo.It("should write files of various sizes, verify size, validate content [Slow]", func() { | |||
ginkgo.It("should write files of various sizes, verify size, validate content [Slow][LinuxOnly]", func() { |
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 it possible to make this testcase work on Windows? Should we open a tracking issue to track all the LinuxOnly test cases and see if it is possible to make it work for 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.
It is possible. But need sometime to find a way to replace those linux commands. I will open an issue for it
test/e2e/storage/utils/utils.go
Outdated
} | ||
|
||
// VerifyExecInPodSucceed verifies shell cmd in target pod succeed | ||
func VerifyExecInPodSucceed(f *framework.Framework, pod *v1.Pod, shExec string) { | ||
stdout, stderr, err := PodExec(f, pod, shExec) | ||
fmt.Printf("shExec %s", shExec) |
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 this for debugging purposes?
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.
removed. thanks!
133dc72
to
b3cf106
Compare
Hmm, there seems to be a build issue? |
b3cf106
to
9ca5cf6
Compare
|
b9b1b4f
to
60ed26e
Compare
60ed26e
to
8143d33
Compare
Update storage test suits to enable some tests on Windows Change-Id: I1f596bb9f7a0f41ae398aec43406ee5c753cdce3
8143d33
to
56373aa
Compare
finally, it passed tests. @Jiawei0227 @msau42 PTAL |
I think it lgtm |
Update storage test suits to enable some tests on Windows
Change-Id: I1f596bb9f7a0f41ae398aec43406ee5c753cdce3
What type of PR is this?
/kind cleanup
What this PR does / why we need it:
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.: