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

⚠️ RSDK-7903 Include unhealthy remotes in resource list with status #4273

Closed
wants to merge 41 commits into from
Closed

⚠️ RSDK-7903 Include unhealthy remotes in resource list with status #4273

wants to merge 41 commits into from
+515 −111

Commits on Oct 3, 2024

  1. get resource names with resource statuses instead 

    removed mutex usage to prevent deadlock
    @maximpertsov
    maximpertsov committed Oct 3, 2024
    Configuration menu
    Copy the full SHA
    968b479 View commit details
    Browse the repository at this point in the history

  2. TestStatusRemote: inject machine status

    @maximpertsov
    maximpertsov committed Oct 3, 2024
    Configuration menu
    Copy the full SHA
    282e057 View commit details
    Browse the repository at this point in the history

  3. filter remotes in client

    @maximpertsov
    maximpertsov committed Oct 3, 2024
    Configuration menu
    Copy the full SHA
    d837e20 View commit details
    Browse the repository at this point in the history

  4. update mocks to include machine status for resources

    @maximpertsov
    maximpertsov committed Oct 3, 2024
    Configuration menu
    Copy the full SHA
    07d9a72 View commit details
    Browse the repository at this point in the history

  5. account for double log in test

    @maximpertsov
    maximpertsov committed Oct 3, 2024
    Configuration menu
    Copy the full SHA
    83cdc93 View commit details
    Browse the repository at this point in the history

  6. WIP: add machine status mocks to sessions remotes

    @maximpertsov
    maximpertsov committed Oct 3, 2024
    Configuration menu
    Copy the full SHA
    cf85b6b View commit details
    Browse the repository at this point in the history

  7. flyby: dedent session table tests

    @maximpertsov
    maximpertsov committed Oct 3, 2024
    Configuration menu
    Copy the full SHA
    f274e56 View commit details
    Browse the repository at this point in the history

  8. whitelist MachineStatus for sessions

    @maximpertsov
    maximpertsov committed Oct 3, 2024
    Configuration menu
    Copy the full SHA
    4754282 View commit details
    Browse the repository at this point in the history

  9. Revert "flyby: dedent session table tests" 

    This reverts commit cb93409.
    @maximpertsov
    maximpertsov committed Oct 3, 2024
    Configuration menu
    Copy the full SHA
    b232589 View commit details
    Browse the repository at this point in the history

  10. remove commented code

    @maximpertsov
    maximpertsov committed Oct 3, 2024
    Configuration menu
    Copy the full SHA
    05a7329 View commit details
    Browse the repository at this point in the history

  11. remove commented code

    @maximpertsov
    maximpertsov committed Oct 3, 2024
    Configuration menu
    Copy the full SHA
    eed4bed View commit details
    Browse the repository at this point in the history

  12. store statuses instead of names in client

    @maximpertsov
    maximpertsov committed Oct 3, 2024
    Configuration menu
    Copy the full SHA
    c93b936 View commit details
    Browse the repository at this point in the history

  13. lint

    @maximpertsov
    maximpertsov committed Oct 3, 2024
    Configuration menu
    Copy the full SHA
    a2d658d View commit details
    Browse the repository at this point in the history

  14. fixup client test

    @maximpertsov
    maximpertsov committed Oct 3, 2024
    Configuration menu
    Copy the full SHA
    d5e9116 View commit details
    Browse the repository at this point in the history

  15. debug

    @maximpertsov
    maximpertsov committed Oct 3, 2024
    Configuration menu
    Copy the full SHA
    8cb9b03 View commit details
    Browse the repository at this point in the history

  16. return fully-qualified remote names

    @maximpertsov
    maximpertsov committed Oct 3, 2024
    Configuration menu
    Copy the full SHA
    fd5da19 View commit details
    Browse the repository at this point in the history

  17. @dgottlieb: add offline remote test 

    Pulled from viamrobotics#4268
    @maximpertsov
    maximpertsov committed Oct 3, 2024
    Configuration menu
    Copy the full SHA
    58f555f View commit details
    Browse the repository at this point in the history

  18. cache full machine status

    @maximpertsov
    maximpertsov committed Oct 3, 2024
    Configuration menu
    Copy the full SHA
    6480d49 View commit details
    Browse the repository at this point in the history

  19. use machine status to decide if remotes need to be updated

    @maximpertsov
    maximpertsov committed Oct 3, 2024
    Configuration menu
    Copy the full SHA
    673ea72 View commit details
    Browse the repository at this point in the history

  20. remove log test 

    the logs we are testing for are output by an internal logger
    @maximpertsov
    maximpertsov committed Oct 3, 2024
    Configuration menu
    Copy the full SHA
    2d18da7 View commit details
    Browse the repository at this point in the history

  21. framesystem -> frame_system

    @maximpertsov
    maximpertsov committed Oct 3, 2024
    Configuration menu
    Copy the full SHA
    49cb0db View commit details
    Browse the repository at this point in the history

  22. flyby: minor clean-up + remove conflicting mod/var names

    @maximpertsov
    maximpertsov committed Oct 3, 2024
    Configuration menu
    Copy the full SHA
    278ad51 View commit details
    Browse the repository at this point in the history

  23. mock machine status for dummy robot

    @maximpertsov
    maximpertsov committed Oct 3, 2024
    Configuration menu
    Copy the full SHA
    3e190ef View commit details
    Browse the repository at this point in the history

  24. lint

    @maximpertsov
    maximpertsov committed Oct 3, 2024
    Configuration menu
    Copy the full SHA
    c5cc1fd View commit details
    Browse the repository at this point in the history

  25. update comment

    @maximpertsov
    maximpertsov committed Oct 3, 2024
    Configuration menu
    Copy the full SHA
    af977e5 View commit details
    Browse the repository at this point in the history

  26. mark disconnected

    @maximpertsov
    maximpertsov committed Oct 3, 2024
    Configuration menu
    Copy the full SHA
    5506d55 View commit details
    Browse the repository at this point in the history

  27. REVERT: less logs

    @maximpertsov
    maximpertsov committed Oct 3, 2024
    Configuration menu
    Copy the full SHA
    426e873 View commit details
    Browse the repository at this point in the history

  28. add Disconnected state

    @maximpertsov
    maximpertsov committed Oct 3, 2024
    Configuration menu
    Copy the full SHA
    0c778b0 View commit details
    Browse the repository at this point in the history

  29. CR@benjirewis+@cheukt: update remote lookup + tests + MachineStatus l… 

    …ogic

Bundled changes from pairing session:

* fix remote node lookup nodes correctly
* update to MachineStatus return an error and no statuses when a client is disconnected
* update tests accordingly
    @maximpertsov
    maximpertsov committed Oct 3, 2024
    Configuration menu
    Copy the full SHA
    c64fe9d View commit details
    Browse the repository at this point in the history

  30. lint

    @maximpertsov
    maximpertsov committed Oct 3, 2024
    Configuration menu
    Copy the full SHA
    b5c4898 View commit details
    Browse the repository at this point in the history

  31. exclude remote apis from machine status resource output

    @maximpertsov
    maximpertsov committed Oct 3, 2024
    Configuration menu
    Copy the full SHA
    c32f367 View commit details
    Browse the repository at this point in the history

  32. revert changes from bad rebase

    @maximpertsov
    maximpertsov committed Oct 3, 2024
    Configuration menu
    Copy the full SHA
    32d33fd View commit details
    Browse the repository at this point in the history

  33. revert to using unhealthy state as disconnected

    @maximpertsov
    maximpertsov committed Oct 3, 2024
    Configuration menu
    Copy the full SHA
    40e0559 View commit details
    Browse the repository at this point in the history

  34. remove log suppression

    @maximpertsov
    maximpertsov committed Oct 3, 2024
    Configuration menu
    Copy the full SHA
    d79ebc4 View commit details
    Browse the repository at this point in the history

  35. update machine status error handling in update remote names

    @maximpertsov
    maximpertsov committed Oct 3, 2024
    Configuration menu
    Copy the full SHA
    d2465ff View commit details
    Browse the repository at this point in the history

  36. remove TODO

    @maximpertsov
    maximpertsov committed Oct 3, 2024
    Configuration menu
    Copy the full SHA
    e401dc0 View commit details
    Browse the repository at this point in the history

  37. Revert "revert to using unhealthy state as disconnected" 

    This reverts commit 34bc7f4.
    @maximpertsov
    maximpertsov committed Oct 3, 2024
    Configuration menu
    Copy the full SHA
    c4c0b89 View commit details
    Browse the repository at this point in the history

  38. nolint (temp)

    @maximpertsov
    maximpertsov committed Oct 3, 2024
    Configuration menu
    Copy the full SHA
    864e1d2 View commit details
    Browse the repository at this point in the history

  39. generate new state

    @maximpertsov
    maximpertsov committed Oct 3, 2024
    Configuration menu
    Copy the full SHA
    c5cf943 View commit details
    Browse the repository at this point in the history

  40. fixup one more test

    @maximpertsov
    maximpertsov committed Oct 3, 2024
    Configuration menu
    Copy the full SHA
    875332c View commit details
    Browse the repository at this point in the history

  41. remove RDK internal resources from test expectation

    @maximpertsov
    maximpertsov committed Oct 3, 2024
    Configuration menu
    Copy the full SHA
    022194a View commit details
    Browse the repository at this point in the history