*** threestrands has quit IRC | 00:25 | |
*** threestrands has joined #openstack-kuryr | 00:28 | |
*** hongbin has joined #openstack-kuryr | 01:16 | |
*** threestrands has quit IRC | 01:35 | |
*** threestrands has joined #openstack-kuryr | 01:35 | |
*** threestrands has quit IRC | 01:36 | |
*** threestrands has joined #openstack-kuryr | 01:37 | |
*** threestrands has quit IRC | 01:37 | |
*** threestrands has joined #openstack-kuryr | 01:37 | |
*** kiennt26 has joined #openstack-kuryr | 01:37 | |
*** threestrands has quit IRC | 01:38 | |
*** threestrands has joined #openstack-kuryr | 01:38 | |
*** threestrands has quit IRC | 01:38 | |
*** threestrands has joined #openstack-kuryr | 01:38 | |
*** threestrands has quit IRC | 01:39 | |
*** threestrands has joined #openstack-kuryr | 01:40 | |
*** threestrands has quit IRC | 01:40 | |
*** threestrands has joined #openstack-kuryr | 01:40 | |
*** threestrands has quit IRC | 01:41 | |
*** threestrands has joined #openstack-kuryr | 02:11 | |
*** threestrands has quit IRC | 02:11 | |
*** threestrands has joined #openstack-kuryr | 02:11 | |
*** kiennt26 has quit IRC | 02:53 | |
*** threestrands has quit IRC | 03:13 | |
*** rh-jelabarre has quit IRC | 03:22 | |
*** rh-jelabarre has joined #openstack-kuryr | 03:31 | |
*** kiennt26 has joined #openstack-kuryr | 03:32 | |
*** yamamoto has joined #openstack-kuryr | 04:00 | |
*** rh-jelabarre has quit IRC | 04:13 | |
*** rh-jelabarre has joined #openstack-kuryr | 04:20 | |
*** hongbin has quit IRC | 04:26 | |
*** gcheresh has joined #openstack-kuryr | 06:25 | |
*** gcheresh_ has joined #openstack-kuryr | 06:33 | |
*** gcheresh has quit IRC | 06:33 | |
*** pcaruana has joined #openstack-kuryr | 06:35 | |
ltomasbo | dmellado, celebdor, dulek, irenab: anyone knows why the patch sets with Zuul+2 are not getting merged? | 06:54 |
---|---|---|
*** celebdor1 has joined #openstack-kuryr | 07:13 | |
irenab | ltomasbo, link? | 07:16 |
ltomasbo | irenab, https://review.openstack.org/#/c/562249/ | 07:21 |
*** jistr|mtgs is now known as jistr | 07:22 | |
dulek | ltomasbo: I don't see a reason there. | 07:42 |
dulek | ltomasbo: Try asking on #openstack-infra. | 07:43 |
ltomasbo | dulek, ok! | 07:43 |
irenab | what about this one: https://review.openstack.org/#/c/562067/? | 07:43 |
irenab | this gets zuul -2, not sure why | 07:43 |
dulek | irenab: One of the tests failed. I'll check it. | 07:44 |
dulek | ltomasbo: Whoops, I've flipped +W flag on 562249 and merge jobs got restarted. | 07:45 |
dulek | ltomasbo: So now we need a bit of luck for them to pass. | 07:46 |
ltomasbo | dulek, ok, will see if it merges this time! | 07:46 |
dulek | irenab: Seems like 562067 is some API timeout on Tempest side. Pretty weird, but should be unrelated. | 07:47 |
*** garyloug has joined #openstack-kuryr | 08:10 | |
*** pmannidi has quit IRC | 08:37 | |
openstackgerrit | Merged openstack/kuryr master: fix tox python3 overrides https://review.openstack.org/572908 | 08:41 |
*** janki has joined #openstack-kuryr | 08:57 | |
dulek | celebdor, celebdor1, dmellado: kuryr-kubernetes-tempest-daemon-containerized-lbaasv2 is broken at the moment. I've looked at it briefly, but I can't find the root cause. Partial cause is K8s API LB not being functional when kuryr-daemon container is started. | 08:59 |
dmellado | huh :\ | 09:00 |
dulek | I'll try providing a patch that switches that gate to Octavia. | 09:00 |
dmellado | either that or make it non-voting for now and add the octavia one | 09:00 |
irenab | dmellado, +1 on non voting | 09:01 |
openstackgerrit | Michał Dulko proposed openstack/kuryr-kubernetes master: Switch containerized job to Octavia https://review.openstack.org/573139 | 09:02 |
dulek | Let's see if that fixes the issue. | 09:02 |
celebdor1 | thanks | 09:03 |
dulek | Anyway if someone wants to dig this: http://logs.openstack.org/67/562067/5/check/kuryr-kubernetes-tempest-daemon-containerized-lbaasv2/e3b50bb/ | 09:04 |
dulek | I don't see any change in neutron-lbaas yesterday. Also at a glance there's nothing special in global requirements log. | 09:04 |
celebdor1 | ok | 09:06 |
*** kiennt26 has quit IRC | 09:07 | |
celebdor1 | dulek: it seems to me like the API is reached | 09:10 |
celebdor1 | but for some reason the pod stays in containercreating | 09:10 |
dulek | celebdor1: http://logs.openstack.org/67/562067/5/check/kuryr-kubernetes-tempest-daemon-containerized-lbaasv2/e3b50bb/controller/logs/kubernetes/pod_logs/kube-system-kuryr-cni-ds-ls2lm.txt.gz | 09:12 |
dulek | celebdor1: Looks like it's reached, but for some reason response is either slow or not coming at all. | 09:12 |
dulek | Current speed: 0 would suggest the latter. | 09:12 |
dulek | (those may be not complete logs - container will get restarted over time) | 09:13 |
celebdor1 | :/ | 09:21 |
openstackgerrit | Daniel Mellado proposed openstack/kuryr-kubernetes master: Fix the experimental multinode jobs https://review.openstack.org/558762 | 10:04 |
openstackgerrit | Luis Tomas Bolivar proposed openstack/kuryr-kubernetes master: Namespace deletion functionality for namespace_subnet driver https://review.openstack.org/562249 | 10:36 |
ltomasbo | dulek, celebdor1, based on infra people advice, I've rebase it ^^ | 10:36 |
ltomasbo | can you add W again | 10:36 |
dulek | ltomasbo: It'll not help until we merge https://review.openstack.org/#/c/573139/, containerized LBaaS job is broken. | 10:38 |
dulek | celebdor1, irenab, dmellado: Looks like this solves the gate issue: https://review.openstack.org/#/c/573139/ | 10:39 |
ltomasbo | ohh | 10:39 |
* dmellado checking | 10:39 | |
dulek | It's a step in right direction, so why don't we merge it and deal with LBaaS later? | 10:39 |
* dulek suspects LBaaS issue is not our fault so maybe it'll fix by itself. | 10:39 | |
dmellado | dulek: lgtm | 10:40 |
irenab | dulek is true believer :-) | 10:40 |
dmellado | I'm also dealing with the multinode thing | 10:40 |
dmellado | seems that the underlying patch changed some defaults | 10:40 |
dmellado | on the services, which affects to the lbaasv2 | 10:40 |
dulek | dmellado: Just disable q-lbaasv2 on the node. | 10:40 |
dmellado | tbh, now I'm wondering whether it'd make sense to even keep lbaasv2 for multinode on masgter | 10:41 |
dmellado | master | 10:41 |
dulek | dmellado: I have a working 2-node setup (HA tests). q-lbaasv2 shouldn't be placed on the node. | 10:42 |
dulek | dmellado: That's all. :P | 10:42 |
dmellado | heh | 10:42 |
dmellado | something has changed as per the definition it shouldn't | 10:43 |
dmellado | it's defined on controller host-vars | 10:43 |
dulek | dmellado: Ah… :( | 10:44 |
openstackgerrit | Daniel Mellado proposed openstack/kuryr-kubernetes master: Fix the experimental multinode jobs https://review.openstack.org/558762 | 10:48 |
celebdor1 | irenab: dulek: dmellado: ltomasbo: Didn't we deprecate n-lbaasv2 during queens? | 10:52 |
celebdor1 | maybe we should consider its removal | 10:52 |
celebdor1 | if not now | 10:52 |
celebdor1 | as soon as there's the rocky release | 10:52 |
dulek | celebdor1: If we did deprecated it, I doubt it's documented anywhere now. | 10:52 |
dulek | Any volunteers to write release note and add notes to docs? | 10:52 |
dulek | irenab: Are you using LBaaS v2? | 10:53 |
dmellado | yeah, that was my idea | 10:53 |
irenab | dulek, only when to want quick deployment of devstack to verify patches | 10:53 |
dulek | Anyone knows if Samsung cares? | 10:54 |
* dulek has a contact there, will try pinging him. | 10:55 | |
*** yamamoto has quit IRC | 11:14 | |
openstackgerrit | Merged openstack/kuryr-kubernetes master: Switch containerized job to Octavia https://review.openstack.org/573139 | 11:45 |
*** garyloug has quit IRC | 11:56 | |
openstackgerrit | Genadi Chereshnya proposed openstack/kuryr-tempest-plugin master: Testing VM to loadBalancer service connectivity https://review.openstack.org/571968 | 11:56 |
*** garyloug has joined #openstack-kuryr | 11:58 | |
*** yamamoto has joined #openstack-kuryr | 12:10 | |
*** maysamacedos has joined #openstack-kuryr | 12:17 | |
*** maysamacedos has quit IRC | 12:19 | |
*** maysamacedos has joined #openstack-kuryr | 12:20 | |
*** yamamoto has quit IRC | 12:21 | |
*** garyloug has quit IRC | 12:39 | |
*** yamamoto has joined #openstack-kuryr | 12:49 | |
*** garyloug has joined #openstack-kuryr | 12:53 | |
*** janki has quit IRC | 13:13 | |
openstackgerrit | Luis Tomas Bolivar proposed openstack/kuryr-kubernetes master: Namespace deletion functionality for namespace_subnet driver https://review.openstack.org/562249 | 13:13 |
*** atoth has joined #openstack-kuryr | 13:19 | |
openstackgerrit | Daniel Mellado proposed openstack/kuryr-kubernetes master: Fix the experimental multinode jobs https://review.openstack.org/558762 | 13:30 |
*** openstackgerrit has quit IRC | 13:34 | |
*** openstackgerrit has joined #openstack-kuryr | 13:45 | |
openstackgerrit | Luis Tomas Bolivar proposed openstack/kuryr-kubernetes master: Ensure different pool related dicts per pool driver https://review.openstack.org/573235 | 13:45 |
*** celebdor1 has quit IRC | 13:50 | |
*** celebdor1 has joined #openstack-kuryr | 14:14 | |
celebdor1 | ltomasbo: regarding https://review.openstack.org/#/c/573235/1/kuryr_kubernetes/controller/drivers/vif_pool.py I always wondered why you have that data declared in the baseclass | 14:22 |
celebdor1 | but now at least it is in the instance | 14:22 |
celebdor1 | so the bug is gone | 14:22 |
celebdor1 | was it in queens? | 14:22 |
ltomasbo | celebdor1, I don't think so | 14:23 |
celebdor1 | I don't think so either | 14:24 |
ltomasbo | celebdor1, I just check, multi pools support was not there | 14:27 |
ltomasbo | celebdor1, but the variables were on the baseclase on queens (since the pools functionality was merged) | 14:28 |
ltomasbo | celebdor1, but it does not have any side effect unless we have multipools | 14:28 |
openstackgerrit | Merged openstack/kuryr-kubernetes master: devstack: fix octavia allocation ranges https://review.openstack.org/562067 | 14:29 |
*** gcheresh_ has quit IRC | 14:43 | |
*** garyloug has quit IRC | 15:05 | |
*** garyloug has joined #openstack-kuryr | 15:10 | |
*** garyloug has quit IRC | 15:43 | |
*** garyloug has joined #openstack-kuryr | 15:45 | |
*** janki has joined #openstack-kuryr | 15:57 | |
*** janki has quit IRC | 16:11 | |
*** AlexeyPerevalov has left #openstack-kuryr | 16:13 | |
*** yamamoto has quit IRC | 16:14 | |
*** yamamoto has joined #openstack-kuryr | 16:21 | |
*** yamamoto has quit IRC | 16:26 | |
openstackgerrit | Michał Dulko proposed openstack/kuryr-kubernetes master: kuryr-controller A/P HA https://review.openstack.org/573312 | 16:29 |
dulek | celebdor1: So you won't miss it: ^^ :) | 16:29 |
*** garyloug has quit IRC | 16:30 | |
celebdor1 | oh!!! | 16:39 |
celebdor1 | I'm pumped | 16:39 |
dulek | celebdor1: It turned out not too bad. At first I've had that implemented by ignoring the events through consumes method, but that wasn't manageable. | 16:48 |
dulek | Version that simply starts and stops the Watcher has minimal amount of code in Python. | 16:48 |
* dulek gotta go. | 16:48 | |
celebdor1 | great | 16:49 |
celebdor1 | less code -> Happy celebdor | 16:50 |
openstackgerrit | Merged openstack/kuryr-kubernetes master: Namespace deletion functionality for namespace_subnet driver https://review.openstack.org/562249 | 17:16 |
*** yamamoto has joined #openstack-kuryr | 17:23 | |
*** atoth has quit IRC | 17:30 | |
*** yamamoto has quit IRC | 17:31 | |
*** celebdor1 has quit IRC | 17:32 | |
*** openstackgerrit has quit IRC | 17:34 | |
*** atoth has joined #openstack-kuryr | 17:42 | |
*** atoth has quit IRC | 17:46 | |
*** atoth has joined #openstack-kuryr | 17:50 | |
*** atoth has quit IRC | 19:22 | |
*** celebdor1 has joined #openstack-kuryr | 20:29 | |
*** pcaruana has quit IRC | 20:39 | |
*** maysamacedos has quit IRC | 21:09 | |
*** celebdor1 has quit IRC | 21:48 | |
*** janki has joined #openstack-kuryr | 21:55 | |
*** janki has quit IRC | 22:52 | |
*** maysamacedos has joined #openstack-kuryr | 23:00 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!