| *** rh-jelabarre has quit IRC | 01:17 | |
| *** hongbin has joined #openstack-kuryr | 02:37 | |
| *** hongbin has quit IRC | 03:24 | |
| *** pcaruana has joined #openstack-kuryr | 04:55 | |
| *** irclogbot_3 has quit IRC | 05:56 | |
| *** irclogbot_2 has joined #openstack-kuryr | 05:57 | |
| *** aperevalov has joined #openstack-kuryr | 07:02 | |
| aperevalov | dulek, hi | 07:53 |
|---|---|---|
| aperevalov | why you still using python version of kuryr-cni cmd line in devstack? | 07:54 |
| dulek | aperevalov: Uhm… I think it's only used when DevStack is running non-containerized? | 07:58 |
| dulek | aperevalov: And there's no good reason, just lack of time to change that path. We mostly focus on containerized. | 07:58 |
| aperevalov | yes, I found it in non-containerized scenario. Also I found the problem with k8s v1.16 it introduces new behaviour, even cniVersion is 0.3.0, kubelet at launch time asks cni plugin a version (CNI_CMD=VERSION), | 08:00 |
| aperevalov | in this case only {cniVersion; "0.4.0"} in the stdin, and kuryr_kubernetes/cni/main.py fails to parse it, due to lack of kuryr_conf field. | 08:00 |
| dulek | aperevalov: Ah, that's not implemented in Python version? | 08:00 |
| aperevalov | yes, we get it out of box with golang's containernetworking library (see github.com/containernetworking/cni/pkg/skel/skel.go VERSION handling). | 08:01 |
| aperevalov | so two way to fix it 1) handle that case in python's 2) use golang version in non-containerized scenario. | 08:02 |
| aperevalov | I already worked it around on my setup where I tested k8s. | 08:03 |
| dulek | aperevalov: I'd say it's better to switch to Golang CNI going forward. | 08:05 |
| aperevalov | so in this case we need to build during devstack deployment | 08:05 |
| dulek | Yeah, probably need additions to bindep then. | 08:07 |
| dulek | aperevalov: If it's easier for you just to create a workaround in Python, I don't mind too much, I can accept that to unblock you and we'll take care of switch later. | 08:08 |
| aperevalov | ok, I'll put it before moving to v1.16, I don't think it's a huge technical debt ;) | 08:13 |
| dulek | It's not, folks mostly run containerized anyway. | 08:13 |
| openstackgerrit | Merged openstack/kuryr-kubernetes stable/train: Avoid race between pod creation retry and namespace deletion https://review.opendev.org/687900 | 08:40 |
| *** takamatsu has joined #openstack-kuryr | 08:48 | |
| *** aperevalov has quit IRC | 10:24 | |
| *** gcheresh_ has joined #openstack-kuryr | 10:52 | |
| *** gcheresh_ has quit IRC | 11:26 | |
| openstackgerrit | Alexey Perevalov proposed openstack/kuryr-kubernetes master: Just check k8s 1.16 https://review.opendev.org/687116 | 11:48 |
| openstackgerrit | Alexey Perevalov proposed openstack/kuryr-kubernetes master: Fix CNI_COMMAND=VERSION case https://review.opendev.org/688108 | 11:48 |
| openstackgerrit | Luis Tomas Bolivar proposed openstack/kuryr-kubernetes master: Avoid race between Retries and Deletion actions https://review.opendev.org/688109 | 12:00 |
| openstackgerrit | Luis Tomas Bolivar proposed openstack/kuryr-kubernetes master: Avoid race between Retries and Deletion actions https://review.opendev.org/688109 | 12:03 |
| *** rh-jelabarre has joined #openstack-kuryr | 12:11 | |
| openstackgerrit | Luis Tomas Bolivar proposed openstack/kuryr-kubernetes master: Avoid race between Retries and Deletion actions https://review.opendev.org/688109 | 12:35 |
| *** aperevalov has joined #openstack-kuryr | 13:36 | |
| openstackgerrit | Luis Tomas Bolivar proposed openstack/kuryr-kubernetes master: Improve LOG messaging https://review.opendev.org/688130 | 14:00 |
| *** FlorianFa has quit IRC | 15:03 | |
| openstackgerrit | Merged openstack/kuryr-kubernetes master: Set subnet_per_namespace https://review.opendev.org/682245 | 15:52 |
| openstackgerrit | Luis Tomas Bolivar proposed openstack/kuryr-kubernetes master: Improve LOG messaging https://review.opendev.org/688130 | 16:01 |
| *** pcaruana has quit IRC | 18:55 | |
| *** rh-jelabarre has quit IRC | 19:45 | |
| *** rh-jelabarre has joined #openstack-kuryr | 19:46 | |
| *** ccamposr has quit IRC | 20:41 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!