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

[docs] Add status header for tanzuobservability exporter #10337

Conversation

Frapschen
Copy link
Contributor

Description:
Add status header for tanzuobservability exporter

@Frapschen Frapschen requested review from a team and Aneurysm9 May 25, 2022 09:54
Copy link
Member

@TylerHelmuth TylerHelmuth left a comment

Choose a reason for hiding this comment

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

/cc @oppegard @thepeterstone @keep94 as code owners to verify beta stability.

@TylerHelmuth TylerHelmuth added the Skip Changelog PRs that do not require a CHANGELOG.md entry label May 25, 2022
@oppegard
Copy link
Member

@TylerHelmuth is this status header something that other exporters are adopting? I haven't seen it before.

@TylerHelmuth
Copy link
Member

@TylerHelmuth is this status header something that other exporters are adopting? I haven't seen it before.

Yes, a new standard being tracked here: #10116

Frapschen and others added 2 commits May 26, 2022 07:24
Co-authored-by: Tyler Helmuth <12352919+TylerHelmuth@users.noreply.github.com>
Copy link
Member

@oppegard oppegard left a comment

Choose a reason for hiding this comment

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

LGTM

@codeboten codeboten merged commit 19009ef into open-telemetry:main May 26, 2022
@Frapschen Frapschen deleted the Add_status_header_for_tanzuobservability branch May 29, 2022 23:46
kentquirk pushed a commit to McSick/opentelemetry-collector-contrib that referenced this pull request Jun 14, 2022
…try#10337)

* Add status header for tanzuobservability

* Apply suggestions from code review

Co-authored-by: Tyler Helmuth <12352919+TylerHelmuth@users.noreply.github.com>

* Update README.md

Co-authored-by: Tyler Helmuth <12352919+TylerHelmuth@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Skip Changelog PRs that do not require a CHANGELOG.md entry
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants