-
Notifications
You must be signed in to change notification settings - Fork 500
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
add script to support launch k8s v1.12 #195
add script to support launch k8s v1.12 #195
Conversation
/run-e2e-tests |
I tested this out. First run When I run
I do see the master running with
|
@gregwebs I haven't encountered this problem in my own test, can you see the log of the docker startup failure? |
The complaint is: I googled that error and didn't come up with anything, any ideas?
|
BTW, I am running Linux. |
…lb/tidb-operator into onlymellb/dind-support-k8s-v1.12
@gregwebs The reason for this problem is that the upstream image is updated. I have updated the startup script and fixed the image to the specific version. You can try again. |
I'm also having error running DinD v1.12 cluster. I'm using NixOS. The error complains Docker can't be started because of dependency failed to start. After some diagnosing, I found that the new version runs containerd service as a dependency of docker. The containerd systemd service file is as follows:
It failed to start because the pre-start command According to this, I've injected the following command which comments out the docker exec ${container_id} sed -i '/ExecStartPre/s/^/#/g' /lib/systemd/system/containerd.service After that, the DinD 1.12 cluster can be started correctly. |
Also the systemd service provided by containerd project uses |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
It works now for me! I will test out deploying the operator & cluster tomorrow and let you know if I come across any issues. |
Some configuration of the extended scheduler needs to be adjusted otherwise the operator may not function correctly. I think we should merge this and fix the scheduler configuration in a later PR. |
@@ -51,253 +52,457 @@ if [[ $(uname) == Linux && -z ${DOCKER_HOST:-} ]]; then | |||
using_local_linuxdocker=1 | |||
fi | |||
|
|||
EMBEDDED_CONFIG=y;DIND_IMAGE=mirantis/kubeadm-dind-cluster:v1.10 | |||
EMBEDDED_CONFIG=y;DIND_IMAGE=mirantis/kubeadm-dind-cluster@sha256:f7c6b21a9a0a55c4bc79678d5b339dea02a6f3aaa3307c0c120c6a9b2cf0f4fc |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is an old version which is incompatible with this new script. The pod network is broken.
I verified I can bring up TiDB on master. |
* add script to support launch k8s v1.12 * fix unbound variable DIND_CRI * fix the problem that containerd failed to start
Signed-off-by: liubo02 <liubo02@pingcap.com>
This PR updates script
dind-cluster-v1.10.sh
to make it consistent with the upstream and adds a scriptdind-cluster-v1.12.sh
to launch k8s v.1.12. We will migrate the CI environment to k8s v1.12 later. resolve #183