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.

Sign up for GitHub

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

Jump to bottom

[SDTEST-409] Send telemetry events in background thread to remove blocking HTTP calls from critical path #3718

Merged
merged 21 commits into from
Jun 25, 2024
Merged

[SDTEST-409] Send telemetry events in background thread to remove blocking HTTP calls from critical path #3718

merged 21 commits into from
Jun 25, 2024
+1,152 −595

Commits on Jun 25, 2024

  1. rename Telemetry::Heartbeat to Telemetry::Worker

    @anmarchenko
    anmarchenko committed Jun 25, 2024
    Configuration menu
    Copy the full SHA
    3db289f View commit details
    Browse the repository at this point in the history

  2. Add queue to telemetry worker. Move sending heartbeat logic to the te… 

    …lemetry worker.
    @anmarchenko
    anmarchenko committed Jun 25, 2024
    Configuration menu
    Copy the full SHA
    b1061ef View commit details
    Browse the repository at this point in the history

  3. move AppStarted telemetry event out of the critical path

    @anmarchenko
    anmarchenko committed Jun 25, 2024
    Configuration menu
    Copy the full SHA
    d4350ca View commit details
    Browse the repository at this point in the history

  4. fix failing tests by waiting for worker startup

    @anmarchenko
    anmarchenko committed Jun 25, 2024
    Configuration menu
    Copy the full SHA
    bb9f888 View commit details
    Browse the repository at this point in the history

  5. debug logging, flushing events, attempt at fixing failing test for wo… 

    …rker
    @anmarchenko
    anmarchenko committed Jun 25, 2024
    Configuration menu
    Copy the full SHA
    a11e48d View commit details
    Browse the repository at this point in the history

  6. don't send heartbeat event if started event wasn't successfully sent

    @anmarchenko
    anmarchenko committed Jun 25, 2024
    Configuration menu
    Copy the full SHA
    6b9204e View commit details
    Browse the repository at this point in the history

  7. fix client_spec

    @anmarchenko
    anmarchenko committed Jun 25, 2024
    Configuration menu
    Copy the full SHA
    dcedde6 View commit details
    Browse the repository at this point in the history

  8. enqueue events to be sent later by worker instead of sending them syn… 

    …chronously
    @anmarchenko
    anmarchenko committed Jun 25, 2024
    Configuration menu
    Copy the full SHA
    3a0256f View commit details
    Browse the repository at this point in the history

  9. rename Telemetry::Client to Telemetry::Component to better reflect it… 

    …s purpose
    @anmarchenko
    anmarchenko committed Jun 25, 2024
    Configuration menu
    Copy the full SHA
    06293a4 View commit details
    Browse the repository at this point in the history

  10. leftover of telemetry component rename

    @anmarchenko
    anmarchenko committed Jun 25, 2024
    Configuration menu
    Copy the full SHA
    d31a088 View commit details
    Browse the repository at this point in the history

  11. add Core::Utils::OnlyOnceSuccessful to execute code with only one suc… 

    …cess
    @anmarchenko
    anmarchenko committed Jun 25, 2024
    Configuration menu
    Copy the full SHA
    4f2aab8 View commit details
    Browse the repository at this point in the history

  12. ensure that app-started event is sent at most once, flush events befo… 

    …re stopping worker
    @anmarchenko
    anmarchenko committed Jun 25, 2024
    Configuration menu
    Copy the full SHA
    8b7a50e View commit details
    Browse the repository at this point in the history

  13. remove Telemetry::Component.started! as right now it just contains de… 

    …pendency collection event logic, move it to the Worker
    @anmarchenko
    anmarchenko committed Jun 25, 2024
    Configuration menu
    Copy the full SHA
    69adca1 View commit details
    Browse the repository at this point in the history

  14. change the wrong expectation in workers spec

    @anmarchenko
    anmarchenko committed Jun 25, 2024
    Configuration menu
    Copy the full SHA
    440746a View commit details
    Browse the repository at this point in the history

  15. send app-dependencies-loaded event right after app-started event in t… 

    …he same thread
    @anmarchenko
    anmarchenko committed Jun 25, 2024
    Configuration menu
    Copy the full SHA
    76bfd9f View commit details
    Browse the repository at this point in the history

  16. add limit option to OnlyOnceSuccessful util

    @anmarchenko
    anmarchenko committed Jun 25, 2024
    Configuration menu
    Copy the full SHA
    cd239cb View commit details
    Browse the repository at this point in the history

  17. limit telemetry app-started event retries

    @anmarchenko
    anmarchenko committed Jun 25, 2024
    Configuration menu
    Copy the full SHA
    71c9e5c View commit details
    Browse the repository at this point in the history

  18. do not instantiate empty array every time when sending events

    @anmarchenko
    anmarchenko committed Jun 25, 2024
    Configuration menu
    Copy the full SHA
    f4d349a View commit details
    Browse the repository at this point in the history

  19. lower HTTP timeout for telemetry worker

    @anmarchenko
    anmarchenko committed Jun 25, 2024
    Configuration menu
    Copy the full SHA
    3ccf58b View commit details
    Browse the repository at this point in the history

  20. do not run httprb spec for jruby

    @anmarchenko
    anmarchenko committed Jun 25, 2024
    Configuration menu
    Copy the full SHA
    5729921 View commit details
    Browse the repository at this point in the history

  21. skip more http specs for jruby

    @anmarchenko
    anmarchenko committed Jun 25, 2024
    Configuration menu
    Copy the full SHA
    3b4e4d4 View commit details
    Browse the repository at this point in the history