Skip to content

Remove most of humility rpc#611

Merged
mkeeter merged 2 commits intomasterfrom
mkeeter/humility-discover
Apr 6, 2026
Merged

Remove most of humility rpc#611
mkeeter merged 2 commits intomasterfrom
mkeeter/humility-discover

Conversation

@mkeeter
Copy link
Copy Markdown
Contributor

@mkeeter mkeeter commented Apr 6, 2026

(this is a rebased version of #610 , which was merged into the wrong branch)

humility rpc --ip=... --call ... is almost identical to humility --ip=... hiffy --call ..., and duplicates the calling implementation. The one exception is being able to execute a udprpc call across every compatible SP. I don't think we use that functionality anywhere (searching the whole oxidecomputer org finds no users of humility rpc), so I'd vote to delete it.

This leaves humility rpc --listen as the only subcommand, which is still useful (and not duplicated elsewhere). It's been renamed to humility discover.

@mkeeter mkeeter enabled auto-merge (squash) April 6, 2026 20:55
@mkeeter mkeeter merged commit c556213 into master Apr 6, 2026
12 checks passed
@mkeeter mkeeter deleted the mkeeter/humility-discover branch April 7, 2026 20:28
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.

1 participant