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

CRI-O systemd unit not picking up /etc/sysconfig/crio.minikube and thus not setting --insecure-registry #11729

Closed
tliron opened this issue Jun 22, 2021 · 0 comments · Fixed by #11968
Labels
co/runtime/crio CRIO related issues help wanted Denotes an issue that needs help from a contributor. Must meet "help wanted" guidelines. kind/bug Categorizes issue or PR as related to a bug. priority/important-longterm Important over the long term, but may not be staffed and/or may need multiple releases to complete.

Comments

@tliron
Copy link

tliron commented Jun 22, 2021

Steps to reproduce the issue:

Started thus:

minikube start \
	--vm-driver=kvm2 \
	--container-runtime=cri-o \
	--cni=bridge \
	--addons=registry \
	--cpus=4 \
	--service-cluster-ip-range=10.96.0.0/12

Now, do minikube ssh and:

sudo systemctl status crio

You will see that the command used is /usr/bin/crio --log-level=debug. However, this is incomplete. Looking at /etc/systemd/system/multi-user.target.wants/crio.service you will see that it refers to /etc/sysconfig/crio.minikube, which importantly has extra arguments for --insecure-registry. However, it seems that /etc/sysconfig/crio.minikube was not used.

This breaks the registry add-on out of the box.

As a workaround, do a quick:

sudo systemctl restart crio
sudo systemctl status crio

And you will see that --insecure-registry has now been set properly.

My guess is that somehow the "crio.minikube" file was create too late during the setup process.

@ilya-zuyev ilya-zuyev added co/runtime/crio CRIO related issues help wanted Denotes an issue that needs help from a contributor. Must meet "help wanted" guidelines. kind/bug Categorizes issue or PR as related to a bug. labels Jun 22, 2021
@spowelljr spowelljr added the priority/important-longterm Important over the long term, but may not be staffed and/or may need multiple releases to complete. label Jul 2, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
co/runtime/crio CRIO related issues help wanted Denotes an issue that needs help from a contributor. Must meet "help wanted" guidelines. kind/bug Categorizes issue or PR as related to a bug. priority/important-longterm Important over the long term, but may not be staffed and/or may need multiple releases to complete.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants