| *** JasonF is now known as JayF | 04:12 | |
| sahid | hberaud: i have noticed that you have +2ed patch on osprofiler that is unblocking CI | 08:55 |
|---|---|---|
| sahid | do you know an other core to approve it? | 08:55 |
| sahid | lools like we don't have lot of contributors for that project | 08:55 |
| hberaud | sahid: I'll +W it too | 09:02 |
| hberaud | done | 09:02 |
| sahid | hberaud: ++ that makes sense I guess | 09:02 |
| opendevreview | Hervé Beraud proposed openstack/oslo.utils master: Implement zoneinfo support to drop dependency to pytz https://review.opendev.org/c/openstack/oslo.utils/+/872862 | 09:12 |
| opendevreview | Hervé Beraud proposed openstack/oslo.utils master: Implement zoneinfo support to drop dependency to pytz https://review.opendev.org/c/openstack/oslo.utils/+/872862 | 09:18 |
| hberaud | fungi: o/ Please can have a look to this patch ^ when you will have spare time. Especially the tzdata part described in my commit message. I think that no time zone data are available by default in our gates env, else we wouldn't get the tzdata error, exact? | 09:21 |
| opendevreview | Rodolfo Alonso proposed openstack/oslo.privsep master: Setup logging without fixing evenlet logging https://review.opendev.org/c/openstack/oslo.privsep/+/864254 | 09:58 |
| ralonsoh | hberaud, is it possible to have ^^ in the next release? That will help with the oslo.log issue solved in https://review.opendev.org/c/openstack/oslo.log/+/864252 | 10:00 |
| ralonsoh | this patch is compatible with oslo.log < 5.0.2 and oslo.log 5.0.2 | 10:01 |
| hberaud | stephenfin: o/ please can you double check? https://review.opendev.org/c/openstack/oslo.log/+/864252 | 10:03 |
| hberaud | ralonsoh: yeah I think we can merge it asap and then release it once merged | 10:03 |
| ralonsoh | hberaud, thanks a lot | 10:03 |
| hberaud | np | 10:04 |
| opendevreview | Merged openstack/osprofiler master: tox: Add functional-py38, functional-py39 envs https://review.opendev.org/c/openstack/osprofiler/+/840711 | 10:47 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/osprofiler master: jaeger: introduce service name prefix https://review.opendev.org/c/openstack/osprofiler/+/872152 | 10:52 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/osprofiler master: Change StrictRedis usage to Redis https://review.opendev.org/c/openstack/osprofiler/+/859543 | 10:53 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/osprofiler master: Add default port to Elasticsearch connection url https://review.opendev.org/c/openstack/osprofiler/+/860412 | 10:55 |
| fungi | hberaud: have you tried adding tzdata to bindep.txt? then you'll see the job install it if it's not already present | 12:52 |
| hberaud | fungi: not to tzdata but to reqs.txt | 13:01 |
| hberaud | I'll try adding it to bindep | 13:01 |
| hberaud | fungi: oh wait, reqs.txt is enough | 13:02 |
| hberaud | the py310 job past | 13:02 |
| hberaud | and it is normal that py38 fail because zoneinfo is added by py39 | 13:03 |
| fungi | hberaud: i thought you were asking about how to get the system package for tzdata installed, but yes i see there's a tzdata package on pypi as well. i have no idea how those differ | 13:04 |
| fungi | Download tzdata | 13:04 |
| fungi | ArchitecturePackage SizeInstalled SizeFiles | 13:05 |
| fungi | er, sorry, pasted from the wrong buffer | 13:05 |
| hberaud | np | 13:05 |
| fungi | https://packages.ubuntu.com/jammy-updates/tzdata | 13:05 |
| fungi | that | 13:05 |
| hberaud | no my question was moslty about if time zone are defined for our gates envs | 13:05 |
| hberaud | to confirm that if we see our lib trying to load tzdata means that those configuration are absent | 13:06 |
| fungi | right, that's what the tzdata package would supply, so you were asking whether we have it preinstalled? | 13:06 |
| hberaud | no | 13:07 |
| hberaud | By default, zoneinfo uses the system’s time zone data if available; if no system time zone data is available, the library will fall back to using the first-party tzdata package available on PyPI. See => https://docs.python.org/3/library/zoneinfo.html | 13:07 |
| fungi | it looks like https://pypi.org/project/tzdata/ is a copy of the same tz database that the distro package supplies | 13:07 |
| hberaud | no I was asking if system's time zone are available, apparantly not | 13:08 |
| fungi | "the system's time zone data" is normally supplied by this package: https://packages.ubuntu.com/jammy-updates/tzdata | 13:08 |
| hberaud | ok | 13:08 |
| fungi | which is why i was saying add it to bindep.txt if you want it to be supplied | 13:08 |
| hberaud | ah I see | 13:09 |
| hberaud | thanks | 13:09 |
| fungi | "is system time zone data available?" is the same as "is the distro's tzdata package installed?" | 13:09 |
| fungi | at least that's the package name for debian derivatives (including ubuntu), but i expect it's the same for red hat derived distros too, and most others | 13:10 |
| fungi | the name of the package on pypi seems to also be identical, presumably because it's a copy of the same database as in the distro packages | 13:11 |
| hberaud | ok, I was thinking that the python doc described another mechanism that the tzdata, as the default, but now I better understand how it works. | 13:14 |
| hberaud | thanks | 13:14 |
| hberaud | (in other, I was thinking the default option was something different than tzdata) | 13:15 |
| hberaud | *words | 13:15 |
| opendevreview | Hervé Beraud proposed openstack/oslo.serialization master: Implement zoneinfo support to drop dependency to pytz https://review.opendev.org/c/openstack/oslo.serialization/+/872915 | 13:40 |
| opendevreview | Hervé Beraud proposed openstack/oslo.serialization master: Implement zoneinfo support to drop dependency to pytz https://review.opendev.org/c/openstack/oslo.serialization/+/872915 | 13:45 |
| opendevreview | Hervé Beraud proposed openstack/oslo.utils master: Implement zoneinfo support to drop dependency to pytz https://review.opendev.org/c/openstack/oslo.utils/+/872862 | 13:46 |
| *** tosky_ is now known as tosky | 14:19 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!