From 64f6dad8f95e4972ee4bef26b9e5da6d6b577f13 Mon Sep 17 00:00:00 2001 From: ludamad Date: Fri, 6 Dec 2024 10:15:38 -0500 Subject: [PATCH] chore(network_test.sh): work around 143 by disabling stern (#10436) Likely will help with the random 143's we see --------- Co-authored-by: Mitch --- yarn-project/end-to-end/scripts/network_test.sh | 12 +++++++----- .../end-to-end/src/spartan/gating-passive.test.ts | 4 ++-- 2 files changed, 9 insertions(+), 7 deletions(-) diff --git a/yarn-project/end-to-end/scripts/network_test.sh b/yarn-project/end-to-end/scripts/network_test.sh index bcbda75d050..b9083df3d7e 100755 --- a/yarn-project/end-to-end/scripts/network_test.sh +++ b/yarn-project/end-to-end/scripts/network_test.sh @@ -53,11 +53,13 @@ if [ "$FRESH_INSTALL" = "true" ]; then kubectl delete namespace "$NAMESPACE" --ignore-not-found=true --wait=true --now --timeout=10m fi -STERN_PID="" +# STERN_PID="" function copy_stern_to_log() { - ulimit -n 4096 - stern spartan -n $NAMESPACE > $SCRIPT_DIR/network-test.log & - STERN_PID=$! + # TODO(AD) we need to figure out a less resource intensive solution than stern + # ulimit -n 4096 + # stern spartan -n $NAMESPACE > $SCRIPT_DIR/network-test.log & + echo "disabled until less resource intensive solution than stern implemented" > $SCRIPT_DIR/network-test.log & + # STERN_PID=$! } function show_status_until_pxe_ready() { @@ -113,7 +115,7 @@ show_status_until_pxe_ready & function cleanup() { # kill everything in our process group except our process - trap - SIGTERM && kill -9 $(pgrep -g $$ | grep -v $$) $(jobs -p) $STERN_PID &>/dev/null || true + trap - SIGTERM && kill -9 $(pgrep -g $$ | grep -v $$) $(jobs -p) &>/dev/null || true if [ "$CLEANUP_CLUSTER" = "true" ]; then kind delete cluster || true diff --git a/yarn-project/end-to-end/src/spartan/gating-passive.test.ts b/yarn-project/end-to-end/src/spartan/gating-passive.test.ts index 2285dac373d..6d8b52261aa 100644 --- a/yarn-project/end-to-end/src/spartan/gating-passive.test.ts +++ b/yarn-project/end-to-end/src/spartan/gating-passive.test.ts @@ -48,8 +48,8 @@ describe('a test that passively observes the network in the presence of network const ETHEREUM_HOST = `http://127.0.0.1:${HOST_ETHEREUM_PORT}`; const PXE_URL = `http://127.0.0.1:${HOST_PXE_PORT}`; - // 50% is the max that we expect to miss - const MAX_MISSED_SLOT_PERCENT = 0.5; + // 60% is the max that we expect to miss + const MAX_MISSED_SLOT_PERCENT = 0.6; afterAll(async () => { await runAlertCheck(config, qosAlerts, debugLogger);