Skip to content

Fix python version. #56

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

Merged
merged 2 commits into from
Feb 14, 2024
Merged

Fix python version. #56

merged 2 commits into from
Feb 14, 2024

Conversation

zzhlogin
Copy link
Contributor

Issue #, if available:

  1. change tox.ini py3{7,8,9,10,11} to 3.{7,8,9,10,11} so that the workflow can run unit test.
  2. change project Dokerfile to use python 3.10, because we are using opentelemetry-exporter-otlp-proto-grpc, it use grpcio as dependencies. It seems like grpc has a strict dependency on the OS / Python version. Also mentioned in Docker build template in upstream.
    When we use python 3.11 to build the docker file, we receive "Cannot import name 'cygrpc' from 'grpc._cython'" in EKS auto-instrumentation injection. As https://github.com/MicrosoftDocs/azure-docs/blob/main/articles/azure-functions/recover-python-functions.md#the-python-interpre[…]tions-python-worker suggested, Azure Functions support python 3.6 - 3.9, but while testing, 3.10 also worked. So change to 3.10 here.
  3. As we change the python version for Dockerfile we also change main_build and release_build workflow to align with Dock build.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@zzhlogin zzhlogin requested a review from a team February 14, 2024 03:05
@zzhlogin zzhlogin merged commit 5b3ed74 into main Feb 14, 2024
@zzhlogin zzhlogin deleted the python_version_fix branch February 14, 2024 18:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants