| opendevreview | Yian Zong proposed openstack/os-brick stable/zed: Dell Powerflex: Add new VOLUME_NOT_MAPPED_ERROR https://review.opendev.org/c/openstack/os-brick/+/915941 | 03:42 |
|---|---|---|
| opendevreview | OpenStack Proposal Bot proposed openstack/cinder master: Imported Translations from Zanata https://review.opendev.org/c/openstack/cinder/+/915796 | 03:42 |
| *** elodilles_pto is now known as elodilles | 07:17 | |
| opendevreview | Christian Rohmann proposed openstack/cinder master: Optimize getting parent backup for new incremental https://review.opendev.org/c/openstack/cinder/+/484729 | 08:16 |
| opendevreview | Christian Rohmann proposed openstack/cinder-specs master: Repropose spec to introduce new backup_status field for volume https://review.opendev.org/c/openstack/cinder-specs/+/915973 | 08:59 |
| opendevreview | Christian Rohmann proposed openstack/cinder-specs master: Repropose spec to introduce new backup_status field for volume https://review.opendev.org/c/openstack/cinder-specs/+/915973 | 09:08 |
| opendevreview | Raghavendra Tilay proposed openstack/cinder master: HPE 3par: PP - Return LUN ids from both arrays https://review.opendev.org/c/openstack/cinder/+/901588 | 09:47 |
| opendevreview | Takashi Kajinami proposed openstack/cinder master: Fix broken swift backup driver with backup_swift_service_auth=True https://review.opendev.org/c/openstack/cinder/+/915979 | 10:11 |
| opendevreview | Niklas Schwarz proposed openstack/cinder master: Ceph: Add option to keep only last n snapshots per backup https://review.opendev.org/c/openstack/cinder/+/810457 | 11:00 |
| zaitcev | Guys, what are our chances to get Cinder tests run on py312? | 20:16 |
| zaitcev | Maybe I ought to ask at the team meeting tomorrow. | 20:17 |
| clarkb | zaitcev: you can push a cahnge to add a job that does it. Then figure out what breaks. I'm guessing its hard to say one way or another until there is data | 20:21 |
| zaitcev | clarkb: I'm one of the oldtimers who still run tox -e py3 on their laptops. | 20:35 |
| eharney | the pep8 tox env currently fails on a python 3.12 environment because of an incompatible dependency | 20:45 |
| eharney | i keep setting basepython=python3.10 in tox.ini to run it | 20:45 |
| zaitcev | Hmm. My magic incantation that fixed everything was this: | 20:46 |
| zaitcev | ln -s /usr/lib/python3.12/site-packages/asyncore .tox/py3/lib/python3.12/site-packages/ | 20:46 |
| zaitcev | But this is obviously not for Zuul. | 20:46 |
| eharney | you can just run "tox -e py310" or 311 for unit tests | 20:47 |
| crohmann | zaitcev: you mean the non-voting openstack-tox-py312 job? | 20:48 |
| crohmann | Since we are talking about tests ... I added the missing unit tests to the https://review.opendev.org/c/openstack/cinder/+/484729 we talked about at the PTG. Could anybody kindly take a peek at the failing CI and tell me if that's really my change causing this? | 20:50 |
| eharney | crohmann: cinder-plugin-ceph-tempest failed there because the volume service hung... that one seems unlikely to be related to your patch | 20:55 |
| zaitcev | Unfortunately, pep8 is no-go on py312 because flake8 uses an obsolete way to get a list of entry points, which no longer works. | 20:59 |
| clarkb | if you add setuptools to the dependency list that may work around ^ as it will pull in pkg_resources. That should still be addressed to use importlib but this is a quick workaround | 22:24 |
| zaitcev | thanks | 22:24 |
| clarkb | zaitcev: fwiw I try to run tox/nox locally myself. The point is more that I doubt many people have tried running under python3.12 yet and getting a CI job running will help expose problems so they can be addressed. | 22:30 |
| clarkb | I also try to keep a number of python versions around for flexibility | 22:30 |
| opendevreview | Pete Zaitcev proposed openstack/cinder master: Output a better availability zone in backup's debug message https://review.opendev.org/c/openstack/cinder/+/913391 | 22:46 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!