Skip to content

Commit d0a08b4

Browse files
committed
Marginally clarify workaround comments
1 parent f5fafe6 commit d0a08b4

2 files changed

Lines changed: 12 additions & 3 deletions

File tree

  • packages/azure/data_stream

packages/azure/data_stream/events/elasticsearch/ingest_pipeline/default.yml

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -44,10 +44,11 @@ processors:
4444
if: 'ctx.tmp_json?.category == null'
4545
ignore_missing: true
4646
description: 'Rename the invalid `Category` field to `category` to apply the correct routing rules.'
47-
# Unfortunately, some Azure services produce logs with `CategoryValue` field
48-
# instead of `Category` field.
47+
# Unfortunately, some Azure services generate logs with
48+
# `CategoryValue` field instead of `Category` field.
4949
#
50-
# We need to rename it to `category` to apply the correct routing rules.
50+
# We need to rename `CategoryValue` as `category` to
51+
# apply the correct routing rules.
5152
#
5253
# Refs;
5354
# - https://github.com/elastic/integrations/issues/15083

packages/azure/data_stream/signinlogs/elasticsearch/ingest_pipeline/default.yml

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -46,6 +46,14 @@ processors:
4646
- drop:
4747
description: Drop non-SignInLogs.
4848
if: ctx.azure?.signinlogs?.category == null || !ctx.azure.signinlogs.category.endsWith('SignInLogs')
49+
# Unfortunately, some Azure services generate logs with
50+
# `created_date_time` field instead of `time` field.
51+
#
52+
# We need to rename `created_date_time` as `time` to
53+
# apply the correct timestamp processing.
54+
#
55+
# Refs;
56+
# - https://github.com/elastic/integrations/issues/15083
4957
- rename:
5058
field: azure.signinlogs.created_date_time
5159
target_field: azure.signinlogs.time

0 commit comments

Comments
 (0)