<div dir="auto"><div>What is extremely weird for me, is that osprofiler is not I requirements at all - only in test-requirements, which should mean that it should not be required anywhere except for package testing? Though I see from code, that osprofiler will just fail with ImportError if opentelemetry is absent, which IMO is quite wrong from the beginning.</div><div dir="auto"><br></div><div dir="auto">And indeed that would make sense to me to use extras_require on setup.cfg to install only drivers that are needed, not * for each service.</div><div dir="auto"><br></div><div dir="auto">Though I am not sure who is motivated enough to invest time into that refactoring...<br><br><div class="gmail_quote" dir="auto"><div dir="ltr" class="gmail_attr">On Fri, Sep 1, 2023, 08:54 Matthias Runge <<a href="mailto:mrunge@matthias-runge.de">mrunge@matthias-runge.de</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 30/08/2023 11:11, Thomas Goirand wrote:<br>
> Hi,<br>
> <br>
> I just saw that the last version of osprofiler requires <br>
> opentelemetry-exporter-otlp and opentelemetry-sdk. Since osprofiler is <br>
> used in almost all projects, and that opentelemetry is really HUGE, I <br>
> wonder if this is all reasonable. This forces downstream distros to do a <br>
> lot of packaging only for that. Is this avoidable?<br>
> <br>
> I started doing the packaging, and then stopped to write this message, <br>
> seeing how much work that would represent.<br>
<br>
Hi zigo,<br>
<br>
from what I have seen here (and you may remember talking about this when <br>
we met at the OpenInfra Summit this year), the change makes sense to me.<br>
<br>
Quickly looking at [1], there may be the possibility to make it optional <br>
though, and that would certainly make sense for the other drivers too.<br>
<br>
Matthias<br>
<br>
<br>
[1] <br>
<a href="https://github.com/openstack/osprofiler/commit/908e7402320eb067db45aa9700d54d31c259f3ca" rel="noreferrer noreferrer" target="_blank">https://github.com/openstack/osprofiler/commit/908e7402320eb067db45aa9700d54d31c259f3ca</a><br>
-- <br>
Matthias Runge <<a href="mailto:mrunge@matthias-runge.de" target="_blank" rel="noreferrer">mrunge@matthias-runge.de</a>><br>
<br>
<br>
</blockquote></div></div></div>