From 1fe294fd4452c216e5e8578a17b9567866f2a56b Mon Sep 17 00:00:00 2001 From: Chris Lu Date: Thu, 2 Jul 2026 15:47:41 -0700 Subject: [PATCH] fix(fuse-tests): drop glog globals from TestWriteBufferCap mount options -logtostderr and -v are glog globals parsed before the subcommand; weed mount's flag set rejects them, so the mount process exited with a usage error and the test silently ran against the local disk (masked until waitForMount started verifying a real mount). glog already writes to stderr here by default, so the options were also redundant. --- test/fuse_integration/write_buffer_cap_test.go | 10 +++------- 1 file changed, 3 insertions(+), 7 deletions(-) diff --git a/test/fuse_integration/write_buffer_cap_test.go b/test/fuse_integration/write_buffer_cap_test.go index 0f7320db2..583b8f183 100644 --- a/test/fuse_integration/write_buffer_cap_test.go +++ b/test/fuse_integration/write_buffer_cap_test.go @@ -88,13 +88,9 @@ func writeBufferCapConfig(debugPort int) *TestConfig { "-writeBufferSizeMB=16", "-debug=true", fmt.Sprintf("-debug.port=%d", debugPort), - // Route glog to stderr so the framework's process log - // capture actually contains something — by default weed - // sends glog to /tmp/weed.* files which the CI artifact - // upload step never sees. Critical for diagnosing - // upload/saveToStorage errors on Linux runs. - "-logtostderr=true", - "-v=2", + // NOTE: glog flags (-logtostderr, -v) are globals parsed + // before the subcommand; passing them here makes weed + // mount exit with a usage error before ever mounting. }, SkipCleanup: false, }