sahid | o/ | 08:58 |
---|---|---|
sahid | tkajinam: any chance that you have a look at https://review.opendev.org/c/openstack/osprofiler/+/877358 ? I'm very sorry annoying you but it's so easy to find reviewer for osprofiler :-) | 08:59 |
opendevreview | Merged openstack/microversion-parse master: Moves supported python runtimes from version 3.8 to 3.10 https://review.opendev.org/c/openstack/microversion-parse/+/879934 | 10:14 |
opendevreview | Merged openstack/etcd3gw master: Moves supported python runtimes from version 3.8 to 3.10 https://review.opendev.org/c/openstack/etcd3gw/+/879933 | 10:14 |
opendevreview | Merged openstack/oslotest master: Moves supported python runtimes from version 3.8 to 3.10 https://review.opendev.org/c/openstack/oslotest/+/879317 | 10:15 |
opendevreview | Merged openstack/oslo.tools master: Moves supported python runtimes from version 3.8 to 3.10 https://review.opendev.org/c/openstack/oslo.tools/+/879313 | 10:15 |
opendevreview | Merged openstack/oslo.upgradecheck master: Moves supported python runtimes from version 3.8 to 3.10 https://review.opendev.org/c/openstack/oslo.upgradecheck/+/879314 | 10:15 |
opendevreview | Merged openstack/castellan master: Moves supported python runtimes from version 3.8 to 3.10 https://review.opendev.org/c/openstack/castellan/+/879932 | 11:13 |
opendevreview | Merged openstack/oslo.rootwrap master: Moves supported python runtimes from version 3.8 to 3.10 https://review.opendev.org/c/openstack/oslo.rootwrap/+/879311 | 11:13 |
opendevreview | Merged openstack/taskflow master: Moves supported python runtimes from version 3.8 to 3.10 https://review.opendev.org/c/openstack/taskflow/+/879929 | 11:15 |
opendevreview | Merged openstack/oslo.versionedobjects master: Moves supported python runtimes from version 3.8 to 3.10 https://review.opendev.org/c/openstack/oslo.versionedobjects/+/879315 | 11:15 |
opendevreview | Merged openstack/oslo.db master: Moves supported python runtimes from version 3.8 to 3.10 https://review.opendev.org/c/openstack/oslo.db/+/879281 | 11:19 |
opendevreview | Merged openstack/oslo.i18n master: tox: Remove 'ignore_base_python_conflict' https://review.opendev.org/c/openstack/oslo.i18n/+/874231 | 11:22 |
opendevreview | Merged openstack/oslo.vmware master: Moves supported python runtimes from version 3.8 to 3.10 https://review.opendev.org/c/openstack/oslo.vmware/+/879316 | 11:23 |
opendevreview | Merged openstack/debtcollector master: Moves supported python runtimes from version 3.8 to 3.10 https://review.opendev.org/c/openstack/debtcollector/+/879927 | 11:24 |
opendevreview | Merged openstack/stevedore master: Catch NotADirectoryError error https://review.opendev.org/c/openstack/stevedore/+/856833 | 11:28 |
opendevreview | Merged openstack/stevedore master: Moves supported python runtimes from version 3.8 to 3.10 https://review.opendev.org/c/openstack/stevedore/+/879931 | 11:28 |
opendevreview | Merged openstack/oslo.service master: Moves supported python runtimes from version 3.8 to 3.10 https://review.opendev.org/c/openstack/oslo.service/+/879312 | 11:28 |
opendevreview | Merged openstack/oslo.limit master: Moves supported python runtimes from version 3.8 to 3.10 https://review.opendev.org/c/openstack/oslo.limit/+/879283 | 11:30 |
opendevreview | Merged openstack/oslo.reports master: Moves supported python runtimes from version 3.8 to 3.10 https://review.opendev.org/c/openstack/oslo.reports/+/879310 | 11:30 |
opendevreview | Merged openstack/oslo.context master: Moves supported python runtimes from version 3.8 to 3.10 https://review.opendev.org/c/openstack/oslo.context/+/879280 | 11:31 |
opendevreview | Merged openstack/oslo.log master: Moves supported python runtimes from version 3.8 to 3.10 https://review.opendev.org/c/openstack/oslo.log/+/879284 | 11:31 |
opendevreview | Merged openstack/oslo.utils master: Moves supported python runtimes from version 3.8 to 3.10 https://review.opendev.org/c/openstack/oslo.utils/+/878749 | 11:31 |
opendevreview | Merged openstack/tooz master: Moves supported python runtimes from version 3.8 to 3.10 https://review.opendev.org/c/openstack/tooz/+/879930 | 11:32 |
opendevreview | Merged openstack/oslo.config master: Moves supported python runtimes from version 3.8 to 3.10 https://review.opendev.org/c/openstack/oslo.config/+/879279 | 11:32 |
opendevreview | Merged openstack/futurist master: tox: Replace whitelist_externals https://review.opendev.org/c/openstack/futurist/+/879941 | 11:33 |
opendevreview | Merged openstack/futurist master: Moves supported python runtimes from version 3.8 to 3.10 https://review.opendev.org/c/openstack/futurist/+/879928 | 11:33 |
tkajinam | sahid, I've left a few comments but I probably have to learn more about relationship between jaeger and otlp | 11:33 |
tkajinam | sahid, as far as I've read some articles it sounded like otlp is more generalized interface which replaces native jaeger library, right ? If we follow that and aim to replace jaeger by otlp in the future then we probably should make otpl driver more generic (at least it should not be jeager_foo) and independent from jaeger imo | 11:34 |
opendevreview | Merged openstack/oslo.serialization master: Moves supported python runtimes from version 3.8 to 3.10 https://review.opendev.org/c/openstack/oslo.serialization/+/878750 | 11:37 |
opendevreview | Merged openstack/oslo.middleware master: Moves supported python runtimes from version 3.8 to 3.10 https://review.opendev.org/c/openstack/oslo.middleware/+/879287 | 11:40 |
sahid | tkajinam: yes i was thinking about to make it more generic, at that point I don't see anything that we can use to consume it. It's more like we want to replace the current jaeger driver | 11:42 |
opendevreview | Merged openstack/oslo.metrics master: Moves supported python runtimes from version 3.8 to 3.10 https://review.opendev.org/c/openstack/oslo.metrics/+/879286 | 11:42 |
sahid | but ti's fair discussion... I don;t know if you feel strong on your idea i will make the change, I guess it's more like I was considering to use the easy pass to avoid build new config options for a new driver | 11:44 |
tkajinam | sahid, ok then I believe it should be generic one and should not depend on jeager driver I believe. For example the current implementation makes that oltp driver referes to [profiler_jaeger] service_name_prefix which likely require redundant steps to drop jaeger | 11:45 |
sahid | that i was not sure it would have made sense for something else than jaeger tracing | 11:45 |
sahid | ok, let'ss build a clear difference | 11:46 |
opendevreview | Merged openstack/oslo.privsep master: Moves supported python runtimes from version 3.8 to 3.10 https://review.opendev.org/c/openstack/oslo.privsep/+/879289 | 11:46 |
sahid | btw how do you feel regarding https://review.opendev.org/c/openstack/osprofiler/+/879318 i would appreciate to get your idea on it when you have a moment | 11:48 |
opendevreview | Merged openstack/oslo.i18n master: Moves supported python runtimes from version 3.8 to 3.10 https://review.opendev.org/c/openstack/oslo.i18n/+/879282 | 11:50 |
tkajinam | sahid, left a few comments but these are not huge blockers | 12:21 |
opendevreview | Hervé Beraud proposed openstack/oslo.messaging stable/2023.1: Moves supported python runtimes from version 3.8 to 3.10 https://review.opendev.org/c/openstack/oslo.messaging/+/880048 | 12:31 |
sahid | tkajinam: thanks a lot ++ | 12:33 |
opendevreview | Merged openstack/oslo.policy master: Moves supported python runtimes from version 3.8 to 3.10 https://review.opendev.org/c/openstack/oslo.policy/+/879288 | 12:54 |
opendevreview | Merged openstack/oslo.concurrency master: Moves supported python runtimes from version 3.8 to 3.10 https://review.opendev.org/c/openstack/oslo.concurrency/+/879277 | 15:26 |
opendevreview | Merged openstack/oslo.messaging stable/2023.1: move py3 jobs from zed to antelope https://review.opendev.org/c/openstack/oslo.messaging/+/875657 | 15:45 |
opendevreview | Tobias Urdin proposed openstack/oslo.messaging master: [WIP] Add sync NATS driver https://review.opendev.org/c/openstack/oslo.messaging/+/878588 | 20:54 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!