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

Windows Installer Script - Implementation #498

Merged
merged 24 commits into from
Mar 31, 2021

Conversation

mattdurham
Copy link
Collaborator

@mattdurham mattdurham commented Mar 29, 2021

PR Description

Create windows installer that supports both silent install and guided mode. This installer should automatically be added to the releases list.

Features:

  • Writes WAL to appdata (c:\ProgramData)
  • Can reinstall cleanly and without overriding config
  • Has Installer and Uninstaller

Which issue(s) this PR fixes

Closes #301

Notes to the Reviewer

PR Checklist

  • CHANGELOG updated
  • Documentation added
  • Tests updated

@mattdurham mattdurham changed the title Checking in modified version of the installer Windows Installer Script - Implementation Mar 29, 2021
@mattdurham mattdurham marked this pull request as ready for review March 30, 2021 13:58
@mattdurham mattdurham requested a review from rfratto March 30, 2021 13:58
Copy link
Member

@rfratto rfratto left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🎉 This is really exciting!

.gitignore Outdated Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
Makefile Outdated Show resolved Hide resolved
docs/windows.md Outdated Show resolved Hide resolved
docs/windows.md Outdated Show resolved Hide resolved
packaging/windows/install_script.nsis Outdated Show resolved Hide resolved
packaging/windows/install_script.nsis Outdated Show resolved Hide resolved
packaging/windows/install_script.nsis Outdated Show resolved Hide resolved
packaging/windows/install_script.nsis Show resolved Hide resolved
packaging/windows/install_script.nsis Outdated Show resolved Hide resolved
@mattdurham mattdurham requested a review from rfratto March 31, 2021 18:08
Copy link
Member

@rfratto rfratto left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Few grammar nits in the docs, but I'm ready for this to be merged.

CHANGELOG.md Outdated Show resolved Hide resolved
docs/windows.md Outdated Show resolved Hide resolved
docs/windows.md Outdated Show resolved Hide resolved
docs/windows.md Outdated Show resolved Hide resolved
docs/windows.md Outdated Show resolved Hide resolved
docs/windows.md Outdated Show resolved Hide resolved
docs/windows.md Show resolved Hide resolved
mattdurham and others added 8 commits March 31, 2021 14:27
Co-authored-by: Robert Fratto <robertfratto@gmail.com>
Co-authored-by: Robert Fratto <robertfratto@gmail.com>
Co-authored-by: Robert Fratto <robertfratto@gmail.com>
Co-authored-by: Robert Fratto <robertfratto@gmail.com>
@mattdurham mattdurham merged commit e83883d into grafana:main Mar 31, 2021
@mattdurham mattdurham deleted the Windows_Install_Script branch March 31, 2021 18:59
@mattdurham mattdurham mentioned this pull request Sep 7, 2021
3 tasks
mattdurham added a commit that referenced this pull request Nov 11, 2021
* Checking in modified version of the installer

* Fix tab issue in make

* Support Versioning

* Add Version to installer output

* Add docs for windows installer

* Do not overwrite agent-config.yaml if it exists

* Use appdata folder instead of program files

* Allow reinstlal

* Stop service on reinstall before copying files

* Remove appdata wal

* Fix documentation

* Update windows.md

Fix type

* Changes based on PR

* Add license information to the install

* Simplify silent install

* Update CHANGELOG.md

Co-authored-by: Robert Fratto <robertfratto@gmail.com>

* Update docs/windows.md

Co-authored-by: Robert Fratto <robertfratto@gmail.com>

* Update docs/windows.md

Co-authored-by: Robert Fratto <robertfratto@gmail.com>

* Update docs/windows.md

Co-authored-by: Robert Fratto <robertfratto@gmail.com>

* Links to and from documentation

* Fix type

* Clean up grammar

* Clean up grammar

Co-authored-by: Robert Fratto <robertfratto@gmail.com>
@github-actions github-actions bot added the frozen-due-to-age Locked due to a period of inactivity. Please open new issues or PRs if more discussion is needed. label Apr 15, 2024
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Apr 15, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
frozen-due-to-age Locked due to a period of inactivity. Please open new issues or PRs if more discussion is needed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Windows MSI installer
2 participants