Skip to content

Output backup utils version on backup and restore#384

Merged
djdefi merged 1 commit into
masterfrom
logutilsversion
Mar 29, 2018
Merged

Output backup utils version on backup and restore#384
djdefi merged 1 commit into
masterfrom
logutilsversion

Conversation

@djdefi

@djdefi djdefi commented Mar 28, 2018

Copy link
Copy Markdown
Member

While we output / log the GitHub Enterprise version, the version of backup-utils is not present in the ghe-backup and ghe-restore output.

This PR adds the backup-utils version to the starting backup / restore messages:

After:

$ bin/ghe-backup 
Starting backup of hostname with backup-utils v2.13.0 in snapshot 20180328T104804
Connect hostname:122 OK (v2.13.0)
Backing up GitHub settings ...
...

Before:

$ bin/ghe-backup 
Starting backup of hostname in snapshot 20180328T105119
Connect hostname:122 OK (v2.13.0)
Backing up GitHub settings ...
...

This will make it easier to determine the backup-utils release when we only have the backup or restore output.

cc: @github/enterprise-support

@djdefi djdefi self-assigned this Mar 28, 2018
@djdefi djdefi requested a review from a team March 28, 2018 17:58

@lildude lildude 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

@djdefi djdefi merged commit 5c5ecc4 into master Mar 29, 2018
@djdefi djdefi deleted the logutilsversion branch March 29, 2018 16:31
@lildude lildude mentioned this pull request May 9, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants