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

Logs SDK - Fix intermittently failing tests #1701

Closed
kaylareopelle opened this issue Aug 28, 2024 · 1 comment · Fixed by #1730
Closed

Logs SDK - Fix intermittently failing tests #1701

kaylareopelle opened this issue Aug 28, 2024 · 1 comment · Fixed by #1730
Assignees

Comments

@kaylareopelle
Copy link
Contributor

kaylareopelle commented Aug 28, 2024

The following test on the logs SDK job fails intermittently:

1) Error:
OpenTelemetry::SDK::Logs::Export::BatchLogRecordProcessor::faulty exporter#test_0001_reports export failures:
MockExpectationError: No more expects available for :error: ["OpenTelemetry error: Unable to export 4 log records"] {}
    /Users/runner/work/opentelemetry-ruby/opentelemetry-ruby/api/lib/opentelemetry.rb:37:in `block in error_handler'
    /Users/runner/work/opentelemetry-ruby/opentelemetry-ruby/api/lib/opentelemetry.rb:45:in `handle_error'
    lib/opentelemetry/sdk/logs/export/batch_log_record_processor.rb:199:in `report_result'
    lib/opentelemetry/sdk/logs/export/batch_log_record_processor.rb:191:in `rescue in export_batch'
    lib/opentelemetry/sdk/logs/export/batch_log_record_processor.rb:186:in `export_batch'
    lib/opentelemetry/sdk/logs/export/batch_log_record_processor.rb:110:in `force_flush'
    lib/opentelemetry/sdk/logs/export/batch_log_record_processor.rb:150:in `shutdown'
    test/opentelemetry/sdk/logs/export/batch_log_record_processor_test.rb:476:in `block (4 levels) in <top (required)>'
    test/opentelemetry/sdk/logs/export/batch_log_record_processor_test.rb:473:in `block (3 levels) in <top (required)>'

There's also an intermittent JRuby failure:

  1) Error:
OpenTelemetry::SDK::Logs::Export::BatchLogRecordProcessor::#on_emit#test_0003_logs a warning if a log record was emitted after the buffer is full:
MockExpectationError: expected warn("1 log record(s) dropped. Reason: buffer-full") => nil
    org/jruby/RubyHash.java:1615:in `each'
    test/opentelemetry/sdk/logs/export/batch_log_record_processor_test.rb:220:in `block in <main>'
    org/jruby/RubyArray.java:1981:in `each'
    org/jruby/RubyArray.java:2803:in `map'
@kaylareopelle kaylareopelle moved this to To Do in Ruby - Logs Aug 28, 2024
@kaylareopelle kaylareopelle changed the title Logs SDK - Fix intermittently failing test Logs SDK - Fix intermittently failing tests Aug 28, 2024
Copy link
Contributor

👋 This issue has been marked as stale because it has been open with no activity. You can: comment on the issue or remove the stale label to hold stale off for a while, add the keep label to hold stale off permanently, or do nothing. If you do nothing this issue will be closed eventually by the stale bot.

@github-actions github-actions bot added the stale label Sep 28, 2024
@kaylareopelle kaylareopelle added keep and removed stale labels Oct 1, 2024
@kaylareopelle kaylareopelle self-assigned this Oct 1, 2024
@github-project-automation github-project-automation bot moved this from To Do to Done in Ruby - Logs Oct 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

1 participant