Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Monitoring main documentation #3050

Open
wants to merge 53 commits into
base: master
Choose a base branch
from
Open

Monitoring main documentation #3050

wants to merge 53 commits into from

Conversation

romansprykee
Copy link
Collaborator

PR labels

When the PR is ready for review, add a TW review needed label. This lets us keep track of PRs that need to be merged and merge them in time.

PR Description

Describe the context for your changes and the changes you've made.

Tickets

If changes are associated with a ticket, add a docs ticket here.

Checklist

  • I agree with the Code Contribution License Agreement in CONTRIBUTING.md

@romansprykee romansprykee added new PR contains a new document updated PR contains meaningful changes to some document labels Feb 11, 2025
## Telemetry data in scope of Spryker Monitoring Integration
The Spryker Monitoring Integration focuses on several key entities to provide comprehensive monitoring:

### Traces and Spans
Copy link
Collaborator

Choose a reason for hiding this comment

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

it is already explained in How to Instrument shall we link to this section?

The default metrics resolution is 60 seconds. All metrics can be split by telemetry-data-account.

## How do I get it?
### Prerequisites
Copy link
Collaborator

Choose a reason for hiding this comment

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

shall we mention also commercials have to be signed before?

Copy link
Contributor

Choose a reason for hiding this comment

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

@romansprykee yes, let's mentioned that as soon as the agreement for this SKU is concluded.

Copy link
Collaborator

@gechetspr gechetspr left a comment

Choose a reason for hiding this comment

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

All good from my side

The Spryker Monitoring Integration focuses on several key entities to provide comprehensive monitoring:

### Traces and Spans
In OpenTelemetry, a **trace** represents the journey of a single request or transaction as it moves through various components of a system, capturing the end-to-end flow. A **span** is a single operation or unit of work within a trace, containing information like the operation name, start and end times, and any relevant metadata. Together, traces and spans provide a detailed view of the interactions and performance of different parts of an application, helping to diagnose issues and optimize performance.
Copy link
Contributor

Choose a reason for hiding this comment

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

@gechetspr let's add here the supported version of OTEL convention - 1.30. As a highlighted box.

Copy link
Contributor

@vlunov-spryker vlunov-spryker left a comment

Choose a reason for hiding this comment

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

Please take a look at minor comments, after those additions let's publish.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new PR contains a new document updated PR contains meaningful changes to some document
Development

Successfully merging this pull request may close these issues.

6 participants