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.

Sign up for GitHub

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

Jump to bottom

fix(packages): dolos reorder steps to resolve start issue #238

Merged

wolf31o2 merged 1 commit into main from fix/packages-dolos-start

May 25, 2024

Member

wolf31o2 commented May 25, 2024

This resolves the issue with Dolos starting up.

Write config file before starting container
Remove static host port mapping to allow multiple instances
Remove redundant dolos- prefix from output names
Restore port lookup to get local port, requires fix: map ports as exposed ports always #237 and reverts fix(packages): dolos outputs with static ports #226

Closes #227


          fix(packages): dolos reorder steps to resolve start issue

a509965

Signed-off-by: Chris Gianelloni <wolf31o2@blinklabs.io>

wolf31o2 requested review from agaffney and verbotenj as code owners

May 25, 2024 17:15

agaffney approved these changes

View reviewed changes

wolf31o2 merged commit ee0cb0b into main

9 checks passed

wolf31o2 deleted the fix/packages-dolos-start branch

May 25, 2024 17:51

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet