Skip to content

Commit

Permalink
PR fixes
Browse files Browse the repository at this point in the history
  • Loading branch information
ymao1 committed Dec 8, 2020
1 parent c0edd6d commit bf14bf7
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 5 deletions.
4 changes: 2 additions & 2 deletions x-pack/plugins/alerts/server/task_runner/task_runner.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -409,7 +409,7 @@ describe('Task Runner', () => {
);
expect(logger.debug).nthCalledWith(
3,
`no actions executed for alert test:1: 'alert-name': alert is muted.`
`no scheduling of actions for alert test:1: 'alert-name': alert is muted.`
);
expect(logger.debug).nthCalledWith(
4,
Expand Down Expand Up @@ -519,7 +519,7 @@ describe('Task Runner', () => {
);
expect(logger.debug).nthCalledWith(
3,
`skipping action execution for '2' in alert test:1: 'alert-name': instance is muted`
`skipping scheduling of actions for '2' in alert test:1: 'alert-name': instance is muted`
);
expect(logger.debug).nthCalledWith(
4,
Expand Down
6 changes: 3 additions & 3 deletions x-pack/plugins/alerts/server/task_runner/task_runner.ts
Original file line number Diff line number Diff line change
Expand Up @@ -260,7 +260,7 @@ export class TaskRunner {
const shouldExecuteAction = !throttled && !muted;
if (!shouldExecuteAction) {
this.logger.debug(
`skipping action execution for '${alertInstanceName}' in alert ${alertLabel}: instance is ${
`skipping scheduling of actions for '${alertInstanceName}' in alert ${alertLabel}: instance is ${
muted ? 'muted' : 'throttled'
}`
);
Expand All @@ -272,7 +272,7 @@ export class TaskRunner {
)
);
} else {
this.logger.debug(`no actions executed for alert ${alertLabel}: alert is muted.`);
this.logger.debug(`no scheduling of actions for alert ${alertLabel}: alert is muted.`);
}

return {
Expand Down Expand Up @@ -555,7 +555,7 @@ function scheduleActionsForRecoveredInstances(params: ScheduleActionsForRecovere
for (const id of recoveredIds) {
if (mutedInstanceIdsSet.has(id)) {
logger.debug(
`skipping action execution for '${id}' in alert ${alertLabel}: instance is muted`
`skipping scheduling of actions for '${id}' in alert ${alertLabel}: instance is muted`
);
} else {
const instance = recoveredAlertInstances[id];
Expand Down

0 comments on commit bf14bf7

Please sign in to comment.