| opendevreview | Merged openstack/python-designateclient stable/ussuri: Update zuul queue configuration https://review.opendev.org/c/openstack/python-designateclient/+/858333 | 00:03 |
|---|---|---|
| opendevreview | Merged openstack/python-designateclient stable/train: Update zuul queue configuration https://review.opendev.org/c/openstack/python-designateclient/+/858379 | 00:04 |
| opendevreview | Merged openstack/designate stable/victoria: Update zuul queue configuration https://review.opendev.org/c/openstack/designate/+/858381 | 00:52 |
| opendevreview | Merged openstack/designate-dashboard master: Switch to 2023.1 Python3 unit tests and generic template name https://review.opendev.org/c/openstack/designate-dashboard/+/857915 | 05:38 |
| opendevreview | Sergey Drozdov proposed openstack/designate-tempest-plugin master: Add tempest api tests for shared zones https://review.opendev.org/c/openstack/designate-tempest-plugin/+/858525 | 12:42 |
| opendevreview | Sergey Drozdov proposed openstack/designate-tempest-plugin master: Add tempest api tests for shared zones https://review.opendev.org/c/openstack/designate-tempest-plugin/+/730370 | 12:58 |
| zigo | Hi there! Building Designate, I have 228 unit test errors of this kind: https://paste.opendev.org/show/bd29Cao9JFhUmX6btuuj/ | 14:12 |
| zigo | Can someone help me fixing this? | 14:12 |
| zigo | frickler: ^ | 14:12 |
| frickler | johnsom is the alchemist in this case I guess. we really only test against versions in upper-constraints, can you check your delta against that? | 14:18 |
| johnsom | zigo Hi, that is interesting. So you are just running the unit test suite? Is this on a host that has a live designate database on it? | 14:32 |
| frickler | I'd assume that to be on a debian builder node | 14:33 |
| johnsom | I'm just trying to get context to help figure this out since we haven't seen it elsewhere | 14:34 |
| frickler | very likely different versions of some module, since debian uses deps from debs, not from pypi | 14:37 |
| johnsom | Yeah, there was also a requirements change associated with that change, so maybe the dependency is missing. I will dig a bit deeper into the code after my meeting. | 14:39 |
| zigo | johnsom: It's just unit test at package build time under Debian Unstable. | 14:41 |
| zigo | So I'm in a schroot env with all depends from unstable + experimental. | 14:42 |
| johnsom | zigo Thanks for the info, is there a link to the build details? I.e. the version list? | 14:42 |
| zigo | If I got a lib in the wrong version, that's very much possible... | 14:42 |
| zigo | johnsom: When my unit test run fails, it outputs "pip freeze" for that purpose (of debugging), so here it is: https://paste.opendev.org/show/b9I8eLo3t9XtHXCrWK5D/ | 14:43 |
| zigo | It's a mix of ilbs from unstable and experimental, and takes all from unstable if I don't have harder constraints. | 14:44 |
| zigo | So for example, oslo.db is the version from unstable. | 14:44 |
| zigo | $ rmadison python3-oslo.db | 14:44 |
| zigo | [...] | 14:44 |
| zigo | python3-oslo.db | 11.2.0-3 | unstable | all | 14:44 |
| zigo | python3-oslo.db | 12.1.0-1 | experimental | all | 14:44 |
| zigo | I can force anything from Experimental if I see it needs it. | 14:45 |
| zigo | In fact, that's the nice thing about this process: it checks we didn't forget to force a new version of a lib that was needed. | 14:45 |
| zigo | And this kind of things are *very* important when doing openstack upgrades. | 14:45 |
| zigo | My OpenStack Q/A page is https://qa.debian.org/developer.php?login=team%2Bopenstack%40tracker.debian.org | 14:46 |
| zigo | (this contains all the packages I maintain for OpenStack in Debian, with versions, etc., missing those I maintain in the Python team) | 14:46 |
| frickler | zigo: johnsom: it is alembic. with 1.7.7 I get the same errors, so we actually need >=1.8.0 | 14:55 |
| zigo | Oh, thanks ! | 14:55 |
| zigo | Let me try upgrading that one then. | 14:55 |
| frickler | I just wanted to ask how feasible that would be, o.k. ;-) | 14:56 |
| johnsom | Thanks for tracking that down. I'm stuck in a meeting at the moment. | 14:57 |
| johnsom | This is why I liked the lower-constraints jobs. sigh | 14:57 |
| johnsom | I will post some patches to update that today | 15:00 |
| zigo | Much better with Alembic 1.8.1, now I only have 2 unit test failures: | 15:09 |
| zigo | https://paste.opendev.org/show/byQHriMEvSMkP94J3AUW/ | 15:09 |
| zigo | frickler: johnsom: Any idea what these are? | 15:09 |
| zigo | I'm headding home, will look at your answer later or tomorrow (or resume the conversation tomorrow if you can help...). | 15:11 |
| zigo | Maybe oslo.messaging 14.0.0 ? | 15:11 |
| johnsom | Yeah, I'm guessing it's an oslo.messaging version issue. | 15:14 |
| zigo | Nop, not that one ... | 15:15 |
| zigo | Ok, got to go... | 15:15 |
| opendevreview | Michael Johnson proposed openstack/designate master: Fix minimum versions in requirements.txt https://review.opendev.org/c/openstack/designate/+/858561 | 17:54 |
| opendevreview | Dr. Jens Harbott proposed openstack/designate master: DNM: CI test https://review.opendev.org/c/openstack/designate/+/858562 | 18:01 |
| frickler | this is a weird failure, I managed to reproduce it, but now I can't get rid of it anymore, even with a completely fresh checkout of the repo. so I'm also not sure yet what triggers it | 18:34 |
| frickler | o.k., so on a fresh vm, "tox -e py3 unit.mdns" fails, full run "tox -e py3" passes. so likely some weird initialization issue | 19:17 |
| frickler | eandersson: ^^ maybe one for you since you did the refactoring there lately | 19:18 |
| johnsom | Ah, a test order issue | 19:21 |
| johnsom | Yeah, I have been looking at this too and hadn't figured it out yet. | 19:21 |
| zigo | frickler: johnsom: Thanks a lot, so I can safely blacklist these unit tests for the moment until you fix it. That's enough for me (ie: to know my package isn't broken...). Best would be to open a bug so I can refer to it... | 20:24 |
| zigo | Designate 15.0.0~rc1-1 uploaded ! :) | 20:30 |
| opendevreview | Thomas Goirand proposed openstack/designate master: requires Alembic 1.8.0 https://review.opendev.org/c/openstack/designate/+/858579 | 20:33 |
| johnsom | zigo https://review.opendev.org/c/openstack/designate/+/858561 | 20:38 |
| johnsom | I wasn't sure if there was another one we needed to update. | 20:39 |
| johnsom | The unit tests are broken, it's a test order issue, so I am comfortable with you skipping them until we get a fix. (I'm working on it) | 20:39 |
| zigo | Abandonned mine. | 20:40 |
| johnsom | Cool | 20:40 |
| johnsom | Thanks for the heads up on these. | 20:40 |
| zigo | Thanks to you, it's super helpful. | 20:40 |
| zigo | All of the packages we use in production are built for Zed at this time. | 20:41 |
| zigo | I'm still having issues building Magnum, Murano, Vitrage and Watcher, everthing else is good to go. | 20:41 |
| zigo | I can already try setting-up a full setup ! :) | 20:42 |
| opendevreview | Michael Johnson proposed openstack/designate master: Fix mdns test_handler setup https://review.opendev.org/c/openstack/designate/+/858588 | 23:03 |
| johnsom | zigo https://review.opendev.org/c/openstack/designate/+/858588 should fix your TRANSPORT issue | 23:03 |
| opendevreview | Merged openstack/designate master: Fix minimum versions in requirements.txt https://review.opendev.org/c/openstack/designate/+/858561 | 23:50 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!