Skip to content

test: fix flaky test-otlp-grpc-metrics#281

Closed
santigimeno wants to merge 1 commit intonode-v22.x-nsolid-v5.xfrom
santi/fix_flaky_otlp_grpc_metrics_test
Closed

test: fix flaky test-otlp-grpc-metrics#281
santigimeno wants to merge 1 commit intonode-v22.x-nsolid-v5.xfrom
santi/fix_flaky_otlp_grpc_metrics_test

Conversation

@santigimeno
Copy link
Copy Markdown
Member

After having enabled ENABLE_ASYNC_EXPORT in opentelemetry-cpp.

@santigimeno santigimeno self-assigned this Mar 24, 2025
@santigimeno santigimeno force-pushed the santi/fix_flaky_otlp_grpc_metrics_test branch from b8c562a to c19d441 Compare April 7, 2025 09:41
Copy link
Copy Markdown
Contributor

@juanarbol juanarbol 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 a question left

Comment thread test/agents/test-otlp-grpc-metrics.mjs Outdated
After having enabled `ENABLE_ASYNC_EXPORT` in `opentelemetry-cpp`.
@santigimeno santigimeno force-pushed the santi/fix_flaky_otlp_grpc_metrics_test branch from c19d441 to f18e72e Compare April 10, 2025 13:17
santigimeno added a commit that referenced this pull request Apr 10, 2025
After having enabled `ENABLE_ASYNC_EXPORT` in `opentelemetry-cpp`.

PR-URL: #281
Reviewed-By: Juan José Arboleda <soyjuanarbol@gmail.com>
@santigimeno
Copy link
Copy Markdown
Member Author

Landed in a8135cd

santigimeno added a commit that referenced this pull request May 1, 2025
After having enabled `ENABLE_ASYNC_EXPORT` in `opentelemetry-cpp`.

PR-URL: #281
Reviewed-By: Juan José Arboleda <soyjuanarbol@gmail.com>
santigimeno added a commit that referenced this pull request May 2, 2025
After having enabled `ENABLE_ASYNC_EXPORT` in `opentelemetry-cpp`.

PR-URL: #281
Reviewed-By: Juan José Arboleda <soyjuanarbol@gmail.com>
santigimeno added a commit that referenced this pull request May 7, 2025
After having enabled `ENABLE_ASYNC_EXPORT` in `opentelemetry-cpp`.

PR-URL: #281
Reviewed-By: Juan José Arboleda <soyjuanarbol@gmail.com>
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.

2 participants