| *** pmannidi|brb is now known as pmannidi | 00:09 | |
| *** pmannidi is now known as pmannidi|AFK | 00:47 | |
| *** pmannidi|AFK is now known as pmannidi | 00:47 | |
| opendevreview | Robin Cernin proposed openstack/kuryr-kubernetes master: move vif_pool oslo_cfg to config https://review.opendev.org/c/openstack/kuryr-kubernetes/+/805331 | 04:52 |
|---|---|---|
| opendevreview | Robin Cernin proposed openstack/kuryr-kubernetes master: Add kuryr-k8s-sanity command https://review.opendev.org/c/openstack/kuryr-kubernetes/+/805100 | 05:00 |
| opendevreview | Robin Cernin proposed openstack/kuryr-kubernetes master: Add kuryr-k8s-sanity command https://review.opendev.org/c/openstack/kuryr-kubernetes/+/805100 | 05:05 |
| opendevreview | Robin Cernin proposed openstack/kuryr-kubernetes master: Add kuryr-k8s-sanity command https://review.opendev.org/c/openstack/kuryr-kubernetes/+/805100 | 05:16 |
| opendevreview | Sunday Mgbogu proposed openstack/kuryr-kubernetes master: DNM: TEST CI https://review.opendev.org/c/openstack/kuryr-kubernetes/+/803902 | 05:17 |
| opendevreview | Robin Cernin proposed openstack/kuryr-kubernetes master: Add kuryr-k8s-sanity command https://review.opendev.org/c/openstack/kuryr-kubernetes/+/805100 | 05:21 |
| opendevreview | Robin Cernin proposed openstack/kuryr-kubernetes master: Add kuryr-k8s-sanity command https://review.opendev.org/c/openstack/kuryr-kubernetes/+/805100 | 05:25 |
| opendevreview | Robin Cernin proposed openstack/kuryr-kubernetes master: Add kuryr-k8s-sanity command https://review.opendev.org/c/openstack/kuryr-kubernetes/+/805100 | 06:29 |
| opendevreview | Robin Cernin proposed openstack/kuryr-kubernetes master: Add kuryr-k8s-sanity command https://review.opendev.org/c/openstack/kuryr-kubernetes/+/805100 | 06:41 |
| *** pmannidi is now known as pmannidi|AFK | 06:43 | |
| digitalsimboja | Hello maysams, just wondering, is there extra configuration that could make the test be skipped for all the other branches | 08:04 |
| digitalsimboja | By default, setting the enable_reconcile to False, should make it work? | 08:04 |
| maysams | digitalsimboja: yes | 08:05 |
| digitalsimboja | I am thinking the setup_credentials needs to also be configured somehow? | 08:06 |
| digitalsimboja | I mean this : | 08:09 |
| digitalsimboja | https://review.opendev.org/c/openstack/kuryr-tempest-plugin/+/803244/29/kuryr_tempest_plugin/tests/scenario/test_service.py#185 | 08:09 |
| maysams | digitalsimboja: I will review the patch in a moment | 08:16 |
| digitalsimboja | Great! | 08:16 |
| digitalsimboja | Hello! maysams, the test is skipped if the CONF is not set on the test | 10:23 |
| maysams | digitalsimboja: That is okay | 10:38 |
| digitalsimboja | Good | 10:38 |
| digitalsimboja | Thanks | 10:38 |
| maysams | digitalsimboja: if you remove the setting on the test the branches tests will stop failing | 10:39 |
| maysams | the tests will run only where they are needed | 10:39 |
| digitalsimboja | Understood | 10:39 |
| digitalsimboja | Got it | 10:39 |
| maysams | digitalsimboja: You can check some other implementations we added before, like https://review.opendev.org/c/openstack/kuryr-tempest-plugin/+/757551 | 10:40 |
| maysams | digitalsimboja: when you have a moment, can you check my comment here https://review.opendev.org/c/openstack/kuryr-kubernetes/+/803870? | 11:35 |
| maysams | thanks | 11:35 |
| digitalsimboja | taking a look... | 11:35 |
| 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: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 | 12: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 | 13:19 |
| opendevreview | Sunday Mgbogu proposed openstack/kuryr-kubernetes master: Update the LoadBalancers Reconciliation Loop https://review.opendev.org/c/openstack/kuryr-kubernetes/+/803870 | 13:37 |
| *** maysams is now known as maysams-afk | 17:47 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!