Skip to content
This repository has been archived by the owner on Oct 31, 2024. It is now read-only.

feat: update APM instrumentation #10

Merged
merged 1 commit into from
Sep 26, 2023

Conversation

sebastiendan
Copy link
Member

Description

This PR updates the APM instrumentation to focus on frontend->backend requests and get a different APM Transaction per request. New labels are also added to make sure we know everything about users' requests.

Fixes TOO-341

Additions and Changes

  • NodeJS APM agent bootstrapped changes from local in main.ts to an actual NestJS service (similarly to what was recently done on the Executor Service)
  • Elastic RUM (browser) instrumentation now creates a different APM transaction per faucet backend request
  • New labels show complete data about users' requests

PR Checklist:

  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have added or updated tests that comprehensively prove my change is effective or that my feature works

@sebastiendan sebastiendan self-assigned this Sep 26, 2023
@sebastiendan sebastiendan merged commit 477fde4 into main Sep 26, 2023
@sebastiendan sebastiendan deleted the feat/update-apm-instrumentation branch September 26, 2023 09:19
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant