| opendevreview | Sunday Mgbogu proposed openstack/kuryr-tempest-plugin master: Add Kuryr-tempest-plugin test for LoadBalancer Reconciliation https://review.opendev.org/c/openstack/kuryr-tempest-plugin/+/803244 | 03:46 |
|---|---|---|
| *** pmannidi is now known as pmannidi|brb | 03:54 | |
| *** pmannidi|brb is now known as pmannidi | 04:58 | |
| opendevreview | Roman Dobosz proposed openstack/kuryr-kubernetes master: [WIP] Switch gates to OVN by default. https://review.opendev.org/c/openstack/kuryr-kubernetes/+/802999 | 05:21 |
| opendevreview | Roman Dobosz proposed openstack/kuryr-kubernetes master: [WIP] Switch gates to OVN by default. https://review.opendev.org/c/openstack/kuryr-kubernetes/+/802999 | 05:43 |
| opendevreview | Roman Dobosz proposed openstack/kuryr-kubernetes master: [WIP] Switch gates to OVN by default. https://review.opendev.org/c/openstack/kuryr-kubernetes/+/802999 | 07:16 |
| digitalsimboja | Hello! Good morning all | 07:45 |
| digitalsimboja | ltomasbo: I would appreciate pointers to how to check connectivity for quicker access | 07:46 |
| ltomasbo | good morning | 07:47 |
| ltomasbo | you can check also the zuul results for your current patch, which fail | 07:47 |
| ltomasbo | the other tests for loadbalancers are also checking connectivity | 07:47 |
| ltomasbo | you should do the same-ish | 07:47 |
| ltomasbo | there is a function called check_service_internal_connectivity | 07:49 |
| digitalsimboja | Great! | 07:56 |
| opendevreview | Sunday Mgbogu proposed openstack/kuryr-kubernetes master: Kuryr Kubernetes Loadbalancers Reconciliation Design https://review.opendev.org/c/openstack/kuryr-kubernetes/+/796234 | 10:18 |
| opendevreview | Roman Dobosz proposed openstack/kuryr-kubernetes master: [WIP] Switch gates to OVN by default. https://review.opendev.org/c/openstack/kuryr-kubernetes/+/802999 | 10:32 |
| *** pmannidi is now known as pmannidi|brb | 10:57 | |
| digitalsimboja | Hello ltomasbo | 11:44 |
| ltomasbo | o/ | 11:44 |
| digitalsimboja | I want to know: Do I need to call the reconcile_loadbalancers? | 11:45 |
| digitalsimboja | or it would run automatically after the lb['id'] is deleted? | 11:45 |
| gryf | digitalsimboja, do you have reconcile_loadbalancers exposed by kuryr-kubernetes API? | 11:48 |
| digitalsimboja | No! not yet exposed | 11:49 |
| gryf | so there are plans to expose it? | 11:49 |
| digitalsimboja | yeah sure | 11:49 |
| gryf | o really. | 11:50 |
| gryf | I had an impression that it would be an automatic task within kuryr internals. | 11:50 |
| digitalsimboja | Okay | 11:53 |
| digitalsimboja | So I need to wait it to run | 11:54 |
| gryf | that's what I thought. | 11:55 |
| *** pmannidi|brb is now known as pmannidi | 11:55 | |
| ltomasbo | yeah, I thought there was not going to be an API for that | 12:16 |
| ltomasbo | so, I thought you would simply wait for the reconciliation look time | 12:16 |
| ltomasbo | meaning if the loop runs every 10 mins, you will wait 10 mins to check the connectivity again | 12:17 |
| ltomasbo | perhaps the reconciliation loop timer can be exposed as a config option, and run it more frequently on the gates to limit the wasted time | 12:17 |
| gryf | ltomasbo, I'm wondering what impact on gate perfromance would it have to make it more frequent, like every 30 seconds. | 12:45 |
| gryf | and btw, centos registry has expired SSL cert couple hours ago - so expect false negatives on gates. | 12:52 |
| gryf | I gave them a heads up, so let's see. | 12:53 |
| opendevreview | Sunday Mgbogu proposed openstack/kuryr-tempest-plugin master: Add Kuryr-tempest-plugin test for LoadBalancer Reconciliation https://review.opendev.org/c/openstack/kuryr-tempest-plugin/+/803244 | 13:10 |
| opendevreview | Sunday Mgbogu proposed openstack/kuryr-tempest-plugin master: Add Kuryr-tempest-plugin test for LoadBalancer Reconciliation https://review.opendev.org/c/openstack/kuryr-tempest-plugin/+/803244 | 13:12 |
| ltomasbo | gryf, yeah, balance between waiting more or doing more actions... though if there is nothing to reconcile... it should not be a lot of impact, just one extra call to OpenStack API every X mins | 13:17 |
| gryf | ltomasbo, +1 | 13:25 |
| ltomasbo | digitalsimboja, please, review the comments dulek left on your (merged) patch for the loadbalancer reconciliation (in case you have not seen it): https://review.opendev.org/c/openstack/kuryr-kubernetes/+/798904 | 13:40 |
| dulek | digitalsimboja: Yep, sorry for not raising them earlier but I was out of office last week and only took a look after the code merged. | 13:50 |
| *** pmannidi is now known as pmannidi|AFK | 13:54 | |
| digitalsimboja | I am taking a look... | 14:01 |
| *** pmannidi|AFK is now known as pmannidi | 23:47 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!