-
Notifications
You must be signed in to change notification settings - Fork 137
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
state: Refactor Task IP sourcing #267
Conversation
/cc @jdef |
6710116
to
77da914
Compare
@@ -1,11 +1,13 @@ | |||
package state | |||
package state_test |
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.
what is the advantage of doing this?
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.
Privacy enforcement.
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.
Package state_test
has no access to private identifiers of package state
.
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.
that's pretty interesting. i'm wondering if I can use this in the k8s-mesos "km" main package since it looks like the _test.go stuff in package main is bleeding into the generated non-test executable.
very elegant. some concerns about aggregation of IPs. |
This commit refactors the Task IP sourcing code to gracefully account for the new `NetworkInfo` source with the same structure and be easily changed to distinguish between IPv4 and IPv6.
77da914
to
6e0b1ae
Compare
This commit refactors the Task IP sourcing code to gracefully account
for the new
NetworkInfo
source with the same structure and be easilychanged to distinguish between IPv4 and IPv6.