python3Packages.opentelemetry-{api, instrumentation}: bump#498050
python3Packages.opentelemetry-{api, instrumentation}: bump#498050CertainLach wants to merge 3 commits into
Conversation
|
Cc: @sinrohit-desco @DanielNoord (from the original PR) |
|
@natsukium as maintainer, any chance we could merge this? |
|
@natsukium gentle ping :) |
5199bd2 to
ca8c9c5
Compare
|
|
@stefanboca What is needed for this to get merged? |
|
I'm not a maintainer so I can't really say :) I guess I should have explained this when I posted my I just tested again after rebasing on nixos-unstable (so I don't have to compile half the universe) and they're still failing. I realize that my
|
|
requests instrumentation hardcodes requests version to python-requests/2.32.3 in all of its tests, but nixpkgs has 2.32.5 packaged We should either patch it with the correct versions, or just disable tests for this package, as those are just bad tests D: |
|
I would greatly appreciate if someone can report this problem to upstream and propose a fix for it, I would then pull this patch into this PR |
|
All of the other opentelemetry packages are failing because they depend on opentelemetry-instrumetation-requests package |
All of the opentelemetry-instrumentation-requests tests are hardcoding requests version, and since requests package in nixpkgs is newer than expected by the package - all of the tests fail. This should be fixed upstream, I do not see a good way to patch that at nixpkgs side.
ca8c9c5 to
56cff7f
Compare
|
@CertainLach @natsukium Is there anything left here? The |
|
I think everything should be fine with the requests instrumentation fix (disabled tests due to hardcoded package versions), and we just have to wait for maintainer |
Supersedes: #489017
Things done
passthru.tests.nixpkgs-reviewon this PR. See nixpkgs-review usage../result/bin/.