Tuesday, 2017-07-18

*** caowei has joined #openstack-kuryr00:54
*** hyunsun has joined #openstack-kuryr01:09
*** kiennt_ has joined #openstack-kuryr01:19
*** yedongcan has joined #openstack-kuryr01:38
openstackgerritOpenStack Proposal Bot proposed openstack/kuryr-kubernetes master: Updated from global requirements  https://review.openstack.org/48118601:47
*** kiennt_ has quit IRC02:10
*** kiennt_ has joined #openstack-kuryr02:10
*** reedip has quit IRC02:41
*** yamamoto has joined #openstack-kuryr02:41
*** reedip has joined #openstack-kuryr02:44
*** hongbin has joined #openstack-kuryr02:59
*** hongbin has quit IRC04:11
*** yedongcan has quit IRC04:13
*** yedongcan has joined #openstack-kuryr04:14
*** s1061123 has quit IRC04:25
*** kiennt_ is now known as kiennt04:38
*** yuanying has quit IRC04:58
*** aojea has joined #openstack-kuryr05:02
*** aojea has quit IRC05:06
*** aojea has joined #openstack-kuryr05:11
*** aojea has quit IRC05:16
*** aojea has joined #openstack-kuryr05:20
*** aojea has quit IRC05:25
*** gouthamr has quit IRC05:25
*** aojea has joined #openstack-kuryr05:29
*** aojea has quit IRC05:34
*** kiennt has quit IRC05:43
*** aojea has joined #openstack-kuryr05:45
*** aojea has quit IRC05:50
*** kiennt has joined #openstack-kuryr06:00
*** aojea has joined #openstack-kuryr06:03
*** kural has joined #openstack-kuryr06:06
*** aojea has quit IRC06:08
*** aojea has joined #openstack-kuryr06:13
*** aojea has quit IRC06:17
*** reedip has quit IRC06:18
*** reedip has joined #openstack-kuryr06:19
*** janki has joined #openstack-kuryr06:21
*** kzaitsev_ws has joined #openstack-kuryr06:36
*** aojea has joined #openstack-kuryr07:20
*** kural has quit IRC07:23
*** kural has joined #openstack-kuryr07:45
*** egonzalez has joined #openstack-kuryr07:45
*** kural has quit IRC07:51
*** kural has joined #openstack-kuryr07:51
*** garyloug has joined #openstack-kuryr08:05
*** kural has quit IRC08:30
*** yamamoto has quit IRC08:36
dulekvikasc: FYI - I've forked pyroute2 (https://github.com/dulek/pyroute2) and removed the line that was causing the problems. Now I'm fighting with OVS sockets permission (ovs-agent isn't running as root in OpenStack-Helm).08:39
*** yamamoto has joined #openstack-kuryr09:01
*** yamamoto has quit IRC09:02
*** yamamoto has joined #openstack-kuryr09:04
*** kural has joined #openstack-kuryr09:14
dulekvikasc: I can ping VMs from containers and vice versa! It finally worked! :)09:59
dulekvikasc: I'll comment on your patch with issues I've found.09:59
vikascdulek, thats great to hear :)10:00
vikascdulek, thanks!10:00
*** s1061123 has joined #openstack-kuryr10:01
*** hyunsun has quit IRC10:10
*** yamamoto has quit IRC10:18
*** yamamoto has joined #openstack-kuryr10:28
*** pc_m has quit IRC10:34
*** yamamoto has quit IRC10:38
*** pc_m has joined #openstack-kuryr10:47
*** tonanhngo has joined #openstack-kuryr10:55
*** pc_m has quit IRC10:56
*** tonanhngo has quit IRC10:56
*** pc_m has joined #openstack-kuryr10:57
*** tonanhngo has joined #openstack-kuryr10:57
*** yedongcan has left #openstack-kuryr11:09
*** rwallner has joined #openstack-kuryr11:18
*** rwallner has quit IRC11:19
*** rwallner has joined #openstack-kuryr11:20
*** aojea has quit IRC11:29
*** yamamoto has joined #openstack-kuryr11:33
*** janki has quit IRC11:38
kzaitsev_wsirenab: are you around?12:01
*** yamamoto has quit IRC12:04
kzaitsev_wsI want to talk through what you suggest on the https://review.openstack.org/#/c/462455 before going in too deep12:06
*** kiennt has quit IRC12:14
kzaitsev_wsltomasbo: ping12:18
*** yamamoto has joined #openstack-kuryr12:19
*** aojea has joined #openstack-kuryr12:24
ltomasbokzaitsev_ws, pong12:24
kzaitsev_wsI have a couple of ideas how to make https://review.openstack.org/#/c/462456 better =)12:25
kzaitsev_wsit's about requesting the VFs and locking12:25
*** caowei has quit IRC12:26
kzaitsev_wsI can wrap the whole locking thing in a context-manager and use a with statement around relevant plugging code12:26
kzaitsev_wsI actually like the idea myself. should turn code more readable12:26
ltomasbokzaitsev_ws, I just saw your comment. I know it is a concurrent problem, but perhaps locking around the dict will help to have the lock just at one place, not across methods12:27
ltomasbobut your idea sounds good12:27
dulekvikasc, apuimedo: https://review.openstack.org/#/c/484736/ - just a very early WiP, but should give idea about how it's supposed to look like.12:28
kzaitsev_wsltomasbo: first thought about pushing the thing to a separate class, but then it's not clear what goes where12:28
kzaitsev_wswith a context manager it would be a single `with get_me_a_vf` thing, that would release the lock on __exit__12:29
*** aojea has quit IRC12:29
kzaitsev_wsso hopefully more readable =)12:29
ltomasbokzaitsev_ws, definitele more readable12:30
vikascthanks dulek , will go through it12:30
ltomasbos/definitele/definitely12:32
dulekvikasc: There's a lot of stuff that should be cleaned up and adapted to OSH standards, so don't be surprised. I've just wanted to put up a PoC ASAP, as I don't think I'll be able to continue that work in my current team.12:32
vikascdulek, sure, noted.12:33
*** aojea has joined #openstack-kuryr12:33
*** aojea has quit IRC12:38
*** aojea has joined #openstack-kuryr12:42
*** aojea has quit IRC12:47
janonymousdmellado: ping12:54
dmelladojanonymous: pong12:54
janonymousdmellado: when we use `systemctl status devstack@kuryr-kubernetes.service` does it use commands from : /etc/systemd/system/devstack@kuryr-kubernetes.service12:55
dmelladohmmmm I'd need to check the code but IIRC yeah12:56
dmelladoI don't have a working dsvm at the moment to check :\12:57
kzaitsev_wsjanonymous: and if you change smth in the service file you have to tell systemd to reload it12:58
janonymousi think it is bug then12:58
janonymousjust a sec, pasting it12:58
kzaitsev_wsltomasbo: usage became clearer, but the function is a bit complicated now %)12:58
openstackgerritJaivish Kothari(janonymous) proposed openstack/kuryr-kubernetes master: Systemd usage of kubernetes startup  https://review.openstack.org/48475412:59
janonymousdmellado: kzaitsev_ws : not sure what would be the right way ..but surely should be changed from previous i guess to something like ^^12:59
janonymousdmellado: kzaitsev_ws : i noted some points on : https://bugs.launchpad.net/kuryr-kubernetes/+bug/170502813:00
openstackLaunchpad bug 1705028 in kuryr-kubernetes "Systemd commands for kubernetes resources have invalid exec commands" [Undecided,New]13:00
ltomasbokzaitsev_ws, there is always a trade-off13:00
dmelladojanonymous: I'll take a look later too, when I get some time13:00
dmelladolet's see if we can sort this out ;)13:00
janonymousdmellado: sure..cool! please feel free to add/change in the patch13:00
*** vikasc has quit IRC13:05
dmelladojanonymous: will do, thanks!13:07
dimsapuimedo : ping about https://review.openstack.org/#/c/484416/ (as i don't see you on -release channel) :)13:09
*** aojea has joined #openstack-kuryr13:09
*** aojea has quit IRC13:14
*** vikasc has joined #openstack-kuryr13:18
*** aojea has joined #openstack-kuryr13:19
openstackgerritKirill Zaitsev proposed openstack/kuryr-kubernetes master: Add SR-IOV binding driver to CNI  https://review.openstack.org/46245613:20
*** aojea has quit IRC13:24
kzaitsev_wsdims: haven't seen him in a couple of days =( I'm going to ping him about my commit as soon as he's back13:25
dimskzaitsev_ws : get some +1's from other folks then?13:26
kzaitsev_wsdims: ok, I will =) I've put the thing up, since this week is deadline for non-client-libraries and next week marks start of requirements freeze. so yeah will get some +1 from folks around here13:28
kzaitsev_wsdims: btw, thanks for having my back with the patch. feels embarrassing (=13:29
dimskzaitsev_ws : no worries at all, i do the same all the time :)13:31
*** vikasc has quit IRC13:44
*** aojea has joined #openstack-kuryr13:47
*** aojea has quit IRC13:47
*** aojea has joined #openstack-kuryr13:47
*** gouthamr has joined #openstack-kuryr13:56
openstackgerritBerezovsky Irena proposed openstack/kuryr-kubernetes master: Fix k8s service subnet devstack setting  https://review.openstack.org/48480414:00
*** janki has joined #openstack-kuryr14:06
*** kzaitsev_ws has quit IRC14:25
*** kzaitsev_ws has joined #openstack-kuryr14:26
kzaitsev_wsirenab: pls ping me when you have time to discuss how you want the drivers to be refactored =)14:30
*** hongbin has joined #openstack-kuryr14:31
*** alraddarla_ has joined #openstack-kuryr15:03
*** kzaitsev_ws has quit IRC15:06
*** mattmceuen has joined #openstack-kuryr15:12
*** yamamoto has quit IRC15:17
*** yamamoto has joined #openstack-kuryr15:21
*** yamamoto has quit IRC15:29
*** aojea has quit IRC16:02
*** egonzalez has quit IRC16:07
*** vikasc has joined #openstack-kuryr16:14
*** yamamoto has joined #openstack-kuryr16:29
*** yamamoto has quit IRC16:37
*** janki has quit IRC16:55
*** yamamoto has joined #openstack-kuryr17:07
*** yamamoto has quit IRC17:12
*** vikasc has quit IRC18:23
*** vikasc has joined #openstack-kuryr18:25
*** kural has quit IRC18:36
*** deep-book-gk_ has joined #openstack-kuryr18:56
*** deep-book-gk_ has left #openstack-kuryr18:57
*** rwallner has quit IRC21:20
*** gouthamr has quit IRC21:53
*** gouthamr has joined #openstack-kuryr22:29
*** mattmceuen has quit IRC23:13
*** garyloug has quit IRC23:29
*** hongbin has quit IRC23:39

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