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.

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

[GH-757] Grab deployment keys from Vault, not secrets #283

Merged
merged 1 commit into from
Dec 14, 2020

Conversation

jack-r-warren
Copy link
Contributor

Purpose

Changes

  • Grabs deployment keys from Vault

Review Instructions

  • The same thing is done in both deployment and the PR tests, so hopefully the test here passes
  • Question: should the dockerhub creedentials used for deployment also be replaced with ones from Vault? That's slightly higher risk
  • After merging this ticket there's a few secrets that we can safely delete, see the ticket for details

@jack-r-warren
Copy link
Contributor Author

Hm, test failing, says the key is invalid format but it isn't when I try it locally--I'm suspecting that the Vault role doesn't have access to read the secret/dsde/gotc/dev/ci/wfl_deploy_keys secret. I've let Rhian know, I think this is probably a simple fix

@jack-r-warren
Copy link
Contributor Author

Hm, test failing, says the key is invalid format but it isn't when I try it locally--I'm suspecting that the Vault role doesn't have access to read the secret/dsde/gotc/dev/ci/wfl_deploy_keys secret. I've let Rhian know, I think this is probably a simple fix

Debugged with Rhian and he realized that some existing keys weren't formatted properly, so I used fold to add newlines and they're all good now. The unused dsde-pipelines key in Vault was formatted even weirder, it had a newline in it, but I don't want to invalidate it or something by messing it up so I just left it.

Copy link
Contributor

@rexwangcc rexwangcc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks!

@jack-r-warren jack-r-warren merged commit 7f9a85a into main Dec 14, 2020
@jack-r-warren jack-r-warren deleted the jack-r-warren/GH-757-use-vault-for-keys branch December 14, 2020 15:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants