versions: Move to k8s 1.10

[jcvenegas]

Move to k8s 1.10

Fixes: #277

Signed-off-by: Jose Carlos Venegas Munoz jose.carlos.venegas.munoz@intel.com

[chavafg]

I was talking with the CRI-O guys and there will be compatibility issues between Openshift and Kubernetes, because latest Openshift v3.9 should use CRI-O 1.9 and k8s 1.10 should use CRI-O 1.10.

There isnt still an Openshift 3.10 out, neither an alpha release.

So I think we will need to have a way to test crio 1.9 with openshift v3.9.0 and crio 1.10 with k8s 1.10

[sboeuf]

@chavafg thanks for the heads up. If we can fit this into our CI without increasing the time taken by the CI, that'd be great. I am saying that because the CI is already pretty slow (around an hour now...)
/cc @egernst

[chavafg]

I was thinking Fedora to run Openshift tests and ubuntu to run k8s, what do you think @sboeuf @egernst @jcvenegas ?

[egernst]

@chavafg this makes a lot of sense to me - a natural division.

[sboeuf]

👍

[jodh-intel]

lgtm

[amshinde]

lgtm

[chavafg]

Using k8s 1.10.0 kubeadm reset doesn't delete the pods created by CRI-O, while k8s 1.9 does it properly. Seems like an issue with cri-o or kuberentes, since I can reproduce the failure using runc. Still investigating...

[chavafg]

The issue is already reported on kubeadm: kubernetes/kubeadm#748

I think the only option here is to add a workaround in the tests to delete the pods after running kubeadm reset using crictl and when this is fixed, remove the workaround.
cc @jcvenegas @egernst @sboeuf @amshinde @jodh-intel

[jodh-intel]

Hi @chavafg - sounds reasonable to me.

[codecov]

Codecov Report

Merging #278 into master will decrease coverage by 0.14%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #278      +/-   ##
==========================================
- Coverage   64.53%   64.38%   -0.15%     
==========================================
  Files          86       86              
  Lines        8397     8432      +35     
==========================================
+ Hits         5419     5429      +10     
- Misses       2404     2428      +24     
- Partials      574      575       +1

Impacted Files	Coverage Δ
virtcontainers/qemu.go	12.01% <0%> (-0.43%)	⬇️
virtcontainers/hyperstart_agent.go	58.58% <0%> (-0.18%)	⬇️
virtcontainers/kata_proxy.go	0% <0%> (ø)	⬆️
virtcontainers/kata_builtin_proxy.go	0% <0%> (ø)	⬆️
virtcontainers/kata_agent.go	29.3% <0%> (+0.02%)	⬆️
virtcontainers/utils/utils.go	71.6% <0%> (+3.11%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f6544a3...5b9cebd. Read the comment docs.

[amshinde]

@chavafg Yes, that sounds fine. Lets create an issue for it so we dont forget.

[chavafg]

opened kata-containers/tests#296 to delete the workaround when the issue gets properly fixed.

[chavafg]

This is ready to be merged, but please when merge this, please also merge this one in tests repository: kata-containers/tests#277