| opendevreview | Merged openstack/tempest master: Add replace location test https://review.opendev.org/c/openstack/tempest/+/861390 | 00:54 |
|---|---|---|
| *** jpena|off is now known as jpena | 08:25 | |
| kopecmartin | johnsom: hi, when you have a sec, could you please check this patch? https://review.opendev.org/c/openstack/designate-tempest-plugin/+/862804 it fixes an issue in python-tempestconf, however, if that fix doesn't belong to designate plugin we will address that somehow in python-tempestconf, thanks | 08:48 |
| frickler | kopecmartin: I can review that, too, although I don't know what python-tempestconf is, can you point me to the issue that you are seeing there? | 09:02 |
| kopecmartin | frickler: thanks, it's a tool that queries a cloud and generates tempest.conf basically | 09:06 |
| kopecmartin | https://zuul.opendev.org/t/openstack/build/61f54d3072d54b1c9bbd825d08af080a | 09:06 |
| frickler | kopecmartin: ah, o.k., I'm trying to see why those checks fail in your deployment but not for designate. maybe amending your test environment would be a better solution | 09:08 |
| kopecmartin | it worked for you because you used tempest.conf which was generated by devstack, python-tempestconf can't discover [dns].nameservers therefore it's not set in tempest.conf and all designate tests fail, my understanding is that even tests which don't require the value fail as well | 09:12 |
| kopecmartin | i think that the whole point is not to fail in init but only when the nameservers are about to be used -> https://review.opendev.org/c/openstack/designate-tempest-plugin/+/862804/8/designate_tempest_plugin/services/dns/query/query_client.py#41 | 09:17 |
| frickler | o.k., thanks for the context, I'll take a deeper look at the patch later | 09:18 |
| kopecmartin | in other words , do we want to make designate-tempest-plugin users to always set the the nameservers value in tempest.conf even if they want to run designate tests which do not require that | 09:21 |
| kopecmartin | frickler: sure, thanks | 09:21 |
| frickler | kopecmartin: is there a reason you don't have a html summary of your tempest run like other jobs? would make it easier to see the number of tests that actually get skipped by your test. but not a blocker, I need to do some local testing anyway | 09:46 |
| kopecmartin | frickler: no reason, let me look into that, it should be a quick fix | 09:49 |
| opendevreview | Martin Kopec proposed openstack/tempest master: Increase timeout for tempest-centos9-stream-fips https://review.opendev.org/c/openstack/tempest/+/863163 | 10:37 |
| opendevreview | Vadym Markov proposed openstack/devstack stable/yoga: Use proper sed separator for paths https://review.opendev.org/c/openstack/devstack/+/863451 | 10:55 |
| dansmith | frickler: kopecmartin: I'd really like to get this in: https://review.opendev.org/c/openstack/devstack/+/860976 | 13:36 |
| dansmith | there's a nova change waiting on it, so that we can test glance changes against nova-ceph-multistore | 13:36 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/devstack-plugin-ceph master: Use default Ubuntu Jammy ceph repos https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/863524 | 14:05 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/devstack-plugin-ceph master: Use default Ubuntu Jammy ceph repos https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/863524 | 15:16 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/devstack-plugin-ceph master: Use default Ubuntu Jammy ceph repos https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/863524 | 15:20 |
| opendevreview | James Parker proposed openstack/whitebox-tempest-plugin master: Update UEFI check when secure is not present https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/863536 | 15:57 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/devstack-plugin-ceph master: Use default Ubuntu Jammy ceph repos https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/863524 | 16:32 |
| *** jpena is now known as jpena|off | 17:22 | |
| *** gibi is now known as gibi_pto | 19:15 | |
| *** arxcruz|ruck is now known as arxcruz | 20:05 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!