Skip to content

Revert "Remove hookshot backup/restore"#136

Merged
lildude merged 1 commit intomasterfrom
revert-134-no-hookshot-backup-restore
Aug 6, 2015
Merged

Revert "Remove hookshot backup/restore"#136
lildude merged 1 commit intomasterfrom
revert-134-no-hookshot-backup-restore

Conversation

@lildude
Copy link
Copy Markdown
Member

@lildude lildude commented Aug 4, 2015

Reverts #134

Following discussions regarding keeping a backup of the hookshot logs, it has been decided to revert this change as the number of logs will be significantly fewer on later releases of GitHub Enterprise.

This does however reintroduce the severe delay when migrating from GitHub Enterprise 11.10.3xx to 2.x as GitHub Enterprise 11.10.3xx doesn't prune the log files like 2.2 does.

@lildude
Copy link
Copy Markdown
Member Author

lildude commented Aug 4, 2015

/cc @tmm1

@lildude
Copy link
Copy Markdown
Member Author

lildude commented Aug 5, 2015

On a side note, I plan to raise a new PR today which limits the number of hookshot logs we backup which will resolve the severe delay 11.10.3xx users will see when migrating. This will be in line with the number of logs GitHub Enterprise 2.2 retains.

@lildude
Copy link
Copy Markdown
Member Author

lildude commented Aug 5, 2015

Looking into this further and we've never backed up the hookshot logs on 11.10.3xx instances so hookshot logs won't be affecting migrations from 11.10.34x to 2.x.

@pengwynn
Copy link
Copy Markdown

pengwynn commented Aug 5, 2015

/fyi @github/platform

@mcolyer
Copy link
Copy Markdown

mcolyer commented Aug 6, 2015

👍 to reverting this.

@lildude
Copy link
Copy Markdown
Member Author

lildude commented Aug 6, 2015

Thanks @mcolyer. Merging.

lildude added a commit that referenced this pull request Aug 6, 2015
…tore

Revert "Remove hookshot backup/restore"
@lildude lildude merged commit da295de into master Aug 6, 2015
@lildude lildude deleted the revert-134-no-hookshot-backup-restore branch August 6, 2015 18:17
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.

3 participants