-
Notifications
You must be signed in to change notification settings - Fork 2
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
[integration] Add support of Graal to ELK & Mordred #4
Comments
@valeriocos @jgbarah @aswanipranjal I have had a look at the codebase of ELK and Mordred. And as we had discussed earlier (in the proposal).
Would love to have suggestions on the approach :) |
@inishchith yes, this is exactly the idea. If you think it can be useful, I can push an example to a branch in my forks of ELK and mordred, what do you think ? |
@inishchith I have just pushed some code :) Graal should be enhanced to mimic the way the Git Perceval backend is called from Mordred. In a nutshell, in the projects.json we declare just the url of the Git repository, the local path where the repo will be cloned ( Here you will find some code that kicstarts the integration between graal and ELK. Feel free to reuse this code and improve it. In order to use that code, you just need to include in the projects.json and setup.cfg the settings below. Then you can launch micro-mordred with the params setup.cfg
projects.json
|
@valeriocos Thanks for sharing this. I've started some work and will update in due time. Thanks a lot :) |
You're welcome @inishchith ! :) |
@valeriocos After getting the initial work executed and tested locally, I had some thoughts over the improvements that could be worked on.
Thanks :) |
Sure @inishchith ! Note that generally an enriched index contains just flat JSON docs (without nested attributes). The enricher itself takes care of flattening the raw data (or just a part of it). However, changes can be made also in Graal, if needed. |
@valeriocos Thanks for the quick response.
Yes. I'm working on a new structure and evaluating them (in the context of both Graal and ELK). I'll share them once I'm done to get some clarity and suggestions :) |
@valeriocos As mentioned earlier, The current structure of analysis attribute -
As we can see the above data is produced per commit ( and a commit can contains changes made to multiple files )
Work Branch - inishchith/grimoirelab-elk#gsoc-graal-2019 Reference Issue: chaoss/metrics#139 |
Closed in reference(above) |
The text was updated successfully, but these errors were encountered: