Skip to content

Commit

Permalink
add ecs-migration.yml
Browse files Browse the repository at this point in the history
  • Loading branch information
graphaelli committed Nov 16, 2018
1 parent a843f56 commit f4ad774
Show file tree
Hide file tree
Showing 5 changed files with 135 additions and 39 deletions.
115 changes: 115 additions & 0 deletions _meta/ecs-migration.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,115 @@
# The ECS migration file contains the information about all the fields which are migrated to ECS in 7.0.
# The goal of the file is to potentially have scripts on top of this information to convert visualisations and templates
# based on this information in an automated way and to keep track of all changes which were applied.
#
# The format of the file is as following:
#
# - from: source-field-in-6.x
# to: target-filed-in-ECS
# # Alias field is useful for fields where there is a 1-1 mapping from old to new
# alias: true-if-alias-is-required-in-6x (default is true)
# # Copy to is useful for fields where multiple fields map to the same ECS field
# copy_to: true-if-field-should-be-copied-to-target-in-6x (default is false)

- from: context.service.agent.name
to: agent.name

- from: context.service.agent.version
to: agent.version

- from: context.system.architecture
to: host.architecture

- from: context.system.ip
to: host.ip

- from: context.system.hostname
to: host.name

- from: context.system.platform
to: host.os.platform

- from: context.request.method
to: http.method

- from: context.request.http_version
to: http.version

- from: context.tags
to: labels
alias: false
copy_to: true

- from: context.process.pid
to: process.pid

- from: context.process.ppid
to: process.ppid

- from: context.process.title
to: process.title

# not in ECS
- from: context.service.environment
to: service.environment

# not in ECS
- from: context.service.framework.name
to: service.framework.name

# not in ECS
- from: context.service.framework.version
to: service.framework.version

# not in ECS
- from: context.service.language.name
to: service.language.name

# not in ECS
- from: context.service.language.version
to: service.language.version

- from: context.service.name
to: service.name

# not in ECS
- from: context.service.runtime.name
to: service.runtime.name

# not in ECS
- from: context.service.runtime.version
to: service.runtime.version

- from: context.request.url.full
to: url.original

- from: context.request.url.hash
to: url.fragment

- from: context.request.url.hostname
to: url.domain

- from: context.request.url.pathname
to: url.path

- from: context.request.url.port
to: url.port
alias: false
copy_to: true

- from: context.request.url.search
to: url.query

- from: context.request.url.protocol
to: url.scheme
alias: false
copy_to: true

- from: context.user.email
to: user.email

- from: context.user.id
to: user.id

- from: context.user.username
to: user.name
33 changes: 8 additions & 25 deletions _meta/fields.common.yml
Original file line number Diff line number Diff line change
Expand Up @@ -446,19 +446,9 @@
type: group
dynamic: false
fields:
- name: href
type: group
fields:
- name: original
type: alias
path: context.request.url.raw

- name: host
type: group
fields:
- name: name
type: alias
path: context.request.url.hostname
- name: domain
type: alias
path: context.request.url.hostname

- name: fragment
type: alias
Expand All @@ -469,12 +459,8 @@
path: context.request.url.full

- name: path
type: group
fields:
- name: original
type: alias
path: context.request.url.pathname
# TODO: multifield original.text
type: alias
path: context.request.url.pathname

# context.request.url.port keyword -> long
- name: port
Expand All @@ -483,12 +469,9 @@
The port of the request, e.g. 443.
- name: query
type: group
fields:
- name: query
type: alias
path: context.request.url.search
# TODO: multifield original.text
type: alias
path: context.request.url.search
# TODO: multifield original.text

# context.request.url.protocol minus the ":"
- name: scheme
Expand Down
11 changes: 0 additions & 11 deletions docs/fields.asciidoc
Original file line number Diff line number Diff line change
Expand Up @@ -685,15 +685,6 @@ type: alias
--
*`context.request.url.raw`*::
+
--
type: alias
--
*`context.request.url.hostname`*::
+
--
Expand All @@ -715,7 +706,6 @@ type: alias
--
*`context.request.url.pathname`*::
+
--
Expand All @@ -733,7 +723,6 @@ The port of the request, e.g. 443.
--
*`context.request.url.search`*::
+
--
Expand Down
2 changes: 1 addition & 1 deletion include/fields.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

13 changes: 11 additions & 2 deletions tests/system/test_ecs_mappings.py
Original file line number Diff line number Diff line change
Expand Up @@ -34,10 +34,9 @@ def test_ecs_migration(self):
all_fields = set()
alias_fields = set()
for f, a in flatmap(yaml.load(self.command_output)["mappings"]["doc"]["properties"]):
all_fields.add(f)
if a.get("type") == "alias":
alias_fields.add(a["path"])
else:
all_fields.add(f)

# fields with special exception, due to mapping type changes, etc
# no comment means unchanged
Expand Down Expand Up @@ -68,6 +67,16 @@ def test_ecs_migration(self):
should_not_be_aliased = alias_fields - all_fields
self.assertFalse(should_not_be_aliased, json.dumps(sorted(should_not_be_aliased)))

# check the migration log too
with open(self._beat_path_join("_meta", "ecs-migration.yml")) as f:
for m in yaml.load(f):
if m.get("alias", True):
self.assertIn(m["from"], alias_fields)
elif m.get("copy_to", False):
self.assertIn(m["from"], all_fields)
self.assertIn(m["to"], all_fields)

# check that all fields are accounted for
not_aliased = all_fields - alias_fields - exception_fields
self.assertFalse(not_aliased,
"\nall fields ({:d}):\n{}\n\naliased ({:d}):\n{}\n\nunaccounted for ({:d}):\n{}".format(
Expand Down

0 comments on commit f4ad774

Please sign in to comment.