set cpu request=limit when cpubind=true (#334)

jschwinger233 · web-flow · commit 566ff668fa48 · 2021-02-04T01:43:07.000+08:00
diff --git a/resources/cpumem/cpumem.go b/resources/cpumem/cpumem.go
@@ -64,6 +64,11 @@ func (cm *cpuMemRequest) Validate() error {
 	if cm.CPUQuotaRequest > 0 && cm.CPUQuotaLimit > 0 && cm.CPUQuotaRequest > cm.CPUQuotaLimit {
 		cm.CPUQuotaLimit = cm.CPUQuotaRequest
 	}
+	// if CPUBind=true, set cpu request=limit to solve the dilemma
+	// only deal with cpu limit>request but not vice versa
+	if cm.CPUBind && cm.CPUQuotaRequest > 0 && cm.CPUQuotaLimit > 0 && cm.CPUQuotaLimit > cm.CPUQuotaRequest {
+		cm.CPUQuotaRequest = cm.CPUQuotaLimit
+	}
 	return nil
 }
 

Original file line number	Diff line number	Diff line change
`@@ -64,6 +64,11 @@ func (cm *cpuMemRequest) Validate() error {`
`64`	`64`	`if cm.CPUQuotaRequest > 0 && cm.CPUQuotaLimit > 0 && cm.CPUQuotaRequest > cm.CPUQuotaLimit {`
`65`	`65`	`cm.CPUQuotaLimit = cm.CPUQuotaRequest`
`66`	`66`	`}`
	`67`	`+ // if CPUBind=true, set cpu request=limit to solve the dilemma`
	`68`	`+ // only deal with cpu limit>request but not vice versa`
	`69`	`+ if cm.CPUBind && cm.CPUQuotaRequest > 0 && cm.CPUQuotaLimit > 0 && cm.CPUQuotaLimit > cm.CPUQuotaRequest {`
	`70`	`+ cm.CPUQuotaRequest = cm.CPUQuotaLimit`
	`71`	`+ }`
`67`	`72`	`return nil`
`68`	`73`	`}`
`69`	`74`