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

py3-grpcio-gcp: Add python multiversion support #34721

Merged
merged 3 commits into from
Nov 25, 2024

Conversation

dannf
Copy link
Contributor

@dannf dannf commented Nov 20, 2024

Related: https://github.com/chainguard-dev/internal-dev/issues/5334

From the individual commits:

  • pip-build-install pipeline: Introduce "source" input variable
    • Python modules may be provided in subdirectories of the upstream source.
  • py3-grpcio-gcp: Build from github repository using py/pip-build-install
    • Building from the tarball results in an unusable package because the included grpc_gcp_pb2.py is generated from an old protobuf. The tarball includes the .proto file we need to generate a new one.
  • py3-grpcio-gcp: Add python multiversion support

dannf added a commit to dannf/os that referenced this pull request Nov 20, 2024
This does not include a conversion to py/pip-build-install because
it does not appear to support building a subdirectory.

python3.13 support is not included because it is blocked by a 3.13
build of py3-grpcio-gcp (wolfi-dev#34721).

Signed-off-by: dann frazier <[email protected]>
dannf added a commit to dannf/os that referenced this pull request Nov 20, 2024
This does not include a conversion to py/pip-build-install because
it does not appear to support building a subdirectory.

python3.13 support is not included because it is blocked by a 3.13
build of py3-grpcio-gcp (wolfi-dev#34721).

Signed-off-by: dann frazier <[email protected]>
dannf added a commit to dannf/os that referenced this pull request Nov 21, 2024
This does not include a conversion to py/pip-build-install because
it does not appear to support building a subdirectory.

python3.13 support is not included because it is blocked by a 3.13
build of py3-grpcio-gcp (wolfi-dev#34721).

Signed-off-by: dann frazier <[email protected]>
@dannf dannf added the approved-to-run A repo member has approved this external contribution label Nov 22, 2024
@octo-sts octo-sts bot added the bincapz/pass bincapz/pass Bincapz (aka. malcontent) scan didn't detect any CRITICALs on the scanned packages. label Nov 22, 2024
@dannf dannf force-pushed the multiversion/py3-grpcio-gcp branch 3 times, most recently from 0f535b4 to 5b4718f Compare November 22, 2024 22:48
@dannf dannf marked this pull request as ready for review November 22, 2024 22:59
@dannf dannf force-pushed the multiversion/py3-grpcio-gcp branch 2 times, most recently from 63eaa40 to 261c4ea Compare November 25, 2024 14:36
Python modules may be provided in subdirectories of the upstream
source.

Signed-off-by: dann frazier <[email protected]>
Building from the tarball results in an unusable package because
the included grpc_gcp_pb2.py is generated from an old protobuf.
The tarball includes the .proto file we need to generate a new
one.

Signed-off-by: dann frazier <[email protected]>
@dannf dannf force-pushed the multiversion/py3-grpcio-gcp branch from 261c4ea to be8c0d3 Compare November 25, 2024 15:56
Copy link
Member

@xnox xnox left a comment

Choose a reason for hiding this comment

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

NICE!!!! love the copy template bit.

@dannf dannf merged commit 0e13d1f into wolfi-dev:main Nov 25, 2024
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved-to-run A repo member has approved this external contribution bincapz/pass bincapz/pass Bincapz (aka. malcontent) scan didn't detect any CRITICALs on the scanned packages.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants