| *** pmannidi is now known as pmannidi|AFK | 01:24 | |
| *** pmannidi|AFK is now known as pmannidi | 02:41 | |
| *** pmannidi is now known as pmannidi|Lunch | 03:25 | |
| *** pmannidi|Lunch is now known as pmannidi|AFK | 04:17 | |
| 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:50 |
|---|---|---|
| *** pmannidi|AFK is now known as pmannidi | 06:20 | |
| opendevreview | Robin Cernin proposed openstack/kuryr-kubernetes master: Remove ep_slices from klb on endpoint delete event https://review.opendev.org/c/openstack/kuryr-kubernetes/+/804205 | 06:42 |
| *** pmannidi is now known as pmannidi|brb | 06:56 | |
| opendevreview | Robin Cernin proposed openstack/kuryr-kubernetes master: Remove ep_slices from klb on endpoint delete event https://review.opendev.org/c/openstack/kuryr-kubernetes/+/804205 | 07:10 |
| opendevreview | Robin Cernin proposed openstack/kuryr-kubernetes master: Remove ep_slices from klb on endpoint delete event https://review.opendev.org/c/openstack/kuryr-kubernetes/+/804205 | 07:16 |
| opendevreview | Robin Cernin proposed openstack/kuryr-kubernetes master: Remove ep_slices from klb on endpoint delete event https://review.opendev.org/c/openstack/kuryr-kubernetes/+/804205 | 07:19 |
| *** pmannidi|brb is now known as pmannidi | 07:47 | |
| simboja | Hello! Good morning | 08:01 |
| simboja | I don't know where I am getting the query_params wrong | 08:01 |
| simboja | https://paste.opendev.org/show/808002/ | 08:01 |
| simboja | It returns this below: | 08:01 |
| simboja | https://paste.opendev.org/show/808003/ | 08:02 |
| simboja | gryf, ltomasbo, maysams: Please take a look | 08:02 |
| simboja | If I list all loadbalancers here | 08:03 |
| simboja | # lb = self.lbaas.list_loadbalancers()\ | 08:03 |
| simboja | I get the loadbalancers returned | 08:03 |
| simboja | Here is how the list is defined: | 08:05 |
| simboja | https://paste.opendev.org/show/808004/ | 08:05 |
| maysams | simboja: it looks ok at a first glance, however I suggest to format the string separetly and then use it in the lbaas call, so you can confirm the query_params contains what you expect | 08:15 |
| maysams | simboja: might be good to go over how that is handled on the Octavia side https://github.com/openstack/octavia-tempest-plugin/blob/058ceaf0e7df0a911a6b07eff74f068c2e54502f/octavia_tempest_plugin/services/load_balancer/v2/base_client.py#L162-L167 | 08:16 |
| simboja | taking a look... | 08:27 |
| opendevreview | Roman Dobosz proposed openstack/kuryr-kubernetes master: [WIP] Switch gates to OVN by default. https://review.opendev.org/c/openstack/kuryr-kubernetes/+/802999 | 08:39 |
| opendevreview | Robin Cernin proposed openstack/kuryr-kubernetes master: Remove ep_slices from klb on endpoint delete event https://review.opendev.org/c/openstack/kuryr-kubernetes/+/804205 | 08:55 |
| opendevreview | Robin Cernin proposed openstack/kuryr-kubernetes master: Remove ep_slices from klb on endpoint delete event https://review.opendev.org/c/openstack/kuryr-kubernetes/+/804205 | 09:25 |
| opendevreview | Robin Cernin proposed openstack/kuryr-kubernetes master: Remove ep_slices from klb on endpoint delete event https://review.opendev.org/c/openstack/kuryr-kubernetes/+/804205 | 09:34 |
| opendevreview | Robin Cernin proposed openstack/kuryr-kubernetes master: Remove ep_slices from klb on endpoint delete event https://review.opendev.org/c/openstack/kuryr-kubernetes/+/804205 | 09:36 |
| simboja | Hello maysams: I was looking at the options of specifying the query_params to the list_loadbalancers: | 09:44 |
| simboja | I realised that I can query with the fields=id&fields=name | 09:44 |
| simboja | which would return a dict of the id and name of all the loadbalancers on OpenStack | 09:45 |
| simboja | Then I can get the id of the loadbalancer I want using the name and delete it | 09:46 |
| simboja | or Option 2: | 09:46 |
| simboja | Get the klb CRD status using the name and get the loadbalancer ID | 09:47 |
| simboja | then I can delete it on OpenStack | 09:47 |
| simboja | Which option is preferred in this case, I guess getting the klb CRD status and the associated loadbalancer Id would be cheaper? | 09:48 |
| simboja | ltomasbo, gryf, maysams ^ | 09:48 |
| ltomasbo | I'd go for second option, getting the ID from the KLB and delete it. Then you can use the id to ensure the loadbalancer is gone from the openstack side (with show_loadbalancer instead of list_loadbalancers) | 09:53 |
| simboja | perfect! | 09:59 |
| *** pmannidi is now known as pmannidi|brb | 10:47 | |
| opendevreview | Robin Cernin proposed openstack/kuryr-kubernetes master: Remove ep_slices from klb on endpoint delete event https://review.opendev.org/c/openstack/kuryr-kubernetes/+/804205 | 11:05 |
| opendevreview | Roman Dobosz proposed openstack/kuryr-kubernetes master: [WIP] Switch gates to OVN by default. https://review.opendev.org/c/openstack/kuryr-kubernetes/+/802999 | 11:18 |
| *** pmannidi|brb is now known as pmannidi | 11:52 | |
| *** pmannidi is now known as pmannidi|AFK | 13:50 | |
| 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 | 18:34 |
| *** pmannidi|AFK is now known as pmannidi | 23:33 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!