luxolus
/
open-nomad
2
0
Fork 0
Code Issues 1 Pull Requests Packages Releases Wiki Activity

Don't rely on CpuStats.Percent 

Just do the math client-side with TotalTicks and Reserved
This commit is contained in:
Michael Lange 2017-10-19 10:43:33 -07:00
parent 18ad13cebf
commit b693ff1886
3 changed files with 9 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
ui/app/models/allocation.js
View File

@ -57,8 +57,13 @@ export default Model.extend({
return used / total; return used / total;
}), }),
percentCPU: computed('stats.ResourceUsage.CpuStats.Percent', function() { percentCPU: computed('cpuUsed', 'taskGroup.reservedCPU', function() {
return this.get('stats.ResourceUsage.CpuStats.Percent') || 0; const used = this.get('cpuUsed');
const total = this.get('taskGroup.reservedCPU');
if (!total || !used) {
return 0;
}
return used / total;
}), }),
stats: computed('node.{isPartial,httpAddr}', function() { stats: computed('node.{isPartial,httpAddr}', function() {

2
ui/tests/acceptance/client-detail-test.js
View File

@ -172,7 +172,7 @@ test('each allocation should have high-level details for the allocation', functi
.find('td:eq(6)') .find('td:eq(6)')
.text() .text()
.trim(), .trim(),
allocStats.resourceUsage.CpuStats.Percent, Math.floor(allocStats.resourceUsage.CpuStats.TotalTicks) / cpuUsed,
'CPU %' 'CPU %'
); );
assert.equal( assert.equal(

2
ui/tests/acceptance/task-group-detail-test.js
View File

@ -217,7 +217,7 @@ test('each allocation should show stats about the allocation, retrieved directly
.find('td:eq(6)') .find('td:eq(6)')
.text() .text()
.trim(), .trim(),
allocStats.resourceUsage.CpuStats.Percent, Math.floor(allocStats.resourceUsage.CpuStats.TotalTicks) / cpuUsed,
'CPU %' 'CPU %'
); );