--host-only-cidr: invalid CIDR address: '192.168.99.1/24'

[jgoeres]

With my ancient minikube version (1.4.0), I had been using the option

--host-only-cidr='192.168.99.1/24' 

which now gives me

X Unable to start VM. Please investigate and run 'minikube delete' if possible: driver start: Error 
setting up host only network on machine start: invalid CIDR address: '192.168.99.1/24'

with both 1.8.1 and 1.8.2.
I used the option to make sure that the minikube VM gets attached to the right host-only network. It now seemed to work fine without it, so this might be no longer needed, but still...

Regards
J

[tstromberg]

Can you please share the entire minikube start output and command-line flags used?

The example --host-only-cidr command works for me on v1.8.3 using hyperkit and virtualbox drivers on macOS.

[afbjorklund]

It seems like the quotes ended up inside the actual configuration, somehow.

package main

import "net"
import "fmt"


func main() {

	hostOnlyCIDR := "'192.168.99.1/24'"
	ip, network, err := net.ParseCIDR(hostOnlyCIDR)
	if err != nil {
		panic(fmt.Sprintf("%v", err))
	}

	fmt.Printf("ip: %v, network: %v\n", ip, network)
}

panic: invalid CIDR address: '192.168.99.1/24'

With just the regular quotes (i.e. not the embedded single quotes), it is fine.

ip: 192.168.99.1, network: 192.168.99.0/24

[afbjorklund]

Unless I am missing something, that is the default option anyway ?

--host-only-cidr='192.168.99.1/24': The CIDR to be used for the minikube VM (virtualbox driver only)

[jgoeres]

I usually used to run

minikube start --disk-size 30G --cpus 6 --insecure-registry <ourInternalRegistry> --memory 12g --host-only-cidr='192.168.99.1/24'

I don't remember the details why I actually started using the host-only-cidr switch - IIRC back when I started using minikube (around v1.4.0) the minikube VM sometimes ended up using the wrong host-only-network.
In the current version I am fine without the switch, cause, as you said, the default works just fine.

[priyawadhwa]

As @afbjorklund commented, I think the only thing to do here is to trim the address of quotes & validate that it is valid. If invalid, return an error message.

        hostOnlyCIDR = strings.Trim(hostOnlyCIDR, "'\"")
	ip, network, err := net.ParseCIDR(hostOnlyCIDR)
	if err != nil {
		return errors.Wrap(err, "invalid address")
	}

this code would probably live in a helper function that would be called here:

minikube/pkg/minikube/registry/drvs/virtualbox/virtualbox.go

Line 53 in 39958f2

func configure(cc config.ClusterConfig, n config.Node) (interface{}, error) {

If anyone is interested in taking this on, feel free to take this issue by commenting /assign

[fejta-bot]

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.

If this issue is safe to close now please do so with /close.

Send feedback to sig-testing, kubernetes/test-infra and/or fejta.
/lifecycle stale

[fejta-bot]

Stale issues rot after 30d of inactivity.
Mark the issue as fresh with /remove-lifecycle rotten.
Rotten issues close after an additional 30d of inactivity.

If this issue is safe to close now please do so with /close.

Send feedback to sig-testing, kubernetes/test-infra and/or fejta.
/lifecycle rotten

[Aut0R3V]

/assign

[bl-ue]

/remove-lifecycle frozen

[fejta-bot]

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.

If this issue is safe to close now please do so with /close.

Send feedback to sig-contributor-experience at kubernetes/community.
/lifecycle stale

[fejta-bot]

Stale issues rot after 30d of inactivity.
Mark the issue as fresh with /remove-lifecycle rotten.
Rotten issues close after an additional 30d of inactivity.

If this issue is safe to close now please do so with /close.

Send feedback to sig-contributor-experience at kubernetes/community.
/lifecycle rotten

[medyagh]

@Aut0R3V are you still working on this ?

[vibecoder]

Hi, I was trying to replicate this issue. The value of fmt.Println(viper.GetString("host-only-cidr")) that I am getting does not have the single quotes on my output hen running the command ./out/minikube start --driver=virtualbox -p virt --host-only-cidr='192.168.99.1/24' and the minikube clusters starts as expected

[tharun208]

yes with the latest minikube I am also able to get the cluster started as expected