Skip to content

Support payload for custom span#1587

Draft
CelianR wants to merge 31 commits into
masterfrom
celian/innovation-week-25-q1-span-payload
Draft

Support payload for custom span#1587
CelianR wants to merge 31 commits into
masterfrom
celian/innovation-week-25-q1-span-payload

Conversation

@CelianR

@CelianR CelianR commented Mar 13, 2025

Copy link
Copy Markdown
Contributor

What and why?

Adds an option to provide a payload such that datadog-ci is executed only once for multiple calls.

It will be useful to avoid extra overhead when uploading several custom spans at once.

Here is an example payload sending two spans:

[
    {
        "name": "hello",
        "startTimeInMs": 42,
        "endTimeInMs": 618,
        "tags": ["hello:world", "super:mario"],
        "measures": ["life:42", "hey:618", "ho:1618"]
    },
    {
        "name": "hello2",
        "startTimeInMs": 43,
        "endTimeInMs": 619,
        "tags": ["bonjour:monde", "super:luigi"],
        "measures": ["life:43", "hey:619", "ho:1619"]
    }
]

How?

Mostly refactoring, the payload is a json file containing an array of arguments.

Review checklist

  • Feature or bugfix MUST have appropriate tests (unit, integration)

CelianR added 26 commits March 10, 2025 10:39
@CelianR CelianR self-assigned this Mar 13, 2025
@CelianR
CelianR changed the base branch from master to celian/innovation-week-25q1-custom-span March 13, 2025 12:36
@datadog-datadog-prod-us1

Copy link
Copy Markdown

Datadog Report

Branch report: celian/innovation-week-25-q1-span-payload
Commit report: 6631a95
Test service: datadog-ci-tests

✅ 0 Failed, 387 Passed, 0 Skipped, 1m 17.76s Total duration (1m 5.84s time saved)

Base automatically changed from celian/innovation-week-25q1-custom-span to master April 15, 2025 07:24
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