Fix cpu counts for sinfo node (#20167)

steveny91 · web-flow · commit a6cf7f3437cd · 2025-04-28T20:33:00.000Z
* fix cpu count for sinfo node

* changelog

* Rename 20166.fixed to 20167.fixed
diff --git a/slurm/changelog.d/20167.fixed b/slurm/changelog.d/20167.fixed
@@ -0,0 +1 @@
+Fix cpu count for sinfo node by using the correct query parameter
diff --git a/slurm/datadog_checks/slurm/constants.py b/slurm/datadog_checks/slurm/constants.py
@@ -5,7 +5,7 @@
     "-ahO",
     "Partition:|,NodeList:|,CPUs:|,Available:|,Memory:|,Cluster:|,NodeAIOT:|,StateLong:|,Nodes:",
 ]
-SINFO_NODE_PARAMS = ["-haNO", "PartitionName:|,Available:|,NodeList:|,NodeAIOT:|,Memory:|,Cluster:"]
+SINFO_NODE_PARAMS = ["-haNO", "PartitionName:|,Available:|,NodeList:|,CPUsState:|,Memory:|,Cluster:"]
 SINFO_ADDITIONAL_NODE_PARAMS = "|,CPUsLoad:|,FreeMem:|,Disk:|,StateLong:|,Reason:|,features_act:|,Threads:"
 GPU_PARAMS = "|,Gres:|,GresUsed:"
 SQUEUE_PARAMS = ["-aho", "%A|%u|%j|%T|%N|%C|%R|%m"]

Original file line number	Diff line number	Diff line change
`@@ -0,0 +1 @@`
	`1`	`+Fix cpu count for sinfo node by using the correct query parameter`
Original file line number	Diff line number	Diff line change
`@@ -5,7 +5,7 @@`
`5`	`5`	`"-ahO",`
`6`	`6`	`"Partition:\|,NodeList:\|,CPUs:\|,Available:\|,Memory:\|,Cluster:\|,NodeAIOT:\|,StateLong:\|,Nodes:",`
`7`	`7`	`]`
`8`		`-SINFO_NODE_PARAMS = ["-haNO", "PartitionName:\|,Available:\|,NodeList:\|,NodeAIOT:\|,Memory:\|,Cluster:"]`
	`8`	`+SINFO_NODE_PARAMS = ["-haNO", "PartitionName:\|,Available:\|,NodeList:\|,CPUsState:\|,Memory:\|,Cluster:"]`
`9`	`9`	`SINFO_ADDITIONAL_NODE_PARAMS = "\|,CPUsLoad:\|,FreeMem:\|,Disk:\|,StateLong:\|,Reason:\|,features_act:\|,Threads:"`
`10`	`10`	`GPU_PARAMS = "\|,Gres:\|,GresUsed:"`
`11`	`11`	`SQUEUE_PARAMS = ["-aho", "%A\|%u\|%j\|%T\|%N\|%C\|%R\|%m"]`