server Docker container should store config and message store in volume to support upgrades #152

Nova issue

dan · 2018-11-01T03:09:16Z

dan comentou

2018-11-01 03:09:16 +00:00

Proprietário

https://stackoverflow.com/questions/26734402/how-to-upgrade-docker-container-after-its-image-changed

GBHC comentou

2018-11-01 17:35:54 +00:00

Yes, apologies.

I didn't want to mess with the golang code itself for t'other ticket.

The following PR resolves both issues; #153

Yes, apologies. I didn't want to mess with the golang code itself for t'other ticket. The following PR resolves both issues; https://git.openprivacy.ca/cwtch.im/cwtch/pulls/153

GBHC comentou

2018-11-01 18:07:04 +00:00

For reference;

# Stop your container
sudo docker stop container1

# Pull the new image
sudo docker pull cwtch:latest

# Create a new container and copy across the volumes
sudo docker create --volumes-from container1 -e CWTCH_CONFIG_DIR=/etc/cwtch/ cwtch:latest

# Start new container
sudo docker start FOO

You should see that the new container (foo) retains the config (hidden service address etc) of the old container (container1).

For reference; ``` # Stop your container sudo docker stop container1 # Pull the new image sudo docker pull cwtch:latest # Create a new container and copy across the volumes sudo docker create --volumes-from container1 -e CWTCH_CONFIG_DIR=/etc/cwtch/ cwtch:latest # Start new container sudo docker start FOO ``` You should see that the new container (foo) retains the config (hidden service address etc) of the old container (container1).

dan comentou

2018-11-01 18:11:54 +00:00

Autor

Proprietário

any reason no to have the DOCKERFILE set the env variable of CWTCH_CONFIG_DIR to keep it even simpler to start? I can add this, I'm just adding the message store and metrics to the config dir too

GBHC comentou

2018-11-01 18:32:58 +00:00

None at all.

An if check in docker-entrypoint would be ideal.

if [ -z "${CWTCH_CONFIG_DIR}" ]; then
CWTCH_CONFIG_DIR=/etc/cwtch/
fi

None at all. An if check in docker-entrypoint would be ideal. ``` if [ -z "${CWTCH_CONFIG_DIR}" ]; then CWTCH_CONFIG_DIR=/etc/cwtch/ fi ```

sarah citou esta issue em um commit

2018-11-02 23:44:17 +00:00

Changes to resolve ticket #152

dan comentou

2018-11-21 21:03:22 +00:00

Autor

Proprietário

done, works now, with #154

dan fechou esta issue

2018-11-21 21:03:38 +00:00

Acesse para participar desta conversação.

Sem etiqueta

ready-for-implementation

Sem marco

Sem responsável

2 participante(s)

Notificações

Data limite

A data limite é inválida ou está fora do intervalo. Por favor, use o formato 'dd/mm/aaaa'.

Data limite não informada.

Dependências

Nenhuma dependência definida.

Referência: cwtch.im/cwtch#152