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

Include Kibana dashboards in the individual Beats packages #4675

Merged
merged 2 commits into from
Jul 17, 2017

Conversation

tsg
Copy link
Contributor

@tsg tsg commented Jul 14, 2017

This is a PoC to:

  • include the kibana directory in the home path of the packages
  • make the setup.dashboards.directory the default instead of the URL
  • remove the snapshot.url and snapshot options
  • keep the setup.dashboards.url, but without a default value
  • keep the beats-dashboards package for the moment, but it won't be easy to use (you
    need to know the URL).

TODOs:

  • docs update
  • changelog

This is a PoC to:

* include the kibana directory in the home path of the packages
* make the `setup.dashboards.directory` the default instead of the URL
* remove the `snapshot.url` and `snapshot` options
* keep the `setup.dashboards.url`, but without a default value
* keep the beats-dashboards package for the moment, but it won't be easy to use (you
  need to know the URL).
@tsg tsg added discuss Issue needs further discussion. in progress Pull request is currently in progress. review labels Jul 14, 2017
@tsg
Copy link
Contributor Author

tsg commented Jul 14, 2017

jenkins, package it

@tsg tsg added v6.0.0-beta1 and removed discuss Issue needs further discussion. in progress Pull request is currently in progress. labels Jul 17, 2017
@exekias exekias merged commit 7d23a6e into elastic:master Jul 17, 2017
@dedemorton dedemorton mentioned this pull request Jul 17, 2017
42 tasks
@exekias
Copy link
Contributor

exekias commented Jul 18, 2017

@tsg, do you have any plan to update packaging to add kibana folder? I think it's not included in current packages by default

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants