| eandersson | What is the issue johnsom? | 01:27 |
|---|---|---|
| eandersson | I am unable to get this to fail. | 01:28 |
| eandersson | I can get it to fail if I create dummy records and then run the test again. | 01:28 |
| eandersson | (e.g. it almost always failed the second CI run because records already existed. | 01:28 |
| eandersson | Maybe related but I re-wrote the middlerware to be more robust https://review.opendev.org/c/openstack/designate/+/796171 and improved the testing | 01:29 |
| opendevreview | Erik Olof Gunnar Andersson proposed openstack/designate master: Updated middleware https://review.opendev.org/c/openstack/designate/+/796171 | 01:31 |
| opendevreview | Erik Olof Gunnar Andersson proposed openstack/designate master: Updated middleware https://review.opendev.org/c/openstack/designate/+/796171 | 01:40 |
| eandersson | I guess the tl;dr is that if you revert my patch and run the CI twice locally it fails 100% of the time | 01:44 |
| eandersson | (my tempest patch) | 01:44 |
| eandersson | ohhh I think I know what it is | 02:30 |
| eandersson | never mind that wasn't it | 02:39 |
| eandersson | ok.. locally it failed at least once.. because there were more than 20 zones | 02:47 |
| eandersson | the more I ran the test the more likely I hit the 20 zone limit | 02:47 |
| eandersson | It takes time for Designate to process zone deletions | 02:48 |
| eandersson | johnsom can you still reproduce it if you set default_limit_v2 to 100? :D | 02:50 |
| eandersson | I think either way we should make sure our tests supports pagination | 03:06 |
| opendevreview | Erik Olof Gunnar Andersson proposed openstack/designate master: Support filtering on zone import/export list https://review.opendev.org/c/openstack/designate/+/789711 | 03:38 |
| opendevreview | Merged openstack/designate-tempest-plugin master: Improve the zone import API tests https://review.opendev.org/c/openstack/designate-tempest-plugin/+/796575 | 05:05 |
| opendevreview | Erik Olof Gunnar Andersson proposed openstack/designate-tempest-plugin master: Properly clean up transfer requests https://review.opendev.org/c/openstack/designate-tempest-plugin/+/797172 | 07:06 |
| eandersson | Lack of pagination support is def causing some weird issues (or maybe the default 20 is just too low?) | 07:22 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!