| *** rh-jelabarre has quit IRC | 00:46 | |
| *** yamamoto has joined #openstack-kuryr | 00:46 | |
| *** yamamoto has quit IRC | 00:52 | |
| *** yamamoto has joined #openstack-kuryr | 01:48 | |
| *** yamamoto has quit IRC | 01:54 | |
| *** yamamoto has joined #openstack-kuryr | 02:50 | |
| *** yamamoto has quit IRC | 02:54 | |
| *** kzaitsev_pi has quit IRC | 03:28 | |
| *** yamamoto has joined #openstack-kuryr | 03:51 | |
| *** yamamoto has quit IRC | 03:56 | |
| *** yamamoto has joined #openstack-kuryr | 04:41 | |
| *** janki has quit IRC | 05:09 | |
| openstackgerrit | Alexey Perevalov proposed openstack/kuryr-kubernetes master: Spec for vhost-user port type https://review.openstack.org/577049 | 05:14 |
|---|---|---|
| openstackgerrit | Yash Gupta proposed openstack/kuryr-kubernetes master: Add multi-vif driver and related handler changes https://review.openstack.org/524590 | 05:51 |
| openstackgerrit | Yash Gupta proposed openstack/kuryr-kubernetes master: Add multi-vif driver and related handler changes https://review.openstack.org/577070 | 05:53 |
| openstackgerrit | Lv Jiawei proposed openstack/kuryr-kubernetes master: Devstack install fail if docker cgroup driver is systemd https://review.openstack.org/575319 | 06:00 |
| *** janki has joined #openstack-kuryr | 06:00 | |
| *** ygupta has joined #openstack-kuryr | 06:05 | |
| ygupta | Hello, I have uploaded an implementation for multi-vif support from our work at Samsung Electronics | 06:07 |
| ygupta | I request reviews from the community for it: https://review.openstack.org/#/c/577070 | 06:08 |
| ygupta | (Please note that this is about kuryr-kubernetes) | 06:09 |
| openstackgerrit | Yash Gupta proposed openstack/kuryr-kubernetes master: Add multi-vif driver and related handler changes https://review.openstack.org/577070 | 06:20 |
| openstackgerrit | Yash Gupta proposed openstack/kuryr-kubernetes master: Add multi-vif driver and related handler changes https://review.openstack.org/577070 | 06:27 |
| *** janonymous has joined #openstack-kuryr | 06:28 | |
| *** kiseok7 has joined #openstack-kuryr | 06:33 | |
| *** gcheresh_ has joined #openstack-kuryr | 06:34 | |
| *** ygupta has quit IRC | 06:36 | |
| *** gcheresh_ has quit IRC | 06:40 | |
| *** gcheresh has joined #openstack-kuryr | 06:40 | |
| *** AlexeyPerevalov has joined #openstack-kuryr | 07:02 | |
| openstackgerrit | Alexey Perevalov proposed openstack/kuryr-kubernetes master: Spec for vhost-user port type https://review.openstack.org/577049 | 07:10 |
| *** gcheresh has quit IRC | 07:14 | |
| *** gcheresh_ has joined #openstack-kuryr | 07:14 | |
| *** pcaruana has joined #openstack-kuryr | 07:22 | |
| *** pmannidi has quit IRC | 07:51 | |
| openstackgerrit | Daniel Mellado proposed openstack/kuryr-kubernetes master: Deprecate running Kuryr-Kubernetes with lbaasv2 https://review.openstack.org/576486 | 08:00 |
| *** garyloug has joined #openstack-kuryr | 08:01 | |
| dulek | dmellado: Hi, can you take a look on https://review.openstack.org/#/c/576071/ ? | 08:02 |
| dmellado | dulek: sure | 08:02 |
| dmellado | dulek: +A | 08:03 |
| dmellado | my only concern was the pprint import / usage | 08:03 |
| dmellado | but you already addressed it | 08:03 |
| dulek | Great, thanks! | 08:03 |
| *** yboaron_ has joined #openstack-kuryr | 08:04 | |
| dmellado | btw dulek if you could also take a look at my reno | 08:16 |
| dmellado | which also kicks out the lbaasv2 gates it'd be great | 08:17 |
| dmellado | I don't want to get those out before you're done with the investigation (if still on it) | 08:17 |
| dulek | dmellado: Ah, right. I think I'm done, I'm pretty sure that the issue is just timeout set for too low value. | 08:17 |
| dulek | Because I don't see it anymore. | 08:18 |
| dulek | dmellado: I'll take a look. | 08:18 |
| dmellado | dulek: thanks! | 08:18 |
| *** apuimedo has joined #openstack-kuryr | 08:36 | |
| *** kzaitsev_pi has joined #openstack-kuryr | 08:39 | |
| *** gcheresh has joined #openstack-kuryr | 08:48 | |
| *** gcheresh_ has quit IRC | 08:48 | |
| openstackgerrit | Daniel Mellado proposed openstack/kuryr-kubernetes master: Deprecate running Kuryr-Kubernetes with lbaasv2 https://review.openstack.org/576486 | 08:51 |
| apuimedo | irenab: can you review https://review.openstack.org/#/c/575771/ ? | 09:01 |
| irenab | sure, asap | 09:01 |
| apuimedo | thanks! | 09:01 |
| openstackgerrit | sunqingliang proposed openstack/kuryr-kubernetes master: fix typos https://review.openstack.org/577099 | 09:13 |
| dulek | dmellado: What do you think of leaving one or two LBaaS v2 jobs as non-voting? | 09:23 |
| dulek | dmellado: Deprecation != dropping support. | 09:23 |
| dmellado | dulek: I wouldn't mind to leave a few for now | 09:23 |
| dmellado | dulek: in any case | 09:23 |
| dmellado | my overall idea was to leave that on | 09:23 |
| dmellado | on the stable branches | 09:23 |
| dmellado | but drop it on master | 09:23 |
| dmellado | I'll need to check how to run those as periodic jobs | 09:24 |
| dulek | apuimedo, irenab: Your thoughts? I'd leave daemon-lbaasv2 job as non-voting on check. Just to have some feedback if it's working fine. | 09:25 |
| openstackgerrit | sunqingliang proposed openstack/kuryr-kubernetes master: fix typos https://review.openstack.org/577099 | 09:26 |
| openstackgerrit | Michał Dulko proposed openstack/kuryr-kubernetes master: Provide data timeouts for K8s API LB https://review.openstack.org/575188 | 09:51 |
| openstackgerrit | Michał Dulko proposed openstack/kuryr-kubernetes master: Use K8s API LB IP instead of HOST_IP in DevStack https://review.openstack.org/569095 | 09:51 |
| dulek | FYI - seems like containerized gates are broken at the moment due to long time taking to build the containers. It's most likely some networking issue on infra side. | 10:17 |
| dulek | Yep, looks like there's some discussion on packages mirroring on #openstack-infra, so it's most likely that. | 10:18 |
| openstackgerrit | Peng Liu proposed openstack/kuryr-kubernetes master: Network Custom Resource Definition De-facto Standard support https://review.openstack.org/574131 | 10:21 |
| apuimedo | dulek: they should be voting until rocky release is cut | 10:23 |
| apuimedo | since in rocky it is deprecated but supported | 10:23 |
| apuimedo | after rocky, they go non-voting | 10:23 |
| apuimedo | until we decide to drop the compatibility altogether | 10:24 |
| dulek | apuimedo: So I guess your +2 should be withdrawn? ;) | 10:24 |
| apuimedo | ehhh | 10:24 |
| apuimedo | link? | 10:24 |
| dulek | apuimedo: https://review.openstack.org/#/c/576486 | 10:25 |
| dulek | The patch removes all of them. | 10:25 |
| * dulek will be back in ~20 minutes. | 10:26 | |
| openstackgerrit | Shachar Snapiri proposed openstack/kuryr-kubernetes master: Split .zuul.yaml into smaller files https://review.openstack.org/548309 | 10:28 |
| *** snapiri has joined #openstack-kuryr | 10:29 | |
| apuimedo | dulek: done. THanks | 10:31 |
| openstackgerrit | Daniel Mellado proposed openstack/kuryr-kubernetes master: Deprecate running Kuryr-Kubernetes with lbaasv2 https://review.openstack.org/576486 | 10:35 |
| openstackgerrit | sunqingliang proposed openstack/kuryr-kubernetes master: fix typos https://review.openstack.org/577099 | 10:43 |
| openstackgerrit | Daniel Mellado proposed openstack/kuryr-kubernetes master: Deprecate running Kuryr-Kubernetes with lbaasv2 https://review.openstack.org/576486 | 10:55 |
| *** gcheresh_ has joined #openstack-kuryr | 11:02 | |
| *** gcheresh has quit IRC | 11:02 | |
| *** yamamoto has quit IRC | 11:05 | |
| *** yamamoto has joined #openstack-kuryr | 11:06 | |
| openstackgerrit | Daniel Mellado proposed openstack/kuryr-kubernetes master: Deprecate running Kuryr-Kubernetes with lbaasv2 https://review.openstack.org/576486 | 11:09 |
| *** yamamoto has quit IRC | 11:10 | |
| *** yboaron_ has quit IRC | 11:17 | |
| *** yboaron has joined #openstack-kuryr | 11:18 | |
| snapiri | dmellado: it seems like there is some problem with the setuptools package in (IMHO, the docker images): http://logs.openstack.org/09/548309/9/check/kuryr-kubernetes-tempest-daemon-containerized-octavia/7b6fd4e/job-output.txt.gz#_2018-06-21_10_57_39_624634 | 11:30 |
| snapiri | see: https://github.com/openaps/openaps/issues/95 | 11:30 |
| dmellado | snapiri: hmmm | 11:35 |
| dmellado | could probably be because of the centos on the images | 11:35 |
| dmellado | dulek: could you take a look at this? | 11:36 |
| dmellado | probably just updating to a newer setuptools will fix it | 11:36 |
| dulek | dmellado: Sure, in 10 minutes. | 11:36 |
| dmellado | dulek: whenever, no hurries | 11:36 |
| dmellado | thanks! | 11:36 |
| dmellado | snapiri: thanks for noticing it! | 11:36 |
| snapiri | My pleasure :) | 11:37 |
| *** rh-jelabarre has joined #openstack-kuryr | 11:51 | |
| openstackgerrit | Daniel Mellado proposed openstack/kuryr-kubernetes master: Fix container image creation. https://review.openstack.org/577132 | 11:52 |
| dmellado | dulek: snapiri ^^ | 11:52 |
| openstackgerrit | Luis Tomas Bolivar proposed openstack/kuryr-tempest-plugin master: Namespace network resources cleanup https://review.openstack.org/576476 | 11:53 |
| dulek | dmellado: Yeah, that should help. Just please address my comments there. :) | 11:54 |
| openstackgerrit | Daniel Mellado proposed openstack/kuryr-kubernetes master: Fix container image creation. https://review.openstack.org/577132 | 11:55 |
| dmellado | dulek: done | 11:56 |
| dmellado | it does fix it, just hit the issue myself while developing | 11:56 |
| dulek | dmellado: Had you tried patchset #2? :P I think it'll fail due to syntax error. | 11:57 |
| dulek | dmellado: Yup. | 11:57 |
| dmellado | dulek: it did work without the \ | 11:58 |
| dmellado | ket me do a quick retest in any case | 11:58 |
| dulek | Error response from daemon: Dockerfile parse error line 9: unknown instruction: && | 11:58 |
| dulek | I guess it might depend on Docker version, though I have a pretty new one. | 11:58 |
| dmellado | let me add it | 11:58 |
| dulek | 18.03.1-ce | 11:58 |
| dmellado | it won't hurt | 11:58 |
| openstackgerrit | Daniel Mellado proposed openstack/kuryr-kubernetes master: Fix container image creation. https://review.openstack.org/577132 | 11:59 |
| dmellado | dulek: there you go | 11:59 |
| dulek | +2 | 12:00 |
| *** yamamoto has joined #openstack-kuryr | 12:04 | |
| *** gcheresh has joined #openstack-kuryr | 12:10 | |
| openstackgerrit | zhangzhaoshan proposed openstack/kuryr master: Add release notes link to README https://review.openstack.org/577134 | 12:10 |
| snapiri | dmellado: +1 :) | 12:11 |
| *** gcheresh_ has quit IRC | 12:11 | |
| *** atoth has joined #openstack-kuryr | 12:11 | |
| openstackgerrit | Shachar Snapiri proposed openstack/kuryr-kubernetes master: Deprecate running Kuryr-Kubernetes with lbaasv2 https://review.openstack.org/576486 | 12:14 |
| openstackgerrit | Shachar Snapiri proposed openstack/kuryr-kubernetes master: Split .zuul.yaml into smaller files https://review.openstack.org/548309 | 12:14 |
| *** pc_m has quit IRC | 12:21 | |
| openstackgerrit | Daniel Mellado proposed openstack/kuryr-kubernetes master: Deprecate running Kuryr-Kubernetes with lbaasv2 https://review.openstack.org/576486 | 12:27 |
| *** yboaron_ has joined #openstack-kuryr | 12:28 | |
| *** pc_m has joined #openstack-kuryr | 12:30 | |
| openstackgerrit | Daniel Mellado proposed openstack/kuryr-kubernetes master: Make ovn gates voting https://review.openstack.org/577142 | 12:30 |
| *** maysamacedos has joined #openstack-kuryr | 12:31 | |
| *** yboaron has quit IRC | 12:32 | |
| *** yamamoto has quit IRC | 12:51 | |
| *** yboaron has joined #openstack-kuryr | 13:12 | |
| *** yboaron_ has quit IRC | 13:14 | |
| *** maysamacedos has quit IRC | 13:22 | |
| *** maysamacedos has joined #openstack-kuryr | 13:22 | |
| *** yamamoto has joined #openstack-kuryr | 13:25 | |
| *** yamamoto has quit IRC | 13:36 | |
| openstackgerrit | zhangzhaoshan proposed openstack/kuryr-tempest-plugin master: Update url in HACKING.rst https://review.openstack.org/577165 | 13:43 |
| *** yamamoto has joined #openstack-kuryr | 13:48 | |
| *** jistr is now known as jistr|mtg | 13:49 | |
| dulek | dmellado: Hey, remember good times when gate issues came in pairs? | 13:50 |
| dulek | dmellado: Looks like we have it again: http://logs.openstack.org/32/577132/3/check/kuryr-kubernetes-tempest-daemon-containerized-octavia/0a33bf3/controller/logs/screen-o-api.txt.gz | 13:50 |
| apuimedo | dulek: to be like the good old times, it should have been on a Friday | 13:52 |
| *** janki has quit IRC | 13:56 | |
| dmellado | dulek: heh, what happened now? xD | 13:58 |
| dulek | dmellado: Not really sure - I thought apuimedo fixed IP conflict issue. | 13:59 |
| dmellado | yeah, that kinda brings back memories... | 14:00 |
| dmellado | apuimedo: can you take a look and check if that isn't a botched workaround? | 14:00 |
| apuimedo | dmellado: so little trust in my workaround... So sad... | 14:01 |
| apuimedo | xD | 14:01 |
| apuimedo | what should I look for? | 14:01 |
| dmellado | apuimedo: check dulek logs | 14:02 |
| openstackgerrit | Luis Tomas Bolivar proposed openstack/kuryr-tempest-plugin master: Namespace network resources cleanup https://review.openstack.org/576476 | 14:04 |
| apuimedo | dmellado: they are colorful | 14:04 |
| dmellado | apuimedo: of course | 14:05 |
| openstackgerrit | Luis Tomas Bolivar proposed openstack/kuryr-tempest-plugin master: Remove sleep 3600 when creating container https://review.openstack.org/577169 | 14:06 |
| dulek | :D | 14:07 |
| apuimedo | I'm almost tempted to say that it is a new Octavia bug | 14:08 |
| apuimedo | since it worked consistently for so long | 14:08 |
| apuimedo | :) | 14:08 |
| openstackgerrit | Luis Tomas Bolivar proposed openstack/kuryr-tempest-plugin master: Namespace network resources cleanup https://review.openstack.org/576476 | 14:08 |
| apuimedo | ltomasbo: did you find out why that sleep was put there? | 14:08 |
| apuimedo | Maybe it is from before we used the image I created | 14:09 |
| apuimedo | and we were using an empty image then | 14:09 |
| apuimedo | in any case | 14:09 |
| apuimedo | this needs merging | 14:09 |
| ltomasbo | yep | 14:09 |
| ltomasbo | and I just tested on dmellado's env my patch | 14:09 |
| ltomasbo | it was working... | 14:09 |
| dmellado | ltomasbo: send a patch and we'll merge it asap | 14:09 |
| ltomasbo | half a day lost in this shi*** | 14:09 |
| ltomasbo | dmellado, https://review.openstack.org/#/c/577169/ | 14:10 |
| ltomasbo | apuimedo, ^^ | 14:10 |
| apuimedo | ltomasbo: it was not so hidden :P | 14:10 |
| apuimedo | ltomasbo: but it does raise questions about the existing tests that checked things with curl | 14:10 |
| ltomasbo | yep, I know, I was just not looking there... | 14:10 |
| ltomasbo | yes, totally | 14:11 |
| ltomasbo | not sure how they were working | 14:11 |
| dmellado | maybe that has something to do with the change | 14:11 |
| dmellado | in our image? | 14:11 |
| dmellado | nevertheless, let's wait on CI and merge this | 14:11 |
| ltomasbo | I see most of then create 2 pods and look for different replies... | 14:11 |
| apuimedo | dmellado: ltomasbo: maybe the previous image was setting ENTRYPOINT instead of CMD and it was ignored :P | 14:12 |
| ltomasbo | that was also delaying me from finding this, the gate sometimes times out, and now I think it is just broken as the kuryr-controller ocntainer cannot be build unless setuptools is updated | 14:12 |
| dmellado | apuimedo: could totally be | 14:13 |
| dmellado | in any case good to know this and remove so we can now go and enhance our testing | 14:13 |
| ltomasbo | also, that could be the reason why we had our lbaasV2 gate broken? | 14:13 |
| dmellado | now we just need to fix apuimedo octavia's botched job | 14:13 |
| dmellado | ltomasbo: but it did use to work properly | 14:13 |
| dmellado | that's what I find quite odd | 14:13 |
| ltomasbo | perhaps as it was wait 3600, for the octavia being slow was helping them... | 14:14 |
| ltomasbo | yes, it is really weird... | 14:14 |
| apuimedo | ltomasbo: which container can't be built? | 14:14 |
| apuimedo | the kuryr-kubernetes one? | 14:14 |
| ltomasbo | kuryr-controller | 14:14 |
| dmellado | apuimedo: ltomasbo | 14:14 |
| apuimedo | we could change it to fedora | 14:14 |
| dmellado | I've a patch to fix this | 14:14 |
| dmellado | did you see it? | 14:14 |
| apuimedo | dmellado: did you add us as reviewers? | 14:14 |
| dmellado | https://review.openstack.org/#/c/577132/ | 14:15 |
| dmellado | I was speaking with dulek a while ago | 14:15 |
| dmellado | thought you saw it | 14:15 |
| dmellado | but there you go | 14:15 |
| dmellado | bug+fix | 14:15 |
| apuimedo | dmellado: if I'm not a reviewer, I haven't reviewed it | 14:15 |
| apuimedo | :-) | 14:15 |
| dmellado | apuimedo: now you will | 14:15 |
| dmellado | xD | 14:15 |
| apuimedo | dmellado: done | 14:16 |
| apuimedo | you're in charge of rechecks | 14:16 |
| dmellado | lol | 14:17 |
| *** janki has joined #openstack-kuryr | 14:19 | |
| ltomasbo | apuimedo, dmellado let me change the complete thing for the tempest fix | 14:24 |
| dmellado | ltomasbo: go for it | 14:24 |
| ltomasbo | now we'll see if all of them were really passing... xD | 14:25 |
| openstackgerrit | Luis Tomas Bolivar proposed openstack/kuryr-tempest-plugin master: Remove sleep 3600 when creating container https://review.openstack.org/577169 | 14:26 |
| ltomasbo | dmellado, apuimedo, ^^ | 14:26 |
| dmellado | let's see about CI | 14:27 |
| ltomasbo | yep, it should work... | 14:27 |
| *** pcaruana has quit IRC | 14:56 | |
| *** janonymous has quit IRC | 14:58 | |
| *** gcheresh has quit IRC | 15:00 | |
| *** yboaron has quit IRC | 15:27 | |
| apuimedo | ltomasbo: you are not checking that the LBs are gone in https://review.openstack.org/#/c/576476/10/kuryr_tempest_plugin/tests/scenario/test_namespace.py | 15:30 |
| apuimedo | only that the subnet (and thus the ports) are gone | 15:30 |
| apuimedo | so it could be that just the LB remained (although without backends) | 15:31 |
| ltomasbo | apuimedo, but that will be a kubernetes issue, nothing to be we the namespace funcionality | 15:32 |
| ltomasbo | that should be checked on the services tempest | 15:32 |
| apuimedo | ltomasbo: you are right | 15:33 |
| apuimedo | I guess I just want bigger coverage overall | 15:33 |
| apuimedo | :P | 15:33 |
| ltomasbo | apuimedo, I agree! we should really increase the coverage! | 15:34 |
| ltomasbo | perhaps we can have a half a day tempest testing per week until the coverage is decent | 15:35 |
| ltomasbo | xD | 15:35 |
| *** jistr|mtg is now known as jistr | 15:36 | |
| openstackgerrit | Luis Tomas Bolivar proposed openstack/kuryr-kubernetes master: Kuryr Kubernetes Port and PortPool CRD design reference document https://review.openstack.org/556181 | 15:40 |
| *** jistr is now known as jistr|off | 15:47 | |
| *** maysamacedos has quit IRC | 16:00 | |
| openstackgerrit | Merged openstack/kuryr-tempest-plugin master: Remove sleep 3600 when creating container https://review.openstack.org/577169 | 16:32 |
| apuimedo | ltomasbo: yay ^^ | 16:35 |
| ltomasbo | xD | 16:35 |
| ltomasbo | apuimedo, now I need dmellado; s one to get in so that I can test my namespace tempest extension... | 16:35 |
| *** raopajay has joined #openstack-kuryr | 16:37 | |
| *** yamamoto has quit IRC | 16:38 | |
| *** yamamoto has joined #openstack-kuryr | 16:46 | |
| *** yamamoto has quit IRC | 16:51 | |
| *** maysamacedos has joined #openstack-kuryr | 16:54 | |
| *** yamamoto has joined #openstack-kuryr | 16:55 | |
| *** yamamoto has quit IRC | 17:00 | |
| openstackgerrit | Merged openstack/kuryr-kubernetes master: Fix container image creation. https://review.openstack.org/577132 | 17:07 |
| *** yamamoto has joined #openstack-kuryr | 17:56 | |
| *** yamamoto has quit IRC | 18:04 | |
| *** janki has quit IRC | 18:25 | |
| *** yamamoto has joined #openstack-kuryr | 19:01 | |
| *** yamamoto has quit IRC | 19:06 | |
| *** maysamacedos has quit IRC | 19:22 | |
| *** mfedosin has quit IRC | 19:32 | |
| *** mfedosin has joined #openstack-kuryr | 19:44 | |
| *** yamamoto has joined #openstack-kuryr | 20:02 | |
| *** yamamoto has quit IRC | 20:07 | |
| *** aojea has joined #openstack-kuryr | 20:51 | |
| *** yamamoto has joined #openstack-kuryr | 21:04 | |
| *** yamamoto has quit IRC | 21:09 | |
| openstackgerrit | Antoni Segura Puimedon proposed openstack/kuryr-kubernetes master: Fetch CONTAINERID from Docker API in kuryr-cni https://review.openstack.org/576071 | 21:39 |
| *** irenab has quit IRC | 21:44 | |
| *** irenab has joined #openstack-kuryr | 21:45 | |
| *** apuimedo has quit IRC | 21:49 | |
| *** lihi has quit IRC | 21:50 | |
| *** lihi has joined #openstack-kuryr | 21:51 | |
| *** yamamoto has joined #openstack-kuryr | 22:06 | |
| *** yamamoto has quit IRC | 22:11 | |
| *** aojea has quit IRC | 22:39 | |
| *** yamamoto has joined #openstack-kuryr | 23:07 | |
| *** yamamoto has quit IRC | 23:12 | |
| *** pmannidi has joined #openstack-kuryr | 23:25 | |
| *** yamamoto has joined #openstack-kuryr | 23:36 | |
| *** garyloug has quit IRC | 23:50 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!