Call peach_stats and peach_network directly (remove JSON-RPC client calls) #62

2022-01-06T10:03:08Z

glyph commented

2022-01-06 10:03:08 +00:00

Another step towards making peach-web leaner.

src/routes/status/device.rs
- Make direct calls to peach-stats (instead of client calls via peach-lib)
src/routes/status/network.rs
- Make direct calls to peach-network (instead of client calls via peach-lib)
- Add IfaceTraffic struct and convert_traffic() to modify traffic values & assign measurement units
templates/status/device.html.tera
- Remove conditional border colour rendering based on peach_stats RPC server status

Another step towards making peach-web leaner. - `src/routes/status/device.rs` - Make direct calls to peach-stats (instead of client calls via peach-lib) - `src/routes/status/network.rs` - Make direct calls to peach-network (instead of client calls via peach-lib) - Add `IfaceTraffic` `struct` and `convert_traffic()` to modify traffic values & assign measurement units - `templates/status/device.html.tera` - Remove conditional border colour rendering based on peach_stats RPC server status CC: @notplants

glyph added 6 commits 2022-01-06 10:03:08 +00:00

add deps for network and stats a0e80fcda7

replace stats rpc client calls with direct calls to peach_stats 3ab3e65eb7

replace network rpc client calls with direct calls to peach_network 567b0bbc2a

Merge branch 'main' into direct_call_net_stats 17d52c771f

Merge crypto library update for peach-lib.

cleanup use paths and leave network_ping note 49ad74595c

add fullstop to docs sentence 57ed0ab66a

glyph requested review from notplants 2022-01-06 10:03:14 +00:00

glyph added the peach-web label 2022-01-06 10:03:30 +00:00

notplants commented

2022-01-06 15:48:49 +00:00

LGTM

is peach fully functional without the microservices now? or work is still needed?
once it is fully functional,
we should at some point,
work on updating peach-config to install and update without the microservices,
and also modify the python package building scripts too.

LGTM is peach fully functional without the microservices now? or work is still needed? once it is fully functional, we should at some point, work on updating peach-config to install and update without the microservices, and also modify the python package building scripts too.

glyph commented

2022-01-07 10:01:15 +00:00

@notplants

is peach fully functional without the microservices now?

Yes, I believe so :)

we should at some point,
work on updating peach-config to install and update without the microservices,
and also modify the python package building scripts too.

Agreed. I think we can pivot to that once we have the sbot functionality incorporated into peach-web, as well as a general config file for PeachCloud.

@notplants > is peach fully functional without the microservices now? Yes, I believe so :) > we should at some point, work on updating peach-config to install and update without the microservices, and also modify the python package building scripts too. Agreed. I think we can pivot to that once we have the sbot functionality incorporated into peach-web, as well as a general config file for PeachCloud.

glyph merged commit 42774674e5 into main

2022-01-07 10:01:24 +00:00

glyph deleted branch direct_call_net_stats

2022-01-07 10:01:24 +00:00

glyph referenced this issue from a commit

2022-01-07 10:01:24 +00:00

Merge pull request 'Call peach_stats and peach_network directly (remove JSON-RPC client calls)' (#62) from direct_call_net_stats into main

glyph referenced this pull request

2022-01-12 09:58:18 +00:00

Fig regression of peach-dyndns-updater #63

Sign in to join this conversation.