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

test: Truffle metrics #14

Closed
wants to merge 15 commits into from
Closed

test: Truffle metrics #14

wants to merge 15 commits into from

Conversation

kaylareopelle
Copy link
Owner

No description provided.

pantuza and others added 9 commits October 1, 2024 10:08
…d operation (open-telemetry#1713)

* feature: adds on_ending method as an optional callback for span processors

Signed-off-by: Gustavo Pantuza <[email protected]>

* feature: calls every span processor that has on_ending implemented right after setting the end timestamp

Signed-off-by: Gustavo Pantuza <[email protected]>

* test: adds tests for the new on_ending method from span processors

Signed-off-by: Gustavo Pantuza <[email protected]>

* docs: Informs the on_ending callback is an experimental features from official spec

Signed-off-by: Gustavo Pantuza <[email protected]>

* Update sdk/lib/opentelemetry/sdk/trace/span_processor.rb

Co-authored-by: Kayla Reopelle <[email protected]>

* Update sdk/lib/opentelemetry/sdk/trace/span_processor.rb

Co-authored-by: Kayla Reopelle <[email protected]>

* refactor: switch method name from on_ending to on_finish to comply with project name strategy

Signed-off-by: Gustavo Pantuza <[email protected]>

---------

Signed-off-by: Gustavo Pantuza <[email protected]>
Co-authored-by: Kayla Reopelle <[email protected]>
Some tests in BatchLogRecordProcessor had timing issues due to
the work method. Since we don't rely on that method for these tests,
stub the work method to do nothing.

Co-authored-by: Tanna McClure <[email protected]>
Co-authored-by: Matthew Wear <[email protected]>
* fix: update references to logging exporter

This exporter has been replaced by the debug exporter and will be removed soon

Signed-off-by: Alex Boten <[email protected]>

* update example to use v0.109.0 of the collector

Signed-off-by: Alex Boten <[email protected]>

---------

Signed-off-by: Alex Boten <[email protected]>
Co-authored-by: Matthew Wear <[email protected]>
* feat: add basic metrics view support

* feat: lint

* feat: metric view lint

* feat: metrics - use flatten for 1-d array MetricData

* Update metrics_sdk/lib/opentelemetry/sdk/metrics/state/metric_stream.rb

Co-authored-by: Kayla Reopelle <[email protected]>

* Update metrics_sdk/lib/opentelemetry/sdk/metrics/meter_provider.rb

Co-authored-by: Kayla Reopelle <[email protected]>

* update doc

* revision

* align the test case

* Update metrics_sdk/lib/opentelemetry/sdk/metrics/meter_provider.rb

Co-authored-by: Kayla Reopelle <[email protected]>

* Update metrics_sdk/lib/opentelemetry/sdk/metrics/meter_provider.rb

Co-authored-by: Kayla Reopelle <[email protected]>

* Update metrics_sdk/lib/opentelemetry/sdk/metrics/meter_provider.rb

Co-authored-by: Kayla Reopelle <[email protected]>

* Update metrics_sdk/lib/opentelemetry/sdk/metrics/view.rb

Co-authored-by: Kayla Reopelle <[email protected]>

* Update metrics_sdk/lib/opentelemetry/sdk/metrics/view/registered_view.rb

Co-authored-by: Kayla Reopelle <[email protected]>

* refactor view and add test

* revision

---------

Co-authored-by: Kayla Reopelle <[email protected]>
Co-authored-by: Matthew Wear <[email protected]>
* feat: Add log record attribute limits

Similar to SpanLimits, add a LogRecordLimits class that handles
configuration of attribute count and value length values.

* Update logs_sdk/test/opentelemetry/sdk/logs/log_record_test.rb

* Update logs_sdk/lib/opentelemetry/sdk/logs/log_record.rb

* Update logs_sdk/lib/opentelemetry/sdk/logs/log_record.rb

---------

Co-authored-by: Matthew Wear <[email protected]>
* fix: add mTLS for metrics exporter

* fix: add second pem for parameter override

* update test case

* Update exporter/otlp-metrics/README.md

Co-authored-by: Bart de Water <[email protected]>

---------

Co-authored-by: Bart de Water <[email protected]>
Co-authored-by: Matthew Wear <[email protected]>
Validate pull request titles to ensure they adhere to conventional
commits. This matches the validation performed in contrib.

Co-authored-by: Matthew Wear <[email protected]>
@kaylareopelle kaylareopelle changed the title Truffle metrics test: Truffle metrics Oct 16, 2024
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.

5 participants