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

Introduce Testbed #51

Merged

Conversation

tigrannajaryan
Copy link
Member

Imported Testbed from Collector core repo and added one perf
test from core scenarios to demonstrate that the testbed works.

The plan is to add tests for other protocols in this contrib repo
using the Testbed.

Imported Testbed from Collector core repo and added one perf
test from core scenarios to demonstrate that the testbed works.

The plan is to add tests for other protocols in this contrib repo
using the Testbed.
Copy link
Contributor

@pjanotti pjanotti left a comment

Choose a reason for hiding this comment

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

LGTM. Just one Q about the possibility of using some Makefile defs from Makefile.Common

testbed/Makefile Outdated Show resolved Hide resolved
@pjanotti pjanotti merged commit 9707a27 into open-telemetry:master Dec 9, 2019
@tigrannajaryan tigrannajaryan deleted the feature/tigran/add-test-bed branch December 9, 2019 18:31
mxiamxia referenced this pull request in mxiamxia/opentelemetry-collector-contrib Jul 22, 2020
mxiamxia referenced this pull request in mxiamxia/opentelemetry-collector-contrib Jul 22, 2020
Improved how translators handling mapping status codes and messages to and from OC

translator/trace/README.md documents the new behaviour
ljmsc referenced this pull request in ljmsc/opentelemetry-collector-contrib Feb 21, 2022
bogdandrutu pushed a commit that referenced this pull request May 12, 2022
* Change entry.Labels to entry.Attributes
jj22ee pushed a commit to jj22ee/opentelemetry-collector-contrib that referenced this pull request Sep 21, 2023
…nhnanced-metrics2

do not emit enhanced metrics unless we are already emitting control plane metrics
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.

3 participants