Thursday, 2018-11-15

*** aojea has joined #openstack-kuryr00:50
*** aojea has quit IRC01:23
*** aojea has joined #openstack-kuryr01:51
*** aojea has quit IRC02:23
*** aojea has joined #openstack-kuryr03:14
*** aojea has quit IRC03:40
*** aojea has joined #openstack-kuryr03:40
*** aojea has quit IRC03:48
*** gkadam has joined #openstack-kuryr04:23
*** gkadam has quit IRC04:27
*** janki has joined #openstack-kuryr04:33
*** aojea has joined #openstack-kuryr04:48
*** aojea has quit IRC05:12
*** yboaron has joined #openstack-kuryr05:27
*** aojea has joined #openstack-kuryr06:13
*** aojea has quit IRC06:16
*** aojea_ has joined #openstack-kuryr06:16
*** aojea_ has quit IRC06:20
*** aojea has joined #openstack-kuryr06:31
*** aojea has quit IRC06:32
openstackgerritMerged openstack/kuryr-kubernetes master: Use show_floatingip method instead of get_floatingip  https://review.openstack.org/61791906:39
*** itzikb has joined #openstack-kuryr06:52
openstackgerritYossi Boaron proposed openstack/kuryr-tempest-plugin master: Add connectivity test for service UDP  https://review.openstack.org/61289307:17
*** pcaruana has joined #openstack-kuryr07:19
*** yboaron has quit IRC07:32
*** maysams has joined #openstack-kuryr07:34
*** aojea has joined #openstack-kuryr07:43
*** ccamposr has joined #openstack-kuryr07:46
*** irenab has joined #openstack-kuryr07:47
*** aojea has quit IRC08:13
*** yboaron has joined #openstack-kuryr08:20
*** livelace has joined #openstack-kuryr08:34
*** gkadam has joined #openstack-kuryr08:38
*** itzikb has quit IRC08:50
*** yboaron_ has joined #openstack-kuryr09:02
*** yboaron has quit IRC09:04
*** aojea has joined #openstack-kuryr09:06
*** livelace has quit IRC09:13
*** celebdor has joined #openstack-kuryr09:14
*** celebdor has quit IRC09:14
*** celebdor has joined #openstack-kuryr09:14
*** yboaron_ has quit IRC09:18
*** yboaron_ has joined #openstack-kuryr09:18
*** garyloug has joined #openstack-kuryr09:27
openstackgerritAshish Billore proposed openstack/kuryr-kubernetes master: Add missing key check while lbaas member add  https://review.openstack.org/60201709:31
openstackgerritMerged openstack/kuryr-kubernetes master: Link to public document instead of private  https://review.openstack.org/61802309:34
*** celebdor has quit IRC09:39
*** aojea has quit IRC09:40
*** celebdor has joined #openstack-kuryr09:41
*** AndroUser2 has joined #openstack-kuryr09:44
*** AndroUser2 has quit IRC09:55
*** gkadam has quit IRC10:13
*** gkadam has joined #openstack-kuryr10:14
openstackgerritLuis Tomas Bolivar proposed openstack/kuryr-kubernetes master: Support network policy update  https://review.openstack.org/60515010:23
openstackgerritLuis Tomas Bolivar proposed openstack/kuryr-kubernetes master: Add security groups driver for NP  https://review.openstack.org/60861610:23
openstackgerritLuis Tomas Bolivar proposed openstack/kuryr-kubernetes master: Ensure existing pods use the right network policy  https://review.openstack.org/61626110:23
ltomasbocelebdor, I addressed your comments on dmellado's patch (all but the one about {} on the ingress rules that I'm not completely sure about)10:25
openstackgerritYossi Boaron proposed openstack/kuryr-tempest-plugin master: Add connectivity test for service UDP  https://review.openstack.org/61289310:32
*** aojea has joined #openstack-kuryr10:36
openstackgerritLuis Tomas Bolivar proposed openstack/kuryr-kubernetes master: Ensure floatingip information is retrieved  https://review.openstack.org/61812310:38
openstackgerritGenadi Chereshnya proposed openstack/kuryr-tempest-plugin master: Update port pool with non-default values  https://review.openstack.org/60312110:42
dmelladoltomasbo: celebdor what's the deal on {} rules10:46
dmelladoI didn't get to see almost any patch this week10:47
celebdorltomasbo: well, my point is, is it possible to get None or not?10:47
dmelladowhat do you mean by that10:47
ltomasbodmellado, look at https://review.openstack.org/#/c/605150, celebdor's comment on line 17310:47
dmelladoltomasbo: which file?10:48
ltomasbodmellado, there is only one comment on L173 in all the files... xD but drivers/network_policy.py10:49
dmelladohttps://review.openstack.org/#/c/605150/17/kuryr_kubernetes/controller/drivers/network_policy.py maybe on another patchset?10:50
ltomasbolook at the comments it was in a previous patch set10:55
ltomasbohttps://review.openstack.org/#/c/605150/15/kuryr_kubernetes/controller/drivers/network_policy.py10:55
*** aojea has quit IRC10:57
celebdorltomasbo: note that in https://review.openstack.org/#/c/618123/1 you make reference to a variable you deleted in your patch. I proposed a different block11:02
ltomasbocelebdor, checking...11:11
ltomasbodamn... I'll respin..11:11
dmelladoso, read it11:12
dmelladodon't think about dictionary11:12
dmelladowe can also add an assert to make sure it's empty but that's a harcoded specific usecase11:12
dmelladoso I'd leave it as it is11:12
dmelladoin any case, I'll get to go thru the reviews after I'm back from the summit11:12
dmelladowill comment it there11:12
openstackgerritAntoni Segura Puimedon proposed openstack/kuryr-kubernetes master: Delete duplicate words in network_policy.rst  https://review.openstack.org/61761411:14
openstackgerritLuis Tomas Bolivar proposed openstack/kuryr-kubernetes master: Ensure floatingip information is retrieved  https://review.openstack.org/61812311:17
openstackgerritMerged openstack/kuryr-kubernetes master: Omit the twice occured words in  npwg_spec_support.rst  https://review.openstack.org/61761211:21
openstackgerritMerged openstack/kuryr-kubernetes master: Update the HTTP links to HTTPS in manual.rstl.  https://review.openstack.org/61782311:31
*** aojea has joined #openstack-kuryr11:49
openstackgerritLuis Tomas Bolivar proposed openstack/kuryr-kubernetes master: Support network policy update  https://review.openstack.org/60515012:02
*** aojea has quit IRC12:20
*** yboaron_ has quit IRC12:45
*** yboaron_ has joined #openstack-kuryr12:45
*** rh-jelabarre has joined #openstack-kuryr12:56
*** rh-jelabarre has quit IRC13:03
*** aojea has joined #openstack-kuryr13:13
openstackgerritLuis Tomas Bolivar proposed openstack/kuryr-kubernetes master: Ensure floatingip information is retrieved  https://review.openstack.org/61812313:14
ltomasbocelebdor, soory, I forgot to update unittest...13:15
openstackgerritLuis Tomas Bolivar proposed openstack/kuryr-kubernetes master: Ensure floatingip information is retrieved  https://review.openstack.org/61812313:17
*** yboaron_ has quit IRC13:21
*** garyloug_ has joined #openstack-kuryr13:30
*** yboaron_ has joined #openstack-kuryr13:32
*** garyloug has quit IRC13:34
*** celebdor has quit IRC13:36
*** celebdor has joined #openstack-kuryr13:38
openstackgerritMerged openstack/kuryr-kubernetes master: Delete duplicate words in network_policy.rst  https://review.openstack.org/61761413:40
*** aojea has quit IRC13:47
*** garyloug__ has joined #openstack-kuryr14:32
*** garyloug_ has quit IRC14:35
*** aojea has joined #openstack-kuryr14:36
*** yboaron_ has quit IRC14:42
*** yboaron_ has joined #openstack-kuryr14:42
*** janki has quit IRC15:28
*** gkadam has quit IRC16:18
*** jistr is now known as jistr|off16:25
*** aojea has quit IRC16:33
*** rh-jelabarre has joined #openstack-kuryr16:33
*** aojea has joined #openstack-kuryr16:37
*** celebdor has quit IRC16:39
*** maysams has quit IRC16:44
*** aojea has quit IRC17:27
*** aojea has joined #openstack-kuryr17:27
*** aojea has quit IRC17:32
*** livelace has joined #openstack-kuryr17:38
*** yboaron_ has quit IRC17:45
*** aojea has joined #openstack-kuryr18:05
*** garyloug__ has quit IRC18:10
*** aojea has quit IRC18:19
*** ccamposr has quit IRC18:51
*** aojea has joined #openstack-kuryr19:28
*** jistr|off is now known as jistr19:32
*** aojea has quit IRC19:59
*** livelace has quit IRC20:08
*** pcaruana has quit IRC20:14
*** pcaruana has joined #openstack-kuryr20:27
*** aojea has joined #openstack-kuryr20:33
*** aojea has quit IRC21:04
*** yboaron_ has joined #openstack-kuryr21:46
*** yboaron_ has quit IRC21:49
*** pcaruana has quit IRC21:55
*** aojea has joined #openstack-kuryr21:58
*** aojea has quit IRC22:31
*** aojea has joined #openstack-kuryr23:23
*** aojea has quit IRC23:45

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