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

Ensure the task group is present whenever an allocation row is rendered

Browse source
This commit is contained in:
Michael Lange 2018-03-15 14:03:14 -07:00
parent c8b3e91d4a
commit 23cfbb9c27
2 changed files with 8 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
ui/app/components/allocation-row.js
View file

@ -58,6 +58,9 @@ function qualifyJob() {
job, job,
originalJobId: null, originalJobId: null,
}); });
if (job.get('isPartial')) {
job.reload();
}
} else { } else {
this.get('store') this.get('store')
.findRecord('job', allocation.get('originalJobId')) .findRecord('job', allocation.get('originalJobId'))

5
ui/app/models/job.js
View file

@ -84,6 +84,11 @@ export default Model.extend({
taskGroups: fragmentArray('task-group', { defaultValue: () => [] }), taskGroups: fragmentArray('task-group', { defaultValue: () => [] }),
summary: belongsTo('job-summary'), summary: belongsTo('job-summary'),
// A job model created from the jobs list response will be lacking
// task groups. This is an indicator that it needs to be reloaded
// if task group information is important.
isPartial: equal('taskGroups.length', 0),
// If a job has only been loaded through the list request, the task groups // If a job has only been loaded through the list request, the task groups
// are still unknown. However, the count of task groups is available through // are still unknown. However, the count of task groups is available through
// the job-summary model which is embedded in the jobs list response. // the job-summary model which is embedded in the jobs list response.