| opendevreview | Michal Nasiadka proposed openstack/magnum-tempest-plugin master: WIP: k8s driver CI tests https://review.opendev.org/c/openstack/magnum-tempest-plugin/+/893131 | 07:28 |
|---|---|---|
| opendevreview | Michal Nasiadka proposed openstack/magnum-tempest-plugin master: WIP: k8s driver CI tests https://review.opendev.org/c/openstack/magnum-tempest-plugin/+/893131 | 07:28 |
| opendevreview | Michal Nasiadka proposed openstack/magnum-tempest-plugin master: WIP: k8s driver CI tests https://review.opendev.org/c/openstack/magnum-tempest-plugin/+/893131 | 07:28 |
| opendevreview | Michal Nasiadka proposed openstack/magnum-tempest-plugin master: WIP: k8s driver CI tests https://review.opendev.org/c/openstack/magnum-tempest-plugin/+/893131 | 07:29 |
| opendevreview | Michal Nasiadka proposed openstack/magnum-tempest-plugin master: WIP: k8s driver CI tests https://review.opendev.org/c/openstack/magnum-tempest-plugin/+/893131 | 07:29 |
| opendevreview | Michal Nasiadka proposed openstack/magnum-tempest-plugin master: WIP: k8s driver CI tests https://review.opendev.org/c/openstack/magnum-tempest-plugin/+/893131 | 08:12 |
| opendevreview | John Garbutt proposed openstack/magnum master: Stop test_delete_loadbalancers_timeout waiting https://review.opendev.org/c/openstack/magnum/+/877115 | 10:12 |
| johnthetubaguy | It would be great to get this one merged or rejected please? It really speeds up running unit tests, for me at least: https://review.opendev.org/c/openstack/magnum/+/877115 | 10:14 |
| opendevreview | John Garbutt proposed openstack/magnum master: Add feature flag for beta drivers https://review.opendev.org/c/openstack/magnum/+/891722 | 10:17 |
| mnasiadka | johnthetubaguy: and amphora stands up in 1 second? | 10:48 |
| mnasiadka | ah, that's pre delete timeout | 10:49 |
| johnthetubaguy | If we want to test we can sleep for 60 seonds, that is fine, please -2 the patch. In the past I always tweak the unit test to reduce the sleep, either by mocking sleep or overriding config. Happy to follow the local tyle. | 10:49 |
| johnthetubaguy | s/tyle/style/ | 10:49 |
| mnasiadka | I just meant the commit message didn't mention it's a pre-delete timeout, so I got a bit confused | 10:51 |
| johnthetubaguy | Happy to tweak that, let me know some suggested wording please. Afraid I don't remember any context now, beyond I found a sleep runing for ages in the unit test. | 10:54 |
| opendevreview | John Garbutt proposed openstack/magnum master: Stop test_delete_loadbalancers_timeout waiting https://review.opendev.org/c/openstack/magnum/+/877115 | 10:57 |
| jakeyip | johnthetubaguy: I think that looks reasonable | 11:26 |
| opendevreview | Michal Nasiadka proposed openstack/magnum-tempest-plugin master: WIP: k8s driver CI tests https://review.opendev.org/c/openstack/magnum-tempest-plugin/+/893131 | 11:40 |
| opendevreview | Merged openstack/magnum master: Stop test_delete_loadbalancers_timeout waiting https://review.opendev.org/c/openstack/magnum/+/877115 | 12:33 |
| opendevreview | Michal Nasiadka proposed openstack/magnum-tempest-plugin master: WIP: k8s driver CI tests https://review.opendev.org/c/openstack/magnum-tempest-plugin/+/893131 | 13:53 |
| opendevreview | Michal Nasiadka proposed openstack/magnum-tempest-plugin master: WIP: k8s driver CI tests https://review.opendev.org/c/openstack/magnum-tempest-plugin/+/893131 | 13:57 |
| mnasiadka | ricolin: do you have an idea why https://opendev.org/openstack/magnum-tempest-plugin/src/commit/ef903363153ef34abf79afb6ea7810ee6663525f/magnum_tempest_plugin/tests/api/v1/test_cluster.py#L78 (basically both actions in this tearDownClass) - are failing with 401 (Unauthorized)? | 14:37 |
| jakeyip | mnasiadka: are you able to read the error message? is it a keystone unauthorised or does it say something like catalog empty? | 14:41 |
| mnasiadka | It’s keystone unauthorized, probably we’re using the wrong user - will check once my lab vm is up | 15:42 |
| opendevreview | Travis Holton proposed openstack/magnum master: Add flatcar as os for cluster api driver https://review.opendev.org/c/openstack/magnum/+/887545 | 19:55 |
| opendevreview | Travis Holton proposed openstack/magnum master: use internal oci registry https://review.opendev.org/c/openstack/magnum/+/888972 | 19:55 |
| opendevreview | Travis Holton proposed openstack/magnum master: Remove fixme comment https://review.opendev.org/c/openstack/magnum/+/888973 | 19:55 |
| opendevreview | Travis Holton proposed openstack/magnum master: Add label to toggle keystone auth flag and pass to helm values https://review.opendev.org/c/openstack/magnum/+/895738 | 19:55 |
| opendevreview | Travis Holton proposed openstack/magnum master: Temporarily add config for k8s-keystone-auth webhook https://review.opendev.org/c/openstack/magnum/+/895739 | 19:55 |
| opendevreview | Travis Holton proposed openstack/magnum master: Fetch values from context https://review.opendev.org/c/openstack/magnum/+/895740 | 19:55 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!