Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Log enhancements #199

Merged
merged 5 commits into from
Nov 14, 2017
Merged

Log enhancements #199

merged 5 commits into from
Nov 14, 2017

Conversation

ncdc
Copy link
Contributor

@ncdc ncdc commented Nov 14, 2017

  • Log when starting a backup in the ark server log (we were previously only logging completion)
  • Log when starting a restore in the ark server log
  • Log when a restore has completed in the ark server log
  • Add error and log location hooks to the backup and restore loggers
  • Make logSource more concise by stripping off the leading .../github.com/heptio/ark/

@ncdc ncdc added this to the v0.6.0 milestone Nov 14, 2017
@ncdc ncdc assigned nrb and skriss Nov 14, 2017
ncdc added 4 commits November 14, 2017 14:19
Signed-off-by: Andy Goldstein <andy.goldstein@gmail.com>
Signed-off-by: Andy Goldstein <andy.goldstein@gmail.com>
Signed-off-by: Andy Goldstein <andy.goldstein@gmail.com>
Strip off leading .../github.com/heptio/ark/

Signed-off-by: Andy Goldstein <andy.goldstein@gmail.com>
Signed-off-by: Andy Goldstein <andy.goldstein@gmail.com>
@skriss
Copy link
Contributor

skriss commented Nov 14, 2017

LGTM, merge on travis

@skriss skriss merged commit 01e9c86 into vmware-tanzu:master Nov 14, 2017
@ncdc ncdc deleted the log-enhancements branch December 1, 2017 16:41
shubham-pampattiwar pushed a commit to shubham-pampattiwar/velero that referenced this pull request Sep 26, 2022
Co-authored-by: Emily McMullan <emcmulla@redhat.com>
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