-
Notifications
You must be signed in to change notification settings - Fork 66
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
Add dependencies for Salvo aiming to serve Envoy ecosystem #174
Comments
cc @mum4k |
Thank you @gyohuangxin for starting this discussion. I would also add that Salvo can and likely will develop its own Docker image long-term as the need for more packages / tooling grows. Adding these packages into this existing image now would help us focus our available cycles on developing the main features of Salvo and postpone the investment into our docker image to a time when we have more contributors working on Salvo (eta ~6 months). |
It would be useful to know what impact those dependencies have on the docker image size |
@mum4k @keith Thanks for the useful information and comments. If we add above packages, the uncompressed docker image size will be 7.43GB, the origin
Then for the compressed docker image uploaded to docker hub, the image size with above packages will be 2.19 GB, the |
@gyohuangxin why would you need |
@lizan Thanks for the useful information, double-checked, the |
@gyohuangxin Seems Salvo is using Bazel as well, according to https://bazel.build/docs/bazel-and-java if you use |
Title: Add dependencies for Salvo aiming to serve Envoy ecosystem
Description:
Salvo project is the Envoy benchmarking framework aiming to serve Envoy ecosystem with capable of:
In order to integrate well with Envoy CI system, we are migrating Salvo's CI workflow from CircleCI to AZP. Aligning with Envoy's and Nighthawk's CI workflow, we are also using script
run_envoy_docker.sh
to run theenvoyproxy/envoy-build-ubuntu
container to run Salvo's CI workflow.We found below packages are required when running Salvo in
envoyproxy/envoy-build-ubuntu
container:At this stage, we are only focusing on x86_64 arch and ubuntu OS, can we add above packages in
envoyproxy/envoy-build-ubuntu
Ubuntu x86_64 image?Relevant Links:
OSS benchmarking of Envoy Requirements: https://docs.google.com/document/d/1mAma-ksRN0OIBInoZKUdjdaIhK2nTQxFnCujq2fKi4E/edit
The text was updated successfully, but these errors were encountered: