*** aojea has joined #openstack-kuryr | 00:50 | |
*** aojea has quit IRC | 01:23 | |
*** aojea has joined #openstack-kuryr | 01:51 | |
*** aojea has quit IRC | 02:23 | |
*** aojea has joined #openstack-kuryr | 03:14 | |
*** aojea has quit IRC | 03:40 | |
*** aojea has joined #openstack-kuryr | 03:40 | |
*** aojea has quit IRC | 03:48 | |
*** gkadam has joined #openstack-kuryr | 04:23 | |
*** gkadam has quit IRC | 04:27 | |
*** janki has joined #openstack-kuryr | 04:33 | |
*** aojea has joined #openstack-kuryr | 04:48 | |
*** aojea has quit IRC | 05:12 | |
*** yboaron has joined #openstack-kuryr | 05:27 | |
*** aojea has joined #openstack-kuryr | 06:13 | |
*** aojea has quit IRC | 06:16 | |
*** aojea_ has joined #openstack-kuryr | 06:16 | |
*** aojea_ has quit IRC | 06:20 | |
*** aojea has joined #openstack-kuryr | 06:31 | |
*** aojea has quit IRC | 06:32 | |
openstackgerrit | Merged openstack/kuryr-kubernetes master: Use show_floatingip method instead of get_floatingip https://review.openstack.org/617919 | 06:39 |
---|---|---|
*** itzikb has joined #openstack-kuryr | 06:52 | |
openstackgerrit | Yossi Boaron proposed openstack/kuryr-tempest-plugin master: Add connectivity test for service UDP https://review.openstack.org/612893 | 07:17 |
*** pcaruana has joined #openstack-kuryr | 07:19 | |
*** yboaron has quit IRC | 07:32 | |
*** maysams has joined #openstack-kuryr | 07:34 | |
*** aojea has joined #openstack-kuryr | 07:43 | |
*** ccamposr has joined #openstack-kuryr | 07:46 | |
*** irenab has joined #openstack-kuryr | 07:47 | |
*** aojea has quit IRC | 08:13 | |
*** yboaron has joined #openstack-kuryr | 08:20 | |
*** livelace has joined #openstack-kuryr | 08:34 | |
*** gkadam has joined #openstack-kuryr | 08:38 | |
*** itzikb has quit IRC | 08:50 | |
*** yboaron_ has joined #openstack-kuryr | 09:02 | |
*** yboaron has quit IRC | 09:04 | |
*** aojea has joined #openstack-kuryr | 09:06 | |
*** livelace has quit IRC | 09:13 | |
*** celebdor has joined #openstack-kuryr | 09:14 | |
*** celebdor has quit IRC | 09:14 | |
*** celebdor has joined #openstack-kuryr | 09:14 | |
*** yboaron_ has quit IRC | 09:18 | |
*** yboaron_ has joined #openstack-kuryr | 09:18 | |
*** garyloug has joined #openstack-kuryr | 09:27 | |
openstackgerrit | Ashish Billore proposed openstack/kuryr-kubernetes master: Add missing key check while lbaas member add https://review.openstack.org/602017 | 09:31 |
openstackgerrit | Merged openstack/kuryr-kubernetes master: Link to public document instead of private https://review.openstack.org/618023 | 09:34 |
*** celebdor has quit IRC | 09:39 | |
*** aojea has quit IRC | 09:40 | |
*** celebdor has joined #openstack-kuryr | 09:41 | |
*** AndroUser2 has joined #openstack-kuryr | 09:44 | |
*** AndroUser2 has quit IRC | 09:55 | |
*** gkadam has quit IRC | 10:13 | |
*** gkadam has joined #openstack-kuryr | 10:14 | |
openstackgerrit | Luis Tomas Bolivar proposed openstack/kuryr-kubernetes master: Support network policy update https://review.openstack.org/605150 | 10:23 |
openstackgerrit | Luis Tomas Bolivar proposed openstack/kuryr-kubernetes master: Add security groups driver for NP https://review.openstack.org/608616 | 10:23 |
openstackgerrit | Luis Tomas Bolivar proposed openstack/kuryr-kubernetes master: Ensure existing pods use the right network policy https://review.openstack.org/616261 | 10:23 |
ltomasbo | celebdor, 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 |
openstackgerrit | Yossi Boaron proposed openstack/kuryr-tempest-plugin master: Add connectivity test for service UDP https://review.openstack.org/612893 | 10:32 |
*** aojea has joined #openstack-kuryr | 10:36 | |
openstackgerrit | Luis Tomas Bolivar proposed openstack/kuryr-kubernetes master: Ensure floatingip information is retrieved https://review.openstack.org/618123 | 10:38 |
openstackgerrit | Genadi Chereshnya proposed openstack/kuryr-tempest-plugin master: Update port pool with non-default values https://review.openstack.org/603121 | 10:42 |
dmellado | ltomasbo: celebdor what's the deal on {} rules | 10:46 |
dmellado | I didn't get to see almost any patch this week | 10:47 |
celebdor | ltomasbo: well, my point is, is it possible to get None or not? | 10:47 |
dmellado | what do you mean by that | 10:47 |
ltomasbo | dmellado, look at https://review.openstack.org/#/c/605150, celebdor's comment on line 173 | 10:47 |
dmellado | ltomasbo: which file? | 10:48 |
ltomasbo | dmellado, there is only one comment on L173 in all the files... xD but drivers/network_policy.py | 10:49 |
dmellado | https://review.openstack.org/#/c/605150/17/kuryr_kubernetes/controller/drivers/network_policy.py maybe on another patchset? | 10:50 |
ltomasbo | look at the comments it was in a previous patch set | 10:55 |
ltomasbo | https://review.openstack.org/#/c/605150/15/kuryr_kubernetes/controller/drivers/network_policy.py | 10:55 |
*** aojea has quit IRC | 10:57 | |
celebdor | ltomasbo: 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 block | 11:02 |
ltomasbo | celebdor, checking... | 11:11 |
ltomasbo | damn... I'll respin.. | 11:11 |
dmellado | so, read it | 11:12 |
dmellado | don't think about dictionary | 11:12 |
dmellado | we can also add an assert to make sure it's empty but that's a harcoded specific usecase | 11:12 |
dmellado | so I'd leave it as it is | 11:12 |
dmellado | in any case, I'll get to go thru the reviews after I'm back from the summit | 11:12 |
dmellado | will comment it there | 11:12 |
openstackgerrit | Antoni Segura Puimedon proposed openstack/kuryr-kubernetes master: Delete duplicate words in network_policy.rst https://review.openstack.org/617614 | 11:14 |
openstackgerrit | Luis Tomas Bolivar proposed openstack/kuryr-kubernetes master: Ensure floatingip information is retrieved https://review.openstack.org/618123 | 11:17 |
openstackgerrit | Merged openstack/kuryr-kubernetes master: Omit the twice occured words in npwg_spec_support.rst https://review.openstack.org/617612 | 11:21 |
openstackgerrit | Merged openstack/kuryr-kubernetes master: Update the HTTP links to HTTPS in manual.rstl. https://review.openstack.org/617823 | 11:31 |
*** aojea has joined #openstack-kuryr | 11:49 | |
openstackgerrit | Luis Tomas Bolivar proposed openstack/kuryr-kubernetes master: Support network policy update https://review.openstack.org/605150 | 12:02 |
*** aojea has quit IRC | 12:20 | |
*** yboaron_ has quit IRC | 12:45 | |
*** yboaron_ has joined #openstack-kuryr | 12:45 | |
*** rh-jelabarre has joined #openstack-kuryr | 12:56 | |
*** rh-jelabarre has quit IRC | 13:03 | |
*** aojea has joined #openstack-kuryr | 13:13 | |
openstackgerrit | Luis Tomas Bolivar proposed openstack/kuryr-kubernetes master: Ensure floatingip information is retrieved https://review.openstack.org/618123 | 13:14 |
ltomasbo | celebdor, soory, I forgot to update unittest... | 13:15 |
openstackgerrit | Luis Tomas Bolivar proposed openstack/kuryr-kubernetes master: Ensure floatingip information is retrieved https://review.openstack.org/618123 | 13:17 |
*** yboaron_ has quit IRC | 13:21 | |
*** garyloug_ has joined #openstack-kuryr | 13:30 | |
*** yboaron_ has joined #openstack-kuryr | 13:32 | |
*** garyloug has quit IRC | 13:34 | |
*** celebdor has quit IRC | 13:36 | |
*** celebdor has joined #openstack-kuryr | 13:38 | |
openstackgerrit | Merged openstack/kuryr-kubernetes master: Delete duplicate words in network_policy.rst https://review.openstack.org/617614 | 13:40 |
*** aojea has quit IRC | 13:47 | |
*** garyloug__ has joined #openstack-kuryr | 14:32 | |
*** garyloug_ has quit IRC | 14:35 | |
*** aojea has joined #openstack-kuryr | 14:36 | |
*** yboaron_ has quit IRC | 14:42 | |
*** yboaron_ has joined #openstack-kuryr | 14:42 | |
*** janki has quit IRC | 15:28 | |
*** gkadam has quit IRC | 16:18 | |
*** jistr is now known as jistr|off | 16:25 | |
*** aojea has quit IRC | 16:33 | |
*** rh-jelabarre has joined #openstack-kuryr | 16:33 | |
*** aojea has joined #openstack-kuryr | 16:37 | |
*** celebdor has quit IRC | 16:39 | |
*** maysams has quit IRC | 16:44 | |
*** aojea has quit IRC | 17:27 | |
*** aojea has joined #openstack-kuryr | 17:27 | |
*** aojea has quit IRC | 17:32 | |
*** livelace has joined #openstack-kuryr | 17:38 | |
*** yboaron_ has quit IRC | 17:45 | |
*** aojea has joined #openstack-kuryr | 18:05 | |
*** garyloug__ has quit IRC | 18:10 | |
*** aojea has quit IRC | 18:19 | |
*** ccamposr has quit IRC | 18:51 | |
*** aojea has joined #openstack-kuryr | 19:28 | |
*** jistr|off is now known as jistr | 19:32 | |
*** aojea has quit IRC | 19:59 | |
*** livelace has quit IRC | 20:08 | |
*** pcaruana has quit IRC | 20:14 | |
*** pcaruana has joined #openstack-kuryr | 20:27 | |
*** aojea has joined #openstack-kuryr | 20:33 | |
*** aojea has quit IRC | 21:04 | |
*** yboaron_ has joined #openstack-kuryr | 21:46 | |
*** yboaron_ has quit IRC | 21:49 | |
*** pcaruana has quit IRC | 21:55 | |
*** aojea has joined #openstack-kuryr | 21:58 | |
*** aojea has quit IRC | 22:31 | |
*** aojea has joined #openstack-kuryr | 23:23 | |
*** aojea has quit IRC | 23:45 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!