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

Update gRPC python version in install_fedora_deps.sh #4241

Merged
merged 3 commits into from
Nov 10, 2023

Conversation

antoninbas
Copy link
Member

The fedora:latest container now comes with Python 3.12. Older gRPC versions do not support this Python version. See
grpc/grpc#33063.

An alternative would be to pin the fedora container version, but then someone has to remember to update it periodically...

@antoninbas antoninbas force-pushed the fix-fedora-ci-build branch 2 times, most recently from 15fafc1 to 32b026b Compare November 10, 2023 19:16
@antoninbas
Copy link
Member Author

This will unstuck pending PRs which have received approval

Copy link
Contributor

@jafingerhut jafingerhut left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The changes look reasonable to me, but after commit 2 it appears that perhaps the BMv2 tests are failing for a different reason than before?

The fedora:latest container now comes with Python 3.12. Older gRPC
versions do not support this Python version. See
grpc/grpc#33063.

An alternative would be to pin the fedora container version, but then
someone has to remember to update it periodically...

Signed-off-by: Antonin Bas <antonin.bas@gmail.com>
Required for more recent versions of Python3

Signed-off-by: Antonin Bas <antonin.bas@gmail.com>
@antoninbas
Copy link
Member Author

The changes look reasonable to me, but after commit 2 it appears that perhaps the BMv2 tests are failing for a different reason than before?

Yes, I'm on commit 3 now. I'm moving from issue to issue, all because of a more recent Python version. But hopefully, the 3rd issue was the last one.

Changed in Python version 3.12: mkdtemp() now always returns an absolute path,
even if dir is relative.

Signed-off-by: Antonin Bas <antonin.bas@gmail.com>
@antoninbas antoninbas merged commit fe7b267 into p4lang:main Nov 10, 2023
13 checks passed
@antoninbas antoninbas deleted the fix-fedora-ci-build branch November 10, 2023 23:32
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.

4 participants