| sahid | morning all, an easy one on osprofiler - https://review.opendev.org/c/openstack/osprofiler/+/877969 | 07:18 |
|---|---|---|
| sahid | it's usefull when working on it and doing multiple stack unstack | 07:19 |
| sahid | an other question, I'm a bit blocked on this https://review.opendev.org/c/openstack/requirements/+/877474 I should avoid to consider using this package? | 09:57 |
| sahid | basically it's to add opentelemetry to requierment | 09:57 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/osprofiler master: devstack: remove jaeger container on unstack https://review.opendev.org/c/openstack/osprofiler/+/877969 | 13:08 |
| sahid | hberaud: fixed and I commented back your review. Thank you ^ | 13:10 |
| hberaud | sahid: concerning your requirement patch, as Sean pointed out this requirement doesn't seems to support the python versions that are ours, so either 1) you work on this requirement to introduce the support of the python versions that we support (https://review.opendev.org/c/openstack/governance/+/872232) and then you came back with an update of your requirement patch that | 13:16 |
| hberaud | introduce these new supported runtime or 2) if possible, you move to another library who offers the same features that you need. | 13:16 |
| hberaud | python3.9, 3.10, and 3.11 seems more or less supported https://github.com/open-telemetry/opentelemetry-python/blob/main/tox.ini#L8 maybe this project simply lack of metadata to reflect these supported runtimes... | 13:18 |
| hberaud | sahid: hopefully my answer can help you | 13:19 |
| opendevreview | Brian Haley proposed openstack/oslo.utils master: Add netutils.get_my_ipv6() https://review.opendev.org/c/openstack/oslo.utils/+/878078 | 13:56 |
| sahid | hberaud: thank a lot for your great explanation (with links...) | 14:25 |
| sahid | but yeah it's weird, looks like opentelemtry os well supporting py3.9 and py3.7 | 14:27 |
| sahid | https://github.com/open-telemetry/opentelemetry-python/blob/main/opentelemetry-sdk/pyproject.toml | 14:27 |
| sahid | I need to double-check the error reported by jenkins | 14:28 |
| hberaud | ack | 14:36 |
| sahid | ok there was a typo on the package name :-) I hope that will be ok now | 14:40 |
| hberaud | crossing fingers :) | 14:42 |
| opendevreview | Eric Harney proposed openstack/taskflow master: Fix doc building with Sphinx 6.0 https://review.opendev.org/c/openstack/taskflow/+/878092 | 15:30 |
| opendevreview | mike_mp@zzzcomputing.com proposed openstack/oslo.db master: Remove sqlalchemy-migrate https://review.opendev.org/c/openstack/oslo.db/+/853025 | 17:42 |
| opendevreview | mike_mp@zzzcomputing.com proposed openstack/oslo.db master: Remove sqlalchemy-migrate https://review.opendev.org/c/openstack/oslo.db/+/853025 | 18:00 |
| opendevreview | mike_mp@zzzcomputing.com proposed openstack/oslo.db master: Get 2.0 to pass fully with test suite https://review.opendev.org/c/openstack/oslo.db/+/875986 | 18:00 |
| opendevreview | mike_mp@zzzcomputing.com proposed openstack/oslo.db master: Get 2.0 to pass fully with test suite https://review.opendev.org/c/openstack/oslo.db/+/875986 | 18:12 |
| opendevreview | mike_mp@zzzcomputing.com proposed openstack/oslo.db master: Get 2.0 to pass fully with test suite https://review.opendev.org/c/openstack/oslo.db/+/875986 | 18:22 |
| opendevreview | mike_mp@zzzcomputing.com proposed openstack/oslo.db master: use SQLAlchemy native pre-ping https://review.opendev.org/c/openstack/oslo.db/+/878146 | 18:28 |
| opendevreview | mike_mp@zzzcomputing.com proposed openstack/oslo.db master: Get 2.0 to pass fully with test suite https://review.opendev.org/c/openstack/oslo.db/+/875986 | 18:34 |
| opendevreview | mike_mp@zzzcomputing.com proposed openstack/oslo.db master: use SQLAlchemy native pre-ping https://review.opendev.org/c/openstack/oslo.db/+/878146 | 18:41 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!