kubernetes/minikube

Make the provisioner interface support more container runtimes

Offen

#10.883 geöffnet am 19.03.2021

 (16 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Go (5.222 Forks)batch import
co/generic-driverco/none-driverco/runtime/crioco/runtime/dockerhelp wantedkind/featurelifecycle/frozenpriority/backlog

Repository-Metriken

Stars
 (31.799 Sterne)
PR-Merge-Metriken
 (Durchschn. Merge 12T 19h) (43 gemergte PRs in 30 T)

Beschreibung

Currently the libmachine interface is quite targetted towards Docker:

        // Do the actual provisioning piece:
        //     1. Set the hostname on the instance.
        //     2. Install Docker if it is not present.
        //     3. Configure the daemon to accept connections over TLS.
        //     4. Copy the needed certificates to the server and local config dir.
        //     5. Configure / activate swarm if applicable.
        Provision(swarmOptions swarm.Options, authOptions auth.Options, engineOptions engine.Options) error

It needs to at least know which runtime to install, during provisioning ?

And the swarmOptions and engineOptions need a generic replacement...


Currently the provisioner in minikube is broken, in that it doesn't provision.

It just assumes that all container runtimes are always present on the node.

The original provisioner would install Docker, if it was not already available:

const (
	DefaultEngineInstallURL = "https://get.docker.com"
)
func installDockerGeneric(p Provisioner, baseURL string) error {
        // install docker - until cloudinit we use ubuntu everywhere so we
        // just install it using the docker repos
        if output, err := p.SSHCommand(fmt.Sprintf("if ! type docker; then curl -sSL %s | sh -; fi", baseURL)); err != nil {
                return fmt.Errorf("error installing docker: %s", output)
        }

        return nil
}

Contributor Guide