Skip to content

Commit

Permalink
Review feedback
Browse files Browse the repository at this point in the history
  • Loading branch information
dgieselaar committed Nov 28, 2021
1 parent 8528192 commit edf303d
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/actions/server/lib/action_executor.ts
Original file line number Diff line number Diff line change
Expand Up @@ -105,7 +105,7 @@ export class ActionExecutor {
name: `execute_action`,
type: 'actions',
labels: {
actions_action_id: actionId,
actions_connector_id: actionId,
},
},
async (span) => {
Expand Down Expand Up @@ -135,7 +135,7 @@ export class ActionExecutor {
if (span) {
span.name = `execute_action ${actionTypeId}`;
span.addLabels({
actions_action_type_id: actionTypeId,
actions_connector_type_id: actionTypeId,
});
}

Expand Down
6 changes: 3 additions & 3 deletions x-pack/plugins/alerting/server/task_runner/task_runner.ts
Original file line number Diff line number Diff line change
Expand Up @@ -883,7 +883,7 @@ function generateNewAndRecoveredInstanceEvents<

if (apm.currentTransaction) {
apm.currentTransaction.addLabels({
alerting_new_instances: newIds.length,
alerting_new_alerts: newIds.length,
});
}

Expand Down Expand Up @@ -1070,8 +1070,8 @@ function logActiveAndRecoveredInstances<

if (apm.currentTransaction) {
apm.currentTransaction.addLabels({
alerting_active_instances: activeInstanceIds.length,
alerting_recovered_instances: recoveredInstanceIds.length,
alerting_active_alerts: activeInstanceIds.length,
alerting_recovered_alerts: recoveredInstanceIds.length,
});
}

Expand Down

0 comments on commit edf303d

Please sign in to comment.