aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorFlorin Coras <fcoras@cisco.com>2025-03-25 23:54:51 -0400
committerDave Wallace <dwallacelf@gmail.com>2025-03-26 17:21:28 +0000
commit020d931d12b39c8d1a9b6012b6660d4ccf9f787c (patch)
tree6b7f4b3b163bec9a08baf362a0ef4303c5d892d6
parent7e295a42f0fd91a266a063b60935f7c8c3f9919e (diff)
hs-test: fix redis test on ubuntu 24.04
Noticed in local runs that after snapshots, vcl crashes due to what appears to be a memory corruption. Given that snapshot are done in a forked child process, probably this is some mishandling of child processes in multi-threaded apps. Disable snapshotting for now. Type: fix Change-Id: Ide2dd3d68911b936742b41992485cc06db737564 Signed-off-by: Florin Coras <fcoras@cisco.com>
-rw-r--r--extras/hs-test/ldp_test.go6
1 files changed, 5 insertions, 1 deletions
diff --git a/extras/hs-test/ldp_test.go b/extras/hs-test/ldp_test.go
index 1933b0b1e8e..6ce0943d6c6 100644
--- a/extras/hs-test/ldp_test.go
+++ b/extras/hs-test/ldp_test.go
@@ -90,7 +90,11 @@ func RedisBenchmarkTest(s *LdpSuite) {
go func() {
defer GinkgoRecover()
- cmd := "redis-server --daemonize yes --protected-mode no --bind " + serverVethAddress + " --loglevel notice --logfile " + s.RedisServerLogFileName(s.Containers.ServerVpp)
+ // Avoid redis warning during startup
+ s.Containers.ServerVpp.Exec(false, "sysctl vm.overcommit_memory=1")
+ // Note: --save "" disables snapshotting which during upgrade to ubuntu 24.04 was
+ // observed to corrupt vcl memory / heap. Needs more debugging.
+ cmd := "redis-server --daemonize yes --protected-mode no --save \"\" --bind " + serverVethAddress + " --loglevel notice --logfile " + s.RedisServerLogFileName(s.Containers.ServerVpp)
s.StartServerApp(s.Containers.ServerVpp, "redis-server", cmd, runningSrv, doneSrv)
}()