-
Notifications
You must be signed in to change notification settings - Fork 17
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
Centralize dynamic config #529
Comments
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
We have some hardcoded dynamic config values for embedded server here: https://github.com/temporalio/features/blob/main/cmd/run.go#L208
As well as a file for the docker way of running things here:
https://github.com/temporalio/features/blob/main/dockerfiles/dynamicconfig/docker.yaml
We should use one file, I suppose keep the yaml format, so that we always use the same options.
Another motivation for this is that the server folks would like to use it so they know what options should apply to server when they're running it in a full infra environment for nightly pipelines.
The text was updated successfully, but these errors were encountered: