-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
UUID package improvements #792
UUID package improvements #792
Conversation
Fixes distribution#782 Signed-off-by: Darren Shepherd <darren@rancher.com>
Signed-off-by: Stephen J Day <stephen.day@docker.com>
@@ -30,7 +29,7 @@ var ( | |||
|
|||
// Loggerf can be used to override the default logging destination. Such | |||
// log messages in this library should be logged at warning or higher. | |||
Loggerf = log.Printf | |||
Loggerf func(format string, args ...interface{}) = func(format string, args ...interface{}) {} |
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.
no need for type, it will infer from value
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.
fixed.
Do we need a check on whether Loggerf is nil in Generate, before it is used? |
@aaronlehmann No, because it is initialized. |
Signed-off-by: Stephen J Day <stephen.day@docker.com>
7159de6
to
2c9ab4f
Compare
LGTM |
Ah, missed the fact that Loggerf is initialized. LGTM. |
@tiborvass Are you going to update in docker/docker? |
To avoid errant log messages, we've disabled logging by default when uuid is having trouble accessing
/dev/urandom
. This also takes the work from #786 and incorporates the feedback.This carries #784 and #786.
cc @jfrazelle @tiborvass @ibuildthecloud