Skip to content

Conversation

howardjohn
Copy link
Member

For https://github.com/istio/ztunnel.

Fixes istio/ztunnel#4

This adds the rust compiler to the build tools. This brings the image from 3.5gb -> 4gb.

There were some other changes to the build order of trivy to improve the image size in general.

The alternative would be to make a new build-tools-rust adding this. However, since its only 500mb I didn't feel it was worthwhile; the proxy one is 8gb (and isn't even a superset) so it is a massive difference, for this case the overhead doesn't seem useful.

@howardjohn howardjohn requested a review from a team as a code owner October 20, 2022 15:22
@istio-testing istio-testing added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Oct 20, 2022
For https://github.com/istio/ztunnel.

Fixes istio/ztunnel#4

This adds the rust compiler to the build tools. This brings the image
from 3.5gb -> 4gb.

There were some other changes to the build order of trivy to improve the
image size in general.

The alternative would be to make a new build-tools-rust adding this.
However, since its only 500mb I didn't feel it was worthwhile; the proxy
one is 8gb (and isn't even a superset) so it is a massive difference,
for this case the overhead doesn't seem useful.
Copy link
Contributor

@ericvn ericvn left a comment

Choose a reason for hiding this comment

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

LGTM I think it's best to have a single build image for both go and rust, otherwise we will need to do some additional logic to make sure we are calling out the correct one in Makefiles, etc. Also allows one to work on both in a single make shell.

@howardjohn
Copy link
Member Author

/retest

@istio-testing istio-testing merged commit 370c331 into istio:master Oct 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size/M Denotes a PR that changes 30-99 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Either add rust support to build-tools, build-tools-proxy, or make a build-tools-rust
3 participants