Skip to content
This repository has been archived by the owner on Jan 21, 2022. It is now read-only.

Use pivotal-golang for localip [#82988760] #8

Merged

Conversation

Amit-PivotalLabs
Copy link

Signed-off-by: Kris Hicks krishicks@gmail.com

Signed-off-by: Kris Hicks <krishicks@gmail.com>
@cfdreddbot
Copy link

Hey Amit-PivotalLabs!

Thanks for submitting this pull request! I'm here to inform the recipients of the pull request that you've already signed the CLA.

@jmtuley
Copy link
Contributor

jmtuley commented Dec 4, 2014

Hey @Amit-PivotalLabs and @krishicks,

We'd really like to accept this PR, but the tests fail consistently on Travis. (Pay no attention to the allowed failure on go-tip.)

Can you take care of that, and then we'll merge this in?

Thanks!
John and @wfernandes

@jmtuley
Copy link
Contributor

jmtuley commented Dec 4, 2014

@Amit-PivotalLabs Never mind; it looks like something changed in a version of testify that came out four hours after the last pass of loggregatorlib. We'll look into the failure.

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling 5efa4f6 on Amit-PivotalLabs:pivotal-golang-localip into 1c010a6 on cloudfoundry:master.

jmtuley added a commit that referenced this pull request Dec 4, 2014
Use pivotal-golang for localip [#82988760]
@jmtuley jmtuley merged commit 702fc9d into cloudfoundry-attic:master Dec 4, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants