Tuesday, 2023-04-11

sahido/08:58
sahidtkajinam: 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
opendevreviewMerged openstack/microversion-parse master: Moves supported python runtimes from version 3.8 to 3.10  https://review.opendev.org/c/openstack/microversion-parse/+/87993410:14
opendevreviewMerged openstack/etcd3gw master: Moves supported python runtimes from version 3.8 to 3.10  https://review.opendev.org/c/openstack/etcd3gw/+/87993310:14
opendevreviewMerged openstack/oslotest master: Moves supported python runtimes from version 3.8 to 3.10  https://review.opendev.org/c/openstack/oslotest/+/87931710:15
opendevreviewMerged openstack/oslo.tools master: Moves supported python runtimes from version 3.8 to 3.10  https://review.opendev.org/c/openstack/oslo.tools/+/87931310:15
opendevreviewMerged openstack/oslo.upgradecheck master: Moves supported python runtimes from version 3.8 to 3.10  https://review.opendev.org/c/openstack/oslo.upgradecheck/+/87931410:15
opendevreviewMerged openstack/castellan master: Moves supported python runtimes from version 3.8 to 3.10  https://review.opendev.org/c/openstack/castellan/+/87993211:13
opendevreviewMerged openstack/oslo.rootwrap master: Moves supported python runtimes from version 3.8 to 3.10  https://review.opendev.org/c/openstack/oslo.rootwrap/+/87931111:13
opendevreviewMerged openstack/taskflow master: Moves supported python runtimes from version 3.8 to 3.10  https://review.opendev.org/c/openstack/taskflow/+/87992911:15
opendevreviewMerged openstack/oslo.versionedobjects master: Moves supported python runtimes from version 3.8 to 3.10  https://review.opendev.org/c/openstack/oslo.versionedobjects/+/87931511:15
opendevreviewMerged openstack/oslo.db master: Moves supported python runtimes from version 3.8 to 3.10  https://review.opendev.org/c/openstack/oslo.db/+/87928111:19
opendevreviewMerged openstack/oslo.i18n master: tox: Remove 'ignore_base_python_conflict'  https://review.opendev.org/c/openstack/oslo.i18n/+/87423111:22
opendevreviewMerged openstack/oslo.vmware master: Moves supported python runtimes from version 3.8 to 3.10  https://review.opendev.org/c/openstack/oslo.vmware/+/87931611:23
opendevreviewMerged openstack/debtcollector master: Moves supported python runtimes from version 3.8 to 3.10  https://review.opendev.org/c/openstack/debtcollector/+/87992711:24
opendevreviewMerged openstack/stevedore master: Catch NotADirectoryError error  https://review.opendev.org/c/openstack/stevedore/+/85683311:28
opendevreviewMerged openstack/stevedore master: Moves supported python runtimes from version 3.8 to 3.10  https://review.opendev.org/c/openstack/stevedore/+/87993111:28
opendevreviewMerged openstack/oslo.service master: Moves supported python runtimes from version 3.8 to 3.10  https://review.opendev.org/c/openstack/oslo.service/+/87931211:28
opendevreviewMerged openstack/oslo.limit master: Moves supported python runtimes from version 3.8 to 3.10  https://review.opendev.org/c/openstack/oslo.limit/+/87928311:30
opendevreviewMerged openstack/oslo.reports master: Moves supported python runtimes from version 3.8 to 3.10  https://review.opendev.org/c/openstack/oslo.reports/+/87931011:30
opendevreviewMerged openstack/oslo.context master: Moves supported python runtimes from version 3.8 to 3.10  https://review.opendev.org/c/openstack/oslo.context/+/87928011:31
opendevreviewMerged openstack/oslo.log master: Moves supported python runtimes from version 3.8 to 3.10  https://review.opendev.org/c/openstack/oslo.log/+/87928411:31
opendevreviewMerged openstack/oslo.utils master: Moves supported python runtimes from version 3.8 to 3.10  https://review.opendev.org/c/openstack/oslo.utils/+/87874911:31
opendevreviewMerged openstack/tooz master: Moves supported python runtimes from version 3.8 to 3.10  https://review.opendev.org/c/openstack/tooz/+/87993011:32
opendevreviewMerged openstack/oslo.config master: Moves supported python runtimes from version 3.8 to 3.10  https://review.opendev.org/c/openstack/oslo.config/+/87927911:32
opendevreviewMerged openstack/futurist master: tox: Replace whitelist_externals  https://review.opendev.org/c/openstack/futurist/+/87994111:33
opendevreviewMerged openstack/futurist master: Moves supported python runtimes from version 3.8 to 3.10  https://review.opendev.org/c/openstack/futurist/+/87992811:33
tkajinamsahid, I've left a few comments but I probably have to learn more about relationship between jaeger and otlp11:33
tkajinamsahid, 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 imo11:34
opendevreviewMerged openstack/oslo.serialization master: Moves supported python runtimes from version 3.8 to 3.10  https://review.opendev.org/c/openstack/oslo.serialization/+/87875011:37
opendevreviewMerged openstack/oslo.middleware master: Moves supported python runtimes from version 3.8 to 3.10  https://review.opendev.org/c/openstack/oslo.middleware/+/87928711:40
sahidtkajinam: 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
opendevreviewMerged openstack/oslo.metrics master: Moves supported python runtimes from version 3.8 to 3.10  https://review.opendev.org/c/openstack/oslo.metrics/+/87928611:42
sahidbut 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 driver11:44
tkajinamsahid, 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 jaeger11:45
sahidthat i was not sure it would have made sense for something else than jaeger tracing11:45
sahidok, let'ss build a clear difference 11:46
opendevreviewMerged openstack/oslo.privsep master: Moves supported python runtimes from version 3.8 to 3.10  https://review.opendev.org/c/openstack/oslo.privsep/+/87928911:46
sahidbtw 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 moment11:48
opendevreviewMerged openstack/oslo.i18n master: Moves supported python runtimes from version 3.8 to 3.10  https://review.opendev.org/c/openstack/oslo.i18n/+/87928211:50
tkajinamsahid, left a few comments but these are not huge blockers12:21
opendevreviewHervĂ© 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/+/88004812:31
sahidtkajinam: thanks a lot ++12:33
opendevreviewMerged openstack/oslo.policy master: Moves supported python runtimes from version 3.8 to 3.10  https://review.opendev.org/c/openstack/oslo.policy/+/87928812:54
opendevreviewMerged openstack/oslo.concurrency master: Moves supported python runtimes from version 3.8 to 3.10  https://review.opendev.org/c/openstack/oslo.concurrency/+/87927715:26
opendevreviewMerged openstack/oslo.messaging stable/2023.1: move py3 jobs from zed to antelope  https://review.opendev.org/c/openstack/oslo.messaging/+/87565715:45
opendevreviewTobias Urdin proposed openstack/oslo.messaging master: [WIP] Add sync NATS driver  https://review.opendev.org/c/openstack/oslo.messaging/+/87858820:54

Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!