| opendevreview | Takashi Kajinami proposed openstack/tooz master: Deprecate zake driver because of unmaintained library https://review.opendev.org/c/openstack/tooz/+/901214 | 03:33 |
|---|---|---|
| opendevreview | Takashi Kajinami proposed openstack/tooz master: Deprecate zake driver because of unmaintained library https://review.opendev.org/c/openstack/tooz/+/901214 | 06:08 |
| opendevreview | Takashi Kajinami proposed openstack/tooz master: tox: Bump python runtime versions in the default envlist https://review.opendev.org/c/openstack/tooz/+/903855 | 06:14 |
| opendevreview | Philippe SÉRAPHIN proposed openstack/oslo.config master: Add choices parameter for IntOpt class https://review.opendev.org/c/openstack/oslo.config/+/903859 | 09:02 |
| opendevreview | Philippe SÉRAPHIN proposed openstack/oslo.config master: Add choices parameter for IntOpt class https://review.opendev.org/c/openstack/oslo.config/+/903862 | 09:24 |
| opendevreview | Takashi Kajinami proposed openstack/pycadf master: doc: Drop DJANGO_SETTINGS_MODULE environment https://review.opendev.org/c/openstack/pycadf/+/903876 | 12:06 |
| opendevreview | Rodolfo Alonso proposed openstack/oslo.utils master: Python-3.12: do not use datetime.datetime.utcnow() https://review.opendev.org/c/openstack/oslo.utils/+/903882 | 12:32 |
| tkajinam | stephenfin, this is probably a dumb question, but do we need __init__.py in db.migrations and db.migrations.versions ? | 12:34 |
| tkajinam | I found it exists in cinder but does not in nova/heat | 12:34 |
| tkajinam | https://github.com/openstack/cinder/blob/master/cinder/db/migrations/__init__.py | 12:34 |
| stephenfin | to the best of my knowledge, alembic doesn't need it, no | 12:34 |
| stephenfin | the only thing that might is the test runner, but it shouldn't be executing any of that stuff directly | 12:34 |
| tkajinam | yeah, iiuc we do not import these modules so I believe we can omit it. | 12:35 |
| opendevreview | Rodolfo Alonso proposed openstack/oslo.utils master: Python-3.12: do not use datetime.datetime.utcnow() https://review.opendev.org/c/openstack/oslo.utils/+/903882 | 12:35 |
| tkajinam | yeah, I agree | 12:35 |
| ralonsoh | zigo, ^^ | 12:35 |
| zigo | ralonsoh: Oh, good catch ! :) | 12:36 |
| tkajinam | zigo, I guess you have to mimic the usage of replace in your patch for keystoneauth. IIRC utcnow returns timezone without tz while now returns one with tz and we need conversions to avoid math operation between one with tz and the other without tz | 12:41 |
| zigo | ralonsoh: FYI, it built fine for me under Python 3.12, I'm uploading the package with your patch to avoid the annoying warning. | 12:43 |
| tkajinam | keystonemiddleware patch I mean | 12:43 |
| ralonsoh | ok | 12:43 |
| zigo | I'm not sure I'll have the available time... :/ | 12:45 |
| opendevreview | Rodolfo Alonso proposed openstack/oslo.utils master: Python-3.12: do not use datetime.datetime.utcnow() https://review.opendev.org/c/openstack/oslo.utils/+/903882 | 14:47 |
| opendevreview | Merged openstack/tooz master: Deprecate zake driver because of unmaintained library https://review.opendev.org/c/openstack/tooz/+/901214 | 17:58 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!