Skip to content
This repository has been archived by the owner on Apr 12, 2023. It is now read-only.

behavior change: relay integration and proxy health check #27

Closed
xmonader opened this issue Feb 28, 2023 · 1 comment
Closed

behavior change: relay integration and proxy health check #27

xmonader opened this issue Feb 28, 2023 · 1 comment
Assignees
Labels
type_bug Something isn't working
Milestone

Comments

@xmonader
Copy link
Contributor

xmonader commented Feb 28, 2023

the health check was doing the following

  • grid proxy api itself: reaching the gridproxy api e.g /farms /nodes to make sure the server is responding (there's a ping endpoint now too on the gridproxy)
  • random nodes connectivity to yggdrassil: by trying to proxy simple command like zos.system.version on nodes via the proxy

Now with the Relay integrated, nodes are reachable through the relay, and we removed the proxying features of ygg threefoldtecharchive/tfgridclient_proxy#292 so this will always fail in the current checks

So, let's keep the gridproxy api health check only and remove the nodes checks

https://github.com/threefoldtech/tf_operations/issues/1360

@rawdaGastan rawdaGastan added the type_bug Something isn't working label Mar 5, 2023
@rawdaGastan rawdaGastan added this to the 1.0.0 milestone Mar 5, 2023
@A-Harby
Copy link

A-Harby commented Mar 6, 2023

Verified.
with new twin relay changes, all functionality works fine.
image
image

@A-Harby A-Harby closed this as completed Mar 6, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
type_bug Something isn't working
Projects
None yet
Development

No branches or pull requests

4 participants