Use the correct Message property instead of Reason in scale POST request

Also use a more informative default message (one that includes the new
count)
This commit is contained in:
Michael Lange
2020-07-30 08:43:15 -07:00
parent 1205387f83
commit 45e1c652c6
3 changed files with 7 additions and 6 deletions

View File

@@ -69,12 +69,12 @@ export default class JobAdapter extends WatchableNamespaceIDs {
});
}
scale(job, group, count, reason) {
scale(job, group, count, message) {
const url = addToPath(this.urlForFindRecord(job.get('id'), 'job'), '/scale');
return this.ajax(url, 'POST', {
data: {
Count: count,
Reason: reason,
Message: message,
Target: {
Group: group,
},

View File

@@ -246,8 +246,9 @@ export default class Job extends Model {
return promise;
}
scale(group, count, reason = 'Manual scaling event from the Nomad UI') {
return this.store.adapterFor('job').scale(this, group, count, reason);
scale(group, count, message) {
if (message == null) message = `Manually scaled to ${count} from the Nomad UI`;
return this.store.adapterFor('job').scale(this, group, count, message);
}
setIdByPayload(payload) {

View File

@@ -59,7 +59,7 @@ export default class TaskGroup extends Fragment {
return maybe(this.get('job.scaleState.taskGroupScales')).findBy('name', this.name);
}
scale(count, reason) {
return this.job.scale(this.name, count, reason);
scale(count, message) {
return this.job.scale(this.name, count, message);
}
}