Skip to content
This repository was archived by the owner on Feb 5, 2020. It is now read-only.

Only one command needs to have backup.db owned by core user #17

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

cehoffman
Copy link
Contributor

The less needed to do during a recovery better and I think leaving the etcd backup readable by core is bad. Arguably you don't even need it in the one case it is used if you are ok with all the recovery files being owned by root instead of core.

Personally I do the recover command as root because I find it less error prone to start the recovery by first running sudo su - and running each step as root and dropping sudo all together. The last thing you want is a child process or a piped command to fail because sudo wasn't tacked on.

@zbwright zbwright requested a review from aaronlevy September 12, 2017 21:47
@joshix joshix self-requested a review October 5, 2017 05:24
@cehoffman
Copy link
Contributor Author

Our process revolved around sudo su initially before any session because it was preferable to failing pipe commands.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant