search

elastic / beats-dashboards

DEPRECATED. Moved to https://github.com/elastic/beats. Please use the new repository to add new issues.
Other
196 stars 73 forks source link

Add load script for Windows #64

Open monicasarbu opened 8 years ago

monicasarbu commented 8 years ago

Merge to the 1.1 branch the PR: https://github.com/elastic/beats-dashboards/pull/57

andrewkroh commented 8 years ago

I found an issue with the script #66 so we may want to hold off on merging to 1.1 until it gets fixed.

andrewkroh commented 8 years ago

This will need documented in the libbeat Getting Started guide where it shows how to load the dashboards on unix.

andrewkroh commented 8 years ago

Also the projects README should be updated to show instructions for Windows.

dedemorton commented 8 years ago

@andrewkroh I'm updating the doc to tell users how to download and run the script. Will there be a downloadable zip file somewhere equivalent to what we provide here for other platforms: http://download.elastic.co/beats/dashboards/beats-dashboards-1.1.0.tar.gz

andrewkroh commented 8 years ago

A tar.gz isn't very Windows friendly so maybe we should only package the dashboards as a zip file.

@monicasarbu @tsg Any issues with me changing the release script to build a zip instead of tar.gz?

monicasarbu commented 8 years ago

@andrewkroh Good point, Andrew. Indeed it is strange to offer tar.gz for Windows. I totally agree that we should provide a zip archive.

damm commented 8 years ago

Windows can handle .tar.gz by default now no? I had no problem with Windows 8 in a VM with them

dedemorton commented 8 years ago

@monicasarbu @andrewkroh Is there a final resolution on this? If we change the script to build a zip, do we plan to do it before 1.1 ships?

dedemorton commented 8 years ago

Just noting that the script has been updated to be a zip through this PR: https://github.com/elastic/beats-dashboards/pull/70