Uploaded image for project: 'Data Management'
  1. Data Management
  2. DM-5291

Docker-ready configuration system for LTD Keeper

    Details

      Description

      To deploy LTD Keeper in a Docker container (DM-5194), it’s best practice to handle all configurations through environment variables. In DM-4950, LTD Keeper was configured through files for test and dev/deployment profiles. What we should do is continue to allow hard-coded configurations for test and dev environments, but have a third fully fledged configuration environment that’s driven entirely by environment variables.

      The environment variables should allow fine grained configuration (for example, to turn off calls to individual external services for testing).

      This should also resolve how to deal with Google Container Engine/Kubernetes auth flow works with environment variables, config files, and profiles.

        Attachments

          Issue Links

            Activity

            Hide
            jsick Jonathan Sick added a comment -

            Thanks for the comments J Matt Peterson [X]!

            Show
            jsick Jonathan Sick added a comment - Thanks for the comments J Matt Peterson [X] !
            Hide
            jmatt J Matt Peterson [X] (Inactive) added a comment -

            Reviewed. Comments on the GitHub pull request.

            Show
            jmatt J Matt Peterson [X] (Inactive) added a comment - Reviewed. Comments on the GitHub pull request.
            Hide
            jsick Jonathan Sick added a comment -

            This is a super simple quick 5-minute review. I just changed how configurations are done towards using configuration classes.

            Show
            jsick Jonathan Sick added a comment - This is a super simple quick 5-minute review. I just changed how configurations are done towards using configuration classes.

              People

              • Assignee:
                jsick Jonathan Sick
                Reporter:
                jsick Jonathan Sick
                Reviewers:
                J Matt Peterson [X] (Inactive)
                Watchers:
                J Matt Peterson [X] (Inactive), Jonathan Sick
              • Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Summary Panel