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.

Sign up for GitHub

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

Jump to bottom

feat(security): Remove Vault dependency on Consul by using file backend #2886

Merged
merged 2 commits into from
Dec 10, 2020
Merged

feat(security): Remove Vault dependency on Consul by using file backend #2886

merged 2 commits into from
Dec 10, 2020

Commits on Dec 10, 2020

  1. feat(security): Remove Vault dependency on Consul by using file backe… 

    …nd instead of consul

 Change the Vault backend storage configuration from Consul to Filesystem so that we can remove the Vault dependency on Consul.

 The docker-compose file will also need to change to remove the -consul dependency from repo developer-script.

 The current edgex-consul healthy check on Vault also need to be remove as that check becomes superfluous.

 Closes: edgexfoundry#2882

Signed-off-by: Jim Wang <yutsung.jim.wang@intel.com>
    @jim-wang-intel
    jim-wang-intel committed Dec 10, 2020
    Configuration menu
    Copy the full SHA
    560fc2a View commit details
    Browse the repository at this point in the history

  2. feat(security): Address PR feedback on /vault/file 

    Resolve conflicts with master

Signed-off-by: Jim Wang <yutsung.jim.wang@intel.com>
    @jim-wang-intel
    jim-wang-intel committed Dec 10, 2020
    Configuration menu
    Copy the full SHA
    565a8fe View commit details
    Browse the repository at this point in the history