HomePhabricator
Diffusion Comm 370381a0bf4c

tunnelbroker: Merge tunnelbroker-base and tunnelbroker-server images

Description

tunnelbroker: Merge tunnelbroker-base and tunnelbroker-server images

Summary:
Discussed with karol-bisztyga in chat that the original reason for splitting the images was for layer caching. This resolves the layer caching problem and merges the images.

This also removes the base image from docker-compose.yml, requiring that scripts/build_base_image.sh be run explicitly before docker-compose commands or implicitly using the run and test scripts.

I merged/flattened the directory structures in a way that I like, but this is open to discussion.

Only does this for one service, tunnelbroker, as it is a proof-of-concept in a series of diffs.

Test Plan: Run tunnelbroker service launch script

Reviewers: geekbrother, karol-bisztyga, varun, ashoat

Reviewed By: geekbrother, karol-bisztyga, ashoat

Subscribers: palys-swm, Adrian, atul, benschac

Differential Revision: https://phabricator.ashoat.com/D3234

Details

Provenance
jimAuthored on Mar 4 2022, 12:40 PM
Reviewer
max
Differential Revision
D3234: tunnelbroker: Merge tunnelbroker-base and tunnelbroker-server images
Parents
rCOMM71290bcc073c: [web] [chore] add remaining icons to the project icon component
Branches
Unknown
Tags
Unknown