Skip to content
This repository has been archived by the owner on Apr 3, 2024. It is now read-only.

Add detach option #139

Closed
lorensr opened this issue Sep 23, 2022 · 2 comments
Closed

Add detach option #139

lorensr opened this issue Sep 23, 2022 · 2 comments
Labels
enhancement New feature or request

Comments

@lorensr
Copy link

lorensr commented Sep 23, 2022

Is your feature request related to a problem? Please describe.

In order to not see output, I have to add > /dev/null 2>&1 & like:

temporalite start --namespace default --ephemeral > /dev/null 2>&1 &

Describe the solution you'd like

Something like docker-compose's -d flag:

  -d, --detach                    Detached mode: Run containers in the background
@lorensr lorensr added the enhancement New feature or request label Sep 23, 2022
@cretz
Copy link
Member

cretz commented Sep 23, 2022

As with our production Temporal server and most binaries out there, we should leave stdio redirection and backgrounding to the caller IMO. Docker compose's option is not a common one among binaries like this.

@jlegrone
Copy link
Collaborator

I agree with @cretz on this one; docker-compose also ships a bunch of subcommands like ls which are used for enumerating and managing those background processes. I'd prefer to avoid this complexity in Temporalite.

Closing for now, but I'm happy to reconsider if this ends up being a really popular community request.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants