Depends on D4858
I added curl to the base image since it is needed to install rust.
The reason I didn't add rust right away to the base image is that we'll be using rust in the backup and tunnelbroker which makes it redundant in the blob service.
Details
Details
- Reviewers
tomek • jon - Commits
- rCOMM14efcd6e0c83: [services] Add curl to the base image
services build
Diff Detail
Diff Detail
- Repository
- rCOMM Comm
- Lint
No Lint Coverage - Unit
No Test Coverage
Event Timeline
Comment Actions
Nice, so we can move Rust install from the backup / Tunnelbroker images since we can just do it in the base image now, right?
Comment Actions
As I said in the description
The reason I didn't add rust right away to the base image is that we'll be using rust in the backup and tunnelbroker which makes it redundant in the blob service.
So if we're okay with having rust in blob even though we do not use it there then I can move it (rust installation) up to the base image.
Comment Actions
Ah, I see... we're planning to continue using gRPC C++ Reactors in blob for now, since it doesn't have the same problems. I think it would be good to eventually move that one over as well, but it makes sense to sequence it later