Support cores and memory overselling by scaling #79
Conversation
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: yuntianfeijing The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
|
Welcome @yuntianfeijing! It looks like this is your first PR to Project-HAMi/volcano-vgpu-device-plugin 🎉 |
Signed-off-by: zhangxl56 <[email protected]>
Signed-off-by: zhangxl56 <[email protected]>
| } | ||
| for _, val := range dlist { | ||
| tmp += val.Id + "," + strconv.FormatInt(int64(val.Count), 10) + "," + strconv.Itoa(int(val.Devmem)) + "," + val.Type + "," + strconv.FormatBool(val.Health) + "," + val.Mode + ":" | ||
| tmp += val.Id + "," + strconv.FormatInt(int64(val.Count), 10) + "," + strconv.Itoa(int(val.Devmem)) + "," + core + "," + val.Type + "," + strconv.FormatBool(val.Health) + "," + val.Mode + ":" |
There was a problem hiding this comment.
The string format here needs to be modified very carefully. It should be modified in synchronization with the volcano-scheduler and keep compatible.
There was a problem hiding this comment.
volcano APPROVED the PR:
volcano-sh/volcano#4597
2 participants
No description provided.