Skip to content

edot: add connector/profilingmetrics#9887

Merged
cmacknz merged 7 commits intomainfrom
edot-profilingmetrics
Sep 23, 2025
Merged

edot: add connector/profilingmetrics#9887
cmacknz merged 7 commits intomainfrom
edot-profilingmetrics

Conversation

@florianl
Copy link
Member

What does this PR do?

Adds the profilingmetrics OTel component.

Why is it important?

We want to use the profiling receiver to generate actionable metrics via profilingmetrics.

Checklist

  • I have read and understood the pull request guidelines of this project.
  • My code follows the style guidelines of this project
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have made corresponding change to the default configuration files
  • I have added tests that prove my fix is effective or that my feature works
  • I have added an entry in ./changelog/fragments using the changelog tool
  • I have added an integration test or an E2E test

Disruptive User Impact

How to test this PR locally

TBA

Signed-off-by: Florian Lehner <florian.lehner@elastic.co>
@florianl florianl requested a review from a team as a code owner September 11, 2025 12:54
@florianl florianl added enhancement New feature or request backport-skip labels Sep 11, 2025
Signed-off-by: Florian Lehner <florian.lehner@elastic.co>
@florianl
Copy link
Member Author

Blocked until #9858 got merged.

@florianl florianl marked this pull request as draft September 11, 2025 15:16
@mergify
Copy link
Contributor

mergify bot commented Sep 14, 2025

This pull request is now in conflicts. Could you fix it? 🙏
To fixup this pull request, you can check out it locally. See documentation: https://help.github.com/articles/checking-out-pull-requests-locally/

git fetch upstream
git checkout -b edot-profilingmetrics upstream/edot-profilingmetrics
git merge upstream/main
git push upstream edot-profilingmetrics
@florianl florianl marked this pull request as ready for review September 22, 2025 15:48
@mergify
Copy link
Contributor

mergify bot commented Sep 22, 2025

This pull request is now in conflicts. Could you fix it? 🙏
To fixup this pull request, you can check out it locally. See documentation: https://help.github.com/articles/checking-out-pull-requests-locally/

git fetch upstream
git checkout -b edot-profilingmetrics upstream/edot-profilingmetrics
git merge upstream/main
git push upstream edot-profilingmetrics
Copy link
Member

@pchila pchila left a comment

Choose a reason for hiding this comment

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

Looks good overall, left a couple of comments about some unexpected dependency changes and also waiting for CI to be green before approving

@pchila pchila removed their assignment Sep 23, 2025
@mergify
Copy link
Contributor

mergify bot commented Sep 23, 2025

This pull request is now in conflicts. Could you fix it? 🙏
To fixup this pull request, you can check out it locally. See documentation: https://help.github.com/articles/checking-out-pull-requests-locally/

git fetch upstream
git checkout -b edot-profilingmetrics upstream/edot-profilingmetrics
git merge upstream/main
git push upstream edot-profilingmetrics
@elasticmachine
Copy link
Contributor

💔 Build Failed

Failed CI Steps

History

cc @florianl

@cmacknz
Copy link
Member

cmacknz commented Sep 23, 2025

The job, Unit tests - Windows 2016, has been canceled as it failed to get an agent after 5 tries.

I am going to force merge past this problem, this PR has been open long enough as is and the failure is unrelated.

@cmacknz cmacknz merged commit 2c23284 into main Sep 23, 2025
20 of 21 checks passed
@cmacknz cmacknz deleted the edot-profilingmetrics branch September 23, 2025 19:30
intxgo pushed a commit to intxgo/elastic-agent that referenced this pull request Sep 24, 2025
* edot: add connector/profilingmetrics

Signed-off-by: Florian Lehner <florian.lehner@elastic.co>

* drop profilingreceiver

Signed-off-by: Florian Lehner <florian.lehner@elastic.co>

* downgrade: go get apikeyauthextension@v0.5.0

Signed-off-by: Florian Lehner <florian.lehner@elastic.co>

---------

Signed-off-by: Florian Lehner <florian.lehner@elastic.co>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport-skip enhancement New feature or request

4 participants