Skip to content

Set restore status on all cluster nodes#274

Merged
snh merged 2 commits into
masterfrom
snh/cluster-restore-status
Dec 12, 2016
Merged

Set restore status on all cluster nodes#274
snh merged 2 commits into
masterfrom
snh/cluster-restore-status

Conversation

@snh

@snh snh commented Nov 20, 2016

Copy link
Copy Markdown
Member

When restoring to a cluster, the restore status file,/data/user/common/ghe-restore-status, which is used by the management console to determine the status of a restore, is currently only written to one node.

This PR modifies the update_restore_status function so that /data/user/common/ghe-restore-status is updated on all online cluster nodes.

/cc @github/backup-utils

@snh snh added the bug label Nov 20, 2016
@snh snh requested a review from rubiojr December 8, 2016 04:55

@rubiojr rubiojr left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@snh snh merged commit d36a2b5 into master Dec 12, 2016
@snh snh deleted the snh/cluster-restore-status branch December 12, 2016 23:27
snh added a commit that referenced this pull request Dec 21, 2016
Bug fixes.

* Set restore status on all cluster nodes #274
* Fix pages backups and restores in GitHub Enterprise 11.10 #275
@snh snh mentioned this pull request Dec 21, 2016
jeluhu pushed a commit that referenced this pull request Jun 12, 2023
@jeluhu jeluhu mentioned this pull request Jun 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants