Merge pull request #194 from chu11/issue193_powerman_bufsize

powerman: increase maximum line length
chaos · Jul 3, 2024 · 8e4dfba · 8e4dfba
2 parents ba6a264 + bd1bd27
commit 8e4dfba
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/src/powerman/client_proto.h b/src/powerman/client_proto.h
@@ -21,7 +21,7 @@
  * If not quit, goto 3
  */
 
-#define CP_LINEMAX  8192                /* max request/response line length */
+#define CP_LINEMAX  131072              /* max request/response line length */
 #define CP_EOL      "\r\n"              /* line terminator */
 #define CP_PROMPT   "powerman> "        /* prompt */
 #define CP_VERSION  "001 %s" CP_EOL

diff --git a/t/t0039-llnl-el-capitan-cluster.t b/t/t0039-llnl-el-capitan-cluster.t
@@ -106,6 +106,12 @@ test_expect_success 'powerman -q shows all off' '
 	makeoutput "" "$ALLSTR" "" >query5.exp &&
 	test_cmp query5.exp query5.out
 '
+test_expect_success 'powerman -q works with giant input' '
+	nodes=$(echo elcap\[$(seq -s, 0 2 16382)\]) &&
+	$powerman -h $testaddr -q $nodes >query6.out &&
+	makeoutput "" "$nodes" "" >query6.exp &&
+	test_cmp query6.exp query6.out
+'
 test_expect_success 'stop powerman daemon' '
 	kill -15 $(cat powermand.pid) &&
 	wait