Tuesday, 2018-04-24

*** pcaruana has quit IRC00:02
*** fabian has joined #openstack-kuryr00:49
*** fabian has quit IRC00:55
*** fabian has joined #openstack-kuryr01:12
*** caisan has joined #openstack-kuryr01:12
*** fabian is now known as chenyb401:23
*** salv-orl_ has joined #openstack-kuryr01:28
*** salv-orlando has quit IRC01:31
*** hongbin_ has joined #openstack-kuryr01:32
*** maysamacedos has quit IRC01:33
*** caowei has joined #openstack-kuryr02:04
*** phuoc_ has joined #openstack-kuryr02:09
*** phuoc has quit IRC02:12
*** yamamoto has quit IRC02:53
*** yamamoto has joined #openstack-kuryr02:54
*** yamamoto has quit IRC02:59
*** premsankar has quit IRC03:13
*** janki has joined #openstack-kuryr03:44
*** hongbin_ has quit IRC03:58
*** gcheresh_ has joined #openstack-kuryr04:19
*** kiennt26 has quit IRC04:25
*** janki has quit IRC04:39
*** gcheresh_ has quit IRC04:49
*** janonymous has joined #openstack-kuryr04:58
*** yboaron has joined #openstack-kuryr05:19
*** gcheresh_ has joined #openstack-kuryr05:26
*** premsankar has joined #openstack-kuryr05:30
*** CrayZee has joined #openstack-kuryr05:31
*** janki has joined #openstack-kuryr05:49
*** yboaron has quit IRC06:17
*** threestrands has quit IRC06:32
*** salv-orl_ has quit IRC06:45
*** salv-orlando has joined #openstack-kuryr06:46
*** threestrands has joined #openstack-kuryr06:50
*** threestrands has quit IRC06:50
*** threestrands has joined #openstack-kuryr06:50
*** salv-orlando has quit IRC06:50
*** pcaruana has joined #openstack-kuryr06:58
*** salv-orlando has joined #openstack-kuryr07:00
*** dangtrinhnt has joined #openstack-kuryr07:02
*** threestrands has quit IRC07:10
*** jchhatbar has joined #openstack-kuryr07:21
*** janki has quit IRC07:23
*** yboaron has joined #openstack-kuryr07:42
*** yboaron_ has joined #openstack-kuryr07:57
*** yboaron has quit IRC08:00
*** jistr is now known as jistr|mtgs08:28
irenabltomasbo, ping08:35
*** yboaron has joined #openstack-kuryr08:41
*** yboaron_ has quit IRC08:42
*** garyloug has joined #openstack-kuryr08:47
ltomasboirenab, pong08:50
irenabltomasbo, I took a look on the per namespace network patches and have a question08:50
irenabyou have network CRD and also annotation in the namespace entity08:51
ltomasbosure!08:51
irenabI am not sure I understand the value of CRD unless we will have pools later08:52
ltomasboirenab, I annotate the kuryrnet-crd link into the namespace08:52
irenabbut instead you canhave network details in the annotation08:52
apuimedoirenab: CRD also helps when the namespace is deleted08:52
apuimedoso we have info still on the system to delete08:52
apuimedothat otherwise we might have missed if the controller went down08:52
ltomasboirenab, yep, specially for the ports pool, where I will retrive the net_id from the kurryrnet CRD08:52
ltomasboirenab, and then remove all the ports from the pools belonging to that network08:53
irenabltomasbo, then maybe using namespace name as partof the network name is not good, since it will be different once pools will be used08:53
irenabltomasbo, so you say in case of namespace deletion, it maybe the case that there won't be an entity to retrive network details?08:54
irenabltomasbo, or it is more for the case when kuryr-controller maybe down and then there should be some sort of garbage collector for the 'zombi' networks?08:55
ltomasbomore about the second I guess08:56
irenabI am just not sure if the new CRD resolves somethig that cannot be done without it. On the other hand, it adds complexity, since there is no additional entity to take care of08:58
*** premsankar has quit IRC08:58
irenabthere is additional entity08:58
ltomasbowell, we can include the complete information into the namespace annotation09:03
ltomasbobut it is a bit more ugly and will not help for garbage collection as you said09:03
ltomasboirenab, ^^why you think it will not work with pools?09:04
openstackgerritDaniel Mellado proposed openstack/kuryr-kubernetes master: Apply ovs-firewall in openshift gates  https://review.openstack.org/56352409:07
openstackgerritMerged openstack/kuryr-kubernetes master: Add better octavia coverage at the gate  https://review.openstack.org/56299809:28
*** pmannidi has joined #openstack-kuryr10:12
apuimedoirenab: dulek: ltomasbo: dmellado: can https://review.openstack.org/#/c/562067/ get some love?10:26
ltomasboxD10:27
* dulek pats the patch on the HEAD.10:27
ltomasboapuimedo, did you check security groups for octavia after applying the subnet split?10:28
ltomasboI'll give it a try with both L3 and L210:29
apuimedoltomasbo: how should l2/l3 affect it?10:30
apuimedothis is for the vip, not for the member connectivity10:30
ltomasboapuimedo, we were enabling the vip subnet on the pods, right?10:31
apuimedoltomasbo: in SG? Yes10:32
*** chenyb4 has quit IRC10:33
ltomasboyes10:33
ltomasbofor L2 I guess it should not matter, for L3, as long as the right service cird, it should work, you are right10:34
apuimedodmellado: irenab: we should also merge https://review.openstack.org/#/c/562366/10:34
apuimedodulek: and it should be backported!10:34
apuimedoqueens supports 3.9 and 3.1010:35
apuimedoprobably also 3.1110:35
*** lxkong has quit IRC10:35
dulekapuimedo: I'm not 100% sure on backporting. We've released with 3.7 in mind, haven't we? And everything was tested on 3.7.10:38
dulekapuimedo: Moreover - 3.9 was not released when Queens was released. :D10:38
irenabltomasbo, regarding the pools, if netwroks are created in advance, you cannot pick the name with namespace name, it can be updated later.10:38
* dulek takes lunch out of the oven.10:39
ltomasboirenab, you don't include the pools into CRDs (with my patch)10:39
ltomasboso, pools will be populated, and loaded10:40
irenabltomasbo,  I know, just wander if this will be the possible perf. improvments10:40
ltomasboand the namespace will have their net_ids associated to it10:40
dulekapuimedo: https://review.openstack.org/#/c/562366 only needs a +W now.10:43
irenabapuimedo, https://review.openstack.org/#/c/562067/ has few unanswered comments10:45
*** pmannidi has quit IRC10:47
openstackgerritMichał Dulko proposed openstack/kuryr-kubernetes master: Raise OpenShift version to 3.9.0  https://review.openstack.org/56236610:49
openstackgerritMichał Dulko proposed openstack/kuryr-kubernetes master: Gate with containerized deployment and OpenShift  https://review.openstack.org/55731310:49
apuimedoirenab: good point10:50
apuimedoI'll check10:50
*** caowei has quit IRC11:13
*** rh-jelabarre has joined #openstack-kuryr11:40
dmelladodulek: o/12:09
dmelladobtw, regarding ovs-firewall nope12:09
dmelladothat approach don't work at all12:09
dmelladoLOL12:09
dmelladothat's what everyone asks12:10
dmelladothe "/" is an specific requirement for that one only12:10
*** caisan has quit IRC12:10
*** yamamoto has joined #openstack-kuryr12:14
apuimedodmellado: "for that one only" dafuq does that mean?12:22
dmelladoapuimedo: it means crappy bullshit on neutron12:22
irenabapuimedo, it sound Hebrew12:23
apuimedoI just don't understand12:24
irenabcontext?12:25
apuimedodmellado: can you translate it to "all my brain blood is in my stomach digesting a big burger"?12:25
apuimedoirenab: about specifying the neutron firewall in gates12:25
irenabapuimedo, probably broken12:26
*** yboaron has quit IRC12:26
dmelladoMine too12:28
dmelladoHold on12:28
dmelladoXd12:28
dmelladoapuimedo: so, check the patch12:33
dmelladohttps://review.openstack.org/#/c/563524/3..4/.zuul.yaml12:33
dmelladothat f**** /Q_bulshit12:33
dmelladoxD12:33
*** portdirect has quit IRC12:36
*** portdirect has joined #openstack-kuryr12:36
apuimedodmellado: so how do we apply it? With a post ansible role?12:36
apuimedoor task12:36
dmelladoit should be parsed by ansible role coming from devstack12:37
dmelladowhich isn't doing a proper job12:37
dmelladoIMHO it's a zuul bug but I'm trying to debug it12:37
*** maysamacedos has joined #openstack-kuryr12:37
openstackgerritMerged openstack/kuryr-kubernetes master: Raise OpenShift version to 3.9.0  https://review.openstack.org/56236612:42
apuimedodmellado: ah, ok. For a moment I thought it was a closed-won't fix there12:42
dulekdmellado: It seems like you haven't tried this:13:01
dulek      devstack_local_conf:13:02
dulek        post-config:13:02
dulek          $Q_PLUGIN_CONF_FILE:13:02
dulek            firewall_driver: openvswitch13:02
dmelladoheh WE TRIED13:03
dmelladoxD13:04
dmelladothere's another patch from ltomasbo when we tried at the same time13:04
dmelladojust like power rangers when they came together to defeat the ultimate evil13:04
dmelladoit was just like the megazord, but with commits13:04
ltomasboxD13:04
dmelladonow seriously, I'm trying to run that on a local machine13:04
dmelladoto see if I can get to debug something13:04
dulekOkay.13:04
dmelladoIMHO I think this hasn't been taken into consideration when designing the parser13:06
dmelladoand it comes from a legacy neutron-plugin devstack thing, but anyways13:06
apuimedoquite likely13:06
apuimedodmellado: to be honest13:06
apuimedothis is all bollocks13:06
apuimedowe should just have sent a patch to neutron devstack ages ago13:07
apuimedoto allow to select the firewall driver13:07
apuimedowith a setting13:07
dmelladototally13:07
dmelladothat's my second option if I get blocked here13:07
*** janonymous has quit IRC13:07
*** salv-orlando has quit IRC13:18
*** salv-orlando has joined #openstack-kuryr13:19
*** atoth has joined #openstack-kuryr13:20
*** salv-orlando has quit IRC13:24
openstackgerritLuis Tomas Bolivar proposed openstack/kuryr-kubernetes master: Run OVN job containerized and daemonized  https://review.openstack.org/54928913:24
*** yamamoto has quit IRC13:28
*** yamamoto has joined #openstack-kuryr13:28
*** yboaron has joined #openstack-kuryr13:37
*** atoth has quit IRC13:43
*** gcheresh has joined #openstack-kuryr13:49
*** gcheresh_ has quit IRC13:49
*** atoth has joined #openstack-kuryr13:58
*** hongbin_ has joined #openstack-kuryr13:59
*** atoth has quit IRC14:04
*** atoth has joined #openstack-kuryr14:16
*** caisan has joined #openstack-kuryr14:16
*** atoth has quit IRC14:16
*** atoth has joined #openstack-kuryr14:17
*** fabian has joined #openstack-kuryr14:19
*** caisan has quit IRC14:31
*** caisan has joined #openstack-kuryr14:32
*** yboaron has quit IRC14:32
*** yboaron has joined #openstack-kuryr14:33
*** fabian has quit IRC14:41
*** gcheresh has quit IRC14:47
*** ajo has quit IRC14:50
*** ajo has joined #openstack-kuryr14:51
dulekapuimedo, dmellado: One more patch for today: https://review.openstack.org/#/c/557313 :)14:52
dulekAll tests are fine and it already had +2*2 and +W. I just needed to rebase it.14:52
apuimedodulek: done from me. Waiting on dmellado now14:53
openstackgerritAntoni Segura Puimedon proposed openstack/kuryr-kubernetes master: devstack: fix octavia allocation ranges  https://review.openstack.org/56206714:57
apuimedodulek: https://review.openstack.org/#/c/562067/3 addresses yboaron's comments14:58
*** yamamoto has quit IRC14:58
dulekapuimedo: Each time I see "whether" I need to check if first "h" is correct. :P14:59
dmelladoDone ;)14:59
apuimedodulek: :-)15:00
*** premsankar has joined #openstack-kuryr15:28
*** salv-orl_ has joined #openstack-kuryr15:35
*** yboaron has quit IRC15:38
*** caisan has quit IRC15:39
*** pcaruana has quit IRC15:51
*** maysamacedos has quit IRC15:51
*** atoth has quit IRC15:53
*** atoth has joined #openstack-kuryr15:55
openstackgerritMerged openstack/kuryr-tempest-plugin master: Testing curl succeeds from pod to the service  https://review.openstack.org/55848215:56
*** yamamoto has joined #openstack-kuryr15:59
openstackgerritMerged openstack/kuryr-kubernetes master: Add namespace handler  https://review.openstack.org/56215916:03
openstackgerritMerged openstack/kuryr-kubernetes master: Gate with containerized deployment and OpenShift  https://review.openstack.org/55731316:03
*** yamamoto has quit IRC16:04
*** jchhatbar has quit IRC16:32
*** garyloug has quit IRC16:32
*** pcaruana has joined #openstack-kuryr16:43
*** salv-orl_ has quit IRC16:48
*** maysamacedos has joined #openstack-kuryr16:53
*** yamamoto has joined #openstack-kuryr17:00
*** atoth has quit IRC17:02
*** yamamoto has quit IRC17:06
*** atoth has joined #openstack-kuryr17:17
*** maysamacedos has quit IRC17:58
*** yamamoto has joined #openstack-kuryr18:03
*** maysamacedos has joined #openstack-kuryr18:03
*** atoth has quit IRC18:08
*** yamamoto has quit IRC18:08
openstackgerritMaysa de Macedo Souza proposed openstack/kuryr-kubernetes master: Fix LB member creation on Nested environment  https://review.openstack.org/56280018:24
*** yboaron has joined #openstack-kuryr18:25
*** pcaruana has quit IRC18:32
*** gcheresh has joined #openstack-kuryr18:34
*** dims has quit IRC19:02
*** yamamoto has joined #openstack-kuryr19:04
*** dims has joined #openstack-kuryr19:09
*** yamamoto has quit IRC19:10
*** salv-orlando has joined #openstack-kuryr19:16
openstackgerritYossi Boaron proposed openstack/kuryr-kubernetes master: OCP-Router: add support for external Ingress Controller's LB creation  https://review.openstack.org/56405119:28
openstackgerritYossi Boaron proposed openstack/kuryr-kubernetes master: OCP-Router: Ingress controller support  https://review.openstack.org/53638719:55
*** yboaron has quit IRC20:03
*** yamamoto has joined #openstack-kuryr20:06
*** gcheresh has quit IRC20:06
*** yamamoto has quit IRC20:11
*** maysamacedos has quit IRC20:26
*** dmellado has quit IRC20:38
*** dmellado has joined #openstack-kuryr20:43
*** yamamoto has joined #openstack-kuryr21:08
*** lxkong has joined #openstack-kuryr21:09
*** yamamoto has quit IRC21:14
*** apuimedo has quit IRC21:32
*** yamamoto has joined #openstack-kuryr22:09
*** yamamoto has quit IRC22:14
*** yamamoto has joined #openstack-kuryr23:10
*** yamamoto has quit IRC23:16
*** hongbin_ has quit IRC23:33
*** yamamoto has joined #openstack-kuryr23:58

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