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

            jsick Jonathan Sick created issue -
            jsick Jonathan Sick made changes -
            Field Original Value New Value
            Epic Link DM-1139 [ 13948 ]
            jsick Jonathan Sick made changes -
            Link This issue relates to DM-4950 [ DM-4950 ]
            jsick Jonathan Sick made changes -
            Link This issue relates to DM-5194 [ DM-5194 ]
            jsick Jonathan Sick made changes -
            Epic Link DM-1139 [ 13948 ] DM-5404 [ 23211 ]
            jsick Jonathan Sick made changes -
            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 AWS auth flow works with environment variables, config files, and profiles.
            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.
            jsick Jonathan Sick made changes -
            Status To Do [ 10001 ] In Progress [ 3 ]
            jsick Jonathan Sick made changes -
            Story Points 1
            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.
            jsick Jonathan Sick made changes -
            Reviewers J Matt Peterson [ jmatt ]
            Status In Progress [ 3 ] In Review [ 10004 ]
            jmatt J Matt Peterson [X] (Inactive) made changes -
            Status In Review [ 10004 ] In Review [ 10004 ]
            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.
            jmatt J Matt Peterson [X] (Inactive) made changes -
            Status In Review [ 10004 ] Reviewed [ 10101 ]
            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] !
            jsick Jonathan Sick made changes -
            Resolution Done [ 10000 ]
            Status Reviewed [ 10101 ] Done [ 10002 ]

              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