| *** pmannidi is now known as pmannidi|AFK | 08:13 | |
| simboja | Hello dulek, good morning | 08:23 |
|---|---|---|
| dulek | simboja: Hi! | 08:24 |
| simboja | please assist to review: https://review.opendev.org/c/openstack/kuryr-kubernetes/+/803870 | 08:24 |
| simboja | Hi | 08:24 |
| dulek | Sure, I'll take a look. | 08:25 |
| simboja | A quick question I have, the periodic task is defined in the k8sService, is there a way I can trigger it to run? | 08:30 |
| dulek | simboja: Like manually? I think you should be able to just call the method. | 08:31 |
| simboja | yeah manually | 08:32 |
| simboja | What if I restart the kuryr-k8s-controller? | 08:34 |
| simboja | service | 08:35 |
| simboja | gryf: I am now getting a setup clients failure https://zuul.opendev.org/t/openstack/build/47d5f944a4d34a4ab0e2bc1a10537a12 | 08:36 |
| simboja | Captured traceback: | 08:36 |
| simboja | ~~~~~~~~~~~~~~~~~~~ | 08:36 |
| simboja | Traceback (most recent call last): | 08:36 |
| simboja | File "/opt/stack/tempest/tempest/test.py", line 181, in setUpClass | 08:36 |
| simboja | raise value.with_traceback(trace) | 08:36 |
| simboja | File "/opt/stack/tempest/tempest/test.py", line 171, in setUpClass | 08:36 |
| simboja | cls.setup_clients() | 08:36 |
| simboja | File "/opt/stack/tempest/.tox/tempest/lib/python3.8/site-packages/kuryr_tempest_plugin/tests/scenario/test_service.py", line 202, in setup_clients | 08:36 |
| simboja | cls.lbaas = cls.os_roles_lb_admin.load_balancer_v2.LoadbalancerClient() | 08:36 |
| simboja | AttributeError: 'Manager' object has no attribute 'load_balancer_v2' | 08:36 |
| gryf | simboja, please, don't paste tracebacks on irc. | 08:37 |
| simboja | Ohh! so sorry | 08:37 |
| simboja | here https://paste.opendev.org/show/808110/ | 08:38 |
| gryf | but on your patchset 19, where you removed surplus method `setup_credentials`, there is no failure regarding lack of missing attribute? | 08:45 |
| simboja | Let me do that, I think got some other failure regarding it. Let me see | 08:47 |
| gryf | btw, you've pasted above, that kuryr-kubernetes-tempest-wallaby complains about lack of lb client. | 08:50 |
| simboja | yes, and I suspect its line 12 for setting up the client | 08:52 |
| gryf | you don't have included octavia-tempest-plugin on ussuri, victoria and wallaby branches. | 08:53 |
| dulek | simboja: If you restart the kuryr-controller you'll need to wait 10 minutes for the task to be run. | 08:55 |
| simboja | Now I see | 08:55 |
| dulek | For debugging you can try setting run_immediately=True to make it run on every restart. | 08:55 |
| simboja | dulek, okay | 08:55 |
| simboja | gryf: Adding octavia-tempest on the branches | 08:56 |
| gryf | hold on. | 09:04 |
| maysams | gryf: I've been chatting with simboja and I suggested another solution for the issues reported here https://review.opendev.org/c/openstack/kuryr-tempest-plugin/+/803244 | 09:22 |
| maysams | gryf: it might be better to make the test run configurable instead of including the dependency to all the other branches | 09:23 |
| maysams | there are users that will run the tempest on another branches where the feature won't be supported | 09:23 |
| maysams | s/another/other | 09:28 |
| gryf | maysams, agree. | 09:29 |
| simboja | Perfect! | 09:30 |
| dulek | simboja: Reviewed! | 11:18 |
| opendevreview | Merged openstack/kuryr-kubernetes master: Remove ep_slices from klb on endpoint delete event https://review.opendev.org/c/openstack/kuryr-kubernetes/+/804205 | 11:33 |
| digitalsimboja | Thanks | 11:58 |
| 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 | 14:46 |
| opendevreview | Merged openstack/kuryr-kubernetes master: Replace duplicate code to get klb crd path https://review.opendev.org/c/openstack/kuryr-kubernetes/+/804593 | 15:14 |
| opendevreview | Merged openstack/kuryr-kubernetes master: Add x-openstack-request-id into Kuryr Logs https://review.opendev.org/c/openstack/kuryr-kubernetes/+/801709 | 15:14 |
| 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:41 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!