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

Remove flush interval from Kafka run script #6831

Merged
merged 1 commit into from Apr 17, 2018

Conversation

Projects
None yet
2 participants
@ruflin
Copy link
Collaborator

commented Apr 12, 2018

The very short flush interval caused the container sometimes to crash. Removing it should make tests more stable.

@ruflin ruflin added the in progress label Apr 12, 2018

@ruflin

This comment has been minimized.

Copy link
Collaborator Author

commented Apr 12, 2018

jenkins, test it

@ruflin ruflin force-pushed the ruflin:skip-kafka-tests branch 2 times, most recently from 00cf515 to 1355b83 Apr 12, 2018

@ruflin ruflin changed the title Skip kafka system tests to see if this fixes CI Remove flush interval from Kafka run script Apr 13, 2018

@ruflin ruflin added review :Testing and removed in progress labels Apr 13, 2018

@ruflin ruflin force-pushed the ruflin:skip-kafka-tests branch from 1355b83 to 3e2b47d Apr 16, 2018

Remove flush interval from Kafka run script
The very short flush interval caused the container sometimes to crash. Removing it should make tests more stable.

@ruflin ruflin force-pushed the ruflin:skip-kafka-tests branch from 3e2b47d to 4995433 Apr 17, 2018

@jsoriano jsoriano merged commit f8425af into elastic:master Apr 17, 2018

6 checks passed

CLA Commit author has signed the CLA
Details
Hound No violations found. Woof!
beats-ci Build finished.
Details
codecov/patch Coverage not affected when comparing fd15ea2...4995433
Details
codecov/project 62.9% (-0.01%) compared to fd15ea2
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.