-
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 (host|control-plane).minikube.internal
to /etc/hosts
#7247
Conversation
/ok-to-test |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: tstromberg 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 |
All Times Minikube (PR 7247): [ 64.669505 73.218519 67.125694] Average minikube: 67.429307 Averages Time Per Log
|
seems much higher than HEAD and it is suspecitous that only lasted 3 mins. maybe need a restart test on a failed machine |
The docker tests are panicking on the dashboard test:
|
The test data yaml files need to be changed for the unit tests to pass, those files are easily the most cumbersome things to maintain in this codebase. |
/retest-this-please |
/retest-this-please |
Times for minikube: [66.384619117 65.66567577500001 65.211770642] Times for Minikube (PR 7247): [67.97513913600001 64.16424909 67.42762365400002] Averages Time Per Log
|
Times for minikube: [66.856650408 66.61344204 67.77245690600002] Times for Minikube (PR 7247): [63.757944927 65.967490875 64.989290255] Averages Time Per Log
|
Times for minikube: [67.39853239399999 67.078169953 67.618235001] Times for Minikube (PR 7247): [67.22304196499998 65.049547914 67.788381618] Averages Time Per Log
|
Times for minikube: [69.667447457 67.69201342200002 64.361016879] Times for Minikube (PR 7247): [66.66178811 69.179885686 67.70493067799998] Averages Time Per Log
|
Times for minikube: [69.52041605299999 65.161501013 63.577017164] Times for Minikube (PR 7247): [67.72041983000001 66.70331142799999 66.64847159500002] Averages Time Per Log
|
Times for minikube: [69.42458749000001 67.02824106000001 65.96256243200003] Times for Minikube (PR 7247): [65.562912532 66.35343435399999 68.42879403900001] Averages Time Per Log
|
Times for minikube: [67.339238802 68.413673488 67.19881655199998] Times for Minikube (PR 7247): [65.37556834300001 67.503662245 64.12870686200002] Averages Time Per Log
|
Times for minikube: [65.269755022 66.690933757 67.497139295] Times for Minikube (PR 7247): [69.34340876 71.05444056900001 66.714565125] Averages Time Per Log
|
Times for minikube: [67.66677584500002 66.50537875900001 67.101002308] Times for Minikube (PR 7247): [65.510121122 65.26630127899999 65.748309034] Averages Time Per Log
|
Times for minikube: [68.05250180000002 68.579297108 65.881122631] Times for Minikube (PR 7247): [69.928223101 67.930680504 68.855569181] Averages Time Per Log
|
kvm2 Driver Times for Minikube (PR 7247): [63.740889460000005 62.598199199999996 62.677469922] Averages Time Per Log
docker Driver Times for Minikube (PR 7247): [45.725393477 44.357729586 42.787538997] Averages Time Per Log
|
(host|control-plane).minikube.internal
to /etc/hosts
This is now ready for code-review! |
kvm2 Driver Times for Minikube (PR 7247): [65.06252022499999 62.515077121000004 63.52880126099999] Averages Time Per Log
docker Driver Times for Minikube (PR 7247): [26.116884622000004 26.269294616 26.369339754] Averages Time Per Log
|
kvm2 Driver Times for Minikube (PR 7247): [64.698896823 65.055561857 67.230453205] Averages Time Per Log
docker Driver Times for Minikube (PR 7247): [27.282102387000002 26.410886307 27.362894135000005] Averages Time Per Log
|
Codecov Report
@@ Coverage Diff @@
## master #7247 +/- ##
==========================================
- Coverage 35.61% 35.57% -0.05%
==========================================
Files 148 148
Lines 9303 9314 +11
==========================================
Hits 3313 3313
- Misses 5593 5604 +11
Partials 397 397
|
Test failures appear to be due to #7874
|
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
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.
looks good depending on KVM tests, please merge with upstream, to see the tests results one more time
KVM test looks scary but maybe they will go away if you merge with upstream |
/retest-this-please |
Error: running mkcmp |
Adds two new hostname aliases that are available to all nodes:
The first is used so that nodes have consistent connectivity to the apiserver across IP changes. Fixes #7246 -- this means that our configurations do not have to be rewritten when the IP changes.
The second is used so that users have an easier time accessing host services. This will allow us to improve this workflow: https://minikube.sigs.k8s.io/docs/handbook/host-access/