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

Clarify environment variable usage when running in Docker #6181

Merged
merged 1 commit into from
Oct 30, 2019
Merged

Clarify environment variable usage when running in Docker #6181

merged 1 commit into from
Oct 30, 2019

Conversation

antonok-edm
Copy link
Contributor

This error message was quite confusing when setting up a Synapse server for the first time:

SYNAPSE_SERVER_NAME and SYNAPSE_CONFIG_PATH are mutually exclusive except in generate or migrate_config mode.

As someone who has had not used Synapse before, I wasn't sure how to resolve the error, and I ended up unsetting SYNAPSE_CONFIG_PATH. I figured this was safe because my config file was in the default location.

The server started working, but it was not until much later that I realized the config was not being used at all.

I've reworked the message to the following:

SYNAPSE_SERVER_NAME can only be combined with SYNAPSE_CONFIG_PATH in generate or migrate_config mode. To start synapse using a config file, unset the SYNAPSE_SERVER_NAME environment variable.

This should make it more clear as to which variable is the preferred method of configuration.

Copy link
Member

@richvdh richvdh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks, that is indeed clearer. Please could you sign off your contribution as per https://github.com/matrix-org/synapse/blob/master/CONTRIBUTING.rst#sign-off?

@richvdh
Copy link
Member

richvdh commented Oct 29, 2019

@antonok-edm : I'm afraid we can't use this without a sign-off. please could you add one?

Let us know if the process isn't clear.

@antonok-edm
Copy link
Contributor Author

I'd prefer not to leave my email laying around, but feel free to either use this or claim the contribution as your own:

Signed-off-by: Anton Lazarev https://antonok.com

@richvdh
Copy link
Member

richvdh commented Oct 30, 2019

That's fine, we don't need an email address. Thanks!

@richvdh richvdh merged commit 213d7eb into matrix-org:master Oct 30, 2019
@antonok-edm antonok-edm deleted the clarify-docker-env-vars branch November 1, 2019 05:19
babolivier pushed a commit that referenced this pull request Sep 1, 2021
* commit '213d7eb22':
  Clarify environment variable usage when running in Docker (#6181)
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