Skip to content

feat: support cross-linking synthetics & apm#595

Open
vigneshshanmugam wants to merge 4 commits into
elastic:mainfrom
vigneshshanmugam:apm-integration
Open

feat: support cross-linking synthetics & apm#595
vigneshshanmugam wants to merge 4 commits into
elastic:mainfrom
vigneshshanmugam:apm-integration

Conversation

@vigneshshanmugam

@vigneshshanmugam vigneshshanmugam commented Sep 14, 2022

Copy link
Copy Markdown
Member
  • fix [Proposal] Crosslinking Synthetics with APM #265

  • Relies on the properties being injected from the Heartbeat side to make it work seamless.

  • Adds support for crosslinking the Synthetics Waterfall information with the APM data to be able to understand the performance monitoring data across Elastic observability solutions. The key fields that are added to make the transition easier are

    • synthetics.monitor.id - monitor id of the synthetics monitor
    • syntheitcs.monitor.type - type of the synthetics monitor, can be browser or http
    • synthetics.trace.id - checkgroup/exec id that denotes execution of a journey at any point
    • synthetics.monitor.location - location id where the monitor was run from
    • tracestate: es=s:1 - Sampling decision is propagated via tracestate header which is understood by APM agents
  • Adds the relevant headers only for the routes/urls that are configured via the apm.traceUrls. Sampling decision can also be propagated via apm.sampleRate. These options can be modified via CLI or Synthetics config file.

journey("test", () => {
   step("trace check", async() => {
      await page.goto("https://example.com")
   });
});

// config 
{
  "apm": { "traceUrls":  ["**/*"] } // add to all requests
}

// Headers
baggage: `synthetics.trace.id=<test>;synthetics.monitor.id=<test>;`
tracestate: `es=s:1`

@ghost

ghost commented Sep 14, 2022

Copy link
Copy Markdown

💔 Build Failed

the below badges are clickable and redirect to their specific view in the CI or DOCS
Pipeline View Test View Changes Artifacts preview preview

Expand to view the summary

Build stats

  • Start Time: 2022-09-14T06:01:55.758+0000

  • Duration: 5 min 2 sec

Steps errors 1

Expand to view the steps failures

Checks linting
  • Took 0 min 4 sec . View more details here
  • Description: npm run-script lint

🤖 GitHub comments

To re-run your PR in the CI, just comment with:

  • /test : Re-trigger the build.

Comment thread src/plugins/apm.ts Outdated
@syepes

syepes commented Jan 18, 2024

Copy link
Copy Markdown

+1

@shahzad31

Copy link
Copy Markdown
Contributor

@vigneshshanmugam is there a plan to revisit this?

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.

[Proposal] Crosslinking Synthetics with APM

6 participants