-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
Better contributing guidelines #14260
Comments
Hey @philippkahr, Thanks for your feedback! Python 2.7 shouldn't be an issue after #14798. If you can, please give a try to the current code. You may need to remove your existing virtual environments. Installation of mage should be easy with
In principle you can enable Travis in your fork in github, and then it should trigger Travis builds in your own account. Let me know if it works for you.
Visualizations must match this regexp: https://github.com/elastic/beats/blob/v7.5.1/dev-tools/mage/check.go#L291 Any contribution there is welcome 🙂
This should be easy to add by copying one of the targets from one of the magefiles that have it. You can also use |
Hi @jsoriano
You are right, I will definitely check this out. Maybe we can add that as a rule for a linter?
Expecting great things from me, I do not feel confident enough in Go yet, to achieve such a task. I did try it but it did not work after a few hours of copying code around I gave up ;). |
No worries 🙂 Did you try to export the dashboards with |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
Describe the enhancement:
Hi,
I just wanted to point out that I had a really hard time, contributing to beats. Setting up go and python environment on my mac was not very clear. I did struggle all the way and it took me nearly 3 hours to set up everything. There were various errors with
mage
andpython2.7
andGOPATH
and so on. it is still broken and does not work, especially the python2.7 part not finding all the modules (likeimport yaml
) and go not being able to find thelibbeat
except I copygithub.com/elastic/beats
to the vendor folder inside the beat I am building.Maybe we should add an ansible playbook that correctly installs GO, set's the python environment and do all the stuff needed? I know that that might break existing settings, but it might help get started? Alternatively, add a guide on how to use the Vagrantfile, and what has to be installed locally to use it. I added a bit more code to the Vagrantfile to make it more useable #14148.
Testing
Would it be possible to trigger the Travis build from outside a pull request? I like to open a pull request once, my entire code is tested and not notify a bunch of people commenting on my pull request that
I fixed the Capitalletter typo
.Dashboards
There is a linter already in place checking for
name of dashboard [beat module] ...
. Maybe adding a new rule fortitle: Capitalletter name of visualisation [beat module] ...
. Would help? In my pull request, I missed a fewCapitalletter
and Travis did not point them out. Alsomage exportDashboard
is not available forwinlogbeat
#14149 (comment).Ressources
#4984
#14135 (comment)
#14148
The text was updated successfully, but these errors were encountered: