Monday, 2018-03-26

*** maysamacedos has joined #openstack-kuryr00:32
*** caowei has joined #openstack-kuryr00:34
*** hongbin has joined #openstack-kuryr01:12
*** salv-orl_ has joined #openstack-kuryr01:30
*** salv-orlando has quit IRC01:33
*** phuoc has quit IRC01:36
*** phuoc has joined #openstack-kuryr01:51
*** maysamacedos has quit IRC01:55
*** premsankar has quit IRC02:06
*** phuoc has quit IRC02:07
*** dangtrinhnt has joined #openstack-kuryr02:11
*** phuoc has joined #openstack-kuryr02:15
*** phuoc has quit IRC02:21
*** phuoc has joined #openstack-kuryr02:37
*** phuoc has quit IRC02:43
*** phuoc has joined #openstack-kuryr02:47
*** dangtrinhnt has quit IRC03:14
*** c00281451 has joined #openstack-kuryr03:22
*** yamamoto has joined #openstack-kuryr03:27
*** yamamoto has quit IRC03:31
*** hongbin has quit IRC03:45
*** yamamoto has joined #openstack-kuryr04:28
*** janonymous has joined #openstack-kuryr04:49
*** gcheresh has joined #openstack-kuryr05:18
*** gcheresh has quit IRC05:25
*** gcheresh has joined #openstack-kuryr05:26
irenableyal, ping05:33
irenableyal, I posted comments on previous patchset of https://review.openstack.org/#/c/526916/, I think they are still relevant on the latest version05:36
*** yboaron has joined #openstack-kuryr06:01
*** phuoc_ has joined #openstack-kuryr06:27
*** phuoc has quit IRC06:29
leyalirenab, thanks ! I will take a look at them today ..06:33
*** pcaruana has joined #openstack-kuryr07:08
*** celebdor has joined #openstack-kuryr07:08
celebdormorning\07:08
*** aojea has joined #openstack-kuryr07:13
*** yboaron has quit IRC07:16
*** aojea has quit IRC07:22
*** aojea has joined #openstack-kuryr07:29
*** aojea has quit IRC07:35
*** aojea has joined #openstack-kuryr07:42
*** aojea has quit IRC07:49
openstackgerritEyal Leshem proposed openstack/kuryr-kubernetes master: Translate k8s policy to SG  https://review.openstack.org/52691607:58
openstackgerritOpenStack Proposal Bot proposed openstack/fuxi master: Updated from global requirements  https://review.openstack.org/51909408:02
openstackgerritOpenStack Proposal Bot proposed openstack/kuryr master: Updated from global requirements  https://review.openstack.org/54953708:06
openstackgerritOpenStack Proposal Bot proposed openstack/kuryr-libnetwork master: Updated from global requirements  https://review.openstack.org/55640708:06
irenableyal, hi08:09
irenabDoes https://review.openstack.org/#/c/545872/ still WIP or shall be reviewed?08:09
openstackgerritEyal Leshem proposed openstack/kuryr-kubernetes master: DisposableWatch for k8s client  https://review.openstack.org/54344808:10
leyalirenab, tests are still missing for that patch08:21
irenabok08:22
*** garyloug has joined #openstack-kuryr08:26
*** celebdor has quit IRC08:32
openstackgerritEyal Leshem proposed openstack/kuryr-kubernetes master: Create network policy handler and driver  https://review.openstack.org/53065508:39
*** janonymous has quit IRC08:39
*** celebdor has joined #openstack-kuryr08:43
openstackgerritMichał Dulko proposed openstack/kuryr-kubernetes master: Execute kuryr-cni as `docker exec`  https://review.openstack.org/55504008:43
*** yboaron has joined #openstack-kuryr08:46
celebdordulek: did you see yboaron's comments to https://review.openstack.org/#/c/555254/2 ?08:47
dulekcelebdor: Yup. And that comment make sense, but I don't really have an idea on solving it.08:49
dulekIt's impossible to run `kubectl logs` with label selector *and* `-f`.08:49
dulekWe can probably just add a playbook that will generate and copy the logs…08:50
celebdorI suppose that would be the best08:55
celebdordmellado: what do you think?08:55
* dmellado on the superfluidity program call08:55
dmelladocelebdor: what's the context?08:55
dmelladolet me check the backlog08:56
dmelladooh, for the logs?08:56
dulekdmellado: Yup.08:57
dmelladodulek: for the logs, maybe what can be done is add a trace08:57
dmelladolike ,for the tempest tests or something08:57
dmelladoso we could filter that08:57
dulekTrace?08:57
dulekI don't think I understand.08:57
dmelladoyeah, so for the logs08:58
dmelladoi.e. log.DEBUG("The tempest tests start here")08:58
dmelladobut in any case I think that's something we can tackle on a a follow up patch08:58
dmelladoyboaron: wdyt?08:58
yboarondmellado, changing to +1 :-)08:59
dmelladobtw dulek celebdor yboaron I'll be on a train during today's meeting and I'm not really confident on its wifi reliability08:59
dmelladocould you please chair today's meeting just in case?08:59
dulekSure.08:59
dmelladothanks!08:59
dmelladoI *should* be able to be there but I'm not trustworthy at all when it comes to the spanish railway company08:59
dmelladoits website it's the worst ever in the history of internet09:00
dmelladoxD09:00
yboaroncelebdor, question regarding : https://bugs.launchpad.net/kuryr-kubernetes/+bug/170794009:02
openstackLaunchpad bug 1707940 in kuryr-kubernetes "lbaas pool members are nameless" [High,Triaged] - Assigned to Antoni Segura Puimedon (celebdor)09:02
celebdortell me09:03
yboaroncelebdor, why did you check 'openstack loadbalancer member list ...' instead of 'neutron lbaas-member-list' ?09:03
celebdordoes it make any difference?09:04
yboaroncelebdor, it's weird , but when running 'neutron lbaas-...' comamnd member's name is correct09:04
yboaroncelebdor, checked Kuryr code and all looks fine to me09:05
yboaroncelebdor, looks like member name not presented from some reason when using 'openstack loadbalancer ..' API for LBAASV209:06
yboaroncelebdor, I guess we should use the 'neutron lbaas-' API , right ?09:07
celebdoryboaron: Okay, let's do this09:08
celebdorcheck if the name is in mysql octavia tables or only on the neutron ones09:09
celebdorand depending on the result we'll move the bug to octavia09:09
yboaroncelebdor, NP , will check it.09:10
openstackgerritBerezovsky Irena proposed openstack/kuryr-kubernetes master: DisposableWatch for k8s client  https://review.openstack.org/54344809:16
celebdorthanks yboaron09:18
openstackgerritEyal Leshem proposed openstack/kuryr-kubernetes master: Create network policy handler and driver  https://review.openstack.org/53065509:26
*** kiennt26 has joined #openstack-kuryr09:31
*** kiennt26 has quit IRC09:54
celebdoryboaron: excuse me if I'm asking something a bit stupid, since I am not too familiar with this code10:03
dmelladobtw celebdor irenab10:03
dmelladoI was wondering about what we should do re: dpdk10:03
celebdorin https://review.openstack.org/#/c/554070/1/kuryr_kubernetes/controller/handlers/lbaas.py what is waiting for the lbaas to be active?10:03
dmelladoI've been waiting on the intel folks to give some ack since PTG10:03
dmelladoI'll reply back to them to check options about what to do10:03
dmelladoworst case I'd just take over their patches but I'd first like to sync with them10:04
yboaroncelebdor, min10:04
dmelladoI haven't seem gary for a long time10:04
dmellado;\10:04
*** janonymous has joined #openstack-kuryr10:06
yboaroncelebdor, which code you refer to ?10:06
*** kaliya has quit IRC10:11
yboaroncelebdor, did you refer to this ?  : https://github.com/openstack/kuryr-kubernetes/blob/master/kuryr_kubernetes/controller/drivers/lbaasv2.py#L37510:12
celebdorright10:13
celebdorthanks yboaron10:13
celebdoryboaron: what does the 'prev' suffix stand for10:14
yboaroncelebdor, previous10:14
yboaroncelebdor, for the floating IP patch, I just moved the K8S status update after  openstack LB provisioning completes10:15
yboaronyboaron, current code , update status after we we got reply from neutron for 'lb_create'10:16
yboarondmellado, Let me know If I can help with the DPDK patches10:19
dmelladoyboaron: sure, thanks!10:22
celebdorI'm slow this morning I gues10:22
celebdor*guess10:22
celebdorI don't quite see it10:22
*** celebdor has quit IRC10:33
irenabdmellado, once you get some answer from Intel team, lets see how to proceed10:36
dmellado+110:44
*** celebdor has joined #openstack-kuryr10:54
*** caowei has quit IRC11:01
*** maysamacedos has joined #openstack-kuryr11:08
*** dangtrinhnt has joined #openstack-kuryr11:13
celebdoryboaron: so my question on https://review.openstack.org/#/c/554070/1/kuryr_kubernetes/controller/handlers/lbaas.py is. If I undertood correctly, before we'd get the FIP and annotate immediately11:17
celebdorwith the change11:17
celebdorafter associating the FIP11:17
celebdorwe just return11:18
celebdorso when will the the on_present (k8s triggered) be called again so that we put the annotation?11:18
*** jlabarre has joined #openstack-kuryr11:22
*** yamamoto has quit IRC11:27
openstackgerritEyal Leshem proposed openstack/kuryr-kubernetes master: DisposableWatch for k8s client  https://review.openstack.org/54344811:31
yboaroncelebdor, just finished my lunch ...11:32
celebdordulek: why did it it fail the containerized? https://review.openstack.org/#/c/555040/ ?11:32
*** yamamoto has joined #openstack-kuryr11:36
celebdoryboaron: so?11:38
openstackgerritEyal Leshem proposed openstack/kuryr-kubernetes master: Annotate pods that selected by k8s label-selector  https://review.openstack.org/54566311:38
yboaroncelebdor, So ???11:39
celebdorcelebdor: I was asking some questions 12 minutes ago11:40
celebdordid you receive them?11:41
yboaroncelebdor, Yep!!11:41
yboaroncelebdor, for creation  LB in openstack we call neutron to create 1. LB  2.Listener 3. pool 4. members11:41
yboaroncelebdor, for steps 2,3,4 - we have the wait for active loop11:42
yboaroncelebdor, current code for Loadbalancer service type, allocates FIP, bind to VIP , and patch K8S status after step 111:43
yboaroncelebdor, for OCTAVIA case step 2,3,4 could take ~100-200 seconds11:43
yboaroncelebdor, *steps11:43
dulekcelebdor: I think that I simple don't know how to increment in Bash. :P11:44
celebdor:P11:44
dulekcelebdor: ((x++)) returns 1 and I have -e in shebang, so I guess that the cause.11:44
celebdoryboaron: I'm still a bit lost11:45
celebdormind to go over it with me after the weekly?11:45
yboaroncelebdor, my patch just moves the K8s status patching after we step 4  completed11:45
*** yamamoto has quit IRC11:45
celebdorI see the code being put before where it was11:45
yboaroncelebdor, OK, will ping U after weekly11:46
dulekHm, it's probably worth saying - Kuryr's IRC meeting is 14:00 UTC, due to DST it's now 16:00 CET.11:47
celebdoryup11:47
*** maysamacedos has quit IRC11:51
openstackgerritMichał Dulko proposed openstack/kuryr-kubernetes master: Execute kuryr-cni as `docker exec`  https://review.openstack.org/55504011:57
*** atoth has joined #openstack-kuryr11:59
*** yamamoto has joined #openstack-kuryr12:05
*** snapiri has quit IRC12:16
*** dangtrinhnt has quit IRC12:19
*** yamamoto has quit IRC12:30
*** phuoc_ has quit IRC12:39
*** phuoc_ has joined #openstack-kuryr12:39
*** phuoc_ has quit IRC12:47
*** caowei has joined #openstack-kuryr12:58
yboaronping leyal13:00
*** phuoc has joined #openstack-kuryr13:17
*** yamamoto has joined #openstack-kuryr13:20
*** yamamoto has quit IRC13:20
*** yamamoto has joined #openstack-kuryr13:20
*** celebdor has quit IRC13:21
*** celebdor has joined #openstack-kuryr13:30
dulekdmellado: Can you accept https://review.openstack.org/555755 ? I don't want to +2 a patch that I've edited.13:32
dmelladolemme check13:36
dmelladooh, the lower req13:36
dmelladosure13:36
*** janonymous has quit IRC13:49
*** hongbin has joined #openstack-kuryr13:55
irenabdmellado, I can only partially attend the weekly, need to make a short break due to some logistics13:56
dmelladoirenab: no prob, I'll be around the first 10-15' but dulek will be the chairmaster today xD13:57
*** jistr is now known as jistr|mtg14:02
*** yamamoto has quit IRC14:03
*** kong has quit IRC14:13
*** yboaron has quit IRC14:14
*** yboaron has joined #openstack-kuryr14:19
*** jistr|mtg is now known as jistr14:27
openstackgerritMerged openstack/kuryr-kubernetes master: add lower-constraints job  https://review.openstack.org/55575514:41
celebdordulek: could we merge https://review.openstack.org/#/c/555206/ ?14:46
celebdoryboaron already tried it14:46
*** leifmadsen has quit IRC15:02
*** yamamoto has joined #openstack-kuryr15:03
*** leifmadsen has joined #openstack-kuryr15:04
*** gcheresh_ has joined #openstack-kuryr15:08
*** celebdor has quit IRC15:09
*** gcheresh_ has quit IRC15:10
*** gcheresh_ has joined #openstack-kuryr15:11
*** gcheresh has quit IRC15:11
*** caowei has quit IRC15:13
*** yamamoto has quit IRC15:15
*** gcheresh_ has quit IRC15:16
*** pcaruana has quit IRC15:18
*** gcheresh_ has joined #openstack-kuryr15:35
*** yboaron has quit IRC15:42
*** gcheresh_ has quit IRC15:47
openstackgerritMerged openstack/kuryr-kubernetes master: Services: Add support for K8S service's port edit use case  https://review.openstack.org/55177316:09
*** salv-orl_ has quit IRC16:25
*** salv-orlando has joined #openstack-kuryr16:26
*** gcheresh_ has joined #openstack-kuryr16:26
*** salv-orlando has quit IRC16:30
*** gcheresh_ has quit IRC16:38
openstackgerritMichał Dulko proposed openstack/kuryr-libnetwork master: add lower-constraints job  https://review.openstack.org/55575616:42
*** garyloug has quit IRC16:43
*** pcaruana has joined #openstack-kuryr16:44
*** celebdor has joined #openstack-kuryr16:47
*** maysamacedos has joined #openstack-kuryr16:53
*** celebdor has quit IRC17:06
openstackgerritMichał Dulko proposed openstack/kuryr-kubernetes master: Execute kuryr-cni as `docker exec`  https://review.openstack.org/55504017:20
*** celebdor has joined #openstack-kuryr17:22
*** salv-orlando has joined #openstack-kuryr17:26
dulekcelebdor: Any idea on what to do with https://review.openstack.org/#/c/552856/ ?17:28
*** salv-orlando has quit IRC17:31
*** gcheresh_ has joined #openstack-kuryr17:41
celebdordulek: personally I'd merge it and I'd make a follow-up bug for tunneling17:54
celebdorgotta go, ping me on signal if anything needs my attention these days17:54
*** salv-orlando has joined #openstack-kuryr18:02
*** gcheresh_ has quit IRC18:04
*** gcheresh_ has joined #openstack-kuryr18:04
*** phuoc has quit IRC18:12
*** pcaruana has quit IRC18:46
openstackgerritMerged openstack/kuryr-kubernetes master: fix kury-cni exits unexpectedly when MTU of network is 0  https://review.openstack.org/55285618:51
*** snapiri has joined #openstack-kuryr18:58
*** atoth has quit IRC19:00
*** snapiri has quit IRC19:15
*** aojea has joined #openstack-kuryr19:29
*** salv-orlando has quit IRC19:31
*** salv-orlando has joined #openstack-kuryr19:31
*** salv-orlando has quit IRC19:37
*** yamamoto has joined #openstack-kuryr19:45
*** salv-orlando has joined #openstack-kuryr19:46
*** yamamoto has quit IRC19:50
*** maysamacedos has quit IRC20:42
*** dougbtv__ is now known as dougbtv20:42
openstackgerritMichał Dulko proposed openstack/kuryr-kubernetes master: Execute kuryr-cni as `docker exec`  https://review.openstack.org/55504020:49
*** kong has joined #openstack-kuryr21:08
openstackgerritMerged openstack/kuryr-kubernetes master: devstack: add kubelet iface to openshift nodes  https://review.openstack.org/55520622:08
*** aojea has quit IRC22:18
*** gcheresh_ has quit IRC22:20
*** salv-orlando has quit IRC22:22
*** salv-orlando has joined #openstack-kuryr22:23
*** salv-orlando has quit IRC22:27
*** jlabarre has quit IRC22:34
*** celebdor has quit IRC22:36
*** hongbin has quit IRC22:58
*** salv-orlando has joined #openstack-kuryr23:23
*** salv-orlando has quit IRC23:28

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