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

add back iroh status -w feature using new rpc streams #602

Merged
merged 7 commits into from
Dec 19, 2022
Merged

Conversation

ramfox
Copy link
Contributor

@ramfox ramfox commented Dec 15, 2022

@ramfox ramfox requested a review from rklaehn December 15, 2022 05:02
@ramfox ramfox self-assigned this Dec 15, 2022
@ramfox ramfox added the c-rpc label Dec 15, 2022
@ramfox ramfox added this to the v0.2.0 milestone Dec 15, 2022
iroh-gateway/src/rpc.rs Outdated Show resolved Hide resolved
iroh-gateway/src/rpc.rs Outdated Show resolved Hide resolved
iroh-gateway/src/rpc.rs Outdated Show resolved Hide resolved
iroh-gateway/src/rpc.rs Outdated Show resolved Hide resolved
iroh-gateway/src/rpc.rs Outdated Show resolved Hide resolved
iroh-p2p/src/rpc.rs Outdated Show resolved Hide resolved
iroh-rpc-client/src/network.rs Outdated Show resolved Hide resolved
iroh-rpc-client/src/status.rs Show resolved Hide resolved
iroh-rpc-types/src/gateway.rs Outdated Show resolved Hide resolved
iroh-rpc-types/src/gateway.rs Show resolved Hide resolved
iroh-rpc-types/src/p2p.rs Show resolved Hide resolved
@rklaehn
Copy link
Contributor

rklaehn commented Dec 15, 2022

Great work picking up the new stuff so quickly.

@dignifiedquire
Copy link
Contributor

@ramfox you have merge conflicts

iroh-api/src/api.rs Outdated Show resolved Hide resolved
@ramfox ramfox merged commit 6e9dae2 into main Dec 19, 2022
@ramfox ramfox deleted the ramfox/health branch December 19, 2022 16:42
ppodolsky pushed a commit to izihawa/iroh that referenced this pull request Dec 28, 2022
)

- add back `iroh status -w` feature using new rpc streams
- add a `HEALTH_POLL_WAIT` constant for how long to wait between health check attempts
- add a `VERSION` constant in each of `iroh-p2p`, `iroh-gateway`, `iroh-store`
- rename `ServiceStatus` to `StatusType`
- remove unused `StatusType` variants
- rename `StatusRow` to `ServiceStatus`
- rename `StatusTable` to `ClientStatus`
- add a `ServiceType` enum with `P2p`, `Gateway`, and `Store` options
- add `version` field to `ServiceStatus`, remove `number` field
- adjust cli output to show the version & not show the "number"
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.

Refine status monitoring
3 participants