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

Adding integration test for logstash Metricbeat module, xpack code path #15800

Merged
merged 5 commits into from
Jan 30, 2020

Conversation

ycombinator
Copy link
Contributor

@ycombinator ycombinator commented Jan 24, 2020

This PR:

  • adds an integration test for the Metricbeat logstash module when xpack.enabled: true is set in the module configuration,
  • adds two new functions to the integration testing framework, NewReportingMetricSetV2Errors and NewMetricSets, in order to make the above test possible, and
  • does some minor cleanup in the Metricbeat logstash integration test code.

@ycombinator ycombinator requested a review from a team as a code owner January 24, 2020 01:25
@ycombinator ycombinator requested review from a team and removed request for a team January 24, 2020 01:55
@elasticmachine
Copy link
Collaborator

Pinging @elastic/stack-monitoring (Stack monitoring)

@ycombinator
Copy link
Contributor Author

Blocked on #15819.

@ycombinator ycombinator changed the title Adding integration test for Metricbeat logstash module, xpack data path Adding integration test for logstash Metricbeat module, xpack code path Jan 30, 2020
@ycombinator
Copy link
Contributor Author

Travis CI is green and Jenkins CI failures are unrelated. Merging.

@ycombinator ycombinator merged commit 0507118 into elastic:master Jan 30, 2020
@ycombinator ycombinator deleted the mb-ls-xp-int-test branch January 30, 2020 20:32
ycombinator added a commit to ycombinator/beats that referenced this pull request Jan 30, 2020
…th (elastic#15800)

* Adding integration test for logstash module, xpack data path

* Adding godoc

* Refactoring

* More refactoring

* Fixing reference
@ycombinator ycombinator removed the needs_backport PR is waiting to be backported to other branches. label Jan 30, 2020
ycombinator added a commit that referenced this pull request Feb 6, 2020
…th (#15800) (#15968)

* Adding integration test for logstash module, xpack data path

* Adding godoc

* Refactoring

* More refactoring

* Fixing reference
@andresrc andresrc added the Team:Integrations Label for the Integrations team label Mar 6, 2020
ycombinator added a commit to ycombinator/beats that referenced this pull request Apr 8, 2020
…th (elastic#15800)

* Adding integration test for logstash module, xpack data path

* Adding godoc

* Refactoring

* More refactoring

* Fixing reference
ycombinator added a commit that referenced this pull request Apr 8, 2020
…th (#15800) (#17598)

* Adding integration test for logstash module, xpack data path

* Adding godoc

* Refactoring

* More refactoring

* Fixing reference
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.

5 participants