Skip to content

Clarify cicd.pipeline.task.run.id uniqueness#3589

Open
kamphaus wants to merge 2 commits intoopen-telemetry:mainfrom
kamphaus:main
Open

Clarify cicd.pipeline.task.run.id uniqueness#3589
kamphaus wants to merge 2 commits intoopen-telemetry:mainfrom
kamphaus:main

Conversation

@kamphaus
Copy link
Copy Markdown
Contributor

Fixes #3566

Changes

Clarify that cicd.pipeline.task.run.id must be unique within a pipeline run
and that this id may be used for correlating results of a task across several pipeline runs.

Important

Pull requests acceptance are subject to the triage process as described in Issue and PR Triage Management.
PRs that do not follow the guidance above, may be automatically rejected and closed.

Merge requirement checklist

  • CONTRIBUTING.md guidelines followed.
  • Change log entry added, according to the guidelines in When to add a changelog entry.
    • If your PR does not need a change log, start the PR title with [chore]
  • Links to the prototypes or existing instrumentations (when adding or changing conventions) → NA

@kamphaus kamphaus requested review from a team as code owners March 31, 2026 20:37
@github-actions github-actions bot added the enhancement New feature or request label Mar 31, 2026
@lmolkova lmolkova moved this from Untriaged to Awaiting codeowners approval in Semantic Conventions Triage Apr 6, 2026
@github-actions
Copy link
Copy Markdown

This PR has been labeled as stale due to lack of activity. It will be automatically closed if there is no further activity over the next 7 days.

@github-actions github-actions bot added the Stale label Apr 15, 2026
@github-actions github-actions bot removed the Stale label Apr 16, 2026
Comment thread docs/registry/attributes/cicd.md
@kamphaus kamphaus moved this from Awaiting codeowners approval to Needs More Approval in Semantic Conventions Triage Apr 19, 2026
Comment thread model/cicd/registry.yaml
Comment on lines +79 to +80
For a given pipeline run and task the `cicd.pipeline.task.run.id` MUST be unique.
For different pipeline runs of the same pipeline the task run id MAY remain the same and thus MAY allow correlating the `cicd.pipeline.task.run.result`
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
For a given pipeline run and task the `cicd.pipeline.task.run.id` MUST be unique.
For different pipeline runs of the same pipeline the task run id MAY remain the same and thus MAY allow correlating the `cicd.pipeline.task.run.result`
For a given pipeline run and task, the `cicd.pipeline.task.run.id` MUST be unique.
For different runs of the same pipeline, `cicd.pipeline.task.run.id` MAY remain the same and allow correlating with `cicd.pipeline.task.run.result`

I got confused with the multiple "pipeline and pipeline runs" so I rephrased it a bit. Also, I'm not entirely sure about the "correlation" part. Do we really need to include that? I don't particularly get any helpful insight from it from the small text here. Consider either removing it or maybe adding more explanation on how it can be correlated?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:cicd enhancement New feature or request

Projects

Status: Needs More Approval

Development

Successfully merging this pull request may close these issues.

Clarification to cicd.pipeline.task.run.id

4 participants