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

Improve PHP-PFM module #3450

Merged
merged 1 commit into from
Jan 25, 2017
Merged

Improve PHP-PFM module #3450

merged 1 commit into from
Jan 25, 2017

Conversation

ruflin
Copy link
Member

@ruflin ruflin commented Jan 24, 2017

  • Add docker environment for integration and system testing
  • Add system test file to check for correct docs.
  • Brings docs in line with generated output
  • Update data.json
  • Remove hostname fields as already part of metricset
  • Apply schema instead of manual conversion
  • Rename pool.pool to pool.name
  • Remove separate http client as not needed anymore

This is a follow up PR for #3415

@ruflin ruflin force-pushed the php-fpm-cleanup branch 4 times, most recently from 9e921cf to 153b887 Compare January 25, 2017 09:05
* Add docker environment for integration and system testing
* Add system test file to check for correct docs.
* Brings docs in line with generated output
* Update data.json
* Remove hostname fields as already part of metricset
* Apply schema instead of manual conversion
* Rename pool.pool to pool.name
* Remove separate http client as not needed anymore

This is a follow up PR for elastic#3415
@tsg tsg merged commit 5cf6aa8 into elastic:master Jan 25, 2017
douaejeouit pushed a commit to douaejeouit/beats that referenced this pull request Jan 25, 2017
* Add docker environment for integration and system testing
* Add system test file to check for correct docs.
* Brings docs in line with generated output
* Update data.json
* Remove hostname fields as already part of metricset
* Apply schema instead of manual conversion
* Rename pool.pool to pool.name
* Remove separate http client as not needed anymore

This is a follow up PR for elastic#3415
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants