Skip to content

Conversation

@alexjustesen
Copy link
Owner

Description

This PR refactors running an Ookla speedtest. The old process was fractured between running scheduled, manual and CLI tests.

Changelog

Changed

  • replaced app:speedtest with app:run-ookla-speedtest command
  • running a manual, scheduler or CLI speedtest now uses the same process through the application by routing all requests through RunOoklaSpeedtest

Fixed

  • fractured process to run an Ookla speedtest

New process

flowchart TD
    A[Scheduled test] --> C{Due?}
    B[Manual test] --> D
    C -->|Yes| D(Run speedtest command)
    D --> E(Queue speedtest job)
    E --> F{Notify?}
    F -->|Yes| G(Queue notifications)
    G --> H(Send notifications)
    F --> |No| Z[Done]
    C --> |No| Z[Done]
    H --> Z
Loading

@alexjustesen alexjustesen added the chore Nothing fancy, just needs to get done label Dec 21, 2023
@alexjustesen alexjustesen self-assigned this Dec 21, 2023
@alexjustesen alexjustesen merged commit be306a9 into main Dec 21, 2023
@alexjustesen alexjustesen deleted the refactor-running-an-ookla-speedtest branch December 21, 2023 17:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

chore Nothing fancy, just needs to get done

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants