Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@aabmass
Copy link
Member

@aabmass aabmass commented Oct 14, 2021

Fixes #2195

  • Upgrade the dependencies in dev-requirements.txt. I switched a few of them to ~= instead of >=.
  • Regenerate protos with ./scripts/proto_codegen.sh
  • Black was trying to reformat the code in opentelemetry/proto/logs so updated the ignore in pyproject.toml to be more general

I'm wondering if we should pin dev-dependencies.txt and introduce some automation to bump them automatically instead of giving ranges

@aabmass aabmass changed the title Upgrade GRPC/protobuf related dependency, regenerate otlp protobufs Upgrade GRPC/protobuf related dependency and regenerate otlp protobufs Oct 14, 2021
@aabmass aabmass force-pushed the upgrade-grpc-proto-deps branch from e7705bf to 8dc86d9 Compare October 14, 2021 19:21
@aabmass aabmass force-pushed the upgrade-grpc-proto-deps branch from 8dc86d9 to 022ff57 Compare October 14, 2021 19:31
@aabmass aabmass marked this pull request as ready for review October 14, 2021 19:31
@aabmass aabmass requested a review from a team October 14, 2021 19:31
@owais
Copy link
Contributor

owais commented Oct 15, 2021

I'm wondering if we should pin dev-dependencies.txt and introduce some automation to bump them automatically instead of giving ranges

+1 for that. Automatic updates causes a lot of breakage and pain. I'd rather stay on older versions than have CI break every month.

@owais owais added the Approve Public API check This label shows that the public symbols added or changed in a PR are strictly necessary label Oct 15, 2021
@owais owais enabled auto-merge (squash) October 15, 2021 01:15
@owais owais merged commit 39fe4db into open-telemetry:main Oct 15, 2021
@aabmass aabmass deleted the upgrade-grpc-proto-deps branch October 15, 2021 02:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Approve Public API check This label shows that the public symbols added or changed in a PR are strictly necessary

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Upgrade grpcio-tools dev-requirements.txt and re-generate opentelemetry-proto

3 participants