labgrid-project / labgrid

Embedded systems control library for development, testing and installation
https://labgrid.readthedocs.io/
Other
343 stars 179 forks source link

add USBHub resource and export #1548

Open jluebbe opened 6 days ago

jluebbe commented 6 days ago

This resource is a bit unusual, as there is no corresponding driver, as it is only useful to monitor whether the expected USB hubs are detected by an exporter.

In larger labs, there are often USB hubs that are considered part of the infrastructure. When these fail (either temporarily or permanently), the connected devices aren't detected any more. For users of individual places, this leads to unavailable resources without a clear cause.

By configuring USBHub resources for the infrastructure hubs in the exporters, we can start monitoring hub availability centrally via the coordinator.

codecov[bot] commented 6 days ago

Codecov Report

Attention: Patch coverage is 62.50000% with 3 lines in your changes missing coverage. Please review.

Project coverage is 56.5%. Comparing base (b90ac44) to head (8e4ce3d).

:white_check_mark: All tests successful. No failed tests found.

Files with missing lines Patch % Lines
labgrid/resource/udev.py 57.1% 3 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #1548 +/- ## ====================================== Coverage 56.5% 56.5% ====================================== Files 168 168 Lines 13116 13124 +8 ====================================== + Hits 7417 7422 +5 - Misses 5699 5702 +3 ``` | [Flag](https://app.codecov.io/gh/labgrid-project/labgrid/pull/1548/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=labgrid-project) | Coverage Δ | | |---|---|---| | [3.10](https://app.codecov.io/gh/labgrid-project/labgrid/pull/1548/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=labgrid-project) | `56.5% <62.5%> (+<0.1%)` | :arrow_up: | | [3.11](https://app.codecov.io/gh/labgrid-project/labgrid/pull/1548/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=labgrid-project) | `56.5% <62.5%> (+<0.1%)` | :arrow_up: | | [3.12](https://app.codecov.io/gh/labgrid-project/labgrid/pull/1548/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=labgrid-project) | `56.5% <62.5%> (+<0.1%)` | :arrow_up: | | [3.13](https://app.codecov.io/gh/labgrid-project/labgrid/pull/1548/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=labgrid-project) | `56.5% <62.5%> (+<0.1%)` | :arrow_up: | | [3.9](https://app.codecov.io/gh/labgrid-project/labgrid/pull/1548/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=labgrid-project) | `56.5% <62.5%> (+<0.1%)` | :arrow_up: | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=labgrid-project#carryforward-flags-in-the-pull-request-comment) to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.