| opendevreview | Fernando Royo proposed openstack/octavia-tempest-plugin master: Not forcing listener CRUD testing with allowed_cidrs https://review.opendev.org/c/openstack/octavia-tempest-plugin/+/849028 | 11:26 |
|---|---|---|
| gthiemon1e | just FYI the openstack-tox-docs job is failing :/ https://zuul.opendev.org/t/openstack/builds?job_name=openstack-tox-docs&project=openstack/octavia | 13:26 |
| opendevreview | Tom Weininger proposed openstack/octavia master: Add pytest testenv to tox.ini https://review.opendev.org/c/openstack/octavia/+/881739 | 15:28 |
| opendevreview | Tom Weininger proposed openstack/octavia master: Make tests work with pytest runner https://review.opendev.org/c/openstack/octavia/+/881805 | 15:28 |
| opendevreview | Tom Weininger proposed openstack/octavia master: Rate limiting support https://review.opendev.org/c/openstack/octavia/+/933016 | 15:28 |
| opendevreview | Tom Weininger proposed openstack/octavia master: controllers/ratelimitrule.py https://review.opendev.org/c/openstack/octavia/+/941409 | 15:28 |
| gthiemon1e | tkajinam: any ideas why this happens in the octavia doc job? https://zuul.opendev.org/t/openstack/build/4d174d6f0a8143449292e6f879572214 | 15:45 |
| tkajinam | gthiemon1e, checking | 15:47 |
| tkajinam | gthiemon1e, ugh it is a regression in the latest oslo.db. I'm fixing it now | 15:48 |
| gthiemon1e | tkajinam: what changed in olso.db? I tried to find recent updates but I didn't see anything | 15:49 |
| gthiemon1e | oh there was a new release 3 days ago | 15:51 |
| tkajinam | this is the one which added that wrong import https://review.opendev.org/c/openstack/oslo.db/+/936314/1/oslo_db/sqlalchemy/models.py | 15:52 |
| gthiemon1e | I ran pep8 and unit tests in oslo.db and they didn't detect the issue | 15:53 |
| gthiemon1e | tkajinam: thanks! | 15:55 |
| tkajinam | gthiemon1e, that's why we didn't catch it... | 15:55 |
| tkajinam | The issue is specifically seen in that doc job because of class API documentation which inspects the full interface | 15:58 |
| gthiemon1e | tkajinam: will you publish a new release of oslo.db? or we may have to disable our doc job for the week | 16:14 |
| gthiemon1e | https://review.opendev.org/c/openstack/releases/+/942608 | 16:18 |
| gthiemon1e | ++ | 16:18 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!