Skip to content

Daemon sockaddr config#417

Merged
goodboy merged 6 commits intomasterfrom
daemon_sockaddr_config
Nov 10, 2022
Merged

Daemon sockaddr config#417
goodboy merged 6 commits intomasterfrom
daemon_sockaddr_config

Conversation

@goodboy
Copy link
Copy Markdown
Contributor

@goodboy goodboy commented Nov 10, 2022

Add support for a -p <port#> flag to pikerd and piker chart such that hackers can easily run separate actor tree daemon-clusters without affecting a different code-version "production" service instance running locally.

tractor's discovery/registry system distinguishes actor trees on a singleton root-actor socket addr and this patch makes it easy to change that from CLI.

Further changes include exposing piker.open_feed() and open_piker_runtime() entrypoints as part of beginning to offer and document a public API for our data feed layer.

Allows starting UI apps and passing the `pikerd` registry socket-addr
args via `--host` or `--port` such that a separate actor tree can be
started by selecting an unused port. This is handy when hacking new
features but while also wishing to run a more stable version of the code
for trading on the same host.
@goodboy goodboy added (sub-)systems general sw design and eng config labels Nov 10, 2022
@goodboy goodboy requested a review from guilledk November 10, 2022 18:11
Base automatically changed from no_signal_pi_overlays to master November 10, 2022 22:31
@goodboy goodboy merged commit a44b8e3 into master Nov 10, 2022
@goodboy goodboy deleted the daemon_sockaddr_config branch November 10, 2022 22:31
goodboy added a commit that referenced this pull request Nov 12, 2022
This slipped in early from #414 before merge and was likely due to
cherry-picking from #417.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

config (sub-)systems general sw design and eng

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants