Skip to content
This repository has been archived by the owner on Jul 27, 2023. It is now read-only.

Lock down vault configuration file #1494

Merged
merged 1 commit into from
Jun 7, 2016

Conversation

langston-barrett
Copy link
Contributor

@langston-barrett langston-barrett commented Jun 1, 2016

  • Installs cleanly on a fresh build of most recent master branch
  • Upgrades cleanly from the most recent release
  • Updates documentation relevant to the changes

Fixes #1493

@langston-barrett
Copy link
Contributor Author

The CI build is failing unrelatedly, this will need manual testing.

@langston-barrett
Copy link
Contributor Author

This works for me on digitalocean:

$ ansible role=control --become -a 'ls -alh /etc/vault/vault.hcl'
lb-mi-control-01 | success | rc=0 >>
-rw------- 1 root root 334 Jun  1 17:32 /etc/vault/vault.hcl

lb-mi-control-02 | success | rc=0 >>
-rw------- 1 root root 334 Jun  1 17:32 /etc/vault/vault.hcl

lb-mi-control-03 | success | rc=0 >>
-rw------- 1 root root 334 Jun  1 17:32 /etc/vault/vault.hcl

deployed, and everything's healthy!

@langston-barrett langston-barrett force-pushed the fix/vault-config-permissions branch from 187dca8 to fed41d1 Compare June 6, 2016 09:31
@langston-barrett
Copy link
Contributor Author

rebased for automated testing on other clouds

@ChrisAubuchon
Copy link
Contributor

👍

@langston-barrett langston-barrett merged commit bd3a82a into master Jun 7, 2016
@langston-barrett langston-barrett deleted the fix/vault-config-permissions branch June 7, 2016 15:30
@ryane ryane modified the milestone: 1.2 Jun 9, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants