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

Make dev tools optional in generated beats #1810

Merged
merged 1 commit into from
Jun 8, 2016

Conversation

ruflin
Copy link
Contributor

@ruflin ruflin commented Jun 7, 2016

The dev-tools folder is now not shipped by default with the generator anymore. To created it and run the packaging process, make pack must be used.

Further changes:

  • Remove GO15VENDOREXPERIMENT as not needed anymore with 1.6
  • Remove not needed files in generators because now repo is in the beats repo
  • For beat generator, separate test and test-build
  • Add beat template tests to travis build
  • Remove glide.yml from metricset generator as it will be generated automatically
  • Temporarely remove versioning from generated beats as required for tests to have most recent version

@ruflin ruflin added the in progress Pull request is currently in progress. label Jun 7, 2016
@ruflin ruflin force-pushed the optional-dev-tools branch from e38fff2 to d661b47 Compare June 7, 2016 12:09
The dev-tools folder is now not shipped by default with the generator anymore. To created it and run the packaging process, `make pack` must be used.

Further changes:
* Remove GO15VENDOREXPERIMENT as not needed anymore with 1.6
* Remove not needed files in generators because now repo is in the beats repo
* For beat generator, separate test and test-build
* Add beat template tests to travis build
* Remove glide.yml from metricset generator as it will be generated automatically
* Temporarely remove versioning from generated beats as required for tests to have most recent version
@ruflin ruflin force-pushed the optional-dev-tools branch from f3c4444 to 4c7c761 Compare June 7, 2016 12:27
@ruflin ruflin changed the title Optional dev tools Make dev tools optional in generated beats Jun 7, 2016
@ruflin ruflin added review :Generator Related to code generators for building custom Beats or modules. and removed in progress Pull request is currently in progress. labels Jun 7, 2016
@tsg tsg merged commit 37819c7 into elastic:master Jun 8, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Generator Related to code generators for building custom Beats or modules. review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants