Skip to content

feat(config): add OpenTelemetry OTLP config for Datadog - #160

Draft
foukou19 wants to merge 1 commit into
mainfrom
feat/idp-core-observability-otel
Draft

foukou19 wants to merge 1 commit into
mainfrom
feat/idp-core-observability-otel

Conversation

@foukou19

@foukou19 foukou19 commented Oct 1, 2026

Copy link
Copy Markdown
Collaborator

PR Description

What this PR Provides

  • Describe in short sentences the goal of the PR. Use lists.
  • If the PR is linked to an ADR, please provide the link.

Fixes

Review

The reviewer must double-check these points:

  • The reviewer has tested the feature
  • The reviewer has reviewed the implementation of the feature
  • The documentation has been updated
  • The feature implementation respects the Technical Doc / ADR previously produced
  • The Pull Request title has a ! after the type/scope to identify the breaking
    change in the release note and ensure we will release a major version.

How to test

Please refer (copy/paste) the test section from the User Story. This should include

  • The initial state: what should be the status of the system before testing
    (for example, ensure the data xxx exists in idp-back to be able to test the feature)
  • What and how to test: steps to perform to test the feature
    (for example, go to page xxx, fill the xxx field and click the 'send' button)
  • Expected results: what should be observed for success or failure
    (for example, there is a link in the database between component X and component Y.
    You can retrieve the information with a GET request to the API)

Breaking changes (if any)

  • Data loss / modification
  • API JSON schema modification (existing resource / behavior)
  • Behavior modification of a component
  • Others
  • N/A

Context of the Breaking Change

For example: we redefined the component types list in the DPAC referential

Result of the Breaking Change

For example: your component of type xxx will migrate to the type yyy

@foukou19
foukou19 force-pushed the feat/idp-core-observability-otel branch 2 times, most recently from 7dff1ba to 9e4eb90 Compare October 1, 2026 13:55
@github-code-quality

github-code-quality Bot commented Oct 1, 2026 •

Copy link
Copy Markdown

Code Coverage Overview

Languages: Java

Java / code-coverage/jacoco

The overall line coverage in commit 845c0f7 in the feat/idp-core-observ... branch remains at 91%, unchanged from commit 98f5452 in the main branch.

Show a line coverage summary of the most impacted files.
File main 98f5452 feat/idp-core-observ... 845c0f7 +/-
com/decathlon/i...ationUtils.java 87% 87% 0%
com/decathlon/i...Controller.java 0% 22% +22%

Updated October 02, 2026 19:31 UTC

@foukou19
foukou19 marked this pull request as draft October 1, 2026 14:17
@foukou19
foukou19 marked this pull request as ready for review October 1, 2026 15:49
@foukou19
foukou19 force-pushed the feat/idp-core-observability-otel branch 4 times, most recently from 3a8180a to 6713577 Compare October 2, 2026 19:11
@foukou19
foukou19 marked this pull request as draft October 2, 2026 19:12
Signed-off-by: foukou19 <ferial.oukoukas@decathlon.com>
@foukou19
foukou19 force-pushed the feat/idp-core-observability-otel branch from 6713577 to 845c0f7 Compare October 2, 2026 19:27
@sonarqubecloud

sonarqubecloud Bot commented Oct 2, 2026

Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
22.2% Coverage on New Code (required ≥ 80%)

See analysis details on SonarQube Cloud

This branch has not been deployed

No deployments
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.

1 participant