Skip to content

[test fixtures] string -> text/keyword for logstash data - #9799

Merged
jbudz merged 1 commit into
elastic:masterfrom
jbudz:test-fixtures
Jan 9, 2017
Merged

jbudz merged 1 commit into
elastic:masterfrom
jbudz:test-fixtures

Conversation

@jbudz

@jbudz jbudz commented Jan 9, 2017

Copy link
Copy Markdown
Contributor

This updates the mappings for mock logstash-* data used when running integration tests. Closes #9763.

@jbudz jbudz added the review label Jan 9, 2017
@jbudz

jbudz commented Jan 9, 2017

Copy link
Copy Markdown
Contributor Author

Separate test failure, comment detailing at #9763 (comment). The mapping changes do appear to be working.

@stacey-gammon stacey-gammon left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM, and yep, my bad on the new selenium failure, working on it. Thanks for fixing this!!

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