Skip to content

[7.x] update APM migration reindex script - #34801

Merged
graphaelli merged 1 commit into
elastic:7.xfrom
graphaelli:apm-migration-update-7.x
Apr 12, 2019
Merged

graphaelli merged 1 commit into
elastic:7.xfrom
graphaelli:apm-migration-update-7.x

Conversation

@graphaelli

Copy link
Copy Markdown
Contributor

fixes:

  • setting user_agent.original from http headers
  • host.hostname ECS alignment under kubernetes

closes #34259

@elasticmachine

Copy link
Copy Markdown
Contributor

💚 Build Succeeded

@graphaelli
graphaelli requested a review from tylersmalley April 12, 2019 01:55
@graphaelli

Copy link
Copy Markdown
Contributor Author

Intended for backport to 7.0 post-merge

export const apmReindexScript = `
// add ecs version
ctx._source.ecs = ['version': '1.0.0-beta2'];
ctx._source.ecs = ['version': '1.1.0-dev'];

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What is the dev pre-release tag for?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

that matches the version of the published ecs schema 7.0.1 is aligned with. This is something inherited from libbeat, not directly set by apm-server

@tylersmalley tylersmalley left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code LGTM

@graphaelli
graphaelli merged commit bfa3077 into elastic:7.x Apr 12, 2019
@graphaelli
graphaelli deleted the apm-migration-update-7.x branch April 12, 2019 16:16
graphaelli added a commit to graphaelli/kibana that referenced this pull request Apr 12, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants