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.

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

Multihomed transport (server) addrs 🕶️ #367

Merged
merged 24 commits into from
Mar 21, 2025
Merged
Changes from 1 commit
Commits
Show all changes
24 commits
Select commit Hold shift + click to select a range
dec2b1f
Reapply "Port all tests to new `reg_addr` fixture name"
goodboy Mar 20, 2025
7ce4bc4
Init-support for "multi homed" transports
goodboy Sep 27, 2023
ef4c4be
Add libp2p style "multi-address" parser from `piker`
goodboy Sep 29, 2023
4db377c
Rename to `parse_maddr()` and fill out doc strings
goodboy Sep 29, 2023
7246749
Add post-mortem catch around failed transport addr binds to aid with …
goodboy Oct 3, 2023
61d82d4
Oof, default reg addrs needs to be in `list[tuple]` form..
goodboy Oct 7, 2023
2e17b08
Always set default reg addr in `find_actor()` if not defined
goodboy Oct 18, 2023
0246c82
._root: set a `_default_lo_addrs` and apply it when not provided by c…
goodboy Oct 18, 2023
94c89fd
Facepalm, `wait_for_actor()` dun take an addr `list`..
goodboy Oct 18, 2023
6d671f6
Rename fixture `arb_addr` -> `reg_addr` and set the session value glo…
goodboy Oct 18, 2023
f834b35
Ensure `registry_addrs` is always set to something
goodboy Oct 18, 2023
4868bf2
Always dynamically re-read the `._root._default_lo_addrs` value in `f…
goodboy Oct 18, 2023
51bd389
Expose per-actor registry addrs via `.reg_addrs`
goodboy Oct 19, 2023
8b0b4ab
Change remaining internals to use `Actor.reg_addrs`
goodboy Oct 19, 2023
5f0bfea
Test with `any(portals)` since `gather_contexts()` will return `list[…
goodboy Nov 6, 2023
2d541fd
Fix doc string "its" typo..
goodboy Nov 6, 2023
ebf9909
Add `open_root_actor(ensure_registry: bool)`
goodboy Nov 7, 2023
15a4a2a
`.discovery.get_arbiter()`: add warning around this now deprecated usage
goodboy Dec 12, 2023
dbd79d8
Log chan-server-startup failures via `.exception()`
goodboy Jan 2, 2024
f947bdf
Use `import <name> as <name>,` style over `__all__` in pkg mod
goodboy Jan 2, 2024
14f34c1
`_root`: drop unused `typing` import
goodboy Jan 2, 2024
9082efb
Add a `._state._runtime_vars['_registry_addrs']`
goodboy Mar 8, 2024
5ffdda7
More spaceless union type annots
goodboy Mar 11, 2024
2b12444
Unmask `pytest.ini` log-capture lines (again)
goodboy Feb 23, 2024
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
.discovery.get_arbiter(): add warning around this now deprecated usage
  • Loading branch information
goodboy committed Mar 20, 2025

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
commit 15a4a2a51ef23de6aeb8ff75eda37ca11eacd3de
14 changes: 12 additions & 2 deletions tractor/_discovery.py
Original file line number Diff line number Diff line change
@@ -76,8 +76,18 @@ async def get_registry(
yield regstr_ptl


# TODO: deprecate and remove _arbiter form
get_arbiter = get_registry

# TODO: deprecate and this remove _arbiter form!
@acm
async def get_arbiter(*args, **kwargs):
warnings.warn(
'`tractor.get_arbiter()` is now deprecated!\n'
'Use `.get_registry()` instead!',
DeprecationWarning,
stacklevel=2,
)
async with get_registry(*args, **kwargs) as to_yield:
yield to_yield


@acm