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

Support for Loggregator 2.x? #904

Open
adityajagtiani89 opened this issue Aug 2, 2018 · 6 comments

Comments

@adityajagtiani89
Copy link

commented Aug 2, 2018

Does this Java client include support for Loggregator 2.x? I'm interested in extracting metrics through the nozzle.

Currently using a client based off of your jmx-consumer project and have seen customers run into a memory issue after a few days of running.

Caused by: io.netty.util.internal.OutOfDirectMemoryError: failed to allocate 16777216 byte(s) of direct memory (used: 1845494055, max: 1860698112)
    at io.netty.util.internal.PlatformDependent.incrementMemoryCounter(PlatformDependent.java:640)
    at io.netty.util.internal.PlatformDependent.allocateDirectNoCleaner(PlatformDependent.java:594)
    at io.netty.buffer.PoolArena$DirectArena.allocateDirect(PoolArena.java:764)
    at io.netty.buffer.PoolArena$DirectArena.newChunk(PoolArena.java:740)
    at io.netty.buffer.PoolArena.allocateNormal(PoolArena.java:244)

Any insight would be appreciated.

Thanks,
Aditya

@twoseat twoseat self-assigned this Aug 10, 2018
@twoseat twoseat added this to the 2.33.0.RELEASE milestone Aug 10, 2018
@twoseat

This comment has been minimized.

Copy link
Collaborator

commented Aug 10, 2018

Hi @adityajagtiani89 - we don't currently support it, but I'll start work on it shortly. I've added it to the next release for now, but you shouldn't make any plans based on that as I don't know how easy it will be to add in.

@youngm

This comment has been minimized.

Copy link
Contributor

commented Aug 10, 2018

Just so it is on your radar @twoseat here are 2 other enhancement requests in this area also: #851 and #899. Might be considered dupes.

@twoseat twoseat modified the milestones: 2.34.0.RELEASE, 2.35.0.RELEASE, 3.16.0.RELEASE Dec 21, 2018
@youngm

This comment has been minimized.

Copy link
Contributor

commented Feb 12, 2019

@twoseat @nebhale So, the loggregator team just announced the removal of support for the v1 firehose in 3 months. How is the loggregator v2 support coming?

@twoseat

This comment has been minimized.

Copy link
Collaborator

commented Feb 13, 2019

I saw the announcement - that certainly sharpens the need! It got buried by some other priorities, but clearly it’s at the top of the list now.

@MasslessParticle

This comment has been minimized.

Copy link
Member

commented Feb 15, 2019

@twoseat @nebhale

Hello, I'm the anchor on the Loggregator team.
I wanted to point out work we've done to make consumption of v2 easier (protobuf and certs not required!):

https://github.com/cloudfoundry/loggregator/blob/925b09a107d4d11286e0b84d5f2f090b30f6a397/docs/rlp_gateway.md

As work on this progresses, please reach out If you have any questions about how to consume the v2 api.

@jtuchscherer

@MasslessParticle

This comment has been minimized.

Copy link
Member

commented Mar 19, 2019

@twoseat @nebhale the PR providing the log_stream url has been merged into CAPI master

cloudfoundry/cloud_controller_ng#1301

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants
You can’t perform that action at this time.