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.

Sign up for GitHub

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

Jump to bottom

Add validation for aws module to check all fields are documented in fields.yml #11226

Closed
kaiyan-sheng opened this issue Mar 13, 2019 · 2 comments
Closed

Add validation for aws module to check all fields are documented in fields.yml #11226

kaiyan-sheng opened this issue Mar 13, 2019 · 2 comments
Assignees
@roncohen @kaiyan-sheng
Labels
enhancement Team:Integrations Label for the Integrations team [zube]: Backlog

Comments

@kaiyan-sheng
Copy link
Contributor

#11221 (review)
I found this issue #11219 when testing kibana for 7.0. It would be good to have validation on if all the fields are documented in fields.yml for all aws modules to avoid this problem next time.
@kaiyan-sheng kaiyan-sheng self-assigned this Mar 13, 2019
@kaiyan-sheng kaiyan-sheng added Team:Integrations Label for the Integrations team enhancement labels Mar 13, 2019
This was referenced Mar 13, 2019
Merged
Merged
@kaiyan-sheng
Copy link
Contributor Author

Will start work on this after #11127 is merged.

@alvarolobato alvarolobato self-assigned this Apr 23, 2019
@alvarolobato alvarolobato removed their assignment Jun 3, 2019
@roncohen roncohen self-assigned this Jun 11, 2019
@kaiyan-sheng kaiyan-sheng mentioned this issue Jun 20, 2019
Merged
@kaiyan-sheng
Copy link
Contributor Author

This is done for Metricbeat and Filebeat already.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@roncohen roncohen

@kaiyan-sheng kaiyan-sheng

Labels
enhancement Team:Integrations Label for the Integrations team [zube]: Backlog
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@roncohen @andresrc @kaiyan-sheng @alvarolobato