Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
[8.x] Enhance task failure log to include error source in tags (#199406…
…) (#200082) # Backport This will backport the following commits from `main` to `8.x`: - [Enhance task failure log to include error source in tags (#199406)](#199406) <!--- Backport version: 9.4.3 --> ### Questions ? Please refer to the [Backport tool documentation](https://github.com/sqren/backport) <!--BACKPORT [{"author":{"name":"Mike Côté","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-11-13T19:48:18Z","message":"Enhance task failure log to include error source in tags (#199406)\n\nResolves https://github.com/elastic/kibana/issues/199346\r\n\r\nIn this PR I'm adding `user-error` and `framework-error` tags to the\r\nassociated task failure logs.\r\n\r\n## To verify\r\n\r\nYou can either use the jest test to observe the returned flags or set\r\nyour logging to JSON and use the following code samples to test various\r\ntypes of errors.\r\n\r\nkibana.yml to set logging to JSON\r\n```\r\nlogging:\r\n appenders:\r\n json-layout:\r\n type: console\r\n layout:\r\n type: json\r\n root:\r\n appenders: [json-layout]\r\n```\r\n\r\nCode samples throwing various types of errors.\r\n```\r\ndiff --git a/x-pack/plugins/alerting/server/task_runner/task_runner.ts b/x-pack/plugins/alerting/server/task_runner/task_runner.ts\r\nindex 89432e18220..129b53f71b4 100644\r\n--- a/x-pack/plugins/alerting/server/task_runner/task_runner.ts\r\n+++ b/x-pack/plugins/alerting/server/task_runner/task_runner.ts\r\n@@ -649,6 +649,10 @@ export class TaskRunner<\r\n schedule: taskSchedule,\r\n } = this.taskInstance;\r\n\r\n+ // throw createTaskRunError(new Error('foo'), TaskErrorSource.USER);\r\n+ // throw createTaskRunError(new Error('foo'), TaskErrorSource.FRAMEWORK);\r\n+ // throw new Error('foo');\r\n+\r\n this.logger = createTaskRunnerLogger({ logger: this.logger, tags: [ruleId, this.ruleType.id] });\r\n\r\n let stateWithMetrics: Result<RuleTaskStateAndMetrics, Error>;\r\n```","sha":"544525d497d6c833c9a861061876412d3198c5e9","branchLabelMapping":{"^v9.0.0$":"main","^v8.17.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","Feature:Task Manager","Team:ResponseOps","v9.0.0","backport:version","v8.17.0"],"title":"Enhance task failure log to include error source in tags","number":199406,"url":"https://github.com/elastic/kibana/pull/199406","mergeCommit":{"message":"Enhance task failure log to include error source in tags (#199406)\n\nResolves https://github.com/elastic/kibana/issues/199346\r\n\r\nIn this PR I'm adding `user-error` and `framework-error` tags to the\r\nassociated task failure logs.\r\n\r\n## To verify\r\n\r\nYou can either use the jest test to observe the returned flags or set\r\nyour logging to JSON and use the following code samples to test various\r\ntypes of errors.\r\n\r\nkibana.yml to set logging to JSON\r\n```\r\nlogging:\r\n appenders:\r\n json-layout:\r\n type: console\r\n layout:\r\n type: json\r\n root:\r\n appenders: [json-layout]\r\n```\r\n\r\nCode samples throwing various types of errors.\r\n```\r\ndiff --git a/x-pack/plugins/alerting/server/task_runner/task_runner.ts b/x-pack/plugins/alerting/server/task_runner/task_runner.ts\r\nindex 89432e18220..129b53f71b4 100644\r\n--- a/x-pack/plugins/alerting/server/task_runner/task_runner.ts\r\n+++ b/x-pack/plugins/alerting/server/task_runner/task_runner.ts\r\n@@ -649,6 +649,10 @@ export class TaskRunner<\r\n schedule: taskSchedule,\r\n } = this.taskInstance;\r\n\r\n+ // throw createTaskRunError(new Error('foo'), TaskErrorSource.USER);\r\n+ // throw createTaskRunError(new Error('foo'), TaskErrorSource.FRAMEWORK);\r\n+ // throw new Error('foo');\r\n+\r\n this.logger = createTaskRunnerLogger({ logger: this.logger, tags: [ruleId, this.ruleType.id] });\r\n\r\n let stateWithMetrics: Result<RuleTaskStateAndMetrics, Error>;\r\n```","sha":"544525d497d6c833c9a861061876412d3198c5e9"}},"sourceBranch":"main","suggestedTargetBranches":["8.x"],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/199406","number":199406,"mergeCommit":{"message":"Enhance task failure log to include error source in tags (#199406)\n\nResolves https://github.com/elastic/kibana/issues/199346\r\n\r\nIn this PR I'm adding `user-error` and `framework-error` tags to the\r\nassociated task failure logs.\r\n\r\n## To verify\r\n\r\nYou can either use the jest test to observe the returned flags or set\r\nyour logging to JSON and use the following code samples to test various\r\ntypes of errors.\r\n\r\nkibana.yml to set logging to JSON\r\n```\r\nlogging:\r\n appenders:\r\n json-layout:\r\n type: console\r\n layout:\r\n type: json\r\n root:\r\n appenders: [json-layout]\r\n```\r\n\r\nCode samples throwing various types of errors.\r\n```\r\ndiff --git a/x-pack/plugins/alerting/server/task_runner/task_runner.ts b/x-pack/plugins/alerting/server/task_runner/task_runner.ts\r\nindex 89432e18220..129b53f71b4 100644\r\n--- a/x-pack/plugins/alerting/server/task_runner/task_runner.ts\r\n+++ b/x-pack/plugins/alerting/server/task_runner/task_runner.ts\r\n@@ -649,6 +649,10 @@ export class TaskRunner<\r\n schedule: taskSchedule,\r\n } = this.taskInstance;\r\n\r\n+ // throw createTaskRunError(new Error('foo'), TaskErrorSource.USER);\r\n+ // throw createTaskRunError(new Error('foo'), TaskErrorSource.FRAMEWORK);\r\n+ // throw new Error('foo');\r\n+\r\n this.logger = createTaskRunnerLogger({ logger: this.logger, tags: [ruleId, this.ruleType.id] });\r\n\r\n let stateWithMetrics: Result<RuleTaskStateAndMetrics, Error>;\r\n```","sha":"544525d497d6c833c9a861061876412d3198c5e9"}},{"branch":"8.x","label":"v8.17.0","branchLabelMappingKey":"^v8.17.0$","isSourceBranch":false,"state":"NOT_CREATED"}]}] BACKPORT--> Co-authored-by: Mike Côté <[email protected]>
- Loading branch information