Skip to content

Conversation

@kyungeunni
Copy link
Contributor

@kyungeunni kyungeunni commented Feb 7, 2024

Proposed commit message

Add transaction.profiler_stack_trace_ids field to traces datastream mapping

It requires the package-spec v3.1.0 which should be shipped with the next release of elastic-package(current release is v0.96.1)

Checklist

  • I have reviewed tips for building integrations and this pull request is aligned with them.
  • I have verified that all data streams collect metrics or logs.
  • I have added an entry to my package's changelog.yml file.
  • I have verified that Kibana version constraints are current according to guidelines.

Author's Checklist

  • Wait for the new release of elastic-package which includes package version 3.1.0

How to test this PR locally

clone the elastic-package, then run:

❯ make clean ... ❯ make build ... ❯ make install

This will move the binary built from the latest commit to $GOBIN.

Then go to the apm package directory and checkout to this branch, then run:

❯ elastic-package check ... ❯ elastic-package lint

It shouldn't print any error logs, and look something like this:
image

Test with Registry

❯ elastic-package stack up -v -d --version 8.13.0-SNAPSHOT # once the stack is up(registry must be running), install the package # integrations/packages/apm ❯ elastic-package build ... ❯ elastic-package install 

When installed, Go to Kibana, and navigate to Stack Management > Index Management > Component and search for traces. the component should show the added field with the counted_keyword type.

image

Related issues

@kyungeunni kyungeunni added the enhancement New feature or request label Feb 7, 2024
@kyungeunni kyungeunni self-assigned this Feb 7, 2024
@kyungeunni kyungeunni requested a review from a team as a code owner February 7, 2024 04:34
@kyungeunni
Copy link
Contributor Author

kyungeunni commented Feb 7, 2024

Failure is expected until we have the newer version of elastic-package as the current version doesn't recognize counted_keyword type. => temporarily using the latest main branch to verify it works.

Unrelated package is failed to pass the tests, but I can see it built the apm package successfully: https://buildkite.com/elastic/integrations/builds/8684#018d827b-6911-4deb-83b9-8b82bbc3c160

cc @jsoriano

@kyungeunni kyungeunni requested a review from a team as a code owner February 7, 2024 07:27
@elasticmachine
Copy link

elasticmachine commented Feb 7, 2024

🚀 Benchmarks report

Package windows 👍(1) 💚(4) 💔(3)

Expand to view
Data stream Previous EPS New EPS Diff (%) Result
applocker_msi_and_script 10309.28 8000 -2309.28 (-22.4%) 💔
applocker_packaged_app_execution 22222.22 16949.15 -5273.07 (-23.73%) 💔
sysmon_operational 2865.33 2398.08 -467.25 (-16.31%) 💔

To see the full report comment with /test benchmark fullreport

@carsonip carsonip requested a review from inge4pres February 9, 2024 13:52
Copy link
Contributor

@inge4pres inge4pres left a comment

Choose a reason for hiding this comment

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

🚀

@elasticmachine
Copy link

💚 Build Succeeded

History

cc @kyungeunni

@elastic-sonarqube
Copy link

Quality Gate passed Quality Gate passed

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
No Coverage information No data about Coverage
No Duplication information No data about Duplication

See analysis details on SonarQube

@carsonip carsonip merged commit 13c80a3 into elastic:main Feb 12, 2024
@kyungeunni kyungeunni deleted the apm-add-profiling-stactrace-ids-field branch February 13, 2024 01:40
@andrewkroh andrewkroh added the Integration:apm Elastic APM label Jul 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request Integration:apm Elastic APM

5 participants