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

Getting online domains using peers health check #8795

Closed
Closed
Changes from 1 commit
Commits
Show all changes
41 commits
Select commit Hold shift + click to select a range
355e513
[syft/network] getting online networks for python gateway node
khoaguin May 9, 2024
9316f69
[syft/network] when checking for online domains, if not able to creat…
khoaguin May 9, 2024
7fe68f9
[syft/network] on getting online domains by checking `domain.ping_sta…
khoaguin May 9, 2024
25eaaee
[syft/network] add total line to show number of online networks / all…
khoaguin May 10, 2024
eddc66f
[syft/network] - update ping url for networks without frontend
khoaguin May 10, 2024
8182a14
[syft/network] integrating `background_tasks` into `Orchestra.launch`
khoaguin May 10, 2024
8268c29
Merge branch 'dev' into peer-health-check-online-domains
khoaguin May 13, 2024
da596ff
[syft/network] pass `background_tasks=True` to `worker_class`
khoaguin May 13, 2024
71e4a34
[test/integration] add removing peers to the beginning of some tests
khoaguin May 13, 2024
0f89b8b
[syft/network] revert back to use `ping_status` instead of `ping_stat…
khoaguin May 14, 2024
744a289
[test/integration] add remove existing peers before testing for `test…
khoaguin May 14, 2024
5fd4c31
Merge branch 'dev' into peer-health-check-online-domains
khoaguin May 14, 2024
c3e0e62
[test/integration] trimming down `network/gateway_test`
khoaguin May 14, 2024
c89d671
[test/integration] add some waiting times for local gateway tests bef…
khoaguin May 14, 2024
243ca68
Merge branch 'dev' into peer-health-check-online-domains
khoaguin May 14, 2024
3782033
[test/integration] update gateway tests
khoaguin May 14, 2024
1545ec3
[test/integration] update gateway k8s tests
khoaguin May 14, 2024
b2b057f
Merge branch 'dev' into peer-health-check-online-domains
khoaguin May 14, 2024
2c45fba
[syft/network] - pick the highest priority route to be the oldest by …
khoaguin May 15, 2024
eaf59b5
Merge branch 'dev' into peer-health-check-online-domains
khoaguin May 15, 2024
a6cd9c9
[syft/network] `PeerHealthCheckTask.peer_route_heathcheck` now only
khoaguin May 15, 2024
e20ff1f
Merge branch 'dev' into peer-health-check-online-domains
khoaguin May 15, 2024
231bcbf
fix linting
khoaguin May 15, 2024
095a235
Merge branch 'dev' into peer-health-check-online-domains
khoaguin May 15, 2024
342439f
[syft/network] return Err if the returned node peer is None
khoaguin May 15, 2024
1384e0e
[test/integration] allow running `gateway_local_test.py` in `syft.tes…
khoaguin May 15, 2024
2dd8873
[test/integration] add `@pytest.mark.network` for tests in `gateway_t…
khoaguin May 15, 2024
821efef
Merge branch 'dev' into peer-health-check-online-domains
khoaguin May 15, 2024
abee20d
Merge branch 'dev' into peer-health-check-online-domains
khoaguin May 16, 2024
7507fff
add a try catch block to catch invalid api registry entries
rasswanth-s May 16, 2024
4619473
[test/integration] trimming down `gateway_local_test.py`
khoaguin May 16, 2024
3e69bbc
Merge branch 'dev' into peer-health-check-online-domains
khoaguin May 16, 2024
3e42aad
[test/integration] small fix
khoaguin May 16, 2024
208f7cc
Merge branch 'dev' into peer-health-check-online-domains
khoaguin May 19, 2024
a1d9dab
Merge branch 'dev' into peer-health-check-online-domains
khoaguin May 20, 2024
63e4648
Merge branch 'dev' into peer-health-check-online-domains
shubham3121 May 21, 2024
a138151
Merge branch 'dev' into peer-health-check-online-domains
khoaguin May 22, 2024
96a3253
Merge branch 'dev' into peer-health-check-online-domains
khoaguin May 23, 2024
afd1231
Merge branch 'dev' into peer-health-check-online-domains
khoaguin May 24, 2024
039a76f
Merge branch 'dev' into peer-health-check-online-domains
khoaguin May 24, 2024
dd01b16
Merge branch 'dev' into peer-health-check-online-domains
khoaguin May 27, 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
[test/integration] allow running gateway_local_test.py in `syft.tes…
…t.integration`
khoaguin committed May 15, 2024
commit 1384e0e876c3a6a91db72546f000289d8602e5df
8 changes: 6 additions & 2 deletions packages/syft/src/syft/service/network/network_service.py
Original file line number Diff line number Diff line change
@@ -135,10 +135,14 @@ def update_peer_ping_status(
# get the node peer for the given sender peer_id
result = self.get_by_uid(credentials=credentials, uid=peer.id)
if result.is_err():
return Err(message=f"Failed to query peer from stash. Err: {result.err()}")
return Err(
f"Failed to query peer peer {peer.id} with name '{peer.name}' from stash. Err: {result.err()}"
)
existing = result.ok()
if existing is None:
return Err(message="Failed to query peer from stash: peer is None")
return Err(
f"Failed to query peer {peer.id} with name '{peer.name}' from stash: peer is None"
)
existing.ping_status = peer.ping_status
existing.ping_status_message = peer.ping_status_message
existing.pinged_timestamp = peer.pinged_timestamp
2 changes: 1 addition & 1 deletion tox.ini
Original file line number Diff line number Diff line change
@@ -649,7 +649,7 @@ allowlist_externals =
setenv =
PYTEST_MODULES = {env:PYTEST_MODULES:local_node}
ASSOCIATION_REQUEST_AUTO_APPROVAL = {env:ASSOCIATION_REQUEST_AUTO_APPROVAL:true}
PYTEST_FLAGS = {env:PYTEST_FLAGS:--ignore=tests/integration/local/gateway_local_test.py --ignore=tests/integration/local/job_test.py}
PYTEST_FLAGS = {env:PYTEST_FLAGS:--ignore=tests/integration/local/job_test.py}
commands =
python -c 'import syft as sy; sy.stage_protocol_changes()'