Tuesday, 2018-01-16

*** yamamoto has joined #openstack-kuryr00:36
*** atoth has joined #openstack-kuryr00:56
*** maysamacedos has quit IRC01:16
*** caowei has joined #openstack-kuryr01:25
*** pmannidi has joined #openstack-kuryr01:29
*** caowei has quit IRC01:32
*** kiennt26 has joined #openstack-kuryr01:46
*** hongbin has joined #openstack-kuryr01:56
*** threestrands_ has joined #openstack-kuryr02:02
*** jappleii__ has quit IRC02:04
*** hongbin has quit IRC02:53
*** hongbin has joined #openstack-kuryr02:54
*** atoth has quit IRC03:06
*** kiseok7 has quit IRC03:08
*** hongbin has quit IRC03:19
*** hongbin has joined #openstack-kuryr03:20
openstackgerritOpenStack Proposal Bot proposed openstack/fuxi master: Updated from global requirements  https://review.openstack.org/51909403:21
openstackgerritOpenStack Proposal Bot proposed openstack/kuryr master: Updated from global requirements  https://review.openstack.org/51693603:26
openstackgerritOpenStack Proposal Bot proposed openstack/kuryr-libnetwork master: Updated from global requirements  https://review.openstack.org/52374003:27
openstackgerritOpenStack Proposal Bot proposed openstack/kuryr-kubernetes master: Updated from global requirements  https://review.openstack.org/50978103:27
*** yamamoto_ has joined #openstack-kuryr03:39
*** yamamoto has quit IRC03:43
*** janonymous has joined #openstack-kuryr04:16
*** phuoc has quit IRC04:29
*** hongbin has quit IRC05:04
*** janki has joined #openstack-kuryr05:10
*** phuoc has joined #openstack-kuryr05:27
openstackgerritwanghui proposed openstack/kuryr master: fix Neutron misspelling  https://review.openstack.org/53408705:47
*** yboaron has joined #openstack-kuryr06:12
*** yboaron has quit IRC06:25
*** gcheresh has joined #openstack-kuryr06:26
*** yboaron has joined #openstack-kuryr06:40
*** c00281451_ is now known as zengchen06:52
*** juriarte has joined #openstack-kuryr07:18
*** threestrands_ has quit IRC07:18
*** juriarte has quit IRC07:22
*** juriarte has joined #openstack-kuryr07:37
*** janki has quit IRC07:39
openstackgerritYossi Boaron proposed openstack/kuryr-kubernetes master: Kuryr-Kubernetes openshift route integration devref  https://review.openstack.org/53009507:44
*** aojea_ has joined #openstack-kuryr07:45
*** kaliya has joined #openstack-kuryr08:22
openstackgerritMerged openstack/kuryr master: fix Neutron misspelling  https://review.openstack.org/53408708:23
leyalyboaron , hi08:25
yboaronleyal, hi08:26
leyalyboaron, i am trying to review the ingress spec - and something that i think that i am not understanding08:27
yboaronleyal, OK08:28
leyalyboaron- in use use case example (line 205) , the spec describe that the ingress-spec is triggering  the creation of the lbass polls and listeners08:28
leyalbut isn't the service s1 should be there before the ingress is created ?08:29
leyaland the creation of the service should trigger the creation of lbaas pools ?08:30
yboaronleyal, yes , it;s a bit confusing , take a look at the last figure in spec , the one that describes the neutron lbass entities of both the L7 router and the service08:31
yboaronleyal, #link http://logs.openstack.org/66/529966/17/check/build-openstack-sphinx-docs/bef6873/html/devref/kuryr_kubernetes_ingress_design.html08:32
leyalyboaron, yes i still do not understand why the pools is created by the ingress handler - as they need already created during the service creation in the service handler..08:35
yboaronleyal, did u manage to see the figure under section 'The L7 router driver¶08:36
yboaron' in the link I just sent ?08:36
yboaronleyal, so the service handler will create the  entities colored in grey (LB,listeners,pools,members)08:37
yboaronleyal, there are two pools in that case , one that attached to the user service . this is not relevant for ingress , just lbaas handler as it  supported in current kuryr version08:38
yboaronleyal, the other pool (colored in green) is relevant for ingress08:39
yboaronleyal, this is the link to the relevant figure #link : http://logs.openstack.org/66/529966/17/check/build-openstack-sphinx-docs/bef6873/html/_images/l7_routing_and_user_lb_neutron_entities.svg08:40
leyalyboaron, tell me if i got it right - when service is create it's create a listener on the service ip and port - this listener is connected to pool , and when the ingress is create you create an http listener and new pool that connected to the same members of the service pools?08:42
yboaronleyal, in general that's correct , with  one correction , when the ingress is created we need to create some entity that will run the ingress/L7 rules ,08:44
yboaronleyal, this entity is the L7policy/L7rule , L7 policy pointed the filtered traffic towards relevant pool (the one that attached to L7 router LB08:45
*** janki has joined #openstack-kuryr08:46
*** janki has quit IRC08:46
*** janki has joined #openstack-kuryr08:47
leyalyboaron , cool. so my question is why we can't reuse the pools of service in the l7-listener instead of creating new polls08:47
leyalyboaron , i think it's will follow the logic that ingress maped to service and service is mapped to endpoints08:48
yboaronleyal, that's could be a good idea , but unfortunately  it's invalid in Lbaas/OCTAVIA08:50
openstackgerritwanghui proposed openstack/kuryr master: change http to https  https://review.openstack.org/53419108:50
leyalyboaron, when i create l7 policy by "neutron --policy policy1 lbaas-create-l7policy --name "policy1" --description "My Description" --listener "listener1" --action redirect_to_pool --pool "pool1" --position 1"08:52
leyali am not free to choose my pool ?08:52
yboaronleyal, before directing l7 policy to pool u need to create the pool , every pool must be connected to either LB or listener08:54
yboaronleyal, and it's invalid to point to pool that already used by another LB08:56
yboaronleyal, anyhow , you should consider  the case in which ingress is created before service  ?08:58
leyalyboaron, yep i saw now that it's 1:1 relationship between listener/LB and pool , it's surprises me .08:59
leyalyboaron, is that scenario is valid?09:00
leyalyou will need the members anyway ..09:00
yboaronleyal, the members will be added later , when the service is created09:01
leyalyboaron , any way if you can add 2 listeners to same pools it's not relevant ..09:02
leyals/can/can't09:03
leyalthanks for all explanations :)09:03
yboaronleyal, you're welcome09:04
*** garyloug has joined #openstack-kuryr09:07
*** kiennt26 has quit IRC10:11
*** dougbtv_ has joined #openstack-kuryr10:14
*** dougbtv has quit IRC10:16
*** aojea_ has quit IRC10:16
*** aojea_ has joined #openstack-kuryr10:16
*** dougbtv__ has joined #openstack-kuryr10:28
*** dougbtv_ has quit IRC10:31
*** vkuramshin has joined #openstack-kuryr11:01
*** aojea_ has quit IRC11:19
*** yamamoto_ has quit IRC11:27
*** aojea has joined #openstack-kuryr11:39
*** aojea has quit IRC11:40
*** maysamacedos has joined #openstack-kuryr11:40
*** leyal has quit IRC11:48
*** lihi has quit IRC11:48
*** irenab has quit IRC11:48
*** oanson has quit IRC11:48
*** irenab has joined #openstack-kuryr11:48
*** oanson has joined #openstack-kuryr11:50
*** lihi has joined #openstack-kuryr11:52
openstackgerritMichał Dulko proposed openstack/kuryr-kubernetes master: Use virtualenv to builid kuryr-cni Docker image  https://review.openstack.org/53425312:05
dulekapuimedo: ^12:05
*** pmannidi has quit IRC12:09
*** lihi has quit IRC12:10
*** oanson has quit IRC12:10
*** irenab has quit IRC12:10
*** oanson has joined #openstack-kuryr12:12
*** irenab has joined #openstack-kuryr12:14
*** lihi has joined #openstack-kuryr12:18
*** yamamoto has joined #openstack-kuryr12:26
openstackgerritOpenStack Proposal Bot proposed openstack/fuxi master: Updated from global requirements  https://review.openstack.org/51909412:26
openstackgerritOpenStack Proposal Bot proposed openstack/fuxi master: Updated from global requirements  https://review.openstack.org/51909412:27
openstackgerritOpenStack Proposal Bot proposed openstack/fuxi master: Updated from global requirements  https://review.openstack.org/51909412:27
*** leyal has joined #openstack-kuryr12:27
openstackgerritOpenStack Proposal Bot proposed openstack/kuryr master: Updated from global requirements  https://review.openstack.org/51693612:28
openstackgerritOpenStack Proposal Bot proposed openstack/kuryr-libnetwork master: Updated from global requirements  https://review.openstack.org/52374012:28
apuimedodulek: to builid?12:28
dulekapuimedo: Oh, you know what I've meant. ;)12:29
apuimedo:-)12:30
dulekapuimedo: https://review.openstack.org/#/c/534206/12:30
dulekapuimedo: And this broke Nova, I'm not sure if we won't suffer as well: https://github.com/svinota/pyroute2/pull/44312:31
openstackgerritOpenStack Proposal Bot proposed openstack/kuryr master: Updated from global requirements  https://review.openstack.org/51693612:31
openstackgerritOpenStack Proposal Bot proposed openstack/kuryr-libnetwork master: Updated from global requirements  https://review.openstack.org/52374012:31
*** pmannidi has joined #openstack-kuryr12:32
*** pcaruana has joined #openstack-kuryr12:33
maysamacedosirenab dulek apuimedo: May I ask you to take a look at https://review.openstack.org/#/c/533292/ ? Thank you.12:35
irenabmaysamacedos, asap12:35
maysamacedosirenab: appreciate it12:38
*** pmannidi has quit IRC13:05
yboaronping irenab13:06
irenabyboaron, pong13:10
yboaronirenab, hi13:11
yboaronirenab, regarding your comment #link https://review.openstack.org/#/c/529966/13:11
yboaronirenab, could u please elaborate ?13:11
yboaronirenab, do u suggest to annotate the L7 router details in ingress resources ?13:12
irenabyes, somehow annotating the node (which is physical related entity) does not look the way to go13:14
irenabother option is to make sure that Router is 'created' at deploy time and then populate the details in the kuryr.conf13:15
*** garyloug has quit IRC13:16
yboaronirenab, yes that my plan , the router will be install at deploy time , do u suggest to keep all neutron resources used for L7 router in kuryr.conf file ?13:18
yboaronirenab, I think that having L7_ROUTER CRD  is the most suitable way to solve this issue , but I prefer to have it in next phase13:19
yboaronirenab, considering all the issues we"ll have with introducing new Paas into kuryr sources, I would prefer to have it in next phase13:21
irenabyboaron, agree on CRD, but I think having it in the kuryr.cof initially is prefered over node annotation13:21
irenabIt can be similar to what we have with the service subnet or project id, the neutron details come in kuryr.conf13:23
yboaronirenab, do you mean to keep all  neutron resources relevant for L7 router in kuryr.conf ?13:23
irenabmy suggestion was for the details you have for the initial router details13:23
irenabthe details you planned to keep in node annotation13:23
irenab{"fip": "172.24.4.14",13:24
irenab        "router_lb": {"versioned_object.data": {"id": "90732f0a-651a-4b17-a14e-9b0e01fbe774",13:24
irenab        "ip": "10.0.0.154", "name": "kuryr-l7-router", "port_id": "5c71a29a-0dc1-461e-81ee-2258a7e3842d",13:24
irenab        "project_id": "868307936d384c21824e5eb0425a3f42", "subnet_id": "9f6d8c9f-d22d-480e-80f5-867daa050ff8"13:24
yboaronirenab, don't us think it will be 'dirty' to have all this details in kuryr.conf ?13:25
irenabbut if you think  node annotation is better, we can start with it. I may miss something13:25
irenabThe queston is if you need all the details in the config, or some details can be retrieved given for example the LB-id13:26
irenabor port-id13:26
yboaronirenab, you are not missing anything , it's definitely  a point that should be handled in more appropriate way13:26
yboaronirenab, keep in mind that all the LbaaS drivers drivers works with this objects - I don't want to invent the wheel ..13:27
yboaronirenab, so yes I prefer to keep this information to re-use the Lbaas driver code13:27
yboaronirenab, I meant all the functions of lbaas driver ..13:28
*** atoth has joined #openstack-kuryr13:29
irenabyboaron, arewe talking about my other question on the different parts between OCP Router and Ingress ?13:30
*** garyloug has joined #openstack-kuryr13:30
yboaronirenab, no , we discussed the L7 router annotation13:31
irenabok :-)13:31
yboaronirenab, as per the ingress/route , in openshift deployment we"ll be able to create both ingress and route resources13:31
*** yamamoto_ has joined #openstack-kuryr13:32
yboaronirenab, while in K8s environment only ingress resources will be available13:32
irenabSo each may have its own l7 router, right?13:32
irenabso for OCP is having 2 routers is a valid option, better not go with kuryr.conf13:33
yboaronirenab, no we"ll have a single router ,13:33
irenabok, then it should be ok13:34
irenabBut probably not annotating Ingress/Route, since it will be data duplication13:34
apuimedocan we just forget about multiple routers for now?13:35
*** yamamoto_ has quit IRC13:35
irenabapuimedo, we did :-)13:35
*** yamamoto has quit IRC13:35
irenabnow trying to see the least hacky way to keep L7 Router details13:36
yboaronirenab, single router , annotating node (in first phase) , is it OK with you ?13:37
apuimedowhich details?13:38
*** danil has joined #openstack-kuryr13:38
yboaronapuimedo, neutron LB,listeners , floating IP13:38
irenabI prefer not to bind the L7 Router details to node13:38
yboaronapuimedo, all the objects that used for recovering lbaas resources by lbaas driver13:39
irenabyboaron, given LB uuid, can you get the rest of the LB related details?13:39
apuimedocan't we just tag them in lbaasv2 api?13:40
irenabapuimedo, since L7 router will be created at deploy time, we can put the details in the kuryr.conf13:40
yboaronapuimedo, irenab : we need to support two flows , create and  recover13:41
irenabtags can work as well13:41
irenabyboaron, create is at deploy time13:41
yboaronirenab, apuimedo and unlike k8s services we don't have the VIP for uniqueness13:41
irenabbefore kuryr-controller is running13:41
irenabyboaron, what do you mean by non-unique VIP13:42
irenabapuimedo, going forward CRD probably the way to keep the details13:43
yboaronirenab, for k8s service LB , kubernetes allocates the VIP for this LB , this vip is unique ,for the L7 router we should allocate the VIP13:44
irenabyboaron, but we have the same with k8s api service. apuimedo please correct me if I am wrong13:44
irenabyboaron, apuimedo sorry, have some meeting for the next 1.5 hours. Will follow up13:45
yboaronirenab, OK13:45
*** yamamoto has joined #openstack-kuryr13:47
*** yamamoto has quit IRC13:52
*** kiennt26 has joined #openstack-kuryr13:58
danilhello, folks. Could you please help me? Maybe someone faced this error. Error in "...python2.7/site-packages/oslo_versionedobjects/base.py" AttributeError: Subnet object has no attribute 'ips' while creating pod. Maybe problems with version of oslo_versionedobjects ??13:59
danilthanks13:59
apuimedoxD14:02
apuimedodanil: yes14:02
apuimedomodify requirements.txt14:02
apuimedowhere there is os-vif14:02
apuimedoinstead of >=1.7.014:02
apuimedoput <1.8.014:02
* apuimedo back to meeting14:02
danilapuimedo, thanks a lot. This works. But other errors have appeared "NoMatches: No 'kuryr_kubernetes.vif_translators' driver found, looking for u'binding_failed'"14:07
*** yamamoto has joined #openstack-kuryr14:17
*** danil has quit IRC14:17
*** yamamoto has quit IRC14:22
*** yboaron has quit IRC14:33
*** hongbin has joined #openstack-kuryr14:39
*** garyloug_ has joined #openstack-kuryr14:46
*** garyloug has quit IRC14:49
*** yamamoto has joined #openstack-kuryr15:03
*** yamamoto has quit IRC15:03
*** vkuramshin has left #openstack-kuryr15:11
*** yamamoto has joined #openstack-kuryr15:12
*** yamamoto has quit IRC15:17
*** maysamacedos has quit IRC15:31
*** gcheresh has quit IRC15:32
*** maysamacedos has joined #openstack-kuryr15:36
*** maysamacedos has quit IRC15:39
*** yamamoto has joined #openstack-kuryr15:58
*** yamamoto has quit IRC16:01
*** yamamoto has joined #openstack-kuryr16:01
*** yamamoto has quit IRC16:01
*** kiennt26 has quit IRC16:06
*** gcheresh has joined #openstack-kuryr16:24
*** janki has quit IRC16:29
*** vikasc has quit IRC16:35
*** maysamacedos has joined #openstack-kuryr16:39
*** yboaron has joined #openstack-kuryr16:40
*** maysamacedos has quit IRC16:41
*** yamamoto has joined #openstack-kuryr16:41
openstackgerritEyal Leshem proposed openstack/kuryr-kubernetes master: Translate k8s policy to SG  https://review.openstack.org/52691616:41
*** yboaron has quit IRC16:45
*** yamamoto has quit IRC16:46
*** juriarte has quit IRC16:46
openstackgerritEyal Leshem proposed openstack/kuryr-kubernetes master: Translate k8s policy to SG  https://review.openstack.org/52691616:47
*** vikasc has joined #openstack-kuryr16:51
*** atoth has quit IRC17:03
*** maysamacedos has joined #openstack-kuryr17:04
apuimedodulek: thanks for the quick replies to the patch review :-)17:26
*** openstackgerrit has quit IRC17:33
*** kaliya has quit IRC17:39
*** yamamoto has joined #openstack-kuryr17:44
*** yamamoto has quit IRC17:58
*** gcheresh has quit IRC17:59
*** maysamacedos has quit IRC18:23
*** maysamacedos has joined #openstack-kuryr18:30
*** gcheresh has joined #openstack-kuryr19:26
*** gcheresh has quit IRC19:33
*** maysamacedos has quit IRC19:41
*** maysamacedos has joined #openstack-kuryr19:47
*** pcaruana has quit IRC19:50
*** maysamacedos has quit IRC20:07
*** janonymous has quit IRC20:15
*** dougbtv_ has joined #openstack-kuryr20:40
*** dougbtv__ has quit IRC20:43
*** dougbtv has joined #openstack-kuryr20:44
*** dougbtv_ has quit IRC20:46
*** gcheresh has joined #openstack-kuryr20:50
*** phuoc has quit IRC21:01
*** phuoc has joined #openstack-kuryr21:02
*** gcheresh has quit IRC21:08
*** threestrands_ has joined #openstack-kuryr21:18
*** garyloug_ has quit IRC21:40
*** pmannidi has joined #openstack-kuryr23:17

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