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

Build Wazuh module for Filebeat #233

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Build Wazuh module for Filebeat #233

wants to merge 5 commits into from

Conversation

jesusgn90
Copy link

@jesusgn90 jesusgn90 commented Jul 16, 2019

Hi team,

wazuh/wazuh#3566 needs to build a Filebeat module, I think this repository is the right place to store and to maintain our build script.

The final location for the script may change, even the code of the script because it was designed under DEB instance with some other requirements already installed, please let me know what we need to make it work in Jenkins for example.

Regards

@jesusgn90
Copy link
Author

Update

The expected S3 path would be https://packages.wazuh.com/3.x/filebeat/wazuh-module.tar.gz

@jesusgn90 jesusgn90 marked this pull request as ready for review July 22, 2019 09:01
@BraulioV BraulioV added the type/refactor Enhancement label Jul 22, 2019
@BraulioV BraulioV added this to Wave2 in TBD via automation Jul 22, 2019
Copy link
Contributor

@alberpilot alberpilot left a comment

Choose a reason for hiding this comment

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

Hello @jesusgn90

Thanks for your contribution. The building tools of this repository follow some directives which are applicable to this building tool. Please follow the directions below:

  • Use docker for building the module.
  • Use a script for parameters like version, output folder, output filename, etc. You need to establish the default values.
  • Add the option "-k", the checksum file.
  • Remove the s3 upload options, it is not the responsibility of the building tool.
  • Add readme page and it has to be mentioned in the main readme.

Please see other examples as Wazuhapp, or SplunkApp. Thanks.
Best regards,
Alberto R

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type/refactor Enhancement
Projects
TBD
  
Wave2
Development

Successfully merging this pull request may close these issues.

None yet

4 participants