regenerate config on retry #8782

sharifelgamal · 2020-07-20T22:32:37Z

Example of 1.9 config:

 cat ~/.minikube/profiles/minikube/config.json 
{
        "Name": "minikube",
        "KeepContext": false,
        "EmbedCerts": false,
        "MinikubeISO": "",
        "Memory": 3892,
        "CPUs": 2,
        "DiskSize": 20000,
        "Driver": "docker",
        "HyperkitVpnKitSock": "",
        "HyperkitVSockPorts": [],
        "DockerEnv": null,
        "InsecureRegistry": null,
        "RegistryMirror": null,
        "HostOnlyCIDR": "192.168.99.1/24",
        "HypervVirtualSwitch": "",
        "HypervUseExternalSwitch": false,
        "HypervExternalAdapter": "",
        "KVMNetwork": "default",
        "KVMQemuURI": "qemu:///system",
        "KVMGPU": false,
        "KVMHidden": false,
        "DockerOpt": null,
        "DisableDriverMounts": false,
        "NFSShare": [],
        "NFSSharesRoot": "/nfsshares",
        "UUID": "",
        "NoVTXCheck": false,
        "DNSProxy": false,
        "HostDNSResolver": true,
        "HostOnlyNicType": "virtio",
        "NatNicType": "virtio",
        "KubernetesConfig": {
                "KubernetesVersion": "v1.18.0",
                "ClusterName": "minikube",
                "APIServerName": "minikubeCA",
                "APIServerNames": null,
                "APIServerIPs": null,
                "DNSDomain": "cluster.local",
                "ContainerRuntime": "docker",
                "CRISocket": "",
                "NetworkPlugin": "",
                "FeatureGates": "",
                "ServiceCIDR": "10.96.0.0/12",
                "ImageRepository": "",
                "ExtraOptions": [
                        {
                                "Component": "kubeadm",
                                "Key": "pod-network-cidr",
                                "Value": "10.244.0.0/16"
                        }
                ],
                "ShouldLoadCachedImages": true,
                "EnableDefaultCNI": false,
                "NodeIP": "",
                "NodePort": 0,
                "NodeName": ""
        },
        "Nodes": [
                {
                        "Name": "m01",
                        "IP": "172.17.0.2",
                        "Port": 8443,
                        "KubernetesVersion": "v1.18.0",
                        "ControlPlane": true,
                        "Worker": true
                }
        ],
        "Addons": {
                "default-storageclass": true,
                "storage-provisioner": true
        },
        "VerifyComponents": {
                "apiserver": true,
                "system_pods": true
        }

What a start with retry on failure now looks like:

minikube start --delete-on-failure=true
😄  minikube v1.12.1 on Darwin 10.15.5
🆕  Kubernetes 1.18.3 is now available. If you would like to upgrade, specify: --kubernetes-version=v1.18.3
✨  Using the docker driver based on existing profile
👍  Starting control plane node minikube in cluster minikube
🏃  Updating the running docker "minikube" container ...
❗  Node m01 failed to start, deleting and trying again.
🔥  Deleting "minikube" in docker ...
🔥  Deleting container "minikube" ...
🔥  Removing /Users/selgamal/.minikube/machines/minikube ...
💀  Removed all traces of the "minikube" cluster.
👍  Starting control plane node minikube in cluster minikube
🔥  Creating docker container (CPUs=2, Memory=3892MB) ...
🌐  Found network options:
    ▪ NO_PROXY=172.17.0.2
🐳  Preparing Kubernetes v1.18.0 on Docker 19.03.2 ...
    ▪ env NO_PROXY=172.17.0.2
🔎  Verifying Kubernetes components...
🌟  Enabled addons: default-storageclass, storage-provisioner
🏄  Done! kubectl is now configured to use "minikube"

And the config file after the retry:

cat ~/.minikube/profiles/minikube/config.json 
{
        "Name": "minikube",
        "KeepContext": false,
        "EmbedCerts": false,
        "MinikubeISO": "",
        "KicBaseImage": "gcr.io/k8s-minikube/kicbase:v0.0.10@sha256:f58e0c4662bac8a9b5dda7984b185bad8502ade5d9fa364bf2755d636ab51438",
        "Memory": 3892,
        "CPUs": 2,
        "DiskSize": 20000,
        "VMDriver": "",
        "Driver": "docker",
        "HyperkitVpnKitSock": "",
        "HyperkitVSockPorts": [],
        "DockerEnv": null,
        "InsecureRegistry": null,
        "RegistryMirror": null,
        "HostOnlyCIDR": "192.168.99.1/24",
        "HypervVirtualSwitch": "",
        "HypervUseExternalSwitch": false,
        "HypervExternalAdapter": "",
        "KVMNetwork": "default",
        "KVMQemuURI": "qemu:///system",
        "KVMGPU": false,
        "KVMHidden": false,
        "DockerOpt": null,
        "DisableDriverMounts": false,
        "NFSShare": [],
        "NFSSharesRoot": "/nfsshares",
        "UUID": "",
        "NoVTXCheck": false,
        "DNSProxy": false,
        "HostDNSResolver": true,
        "HostOnlyNicType": "virtio",
        "NatNicType": "virtio",
        "KubernetesConfig": {
                "KubernetesVersion": "v1.18.0",
                "ClusterName": "minikube",
                "APIServerName": "minikubeCA",
                "APIServerNames": null,
                "APIServerIPs": null,
                "DNSDomain": "cluster.local",
                "ContainerRuntime": "docker",
                "CRISocket": "",
                "NetworkPlugin": "",
                "FeatureGates": "",
                "ServiceCIDR": "10.96.0.0/12",
                "ImageRepository": "",
                "LoadBalancerStartIP": "",
                "LoadBalancerEndIP": "",
                "ExtraOptions": [
                        {
                                "Component": "kubeadm",
                                "Key": "pod-network-cidr",
                                "Value": "10.244.0.0/16"
                        }
                ],
                "ShouldLoadCachedImages": true,
                "EnableDefaultCNI": false,
                "CNI": "",
                "NodeIP": "",
                "NodePort": 8443,
                "NodeName": ""
        },
        "Nodes": [
                {
                        "Name": "m01",
                        "IP": "172.17.0.3",
                        "Port": 8443,
                        "KubernetesVersion": "v1.18.0",
                        "ControlPlane": true,
                        "Worker": true
                }
        ],
        "Addons": {
                "ambassador": false,
                "dashboard": false,
                "default-storageclass": true,
                "efk": false,
                "freshpod": false,
                "gvisor": false,
                "helm-tiller": false,
                "ingress": false,
                "ingress-dns": false,
                "istio": false,
                "istio-provisioner": false,
                "kubevirt": false,
                "logviewer": false,
                "metallb": false,
                "metrics-server": false,
                "nvidia-driver-installer": false,
                "nvidia-gpu-device-plugin": false,
                "olm": false,
                "pod-security-policy": false,
                "registry": false,
                "registry-aliases": false,
                "registry-creds": false,
                "storage-provisioner": true,
                "storage-provisioner-gluster": false
        },
        "VerifyComponents": {
                "apiserver": true,
                "system_pods": true
        }

k8s-ci-robot · 2020-07-20T22:32:54Z

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: sharifelgamal

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:

~~OWNERS~~ [sharifelgamal]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

sharifelgamal · 2020-07-20T22:35:48Z

cc @priyawadhwa @medyagh

medyagh

thank you for fixing this, do you mind sharing the before after this PR output?

sharifelgamal · 2020-07-20T22:37:21Z

thank you for fixing this, do you mind sharing the before after this PR output?

There's no difference to minikube output

codecov-commenter · 2020-07-21T00:46:08Z

Codecov Report

❌ Patch coverage is 0% with 11 lines in your changes missing coverage. Please review.
✅ Project coverage is 31.88%. Comparing base (bf38785) to head (c4cad39).
⚠️ Report is 14878 commits behind head on master.

Files with missing lines	Patch %	Lines
cmd/minikube/cmd/start.go	0.00%	8 Missing ⚠️
cmd/minikube/cmd/node_add.go	0.00%	1 Missing ⚠️
cmd/minikube/cmd/node_start.go	0.00%	1 Missing ⚠️
cmd/minikube/cmd/start_flags.go	0.00%	0 Missing and 1 partial ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #8782      +/-   ##
==========================================
- Coverage   32.20%   31.88%   -0.33%     
==========================================
  Files         165      165              
  Lines       10706    10823     +117     
==========================================
+ Hits         3448     3451       +3     
- Misses       6848     6958     +110     
- Partials      410      414       +4

Files with missing lines	Coverage Δ
cmd/minikube/cmd/node_add.go	`17.85% <0.00%> (ø)`
cmd/minikube/cmd/node_start.go	`9.09% <0.00%> (ø)`
cmd/minikube/cmd/start_flags.go	`49.69% <0.00%> (-0.16%)`	⬇️
cmd/minikube/cmd/start.go	`11.16% <0.00%> (-0.17%)`	⬇️

... and 7 files with indirect coverage changes

🚀 New features to boost your workflow:

❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

medyagh · 2020-07-21T01:22:07Z

thank you for fixing this, do you mind sharing the before after this PR output?

There's no difference to minikube output

how about different in the config file ? before and after?

sharifelgamal · 2020-07-21T19:06:06Z

thank you for fixing this, do you mind sharing the before after this PR output?

There's no difference to minikube output

how about different in the config file ? before and after?

Done. Added to PR description.

sharifelgamal · 2020-07-21T19:22:24Z

/ok-to-test

minikube-pr-bot · 2020-07-21T19:34:37Z

kvm2 Driver
docker Driver

regenerate config on retry

f33ac4c

k8s-ci-robot added cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Jul 20, 2020

k8s-ci-robot requested review from RA489 and josedonizetti July 20, 2020 22:32

k8s-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jul 20, 2020

medyagh reviewed Jul 20, 2020

View reviewed changes

properly regenerate config

c4cad39

k8s-ci-robot added the ok-to-test Indicates a non-member PR verified by an org member that is safe to test. label Jul 21, 2020

medyagh approved these changes Jul 21, 2020

View reviewed changes

sharifelgamal merged commit 280c251 into kubernetes:master Jul 21, 2020

fmotrifork mentioned this pull request Aug 6, 2020

minikube 1.12.2 fishworks/fish-food#883

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

regenerate config on retry #8782

regenerate config on retry #8782

Uh oh!

sharifelgamal commented Jul 20, 2020 •

edited

Loading

Uh oh!

k8s-ci-robot commented Jul 20, 2020

Uh oh!

sharifelgamal commented Jul 20, 2020

Uh oh!

medyagh left a comment

Uh oh!

sharifelgamal commented Jul 20, 2020

Uh oh!

codecov-commenter commented Jul 21, 2020 •

edited

Loading

Uh oh!

medyagh commented Jul 21, 2020

Uh oh!

sharifelgamal commented Jul 21, 2020

Uh oh!

sharifelgamal commented Jul 21, 2020

Uh oh!

minikube-pr-bot commented Jul 21, 2020

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

regenerate config on retry #8782

regenerate config on retry #8782

Uh oh!

Conversation

sharifelgamal commented Jul 20, 2020 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

k8s-ci-robot commented Jul 20, 2020

Uh oh!

sharifelgamal commented Jul 20, 2020

Uh oh!

medyagh left a comment

Choose a reason for hiding this comment

Uh oh!

sharifelgamal commented Jul 20, 2020

Uh oh!

codecov-commenter commented Jul 21, 2020 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Codecov Report

Uh oh!

medyagh commented Jul 21, 2020

Uh oh!

sharifelgamal commented Jul 21, 2020

Uh oh!

sharifelgamal commented Jul 21, 2020

Uh oh!

minikube-pr-bot commented Jul 21, 2020

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

sharifelgamal commented Jul 20, 2020 •

edited

Loading

codecov-commenter commented Jul 21, 2020 •

edited

Loading