-
Notifications
You must be signed in to change notification settings - Fork 4.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
Add all mount flags to start command #12930
Conversation
/ok-to-test |
kvm2 driver with docker runtime
Times for minikube start: 49.8s 46.3s 48.5s 45.4s 45.8s Times for minikube ingress: 32.2s 30.8s 31.8s 31.3s 31.8s docker driver with docker runtime
Times for minikube start: 22.0s 21.0s 21.3s 21.9s 20.1s Times for minikube ingress: 26.4s 27.4s 34.9s 27.4s 28.0s docker driver with containerd runtime
Times for minikube start: 28.7s 41.3s 41.5s 41.6s 40.5s Times for minikube (PR 12930) ingress: 28.4s 32.0s 17.4s 17.4s 21.9s |
These are the flake rates of all failed tests.
Too many tests failed - See test logs for more details. To see the flake rates of all tests by environment, click here. |
kvm2 driver with docker runtime
Times for minikube start: 47.2s 45.3s 46.2s 45.7s 47.0s Times for minikube ingress: 32.2s 31.2s 31.3s 31.2s 31.7s docker driver with docker runtime
Times for minikube start: 22.6s 21.0s 20.7s 20.7s 20.7s Times for minikube ingress: 26.4s 25.9s 26.9s 26.9s 25.9s docker driver with containerd runtime
Times for minikube start: 28.5s 41.3s 40.6s 41.0s 40.9s Times for minikube ingress: 27.4s 31.9s 27.4s 18.4s 18.4s |
These are the flake rates of all failed tests.
Too many tests failed - See test logs for more details. To see the flake rates of all tests by environment, click here. |
kvm2 driver with docker runtime
Times for minikube start: 48.0s 46.4s 45.9s 46.8s 46.2s Times for minikube ingress: 32.2s 31.3s 30.2s 30.2s 30.8s docker driver with docker runtime
Times for minikube start: 21.1s 21.7s 20.5s 23.0s 22.1s Times for minikube ingress: 27.4s 26.9s 26.9s 26.9s 34.4s docker driver with containerd runtime
Times for minikube start: 29.6s 41.2s 41.9s 40.7s 42.3s Times for minikube ingress: 28.9s 32.9s 32.9s 32.9s 32.9s |
These are the flake rates of all failed tests.
Too many tests failed - See test logs for more details. To see the flake rates of all tests by environment, click here. |
kvm2 driver with docker runtime
Times for minikube start: 47.9s 48.0s 47.2s 46.9s 47.3s Times for minikube ingress: 31.9s 31.3s 32.9s 32.3s 31.9s docker driver with docker runtime
Times for minikube start: 22.8s 20.9s 21.7s 22.1s 21.6s Times for minikube ingress: 27.5s 34.5s 27.5s 34.5s 27.0s docker driver with containerd runtime
Times for minikube start: 25.4s 42.5s 43.4s 44.5s 41.5s Times for minikube ingress: 71.9s 19.9s 27.4s 28.9s 36.0s |
These are the flake rates of all failed tests.
Too many tests failed - See test logs for more details. To see the flake rates of all tests by environment, click here. |
Waiting for #13018 to be merged to fix mode failure in tests |
2ae1aa8
to
9439da2
Compare
kvm2 driver with docker runtime
Times for minikube start: 48.6s 45.0s 46.0s 46.1s 46.0s Times for minikube ingress: 30.3s 31.7s 30.7s 30.7s 30.2s docker driver with docker runtime
Times for minikube start: 21.3s 20.7s 21.1s 20.7s 21.0s Times for minikube ingress: 25.9s 33.9s 27.4s 27.0s 26.4s docker driver with containerd runtime
Times for minikube start: 41.0s 28.3s 44.5s 41.5s 40.8s Times for minikube (PR 12930) ingress: 32.4s 27.9s 32.4s 32.4s 31.9s |
These are the flake rates of all failed tests.
Too many tests failed - See test logs for more details. To see the flake rates of all tests by environment, click here. |
dd0eff2
to
4e598b6
Compare
kvm2 driver with docker runtime
Times for minikube start: 49.4s 47.1s 47.7s 46.9s 47.0s Times for minikube ingress: 30.8s 30.4s 30.3s 30.8s 30.8s docker driver with docker runtime
Times for minikube start: 22.1s 21.4s 22.1s 21.6s 21.0s Times for minikube ingress: 33.9s 25.9s 27.9s 27.5s 25.9s docker driver with containerd runtime
Times for minikube start: 28.5s 40.8s 40.8s 36.3s 36.6s Times for minikube ingress: 30.7s 29.4s 22.4s 24.9s 23.9s |
These are the flake rates of all failed tests.
To see the flake rates of all tests by environment, click here. |
kvm2 driver with docker runtime
Times for minikube start: 49.1s 47.2s 46.1s 48.5s 48.0s Times for minikube (PR 12930) ingress: 31.0s 32.4s 30.8s 31.9s 31.4s docker driver with docker runtime
Times for minikube start: 22.4s 21.8s 22.1s 21.7s 21.4s Times for minikube ingress: 35.4s 27.9s 34.4s 27.5s 27.4s docker driver with containerd runtime
Times for minikube ingress: 34.9s 32.9s 31.9s 33.0s 36.4s Times for minikube start: 25.9s 41.4s 42.4s 42.5s 41.4s |
These are the flake rates of all failed tests.
Too many tests failed - See test logs for more details. To see the flake rates of all tests by environment, click here. |
a19f53b
to
8091907
Compare
kvm2 driver with docker runtime
Times for minikube start: 45.9s 45.1s 45.3s 44.4s 46.4s Times for minikube ingress: 28.6s 30.1s 29.1s 29.1s 29.5s docker driver with docker runtime
Times for minikube start: 30.8s 26.2s 26.4s 25.6s 26.2s Times for minikube ingress: 21.9s 21.4s 22.0s 22.9s 83.9s docker driver with containerd runtime
Times for minikube start: 30.2s 29.2s 40.7s 30.7s 41.0s Times for minikube (PR 12930) ingress: 28.9s 29.9s 33.4s 29.4s 28.4s |
These are the flake rates of all failed tests.
Too many tests failed - See test logs for more details. To see the flake rates of all tests by environment, click here. |
d3b6ef6
to
7b3377e
Compare
@@ -83,6 +83,15 @@ const ( | |||
imageRepository = "image-repository" | |||
imageMirrorCountry = "image-mirror-country" | |||
mountString = "mount-string" | |||
mount9PVersion = "mount-9p-version" | |||
mountGID = "mount-gid" | |||
mountIPFlag = "mount-ip" |
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.
Some have "Flag" at the end and some do not. Are all of these flags? Could we drop "Flag" for consistency between these?
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.
mountIP
, mountPort
& mountType
are already defined in mount.go
so I had to differentiate them
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.
Alright, works for me.
for _, option := range cc.MountOptions { | ||
args = append(args, fmt.Sprintf("--%s", constants.MountOptionsFlag), option) | ||
} | ||
return args |
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.
Are there any cases where flags could possess inconsistent values?
Should we have logic, to say, for example, that mountSize value is out of range (maybe we want to do this in a separate PR).
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.
Yeah, I can add that is a follow up PR, I created an issue
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.
Makes sense!
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.
Added a few comments, mainly curious about if validating flag values is needed.
kvm2 driver with docker runtime
Times for minikube (PR 12930) start: 53.0s 50.9s 53.0s 54.5s 52.0s Times for minikube ingress: 32.4s 31.9s 32.9s 32.4s 31.4s docker driver with docker runtime
Times for minikube ingress: 35.5s 35.0s 27.0s 35.5s 26.5s Times for minikube start: 25.2s 24.2s 24.0s 24.4s 23.9s docker driver with containerd runtime
Times for minikube ingress: 33.5s 34.0s 28.0s 37.0s 36.5s Times for minikube start: 30.5s 43.9s 43.5s 43.7s 43.0s |
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.
Thanks for fixing this
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: henrik242, klaases, sharifelgamal, spowelljr 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 |
These are the flake rates of all failed tests.
To see the flake rates of all tests by environment, click here. |
6fd621c
to
3ad2f8d
Compare
These are the flake rates of all failed tests.
To see the flake rates of all tests by environment, click here. |
Closes #12802
Implemented all the mount flags from
minikube mount
intominikube start
.