Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

rubiojr
Copy link
Member

@rubiojr rubiojr commented May 9, 2016

Wrong variable name and re-creating the wrong remote directory.

/cc @github/backup-utils

@rubiojr
Copy link
Member Author

rubiojr commented May 9, 2016

Only affects restores to a GitHub Enterprise 2.6 cluster (using the new Pre-receive hooks feature) if (and only if) the data being restored is from a 2.6 VM or cluster.

@rubiojr
Copy link
Member Author

rubiojr commented May 9, 2016

/cc #201

@lildude
Copy link
Member

lildude commented May 9, 2016

LGTM and makes sense too. 👍

The $GHE_DATA_DIR/$GHE_RESTORE_SNAPSHOT/git-hooks path is otherwise
recreated in the remote server and the git-hooks data is transfered to
the wrong directory:

```
root@ghe-test-dgit-fs3:/data/user/git-hooks# find
.
./data
./data/2.6-storage
./data/2.6-storage/current
./data/2.6-storage/current/git-hooks
./data/2.6-storage/current/git-hooks/environments
./data/2.6-storage/current/git-hooks/environments/1
./data/2.6-storage/current/git-hooks/environments/1/0
./data/2.6-storage/current/git-hooks/repos
```
@rubiojr rubiojr merged commit e486e67 into master May 9, 2016
@rubiojr rubiojr deleted the fix-git-hook-restores branch May 9, 2016 19:05
@rubiojr rubiojr mentioned this pull request May 31, 2016
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