Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Unification + expansion of the environment variables #68

Closed
peb-adr opened this issue Jul 5, 2023 · 1 comment
Closed

Unification + expansion of the environment variables #68

peb-adr opened this issue Jul 5, 2023 · 1 comment
Assignees
Labels
Milestone

Comments

@peb-adr
Copy link
Member

peb-adr commented Jul 5, 2023

Please make the following changes to the environment variables:

- remove:
  MEDIA_PRESENTER_HOST
  MEDIA_PRESENTER_PORT
  MEDIA_DATABASE_HOST
  MEDIA_DATABASE_NAME
  MEDIA_DATABASE_PASSWORD_FILE
  MEDIA_DATABASE_PORT
  MEDIA_DATABASE_USER
  • add:
  PRESENTER_HOST
  PRESENTER_PORT
  DATABASE_HOST
  DATABASE_NAME
  DATABASE_PASSWORD_FILE
  DATABASE_PORT
  DATABASE_USER

The env vars for the DB connection should be renamed.

Instead of having MEDIA_PRESENTER_HOST, PRESENTER_HOST should be set
for the media service overriding the value from the default env (analogous for
the port).

More details on these changes are in manage-service issue OpenSlides/openslides-manage-service#224.
Also please provide non-service-specific feedback on these related issues in the
manage-service issue.

@peb-adr peb-adr added this to the 4.1 milestone Jul 5, 2023
@jsangmeister jsangmeister self-assigned this Jul 6, 2023
@jsangmeister
Copy link
Contributor

duplicate of #67 - also, PRESENTER_{HOST,PORT} are already named that way without the prefix.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants