Ed/debug web server #1522

elliedavidson · 2023-08-07T22:31:46Z

This PR:

Reduces the number of successful views needed for the web server, since it will naturally be slower than the memory network
Removes old tests from the web server crate

shenkeyao

LGTM. Maybe sync with main again in case test_with_failures causes the CI to fail.

shenkeyao · 2023-08-07T22:36:06Z

LGTM. Maybe sync with main again in case test_with_failures causes the CI to fail.

Oh there are also format errors.

…ebug_web_server

shenkeyao and others added 6 commits August 7, 2023 12:07

Add to justfile

090d495

Add logging to web server test

f682bed

Delete old web server tests

eabc3e4

Merge branch 'main' into keyao/test_kill_nodes

d68e19e

Update DA size

ec423a6

Reduce successful views needed for web server test

cc649a6

elliedavidson requested a review from shenkeyao August 7, 2023 22:31

shenkeyao previously approved these changes Aug 7, 2023

View reviewed changes

elliedavidson added 2 commits August 7, 2023 18:42

Merge remote-tracking branch 'origin/keyao/test_kill_nodes' into ed/d…

987fdba

…ebug_web_server

Sync with origin/keyao/test_kill_nodes and cargo fmt

3e6a19e

elliedavidson dismissed shenkeyao’s stale review via 3e6a19e August 7, 2023 22:43

elliedavidson added 2 commits August 7, 2023 18:48

One clippy warning

ac5baa4

fmt

1154dc3

elliedavidson requested review from bfish713, DieracDelta, dailinsubjam and kmchicoine August 8, 2023 11:58

bfish713 approved these changes Aug 8, 2023

View reviewed changes

elliedavidson merged commit a9320e2 into main Aug 8, 2023
6 of 7 checks passed

elliedavidson deleted the ed/debug_web_server branch August 8, 2023 13:18

Provide feedback