Closed ringods closed 5 years ago
I like the idea of passing an outside datasource for configuration. That way its up to the user to choose which configuration options are set and how they are structured.
I do think we should include a default configuration to get the user up-and-running.
This ticket is a Request For Comments to discuss the future evolution of this module given the set of CLI arguments or environment variables in upstream Concourse will still evolve quite a bit.
Below is the full list of arguments that
concourse web
supports:Before adding specific support for other sets of options, I would like to discuss whether or not it would be better to pass these options in a more general way.
The configuration for PostgreSQL and TSA are required settings for any Concourse deployment. The configuration of these 2 sets of arguments could remain as is. The current state of the module only supports a subset of the remaining options:
Suppose we would only pass a single
args
parameter, then the templating to combine all the arguments would be outside of this module and specific to the use case. Given the set of options will evolve in the future, this approach will require less maintenance on this module.The current case I'm working on requires: