Skip to content

lua: implement RTL8380M wired provider#249

Draft
cpunt wants to merge 2 commits into
migration-basefrom
switch-migration
Draft

lua: implement RTL8380M wired provider#249
cpunt wants to merge 2 commits into
migration-basefrom
switch-migration

Conversation

@cpunt
Copy link
Copy Markdown
Contributor

@cpunt cpunt commented May 18, 2026

Summary

  • keep the RTL8380M migration as a HAL wired provider, supervised by the existing HAL wired manager
  • add a narrow HTTP ref factory for the wired manager/provider path instead of passing the full bus connection into drivers
  • make the RTL8380M provider call the http service with response_parser=tolerant-http1 for Hydra-style responses
  • add the tolerant HTTP/1 transport under services.http.transport
  • keep HAL wired-provider facts raw so Device curates them into public cap/wired-provider/... for Wired

Verification

  • luacheck src/services/hal/backends/wired/providers/rtl8380m_http.lua src/services/http/policy.lua src/services/http/transport/client.lua src/services/http/transport/tolerant_http1.lua tests/unit/hal/wired_provider_spec.lua tests/unit/http/test_policy.lua tests/unit/http/transport/test_tolerant_http1.lua
  • git diff --check
  • direct module runner for:
    • unit.hal.wired_provider_spec
    • unit.http.transport.test_tolerant_http1
    • unit.device.test_wired_provider_curation
    • unit.wired.test_architecture
    • unit.wired.test_config
    • unit.wired.test_trunk_validation

Notes

The normal tests/run.lua path is blocked in this checkout before filters apply because http.request is not installed, so the focused tests above were run directly.

@lePereT
Copy link
Copy Markdown
Contributor

lePereT commented May 18, 2026

looking good Chris - but a couple of points:

  • there's a few arbitrary formatting changes in hal that show up as diffs just detract from the meaningful changes
  • more broadly i don't think that we should be passing the unfettered bus connection all the way through to drivers - that's a very broadly scoped capability. we should be passing something much narrower, only to drivers (like yours) that actually need this capability - open to what that looks like

@cpunt cpunt changed the title lua: implement RTL8380M wired provider lua: migrate switch service May 18, 2026
@cpunt cpunt changed the title lua: migrate switch service lua: implement RTL8380M wired provider May 18, 2026
@cpunt cpunt force-pushed the switch-migration branch from cac02ac to 59dc611 Compare May 19, 2026 09:24
@cpunt cpunt changed the base branch from rt-net-now to migration-base May 19, 2026 09:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants