Skip to content
This repository has been archived by the owner on Aug 9, 2022. It is now read-only.

Update thaler-testnet.md #223

Closed
wants to merge 1 commit into from
Closed

Conversation

Alive29
Copy link

@Alive29 Alive29 commented Aug 19, 2020

add systemd service

@tomtau
Copy link
Contributor

tomtau commented Aug 20, 2020

@Alive29 thank you for the contribution!
Currently, this tutorial page shows a warning saying:

This page only shows the minimal setup.

You may want to (...) want to execute the processes using supervisor or equivalent etc.

so for executing the processes using systemd (or other supervisor), it'd be better to put them under the "Notes on Production Deployment" page instead (and put a link to that in "Thaler Testnet" page).

The systemd services may also need a few small improvements (~> no need for some of the environment variables; as the services are interdependent, it may not be good to individually restart them etc.)

I'll close this PR for now and I opened this issue to track this: crypto-org-chain/chain-docs#224 (feel free to reopen a revised PR addressing that issue though!)

@tomtau tomtau closed this Aug 20, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants