Skip to content

feat: add pyroscope http_receive and targets support#14

Merged
Frostman merged 1 commit intomasterfrom
dev/frostman/pyroscope
Nov 4, 2025
Merged

feat: add pyroscope http_receive and targets support#14
Frostman merged 1 commit intomasterfrom
dev/frostman/pyroscope

Conversation

@Frostman
Copy link
Copy Markdown
Member

@Frostman Frostman commented Nov 4, 2025

No description provided.

Signed-off-by: Sergei Lukianov <me@slukjanov.name>
@Frostman Frostman requested a review from Copilot November 4, 2025 20:28
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds Pyroscope profiling support to the Alloy configuration system, enabling continuous profiling data collection and forwarding to Pyroscope backends.

  • Adds PyroscopeTarget type with validation for defining Pyroscope write endpoints
  • Implements Pyroscope configuration struct with HTTP receiver settings (address, port)
  • Updates the Alloy template to generate pyroscope.write and pyroscope.receive_http blocks

Reviewed Changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated no comments.

Show a summary per file
File Description
pkg/alloy/targets.go Adds PyroscopeTarget type and validation logic for Pyroscope targets
pkg/alloy/config.go Adds Pyroscope configuration struct with default port constant, removes unused Kube.Validate() method, and applies field alignment
pkg/alloy/config.alloy.tmpl Adds template blocks for Pyroscope write endpoints and HTTP receiver configuration
pkg/alloy/zz_generated.deepcopy.go Auto-generated deepcopy methods for new Pyroscope types
pkg/alloy/config_test.go Adds test coverage for Pyroscope configuration in the full config test case
pkg/alloy/testdata/TestConfigRender_full.expected Updates expected output with rendered Pyroscope configuration blocks

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@Frostman Frostman merged commit 693569d into master Nov 4, 2025
9 checks passed
@Frostman Frostman deleted the dev/frostman/pyroscope branch November 4, 2025 20:40
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.

2 participants