| opendevreview | Robin Cernin proposed openstack/kuryr-kubernetes master: Add klb ep handling vms https://review.opendev.org/c/openstack/kuryr-kubernetes/+/804931 | 00:21 |
|---|---|---|
| opendevreview | Robin Cernin proposed openstack/kuryr-kubernetes master: Add klb ep handling vms https://review.opendev.org/c/openstack/kuryr-kubernetes/+/804931 | 03:41 |
| opendevreview | Robin Cernin proposed openstack/kuryr-kubernetes master: Add klb ep handling vms https://review.opendev.org/c/openstack/kuryr-kubernetes/+/804931 | 04:04 |
| opendevreview | Robin Cernin proposed openstack/kuryr-kubernetes master: Add klb ep handling vms https://review.opendev.org/c/openstack/kuryr-kubernetes/+/804931 | 04:39 |
| opendevreview | Robin Cernin proposed openstack/kuryr-kubernetes master: Add klb ep handling vms https://review.opendev.org/c/openstack/kuryr-kubernetes/+/804931 | 04:52 |
| opendevreview | Sunday Mgbogu proposed openstack/kuryr-kubernetes master: Update the LoadBalancers Reconciliation Loop https://review.opendev.org/c/openstack/kuryr-kubernetes/+/803870 | 07:02 |
| 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:08 |
| 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:16 |
| opendevreview | Roman Dobosz proposed openstack/kuryr-kubernetes master: [WIP] Switch gates to OVN by default. https://review.opendev.org/c/openstack/kuryr-kubernetes/+/802999 | 12:42 |
| opendevreview | Roman Dobosz proposed openstack/kuryr-kubernetes master: [WIP] Switch gates to OVN by default. https://review.opendev.org/c/openstack/kuryr-kubernetes/+/802999 | 12:46 |
| opendevreview | Sunday Mgbogu proposed openstack/kuryr-kubernetes master: Update the LoadBalancers Reconciliation Loop https://review.opendev.org/c/openstack/kuryr-kubernetes/+/803870 | 13:27 |
| digitalsimboja | Hi dulek: please take a look | 13:27 |
| digitalsimboja | Thanks | 13:27 |
| dulek | Will do. | 13:31 |
| opendevreview | Sunday Mgbogu proposed openstack/kuryr-kubernetes master: Check whether Octavia tempest plugin is enabled https://review.opendev.org/c/openstack/kuryr-kubernetes/+/805033 | 14:27 |
| digitalsimboja | maysams: Please take a look. Thanks | 14:27 |
| 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:36 |
| opendevreview | Sunday Mgbogu proposed openstack/kuryr-kubernetes master: Enable reconciliation by default https://review.opendev.org/c/openstack/kuryr-kubernetes/+/805033 | 16: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 | 16:17 |
| 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 | 16:39 |
| digitalsimboja | maysams Hello!: Now to enable the reconciliation to run? | 16:56 |
| digitalsimboja | {0} setUpClass (kuryr_tempest_plugin.tests.scenario.test_service.TestLoadBalancerReconciliation) ... SKIPPED: Reconciliation is not enabled | 16:56 |
| maysams | digitalsimboja: the config should be set to False on the kuryr-tempest-plugin patch as it's depending on another patch on kuryr-kubernetes that enables it | 17:00 |
| digitalsimboja | right within the patch I set it to False? | 17:01 |
| maysams | yes | 17:01 |
| digitalsimboja | now I understand | 17:01 |
| digitalsimboja | Thanks! | 17:01 |
| maysams | and it would be nice to include the blueprint reference on the commit message of those patches | 17:01 |
| maysams | digitalsimboja: ^ | 17:04 |
| digitalsimboja | Great! | 17:04 |
| digitalsimboja | Same blueprint like in the previous patch? | 17:05 |
| maysams | digitalsimboja: yes | 17:29 |
| digitalsimboja | maysams: I am setting the CONF thus: | 17:39 |
| digitalsimboja | https://paste.opendev.org/show/808182/ | 17:39 |
| digitalsimboja | yet failing the test | 17:39 |
| maysams | did you set it back to false on the config? | 17:46 |
| maysams | digitalsimboja: probably the config is not enabled on the tempest.conf file | 17:53 |
| maysams | We should be able to see if it will run on the ci | 17:54 |
| simboja | I need to enable this manually? | 18:04 |
| maysams | yeah, but if this would be enabled by default like you've done https://review.opendev.org/c/openstack/kuryr-tempest-plugin/+/803244/27/kuryr_tempest_plugin/config.py#102 just for testing it wouldn't be needed | 18:07 |
| simboja | I see the mistake | 18:07 |
| *** maysams is now known as maysams-afk | 18:24 | |
| 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 | 19:16 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!