| *** yoctozepto has quit IRC | 00:53 | |
| *** yoctozepto has joined #openstack-kuryr | 00:54 | |
| *** yoctozepto has quit IRC | 01:05 | |
| *** yoctozepto has joined #openstack-kuryr | 01:05 | |
| *** rh-jelabarre has quit IRC | 01:27 | |
| *** gdwornicki has joined #openstack-kuryr | 01:34 | |
| *** gdwornicki has quit IRC | 01:35 | |
| *** brault has quit IRC | 03:13 | |
| *** brault has joined #openstack-kuryr | 03:20 | |
| *** gcheresh_ has joined #openstack-kuryr | 06:37 | |
| openstackgerrit | Roman Dobosz proposed openstack/kuryr-kubernetes master: Update drivers to use OpenStackSDK 1/2. https://review.opendev.org/690562 | 06:38 |
|---|---|---|
| openstackgerrit | Roman Dobosz proposed openstack/kuryr-kubernetes master: Update drivers to use OpenStackSDK 2/2. https://review.opendev.org/690981 | 06:38 |
| dulek | papasuder: I bet it's the other way around - broken pipe errors cause the daemon to go unhealthy. | 06:52 |
| dulek | gwdwornicki: Uhm, no idea really, but we might want to have some semaphore there to prevent saturating that. Good thing is that this should be retried by kubelet. | 06:53 |
| * dulek knows guys are offline, but maybe they're checking out channel logs on eavesdrop.openstack.org. | 06:54 | |
| openstackgerrit | Michał Dulko proposed openstack/kuryr-kubernetes master: Remove Python 2 support https://review.opendev.org/693485 | 07:29 |
| *** aperevalov has joined #openstack-kuryr | 07:42 | |
| *** gcheresh_ has quit IRC | 07:45 | |
| *** gcheresh_ has joined #openstack-kuryr | 07:48 | |
| *** irclogbot_1 has quit IRC | 09:39 | |
| *** irclogbot_2 has joined #openstack-kuryr | 09:40 | |
| *** maysams has joined #openstack-kuryr | 10:00 | |
| *** aperevalov has quit IRC | 10:34 | |
| *** ccamposr__ has quit IRC | 10:57 | |
| *** ccamposr__ has joined #openstack-kuryr | 10:59 | |
| *** openstackgerrit has quit IRC | 12:41 | |
| *** rh-jelabarre has joined #openstack-kuryr | 13:07 | |
| *** pc_m has joined #openstack-kuryr | 13:17 | |
| *** aperevalov has joined #openstack-kuryr | 13:41 | |
| aperevalov | dulek, hi. I'v seen the code in neutrons/notifiers. It uses itself the same code (from neutron_lib.callbacks import registry) as in neutron rpc, but they also sends notification to nova's endpoint. | 13:44 |
| aperevalov | and to ironic endpoint. | 13:45 |
| aperevalov | If we'll implement in the same way as existing notifiers do, is it possible to send message from neutron host to kuryr-kubernetes host in you environment? | 13:46 |
| *** gcheresh_ has quit IRC | 14:27 | |
| *** gcheresh_ has joined #openstack-kuryr | 14:33 | |
| dulek | aperevalov: Hi! Sorry for late reply, was lunching. | 14:42 |
| dulek | aperevalov: So this seems really free-form, the notifiers interface. | 14:43 |
| dulek | So we could add a kuryr.py there with whatever we want. | 14:43 |
| dulek | I was thinking about doing it through K8s API - ie. making sure Neutron node can access K8s API endpoint and then doing whatever PUT request we want. | 14:44 |
| dulek | But that would probably depend on KuryrPort work - ie. extracting pod annotations into a CRD. | 14:44 |
| dulek | As looking up all the pods to find which one has a particular port attached seems suboptimal. | 14:45 |
| *** openstackgerrit has joined #openstack-kuryr | 15:19 | |
| openstackgerrit | Itzik Brown proposed openstack/kuryr-tempest-plugin master: Add a check for ClusterIP IP of LB service https://review.opendev.org/693632 | 15:19 |
| *** pcaruana has joined #openstack-kuryr | 15:28 | |
| openstackgerrit | Michał Dulko proposed openstack/kuryr-kubernetes master: Remove Python 2 support https://review.opendev.org/693485 | 15:42 |
| *** gcheresh_ has quit IRC | 15:51 | |
| *** aperevalov has quit IRC | 16:08 | |
| *** aperevalov has joined #openstack-kuryr | 16:12 | |
| aperevalov | dulek, I was going to implement it as a http endpoint as well as cni daemon. | 16:14 |
| dulek | That would mean it needs to be exposed as a Service. I'm not a fan of that really… | 16:15 |
| aperevalov | Service in term of K8S? | 16:15 |
| dulek | Yeah, I think so. It's easier to expect Neutron will have access to the Octavia LB, than to internal pod IP on one of the networks. | 16:16 |
| dulek | Eh, actually accessibility of both is not granted - the LB can be on any network really. | 16:17 |
| dulek | aperevalov: Had you already dismissed that idea of using notifications, ie. the ones that aren't internal Neutron API? | 16:17 |
| aperevalov | so in case of working through kube-apiserver, ClusterIP (default k8s endpoint) should be available, and you rely on that, right? | 16:17 |
| dulek | aperevalov: True. | 16:18 |
| aperevalov | did you ask about this one notifications "from neutron_lib.callbacks import registry"? | 16:20 |
| dulek | aperevalov: More like those: https://docs.openstack.org/neutron/pike/admin/archives/adv-operational-features.html#notifications | 16:21 |
| dulek | I'm trying to find the code doing them (saw your comment). | 16:21 |
| aperevalov | oh, I still didn't find the code. | 16:22 |
| dulek | aperevalov: The options are taken from oslo_messaging. | 16:23 |
| openstackgerrit | Merged openstack/kuryr-kubernetes master: Ensure LB SG is not updated for egress only policy https://review.opendev.org/689831 | 16:27 |
| dulek | aperevalov: https://github.com/openstack/neutron/blob/801fde7988d431579c821618e97bcd9e3a60ff10/neutron/api/v2/base.py#L434-L436 | 16:30 |
| dulek | It's this stuff IMO. | 16:30 |
| dulek | aperevalov: So as you already have the RPC code in place, this might be better to get started instead of going an extra mile and creating our own Neutron notifier. | 16:31 |
| dulek | https://github.com/openstack/neutron-lib/blob/master/neutron_lib/rpc.py#L63-L64 - yeah, this is the thing. | 16:32 |
| aperevalov | thanks, looks like it better, need to try it to figure out how does it work ) | 16:37 |
| openstackgerrit | Michał Dulko proposed openstack/kuryr-kubernetes master: Remove Python 2 support https://review.opendev.org/693485 | 16:37 |
| *** aperevalov has left #openstack-kuryr | 16:42 | |
| openstackgerrit | Sarka Scavnicka proposed openstack/kuryr-kubernetes master: WiP: Update loadbalancer CRD with service spec and endpoints status https://review.opendev.org/693884 | 17:00 |
| *** pcaruana has quit IRC | 17:03 | |
| *** maysams has quit IRC | 17:06 | |
| openstackgerrit | Sarka Scavnicka proposed openstack/kuryr-kubernetes master: WiP: Update loadbalancer CRD with service spec and endpoints status https://review.opendev.org/693884 | 17:09 |
| *** ccamposr__ has quit IRC | 17:46 | |
| *** ccamposr__ has joined #openstack-kuryr | 17:47 | |
| *** ccamposr has joined #openstack-kuryr | 18:48 | |
| *** ccamposr__ has quit IRC | 18:50 | |
| *** gcheresh_ has joined #openstack-kuryr | 19:47 | |
| openstackgerrit | Roman Dobosz proposed openstack/kuryr-kubernetes master: Fix inconsistency in headlines format. https://review.opendev.org/693927 | 20:45 |
| openstackgerrit | Roman Dobosz proposed openstack/kuryr-kubernetes master: Fix text blocks formatting https://review.opendev.org/693928 | 20:45 |
| openstackgerrit | Roman Dobosz proposed openstack/kuryr-kubernetes master: Fix directives formatting. https://review.opendev.org/693929 | 20:45 |
| openstackgerrit | Roman Dobosz proposed openstack/kuryr-kubernetes master: Explicitly use code-block. https://review.opendev.org/693930 | 20:45 |
| openstackgerrit | Itzik Brown proposed openstack/kuryr-tempest-plugin master: Add a check for ClusterIP IP of LB service https://review.opendev.org/693632 | 21:09 |
| openstackgerrit | Itzik Brown proposed openstack/kuryr-tempest-plugin master: Add a check for ClusterIP IP of LB service https://review.opendev.org/693632 | 21:22 |
| *** pc_m has quit IRC | 21:55 | |
| *** gcheresh_ has quit IRC | 21:58 | |
| openstackgerrit | Merged openstack/kuryr-kubernetes master: Ensure Network Policy handles egress traffic to a SVC https://review.opendev.org/689102 | 22:11 |
| openstackgerrit | Itzik Brown proposed openstack/kuryr-tempest-plugin master: Add a check for ClusterIP IP of LB service https://review.opendev.org/693632 | 22:21 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!