-
Notifications
You must be signed in to change notification settings - Fork 68
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
Update package output structure #514
Update package output structure #514
Conversation
@jen-huang This is related to Phase 3: elastic/beats#19082 You should be able to use this branch to test changes in Kibana. |
Part of #517 |
I'm hitting errors with |
a41fea9
to
cada7b0
Compare
@jen-huang I rebased on master, now things should work as expected. |
This change contains several changes to the output generated for the package API endpoint: * dataset.id is renamed to dataset.name * datasources is renamed to config_templates. Currently only in the output, packages can follow later. * config_templates (aka datasources) do not pregenerate the content anymore to simply config building in Kibana * template_content is removed Further change is done to use `Dataset(s)` instead of `DataSet(s)` in the code for consistency.
3e88577
to
9bf9866
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approved, but please make sure if the import-beats
is not impacted with this change.
This change contains several changes to the output generated for the package API endpoint:
Further change is done to use
Dataset(s)
instead ofDataSet(s)
in the code for consistency.