Skip to content

Include "linux_extra" python artifacts in regular linux build#27279

Merged
jtattermusch merged 1 commit intogrpc:masterfrom
jtattermusch:linux_extra
Sep 9, 2021
Merged

Include "linux_extra" python artifacts in regular linux build#27279
jtattermusch merged 1 commit intogrpc:masterfrom
jtattermusch:linux_extra

Conversation

@jtattermusch
Copy link
Copy Markdown
Contributor

The "linux_extra" artifacts used to be build in a fully emulated environment (which made for a terribly slow build), but now they are being crosscompiled, so their build time is comparable to other python wheels.

If we add linux_extra artifacts back to the "linux" artifacts, it will simplify the release process (avoiding the need to run a separate "linux_extra" kokoro job).

(We still need to check if adding more python artifacts will fit into the build_artifact job timeout, since we're already building a large number of linux wheels there).

@jtattermusch jtattermusch requested a review from gnossen September 8, 2021 07:19
@jtattermusch jtattermusch added release notes: no Indicates if PR should not be in release notes lang/Python labels Sep 8, 2021
@jtattermusch
Copy link
Copy Markdown
Contributor Author

linux build_artifact duration is now ~2h 18min (https://source.cloud.google.com/results/invocations/7df0dff0-289a-4a9f-8222-3532b97d8035/targets), on master the duration is currently between 105 and 126min (https://screenshot.googleplex.com/AQjfMdCvA7gTmPV)

The job timeout is currently 180min:

So this still feels like safe to merge.

The python windows build failure (https://source.cloud.google.com/results/invocations/20ce277a-57bb-4f92-b954-78a8be28b953) is due to prepare build step failing, which is unrelated to this PR.

@jtattermusch jtattermusch merged commit e11dcbc into grpc:master Sep 9, 2021
ashithasantosh pushed a commit to ashithasantosh/grpc that referenced this pull request Sep 10, 2021
lidizheng pushed a commit to lidizheng/grpc that referenced this pull request Sep 23, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lang/Python release notes: no Indicates if PR should not be in release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants