-
Notifications
You must be signed in to change notification settings - Fork 4.7k
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
rsync: output warnings to stdout instead of using glog #5531
Conversation
@fabianofranz ptal |
or @bparees ptal |
lgtm [merge] |
[Test]ing while waiting on the merge queue |
LGTM |
@smarterclayton or @liggitt ptal |
@@ -26,7 +26,7 @@ func (ss copyStrategies) Copy(source, destination *pathSpec, out, errOut io.Writ | |||
err = s.Copy(source, destination, out, errBuf) | |||
if _, isSetupError := err.(strategySetupError); isSetupError { | |||
glog.V(4).Infof("Error output:\n%s", errBuf.String()) | |||
glog.Warningf("Cannot use %s: %v", s.String(), err.Error()) | |||
fmt.Fprintf(out, "WARNING: cannot use %s: %v", s.String(), err.Error()) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should be to stderr - is that the case?
@smarterclayton thx, comments addressed |
LGTM [merge] |
continuous-integration/openshift-jenkins/merge SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pull_requests_origin/6677/) (Image: devenv-rhel7_2631) |
Evaluated for origin merge up to 2355304 |
Evaluated for origin test up to 2355304 |
continuous-integration/openshift-jenkins/test SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pull_requests_origin/6677/) |
Removes the use of glog for warning output