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

implement reading from loggregator v2 api #213

Merged
merged 1 commit into from
Mar 25, 2019

Conversation

MasslessParticle
Copy link
Contributor

Signed-off-by: Travis Patterson tpatterson@pivotal.io

Signed-off-by: Travis Patterson <tpatterson@pivotal.io>
@shinji62
Copy link
Contributor

Thanks for the Pr.

@jtuchscherer
Copy link

It is probably worth noting that this requires the RLP gateway to be deployed, which has been included in cf-deployment 4.5 (https://github.com/cloudfoundry/cf-deployment/releases/tag/v4.5.0)

@jtuchscherer
Copy link

@shinji62 Do you need something else to be able to pull in this PR? We would be happy to help.

@shinji62
Copy link
Contributor

@jtuchscherer No it's fine. I would try to merge this week.

Sorry for the delay I am a bit busy now.

@jtuchscherer
Copy link

No rush. Let us know if you need any help or support.

@shinji62 shinji62 merged commit 0324cba into cloudfoundry-community:develop Mar 25, 2019
@shinji62
Copy link
Contributor

@jtuchscherer
Test are failing on my CI

Failed to compile firehoseclient:

firehoseclient/firehoseclient.go:4:2: cannot find package "code.cloudfoundry.org/go-loggregator" in any of:
	/tmp/build/0d1a4794/gopath/src/github.com/cloudfoundry-community/firehose-to-syslog/vendor/code.cloudfoundry.org/go-loggregator (vendor tree)
	/usr/local/go/src/code.cloudfoundry.org/go-loggregator (from $GOROOT)
	/tmp/build/0d1a4794/gopath/src/code.cloudfoundry.org/go-loggregator (from $GOPATH)
firehoseclient/v2adapter.go:5:2: cannot find package "code.cloudfoundry.org/go-loggregator/conversion" in any of:
	/tmp/build/0d1a4794/gopath/src/github.com/cloudfoundry-community/firehose-to-syslog/vendor/code.cloudfoundry.org/go-loggregator/conversion (vendor tree)
	/usr/local/go/src/code.cloudfoundry.org/go-loggregator/conversion (from $GOROOT)
	/tmp/build/0d1a4794/gopath/src/code.cloudfoundry.org/go-loggregator/conversion (from $GOPATH)
firehoseclient/v2adapter.go:6:2: cannot find package "code.cloudfoundry.org/go-loggregator/rpc/loggregator_v2" in any of:
	/tmp/build/0d1a4794/gopath/src/github.com/cloudfoundry-community/firehose-to-syslog/vendor/code.cloudfoundry.org/go-loggregator/rpc/loggregator_v2 (vendor tree)
	/usr/local/go/src/code.cloudfoundry.org/go-loggregator/rpc/loggregator_v2 (from $GOROOT)
	/tmp/build/0d1a4794/gopath/src/code.cloudfoundry.org/go-loggregator/rpc/loggregator_v2 (from $GOPATH)

Ginkgo ran 7 suites in 39.766801451s
Test Suite Failed
make: *** [Makefile:6: test] Error 1

I guess some vendored dependency are missing.

@jtuchscherer
Copy link

@shinji62 I created this bug to have the team look into this: https://www.pivotaltracker.com/story/show/164885375
cc/ @MasslessParticle

@johncornish
Copy link
Contributor

johncornish commented Mar 29, 2019

@shinji62

I tried a fresh clone of our work and didn't get the error you're seeing, but I did get some strange type errors. I ran GO111MODULE=on go test ./... and it worked--possibly not the answer you're looking for, but could we have more information about how you're running this?

@chentom88
Copy link

@shinji62 @jtuchscherer @MasslessParticle
It was indeed missing dependencies. We added them and opened a new PR:

#214

@sba30
Copy link

sba30 commented Oct 8, 2019

Any idea when i release will be cut with this change?

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.

6 participants