Skip to content
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

Running with containerd starts crio #13433

Closed
afbjorklund opened this issue Jan 22, 2022 · 2 comments · Fixed by #13442
Closed

Running with containerd starts crio #13433

afbjorklund opened this issue Jan 22, 2022 · 2 comments · Fixed by #13442
Assignees
Labels
co/runtime/containerd kind/bug Categorizes issue or PR as related to a bug.

Comments

@afbjorklund
Copy link
Collaborator

afbjorklund commented Jan 22, 2022

This looks broken:

minikube start --no-kubernetes --vm --container-runtime=containerd

🏄 Done! minikube is ready without Kubernetes!

$ minikube ssh -- pstree
systemd-+-acpid
        |-2*[agetty]
        |-crio---11*[{crio}]
        |-dbus-daemon
        |-rpc.mountd
        |-rpc.statd
        |-rpcbind
        |-sshd---sshd---sshd---pstree
        |-systemd-hostnam
        |-systemd-journal
        |-systemd-logind
        |-systemd-network
        |-systemd-resolve
        |-systemd-timesyn---{systemd-timesyn}
        `-systemd-udevd---20*[systemd-udevd]

It was supposed to run containerd, not crio ?

$ minikube ssh -- sudo ctr version
Client:
  Version:  v1.4.12
  Revision: 7b11cfaabd73bb80907dd23182b9347b4245eb5d
  Go version: go1.17

ctr: failed to dial "/run/containerd/containerd.sock": context deadline exceeded
ssh: Process exited with status 1

When running with docker, it would start both.

(This is because CRI-O is started on boot of VM)

@afbjorklund afbjorklund added co/runtime/containerd kind/bug Categorizes issue or PR as related to a bug. labels Jan 22, 2022
@MdSahil-oss
Copy link

/assign

@afbjorklund
Copy link
Collaborator Author

It is missing a call to configureRuntimes, before the configureMounts (and return):

        stopk8s, err := handleNoKubernetes(starter)
        if err != nil {
                return nil, err
        }
        if stopk8s {
                configureMounts(&wg, *starter.Cfg)
                return nil, config.Write(viper.GetString(config.ProfileName), starter.Cfg)
        }

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
co/runtime/containerd kind/bug Categorizes issue or PR as related to a bug.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants