-
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
Sort bind options in JoinMountOptions #69451
Conversation
We were not sorting them previously, which made the order non-deterministic. If we believe the order doesn't matter, let's pick a consistent order to minimize the chances of a rare flake. This also simplifies the unit tests, which were flaking not-very-rarely, e.g. with `bazel test //pkg/volume/awsebs/... --runs_per_test=8`
@justinsb: Adding the "do-not-merge/release-note-label-needed" label because no release-note block was detected, please follow our release note process to remove it. 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. |
/release-note-none |
|
@justinsb yeah the ordering of mount options only matter when they conflict with each other, then last one wins. I think we should be fine. /lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: gnufied, justinsb 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 |
/retest Tests are taking longer and longer to complete, not entirely sure why yet. And so these tests are starting to timeout more often.. |
/sig aws |
We were not sorting them previously, which made the order
non-deterministic. If we believe the order doesn't matter, let's pick a
consistent order to minimize the chances of a rare flake.
This also simplifies the unit tests, which were flaking not-very-rarely,
e.g. with
bazel test //pkg/volume/awsebs/... --runs_per_test=8