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

Draft:(resource/sem) Adding Datacenter Conventions. #1459

Conversation

hdost
Copy link

@hdost hdost commented Feb 23, 2021

  • Would be good to unify potentially with "Cloud" conventions. To try to
    be a little more generic.

Relates to open-telemetry/semantic-conventions#1409

Fixes #

Changes

Please provide a brief description of the changes here. Update the
CHANGELOG.md for non-trivial changes. If CHANGELOG.md is updated,
also be sure to update spec-compliance-matrix.md if necessary.

Related issues #

Related oteps #

* Would be good to unify potentially with "Cloud" conventions. To try to
  be a little more generic.

Relates to #1458

**Description:** Generic DataCenter infrastructure tagging.

<!-- semconv datacenter -->
Copy link
Member

Choose a reason for hiding this comment

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

The semconv HTML comments make the semconv generator tool replace the content in-between with what it generates from the YAML under https://github.com/open-telemetry/opentelemetry-specification/tree/main/semantic_conventions. But we can discuss this later.

@github-actions
Copy link

github-actions bot commented Mar 3, 2021

This PR was marked stale due to lack of activity. It will be closed in 7 days.

@github-actions github-actions bot added the Stale label Mar 3, 2021
@github-actions
Copy link

Closed as inactive. Feel free to reopen if this PR is still being worked on.

@github-actions github-actions bot closed this Mar 10, 2021
@hdost
Copy link
Author

hdost commented Dec 27, 2021

reopen please

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants