Monday, 2018-06-11

*** rh-jelabarre has joined #openstack-kuryr00:23
*** kiennt26 has joined #openstack-kuryr01:02
*** hongbin has joined #openstack-kuryr01:18
*** phuoc has quit IRC01:28
*** phuoc has joined #openstack-kuryr01:28
*** premsankar has joined #openstack-kuryr02:03
*** kiseok7 has quit IRC02:20
*** hongbin has quit IRC03:48
*** rh-jelabarre has quit IRC04:18
*** yamamoto has joined #openstack-kuryr04:28
*** premsankar has quit IRC05:00
*** kiseok7 has joined #openstack-kuryr05:02
openstackgerritPeng Liu proposed openstack/kuryr-kubernetes master: Kubernetes Network Custom Resource Definition De-facto Standard support  https://review.openstack.org/57413105:17
openstackgerritPeng Liu proposed openstack/kuryr-kubernetes master: Network Custom Resource Definition De-facto Standard support  https://review.openstack.org/57413105:18
*** gcheresh_ has joined #openstack-kuryr06:07
*** gcheresh_ has quit IRC06:13
*** gcheresh_ has joined #openstack-kuryr06:13
openstackgerritPeng Liu proposed openstack/kuryr-kubernetes master: Network Custom Resource Definition De-facto Standard support  https://review.openstack.org/57413106:13
*** janki has joined #openstack-kuryr06:24
*** pcaruana has joined #openstack-kuryr06:26
*** NatanBro has joined #openstack-kuryr06:32
*** yamamoto has quit IRC06:35
*** kiennt26 has quit IRC06:37
*** yamamoto has joined #openstack-kuryr06:41
openstackgerritPeng Liu proposed openstack/kuryr-kubernetes master: Network Custom Resource Definition De-facto Standard support  https://review.openstack.org/57413106:49
openstackgerritLuis Tomas Bolivar proposed openstack/kuryr-kubernetes master: Run OVN job containerized and daemonized  https://review.openstack.org/54928907:22
*** janki has quit IRC07:36
*** yamamoto has quit IRC07:50
*** yamamoto has joined #openstack-kuryr07:51
*** yamamoto has quit IRC07:54
*** yamamoto has joined #openstack-kuryr07:54
*** yamamoto has quit IRC07:54
*** yamamoto has joined #openstack-kuryr07:56
*** janki has joined #openstack-kuryr07:59
openstackgerritPeng Liu proposed openstack/kuryr-kubernetes master: Network Custom Resource Definition De-facto Standard support  https://review.openstack.org/57413108:19
*** celebdor1 has joined #openstack-kuryr08:26
*** janki has quit IRC08:38
*** janki has joined #openstack-kuryr08:40
openstackgerritDaniel Mellado proposed openstack/kuryr-kubernetes master: Make tempest-daemon-lbaasv2 gate non-voting  https://review.openstack.org/57417108:40
*** garyloug has joined #openstack-kuryr08:43
celebdor1irenab: ^^08:44
*** janki has quit IRC08:52
*** salv-orlando has joined #openstack-kuryr09:05
celebdor1dulek: you around?09:16
*** salv-orlando has quit IRC09:18
celebdor1pliu: thanks for the spec!09:20
openstackgerritAntoni Segura Puimedon proposed openstack/kuryr-kubernetes master: OCP-Router: add support for external Ingress Controller's LB creation  https://review.openstack.org/56405109:32
openstackgerritAntoni Segura Puimedon proposed openstack/kuryr-kubernetes master: OCP-Router: Ingress controller support  https://review.openstack.org/53638709:32
openstackgerritAntoni Segura Puimedon proposed openstack/kuryr-kubernetes master: OCP-Router: OCP-Route and Ingress LBaaS handlers  https://review.openstack.org/53857709:32
celebdor1ltomasbo: I rebased yossi's patches09:32
celebdor1dmellado: irenab: let's try to get this in already09:32
dmelladoI'll take a look09:33
dmelladobtw, I'm also rebasing and refactoring network policy support09:33
dmelladoI've a wip patch that could use some eyes on it09:33
ltomasbocelebdor1, I'll take a look asap09:34
ltomasbocelebdor1, some of them already have comments from my side09:36
celebdor1ltomasbo: I saw09:36
celebdor1ltomasbo: dmellado: dulek: irenab: Do you remember I did a hack for neutron client to talk to Octavia?09:44
dmelladoyeah quite while back there09:44
celebdor1As Rocky is drawing nearer to release09:44
celebdor1maybe we should revisit that09:44
celebdor1The options are:09:45
dmelladoI guess it'd be worth a proper refactor09:45
celebdor1a) keep it09:45
ltomasboI don't quite remember...09:45
celebdor1b) use octavia client only09:45
celebdor1c) use neutron + octavia client09:45
celebdor1d) Fsck clients and their dependencies, we just send http requests09:45
ltomasboahh, now I do, the lbaas client to use either neutron or octavia based on the env..09:45
dmelladocelebdor1: as octavia will be the default09:45
dmelladoI'd just go for octavia only09:45
dmelladofrom rocky on09:45
celebdor1dmellado: that means dropping lbaasv2 entirely09:46
dmelladocelebdor1: yeah09:46
dmelladoI meant to do that09:46
dmelladoand keep lbaas only in stable branches09:46
celebdor1Good Lord gracious, that's some list of requirements these clients have09:47
ltomasbolbaasv2 is already deprecated and now new features are accepted (for instance to make it work with ovn), so I tend to agree with dmellado and start caring only about octavia09:47
celebdor1what's the status on that ovn + octavia front?09:47
celebdor1Is it making ROcky?09:47
dmelladoit rains rocks09:48
celebdor1?09:48
dmelladoltomasbo: how was that refactor09:48
dmelladodid it fail?09:48
ltomasbothe multidriver thing for octavia? I hope so, but not sure09:48
dmelladonope, the rebase on the ovn gate09:48
ltomasbodmellado, I just rebased the ovn patchset, and it of course failed09:49
ltomasbogoing to check why09:49
dmelladoof course, why would we be expecting any difference09:49
ltomasboxD09:50
ltomasbodmellado, seems to be a sg issue http://logs.openstack.org/89/549289/16/check/kuryr-kubernetes-tempest-daemon-containerized-ovn/ef5f5bd/testr_results.html.gz09:50
dmelladohmmm I kinda recall talking about this with dalvarez09:51
ltomasbodmellado, VM cannot be pinged from pod09:52
ltomasboprobably the VM sg needs to enable ping from remote09:52
dmelladoltomasbo: that's not really it09:53
dmelladoI mean, by default, all those tempest vms enable icmp and ssh09:53
dmelladootherwise the test would be failing for all gates09:53
ltomasboare you sure?09:53
dmelladototally09:53
dmelladoso the issue seems to be something on the ovn side09:54
ltomasbobut the default security group rules could be different for ovn than ovs09:54
ltomasbothe VM can ping the pod in that test (as it passes)09:54
ltomasboso...09:54
ltomasboI bet that function to create a default security group is slightly different for ovn than ovs09:55
dmelladolet me search for it09:55
dmelladoshould be the same09:55
openstackgerritAntoni Segura Puimedon proposed openstack/kuryr-kubernetes master: OCP-Router: Ingress controller support  https://review.openstack.org/53638709:55
ltomasbodmellado, http://git.openstack.org/cgit/openstack/tempest/tree/tempest/api/compute/admin/test_security_group_default_rules.py09:56
ltomasboseems default rules only allow ping from 10.10.0.0/24 cird09:57
dmelladoltomasbo: but that's for tests sg rules09:57
dmelladohold on09:57
dmelladothe code is not there ;)09:57
dmelladohttps://github.com/openstack/tempest/blob/master/tempest/scenario/manager.py#L11510:00
celebdor1ltomasbo: do you know what's the point of having https://review.openstack.org/#/c/536387/31/kuryr_kubernetes/tests/unit/controller/handlers/test_lbaas.py10:00
dmelladohm I kinda recalled tht this enabled icmp and ssh10:00
celebdor1all this "pass" tests10:00
dmelladofor sure, but let me double check10:00
ltomasbocelebdor1, I think he was creating a fake driver for the unittesting10:01
ltomasbocelebdor1, so that avoids some mocking and repeating code (I suppose)10:01
dmelladohttps://github.com/openstack/tempest/blob/master/tempest/scenario/manager.py#L1054-L110710:01
celebdor1aha10:02
celebdor1ah right10:02
celebdor1yes10:02
celebdor1thanks ltomasbo10:02
ltomasbonp10:02
openstackgerritAntoni Segura Puimedon proposed openstack/kuryr-kubernetes master: OCP-Router: Ingress controller support  https://review.openstack.org/53638710:10
openstackgerritLuis Tomas Bolivar proposed openstack/kuryr-tempest-plugin master: DNM: Testing OVN gate  https://review.openstack.org/57420010:18
*** salv-orlando has joined #openstack-kuryr10:18
*** salv-orlando has quit IRC10:24
ltomasbodmellado, celebdor1: so this is why ovn gate was not working: https://mail.openvswitch.org/pipermail/ovs-discuss/2018-March/046404.html10:25
ltomasboseems we cannot use ping -c1...10:25
dmelladodafuq10:25
dmelladoxD10:25
dmelladolet's change it to -c2 xD10:25
ltomasbolet me retry my DNM patch with that10:26
ltomasbodmellado, ^^10:26
dmelladogo ahead ltomasbo10:26
dmelladoirenab: oanson now that we speak about gates10:27
dmelladoany chance that you could get to have a look at the dragonflow one?10:27
celebdor1:O10:27
openstackgerritLuis Tomas Bolivar proposed openstack/kuryr-tempest-plugin master: DNM: Testing OVN gate  https://review.openstack.org/57420010:30
openstackgerritLuis Tomas Bolivar proposed openstack/kuryr-tempest-plugin master: DNM: Testing OVN gate  https://review.openstack.org/57420010:33
oansondmellado, due to the downsizing in Dragonflow, it will take us a while to reach that bug.10:35
oansonSorry.10:35
dmelladooanson: no worries, I'm aware of your current circumstances, thanks!10:36
oansonIt's not forgotten, but to make the most of our time, there are a few things we need to do before that10:36
openstackgerritMerged openstack/kuryr-kubernetes master: Add ports pool clean up support to namespace deletion  https://review.openstack.org/56414810:41
openstackgerritMerged openstack/kuryr-kubernetes master: Retry namespace deletion to mitigate cascading race  https://review.openstack.org/57234410:41
*** NatanBro has quit IRC11:09
openstackgerritPeng Liu proposed openstack/kuryr-kubernetes master: Network Custom Resource Definition De-facto Standard support  https://review.openstack.org/57413111:11
*** atoth has joined #openstack-kuryr11:17
*** salv-orlando has joined #openstack-kuryr11:20
*** salv-orlando has quit IRC11:24
*** rh-jelabarre has joined #openstack-kuryr11:42
ltomasbodmellado, I got the green light on the ovn gate!11:46
dmelladoltomasbo: awesome11:47
dmelladowith the -c4?11:47
ltomasboI'll update the patch description to modify it everywhere11:47
ltomasboyes11:47
dmelladoglad to hear it11:47
dmelladowe have had the ovn gate patch working for a long time11:47
ltomasboand -w4 too, that was set in one or two cases11:47
dmelladoif only it wasn't for the -c411:47
ltomasboyep11:47
openstackgerritLuis Tomas Bolivar proposed openstack/kuryr-tempest-plugin master: Increase ping counter to avoid tests flakiness  https://review.openstack.org/57420011:55
ltomasbodmellado, ^^11:55
openstackgerritLuis Tomas Bolivar proposed openstack/kuryr-kubernetes master: Run OVN job containerized and daemonized  https://review.openstack.org/54928912:02
openstackgerritLuis Tomas Bolivar proposed openstack/kuryr-kubernetes master: Experimental gate for namespace subnet driver  https://review.openstack.org/57036512:10
*** dougbtv has quit IRC12:18
irenabdmellado, I will be late for today's weekly12:18
dmelladoirenab: no worries! thanks for letting me know12:18
*** dougbtv has joined #openstack-kuryr12:18
*** salv-orlando has joined #openstack-kuryr12:20
*** salv-orlando has quit IRC12:26
openstackgerritMerged openstack/kuryr-tempest-plugin master: Add tempest coverage for namespace creation  https://review.openstack.org/56717912:37
*** pcaruana has quit IRC12:39
*** jistr is now known as jistr|mtg12:41
*** jistr|mtg is now known as jistr12:57
*** salv-orlando has joined #openstack-kuryr13:22
*** salv-orlando has quit IRC13:27
*** pcaruana has joined #openstack-kuryr13:32
openstackgerritLuis Tomas Bolivar proposed openstack/kuryr-kubernetes master: Run OVN job containerized and daemonized  https://review.openstack.org/54928913:43
openstackgerritGary Loughnane proposed openstack/kuryr-kubernetes master: Add DPDK support for nested pods  https://review.openstack.org/55936313:45
openstackgerritPeng Liu proposed openstack/kuryr-kubernetes master: Network Custom Resource Definition De-facto Standard support  https://review.openstack.org/57413113:46
openstackgerritGary Loughnane proposed openstack/kuryr-kubernetes master: Add DPDK support for nested pods  https://review.openstack.org/55936313:54
snapiridmellado: when is the weekly today? Thought to drop by :)14:00
dmelladosnapiri: just now! xD14:01
*** yamamoto has quit IRC14:11
*** yamamoto has joined #openstack-kuryr14:12
*** yamamoto has quit IRC14:19
*** salv-orlando has joined #openstack-kuryr14:23
*** salv-orlando has quit IRC14:28
*** yamamoto has joined #openstack-kuryr14:32
*** juriarte3 has joined #openstack-kuryr14:32
*** maysamacedos has joined #openstack-kuryr14:40
*** hongbin has joined #openstack-kuryr14:46
openstackgerritMerged openstack/kuryr-tempest-plugin master: Increase ping counter to avoid tests flakiness  https://review.openstack.org/57420014:56
*** gcheresh_ has quit IRC15:04
*** salv-orlando has joined #openstack-kuryr15:24
*** salv-orlando has quit IRC15:29
*** rh-jelabarre has quit IRC15:30
*** rh-jelabarre has joined #openstack-kuryr15:32
openstackgerritMerged openstack/kuryr-kubernetes master: Experimental gate for namespace subnet driver  https://review.openstack.org/57036516:02
*** maysamacedos has quit IRC16:05
*** salv-orlando has joined #openstack-kuryr16:25
*** salv-orlando has quit IRC16:30
openstackgerritLuis Tomas Bolivar proposed openstack/kuryr-kubernetes master: Run OVN job containerized and daemonized  https://review.openstack.org/54928916:36
*** juriarte3 has quit IRC16:48
*** maysamacedos has joined #openstack-kuryr17:10
*** garyloug has quit IRC17:12
*** salv-orlando has joined #openstack-kuryr17:25
*** salv-orlando has quit IRC17:31
*** salv-orlando has joined #openstack-kuryr18:26
*** salv-orlando has quit IRC18:31
*** mspreitz has joined #openstack-kuryr18:32
*** atoth has quit IRC19:23
*** salv-orlando has joined #openstack-kuryr19:27
*** salv-orlando has quit IRC19:32
*** yamamoto has quit IRC19:51
*** rh-jelabarre has quit IRC19:54
*** maysamacedos has quit IRC20:02
*** rh-jelabarre has joined #openstack-kuryr20:06
*** salv-orlando has joined #openstack-kuryr20:28
*** celebdor1 has quit IRC20:28
*** salv-orlando has quit IRC20:32
*** yamamoto has joined #openstack-kuryr20:51
*** yamamoto has quit IRC20:58
*** mspreitz has quit IRC21:09
*** snapiri has quit IRC22:47
*** snapiri has joined #openstack-kuryr22:47
*** hongbin has quit IRC23:26

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!