| sahid | o/ | 07:58 |
|---|---|---|
| sahid | quick question regqrding https://review.opendev.org/c/openstack/requirements/+/877474 I can see that it received 2x +2 but have not been merged, is there anything missing? | 07:59 |
| sahid | I need it to make progress on a work for osprofiler | 07:59 |
| clarkb | sahid: someone needs to approve it in addition to the +2's. I don't know why they didn't approve it when providing the +2 but my hunch is due to the openstack release happening at roughly the same time. Now that that is done maybe prometheanfire can approve it | 14:52 |
| prometheanfire | for req adds / removals and other larger changes I prefer to have a second set of eyes | 15:03 |
| prometheanfire | tonyb: ^ | 15:03 |
| sahid | ack thank you guys | 15:17 |
| tonyb | sahid: It looks to me like this will replace the jaeger-client impementation | 18:04 |
| tonyb | sahid: from: https://review.opendev.org/c/openstack/osprofiler/+/877358/8/releasenotes/notes/otlp-driver-for-jaeger-tracing-cb932038ad580ac2.yaml you say that jaeget-client is deprecated but they're still doing releases? | 18:04 |
| tonyb | Can you a) clarify the status of jaeger-client ; and b) give us some idea of the deprecation timeline? | 18:05 |
| tonyb | it's not that jaeger-client is a pain for us us but it'd be a good idea to an an idea of when it can/will be dropped | 18:05 |
| tonyb | Also the upgrade plan for consumers of the jaeger driver to jaeger_otlp | 18:06 |
| tonyb | Actually nevermind question 1) https://github.com/jaegertracing/jaeger-client-python/blob/master/README.md makes the status very clear | 18:08 |
| opendevreview | Merged openstack/requirements master: add opentelemetry to global requirements https://review.opendev.org/c/openstack/requirements/+/877474 | 20:02 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!