Skip to content

Add dataflow definitions for opentelemetry#22875

Merged
alexeypilyugin merged 1 commit intomasterfrom
alexeypilyugin/opentelemetry
Mar 13, 2026
Merged

Add dataflow definitions for opentelemetry#22875
alexeypilyugin merged 1 commit intomasterfrom
alexeypilyugin/opentelemetry

Conversation

@alexeypilyugin
Copy link
Copy Markdown
Contributor

Add missing dataflow declarations inferred from integration source/config for logs, metrics, and events where applicable.

What does this PR do?

Adds dataflows to agent integrations based on the following rules:

  • Owned by opentelemetry
  • Doesn't have a dataflows definition yet
  • Dataflow is in the following format %integration-name%-%dataflow-type%, where dataflow-type can be one of [logs, metrics, events]
  • direction is always inbound
  • always_on is true if the flow cannot be turned off once the conf.yaml is set up. For example, metrics are usually always_on: true because they cannot be turned off once the configuration is in place. Logs, however, are always_on: false because log collection can be disabled in the conf.yaml file.

Motivation

https://datadoghq.atlassian.net/wiki/spaces/IXP/pages/5756519690/Describing+your+Integration+Dataflows

Review checklist (to be filled by reviewers)

  • Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
  • Add the qa/skip-qa label if the PR doesn't need to be tested during QA.
  • If you need to backport this PR to another branch, you can add the backport/<branch-name> label to the PR and it will automatically open a backport PR once this one is merged

Add missing dataflow declarations inferred from integration source/config for logs, metrics, and events where applicable.
@alexeypilyugin alexeypilyugin marked this pull request as ready for review March 11, 2026 12:44
@alexeypilyugin alexeypilyugin requested review from a team as code owners March 11, 2026 12:44
@alexeypilyugin alexeypilyugin added the qa/skip-qa Automatically skip this PR for the next QA label Mar 11, 2026
@asopkin asopkin added the assets/no-deploy Prevents APW from deploying this PR in staging label Mar 12, 2026
@asopkin
Copy link
Copy Markdown

asopkin commented Mar 12, 2026

Hey! Adding the assets/no-deploy label to this PR temporarily so I can get a dashboard for mine deployed to staging: #22861

@alexeypilyugin alexeypilyugin added this pull request to the merge queue Mar 13, 2026
Merged via the queue into master with commit 5a11dff Mar 13, 2026
71 of 76 checks passed
@alexeypilyugin alexeypilyugin deleted the alexeypilyugin/opentelemetry branch March 13, 2026 19:16
lukepatrick pushed a commit to lukepatrick/integrations-core that referenced this pull request Mar 17, 2026
Add missing dataflow declarations inferred from integration source/config for logs, metrics, and events where applicable.

Signed-off-by: lukepatrick <lukephilips@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

assets/no-deploy Prevents APW from deploying this PR in staging integration/otel qa/skip-qa Automatically skip this PR for the next QA team/agent-integrations team/opentelemetry

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants