Users are allowed to use high-priority; change WARNING to NOTE (#102)

dgrove-oss · web-flow · commit b9010d87bdd9 · 2024-11-20T12:12:06.000-05:00
diff --git a/tools/cluster-checker/checker.js b/tools/cluster-checker/checker.js
@@ -214,7 +214,7 @@ async function checkUserNamespace (client, namespace, queues) {
 
     // report high-priority workloads
     if (workload.spec.priorityClassName !== 'default-priority' && workload.spec.priorityClassName !== 'low-priority') {
-      console.log(`WARNING: workload "${namespace.metadata.name}/${workload.metadata.name}" has priority "${workload.spec.priorityClassName}"`)
+      console.log(`NOTE: workload "${namespace.metadata.name}/${workload.metadata.name}" has priority "${workload.spec.priorityClassName}"`)
     }
 
     // report unusual conditions

Original file line number	Diff line number	Diff line change
`@@ -214,7 +214,7 @@ async function checkUserNamespace (client, namespace, queues) {`
`214`	`214`
`215`	`215`	`// report high-priority workloads`
`216`	`216`	`if (workload.spec.priorityClassName !== 'default-priority' && workload.spec.priorityClassName !== 'low-priority') {`
`217`		- console.log(`WARNING: workload "${namespace.metadata.name}/${workload.metadata.name}" has priority "${workload.spec.priorityClassName}"`)
	`217`	+ console.log(`NOTE: workload "${namespace.metadata.name}/${workload.metadata.name}" has priority "${workload.spec.priorityClassName}"`)
`218`	`218`	`}`
`219`	`219`
`220`	`220`	`// report unusual conditions`