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.

Sign up for GitHub

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

Jump to bottom

pyroscope.receive_http should support the connect go handlers #2301

Open
simonswine opened this issue Dec 19, 2024 · 0 comments
Open

pyroscope.receive_http should support the connect go handlers #2301

simonswine opened this issue Dec 19, 2024 · 0 comments
Labels
enhancement New feature or request proposal A proposal for new functionality.

Comments

@simonswine
Copy link
Contributor

Background

Pyroscope receive_http only support the /ingest API, which is confusing. We should promote the use of the connect-go handlers and there fore provide a pushv1.PushV1.

Currently it is not possible for example to make pyroscope.ebpf of many agents send to pyroscope.receive_http on a single gateway agent, as this only uses the push.v1.PusherService/Push.

Proposal

Support connect-go on the reciever_http socket.
@simonswine simonswine added the proposal A proposal for new functionality. label Dec 19, 2024
@simonswine simonswine added the enhancement New feature or request label Dec 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request proposal A proposal for new functionality.
Projects
Alloy proposals
Status: Incoming
Milestone
No milestone
Development

No branches or pull requests
1 participant
@simonswine