| fungi | pip 24.3 was just released. they love their weekend releases | 00:16 |
|---|---|---|
| mordred | <fungi> "and yeah, i want to say this has..." <- wat? openstacksdk was *never* supposed to use oslo *anything* | 16:43 |
| mordred | uhm. so - that's coming from keystoneauth: https://zuul.opendev.org/t/opendev/build/9dffc60bd59a435f86b7c9c5b9ba96b9 | 16:51 |
| mordred | I'm *really* confused here:... (full message at <https://matrix.org/oftc/media/v1/media/download/AekbmMG0jFuvIYZDIBro8kcDa1lxf7mPh5yP7T9XNx53SUge8WZMkCSH4HaOk_lNBA5u57aGDPrQtDzK6ToZ2INCeTgyczFQAG1hdHJpeC5vcmcvdUxuSkZia0dYTENZWEtxaFlrTUhjQk9W>) | 16:56 |
| mordred | oh - that on ubuntu noble - trying that, and yes, there is an oslo utils import | 17:01 |
| mordred | someone has added that - it does not exist in keystoneauth | 17:01 |
| mordred | yup. added in an ubuntu patch | 17:05 |
| mordred | jamespage: ^^ patches/datetime-deprecations.patch adds an import of oslo.utils without adding a dependency on the oslo utils package. FWIW - it is very explicitly a keystoneauth design decision to never have runtime depends on any oslo packages, because it is supposed to be hyper lightweight and oslo utils are not intended for clients, they are intended for servers. It would be AWESOME to solve that problem in a way that didn't break that design | 17:09 |
| mordred | contract. HOWEVER - at the very least, if that is a thing that is going to happen in the ubuntu packaging even against the stated design principles of the project (please see the NOTE from morgan in the requirements.txt in the source repo) ... please add an ubuntu package dependency on oslo.utils | 17:09 |
| mordred | but may I suggest that instead of adding oslo.utils - you simply replace datetime.datetime.utcnow() with datetime.datetime.now(datetime.timezone.utc) which solves the same problem without pulling in a whole library? | 17:11 |
| fungi | (which was also the recommendation in the python release notes about utcnow() removal) | 17:18 |
| mordred | https://bugs.launchpad.net/ubuntu/+source/python-keystoneauth1/+bug/2088384 | 17:29 |
| mordred | I have filed a bug for completeness | 17:29 |
| clarkb | it is up to shas with a 392 prefix now | 18:46 |
| *** elodilles is now known as elodilles_pto | 19:12 | |
| jamespage | mordred: yep that just impacts on noble - I'll look to get that resolved. | 21:08 |
| mordred | Thank you sir! | 21:38 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!