-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
csi: fix filesystem write paths in e2e tests #7361
Conversation
76d1bc1
to
f7779d3
Compare
Removing you from review @langmartin until I've got the mount paths issue fixed. |
f7779d3
to
a19e172
Compare
a19e172
to
1bc808b
Compare
The writer jobs for the CSI e2e tests will write to locations in the alloc dir and not arbitrary locations at the root. This changeset mounts the CSI volume at a location within the alloc dir, uses the alloc ID as a better-namespaced identifier of the file we write, and corrects the shell used by the busybox container.
1bc808b
to
9ebeecb
Compare
I've added a note about #7365 where we can improve this, but this should now fix the problem described here. |
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.
lgtm
The writer jobs for the CSI e2e tests will write to locations in the alloc dir and not arbitrary locations at the root. This changeset mounts the CSI volume at a location within the alloc dir, uses the alloc ID as a better-namespaced identifier of the file we write, and corrects the shell used by the busybox container.
The writer jobs for the CSI e2e tests will write to locations in the alloc dir and not arbitrary locations at the root. This changeset mounts the CSI volume at a location within the alloc dir, uses the alloc ID as a better-namespaced identifier of the file we write, and corrects the shell used by the busybox container.
I'm going to lock this pull request because it has been closed for 120 days ⏳. This helps our maintainers find and focus on the active contributions. |
Fixing a few bugs in the e2e test suite for CSI following testing of #7356.
The writer jobs for the CSI e2e tests will write to locations in the alloc dir and not arbitrary locations at the root. This changeset mounts the CSI volume at a location within the alloc dir, uses the alloc ID as a better-namespaced identifier of the file we write, and corrects the shell used by the busybox container.
This PR doesn't quite bring us to a working state; we'll need to fix #7338, #7360, and #7362 at least.