Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

resolve requirement conflicts with grpcio-status in other transient dependancies when using python 3.11 #3258

Merged
merged 4 commits into from
Sep 15, 2023

Conversation

psi29a
Copy link
Contributor

@psi29a psi29a commented Apr 14, 2023

Description

resolve requirement conflicts with grpcio-status in other transient dependencies when using python 3.11

  File "/Users/psi29a/Workspace/backend/venv/lib/python3.11/site-packages/pkg_resources/__init__.py", line 800, in resolve
    raise VersionConflict(dist, req).with_context(dependent_req)
pkg_resources.ContextualVersionConflict: (grpcio-status 1.48.2 (/Users/psi29a/Workspace/backend/venv/lib/python3.11/site-packages), Requirement.parse('grpcio-status<2.0dev,>=1.49.1; python_version >= "3.11" and extra == "grpc"'), {'google-api-core'})

Bumping this also bumps grpcio-status which fixes other packages. There is no change in behavior of code.

Type of change

  • Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

Ran test suite and validate in internal product testing.

@psi29a psi29a requested a review from a team as a code owner April 14, 2023 10:24
@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Apr 14, 2023

CLA Signed

The committers listed above are authorized under a signed CLA.

  • ✅ login: psi29a / name: Bret Curtis (63b6290)
  • ✅ login: ocelotl / name: Diego Hurtado (3a2d8c9)
  • ✅ login: lzchen / name: Leighton Chen (866d820, d69ae23)

@psi29a
Copy link
Contributor Author

psi29a commented Apr 17, 2023

Do this really need a changelog entry?

@srikanthccv
Copy link
Member

No, but other failures should be resolved.

@psi29a
Copy link
Contributor Author

psi29a commented Apr 20, 2023

No, but other failures should be resolved.

That's the only failure is the changelog check.

@srikanthccv
Copy link
Member

Screenshot 2023-04-20 at 4 53 32 PM

@ocelotl ocelotl added the Skip Changelog PRs that do not require a CHANGELOG.md entry label Jun 6, 2023
@lzchen lzchen merged commit f02029c into open-telemetry:main Sep 15, 2023
111 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Skip Changelog PRs that do not require a CHANGELOG.md entry
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants