| stephenfin | mnasiadka: Could I get a +W on https://review.opendev.org/c/openstack/requirements/+/1000078 (I already have 2 +2s)? | 11:53 |
|---|---|---|
| mnasiadka | stephenfin: I think frickler indicated he would like to get the other patches up the chain sorted out before merging this | 11:55 |
| stephenfin | I don't think so? This is needed regardless of whether we change the default rules or not | 11:56 |
| mnasiadka | so be it | 11:58 |
| frickler | stephenfin: my concern was I don't want to have too much delay in getting https://review.opendev.org/c/openstack/requirements/+/1007744/1 added on top of it. so better to have the whole stack ready | 12:00 |
| stephenfin | https://review.opendev.org/c/openstack/requirements/+/1000078/ only changes tox.ini though. The other changes are all separate | 12:01 |
| stephenfin | There's a change to project.py but will disappear on merge since it was fixed elsewhere | 12:01 |
| stephenfin | so I don't think we'd want to blame ignore this change since it's an actually meaningful change | 12:02 |
| frickler | oh, I didn't notice the latter. then it is fine to merge I guess | 12:02 |
| stephenfin | cool cool | 12:03 |
| frickler | stephenfin: btw., did you see the issues with latest sqlalchemy yet? seems 2.1.x needs some adaptions, we have manually pinned to < 2.1 so far. https://review.opendev.org/c/openstack/requirements/+/1008574 should have some relevant logs, but we likely should also do a dedicated bump for easier working on this | 12:07 |
| opendevreview | Dr. Jens Harbott proposed openstack/requirements master: Fixes applied to automated upper-constraints.txt update: https://review.opendev.org/c/openstack/requirements/+/977551 | 12:08 |
| frickler | one example: sqlalchemy.exc.SADeprecationWarning: The noload() option is deprecated and will be removed in a future release. This option produces incorrect results by returning ``None`` for related items. (deprecated since: 2.1) | 12:11 |
| frickler | (from https://zuul.opendev.org/t/openstack/build/066d023eb3fb45dba503299f44e1b48d) | 12:11 |
| stephenfin | yeah, I made deprecation warnings into errors for nova because we missed deprecations in the past and they bit us later on | 12:12 |
| stephenfin | I am unlikely to have time to look into that until later this week though. Would you have time to bring this up on openstack-discuss? It sounds like nova, manila and cinder all have work to do? | 12:13 |
| frickler | I try to avoid sending mails to the list if possible, but I can have a look later. let me propose the single bump first to verify affected projects easier | 12:16 |
| opendevreview | Dr. Jens Harbott proposed openstack/requirements master: Bump SQLAlchemy to 2.1.3 https://review.opendev.org/c/openstack/requirements/+/1008776 | 12:18 |
| mnasiadka | Seems cinder, manila, nova and octavia are affected (or at least their unit tests) | 12:48 |
| opendevreview | Merged openstack/requirements master: Remove backports.txt file, backport handling https://review.opendev.org/c/openstack/requirements/+/1008440 | 12:54 |
| tkajinam | I remember I submitted something for that warning. | 13:05 |
| tkajinam | https://review.opendev.org/q/topic:%22bug/2168669%22 | 13:06 |
| tkajinam | I didn't notice that cinder and manila are also affected by it. I was grepping the code but there might be some patterns that could not catch | 13:07 |
| tkajinam | in general we can replace noload by raiseload but there are number of implementations (at least in octavia) accessing these non-loaded attributes now :-( | 13:07 |
| tkajinam | there are broken by raiseload "correctly" | 13:08 |
| tkajinam | these * | 13:08 |
| tkajinam | ok cinder and manila are broken due to a different reason | 13:11 |
| tkajinam | "AttributeError: module 'sqlalchemy.orm.attributes' has no attribute 'ScalarAttributeImpl'" | 13:11 |
| tkajinam | and the error comes from sqlalchemy-utils | 13:12 |
| opendevreview | Merged openstack/requirements master: Allow projects to drop their Windows specifiers https://review.opendev.org/c/openstack/requirements/+/1008441 | 13:13 |
| opendevreview | Merged openstack/requirements master: test: Remove use of testscenarios https://review.opendev.org/c/openstack/requirements/+/1008442 | 13:13 |
| opendevreview | Merged openstack/requirements master: Remove tomli dependency https://review.opendev.org/c/openstack/requirements/+/1008443 | 13:13 |
| opendevreview | Merged openstack/requirements stable/2026.2: update constraint for tosca-parser to new release 2.15.0 https://review.opendev.org/c/openstack/requirements/+/1007688 | 13:14 |
| opendevreview | Merged openstack/requirements stable/2025.2: update constraint for oslo.messaging to new release 17.1.1 https://review.opendev.org/c/openstack/requirements/+/1008094 | 13:14 |
| opendevreview | Merged openstack/requirements stable/2026.1: update constraint for oslo.messaging to new release 17.3.1 https://review.opendev.org/c/openstack/requirements/+/1008095 | 13:14 |
| opendevreview | Merged openstack/requirements stable/2026.2: update constraint for oslo.log to new release 8.3.2 https://review.opendev.org/c/openstack/requirements/+/1007454 | 13:14 |
| opendevreview | Merged openstack/requirements master: update constraint for os-refresh-config to new release 14.1.0 https://review.opendev.org/c/openstack/requirements/+/1008251 | 13:14 |
| opendevreview | Rodolfo Alonso proposed openstack/requirements master: Remove ``sys_platform`` marker from ``pyroute2`` https://review.opendev.org/c/openstack/requirements/+/1008065 | 13:16 |
| opendevreview | Merged openstack/requirements master: update constraint for os-collect-config to new release 14.1.0 https://review.opendev.org/c/openstack/requirements/+/1008253 | 13:19 |
| opendevreview | Merged openstack/requirements master: update constraint for os-apply-config to new release 14.1.0 https://review.opendev.org/c/openstack/requirements/+/1008252 | 13:19 |
| tkajinam | we have to wait until https://github.com/kvesteri/sqlalchemy-utils/pull/831 it seems | 13:20 |
| tkajinam | wait until that PR is merged, I mean | 13:20 |
| opendevreview | Simon Dodsley proposed openstack/requirements stable/2026.1: Bump py-pure-client upper constraint to 1.89.0 https://review.opendev.org/c/openstack/requirements/+/1008807 | 14:34 |
| opendevreview | OpenStack Proposal Bot proposed openstack/requirements master: update constraint for openstack-release-test to new release 8.6.0 https://review.opendev.org/c/openstack/requirements/+/1008829 | 16:06 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!