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

New Issue

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

dan commented

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

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

GBHC commented

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 commented

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 commented

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

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 commented

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 referenced this issue from a commit

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

Changes to resolve ticket #152

dan commented

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

done, works now, with #154

dan closed this issue

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

Sign in to join this conversation.

No Label

ready-for-implementation

No Milestone

No Assignees

2 Participants

Notifications

Due Date

The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: cwtch.im/cwtch#152