Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

added log info for clarity while running the test with additional params- test huge field #22277

Merged
merged 1 commit into from
Aug 23, 2018

Conversation

rashmivkulkarni
Copy link
Contributor

Adding additional log.info - in case the test fails and has instructions how to run this test with additional params while starting the test.

investigating -- if there is another better way if we can decrease the number of fields or make a change where it's still within the default bounds. Or, can we post to ES directly.

@rashmivkulkarni
Copy link
Contributor Author

jenkins, test this

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Copy link

@LeeDr LeeDr left a comment

Choose a reason for hiding this comment

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

LGTM - Code review and checked these Jenkins results;

01:09:10.416            │ info if there is a failure, start the server with "node scripts/functional_tests_server -- --server.maxPayloadBytes=1648576"
01:09:10.417            └- ✓ pass 
01:09:10.419          └-> "after all" hook

@rashmivkulkarni rashmivkulkarni merged commit 1c22214 into elastic:6.x Aug 23, 2018
rashmivkulkarni added a commit to rashmivkulkarni/kibana that referenced this pull request Aug 23, 2018
rashmivkulkarni added a commit to rashmivkulkarni/kibana that referenced this pull request Aug 23, 2018
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