Thanks to visit codestin.com
Credit goes to github.com

Skip to content

fix MSI installer #19962

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

Merged
merged 11 commits into from
Mar 27, 2025
Merged

fix MSI installer #19962

merged 11 commits into from
Mar 27, 2025

Conversation

ktsaou
Copy link
Member

@ktsaou ktsaou commented Mar 25, 2025

Development

  • allow users to install any version
  • stop netdata before installing the update
  • copy stream.conf to etc/netdata
  • download netdata.conf from the running netdata, if it is not already there.
  • if downloading of netdata.conf fails, create an empty netdata.conf, but only if it is not already there
  • Call Power Shell script after installation is completed

Test Plan

  1. Compile this branch
  2. Run package script
  3. Create installer.
  4. Install netdata and verify the following scenarios:
    • You have stream.conf and netdata.conf: nothing will change in your files
    • You are missing one of the files (or both): They will be created at the end
    • Uninstall: When you remove netdata, both files will be kept.

@ktsaou ktsaou requested a review from a team as a code owner March 25, 2025 14:24
@github-actions github-actions bot added the area/packaging Packaging and operating systems support label Mar 25, 2025
@thiagoftsm

This comment was marked as outdated.

@thiagoftsm thiagoftsm requested a review from ilyam8 March 27, 2025 14:47
@netdata-community-bot
Copy link

This pull request has been mentioned on Netdata Community Forums. There might be relevant details there:

https://community.netdata.cloud/t/updating-windows-agent-not-possible-without-reboot-from-v2-2-6-v3-0/7471/2

@netdata-community-bot

This comment was marked as resolved.

@ilyam8 ilyam8 merged commit e80fdfd into netdata:master Mar 27, 2025
101 checks passed
stelfrag pushed a commit to stelfrag/netdata that referenced this pull request Mar 28, 2025
* allow users to install any version; stop netdata before installing the update

* fix compatibility with wix 4

* fix-msi: Remove conflict between options (https://docs.firegiant.com/wix3/xsd/wix/majorupgrade/)

* fix-msi: Comment not allowed code. We cannot have more than one primary key listed (FileInUse)

* fix-msi: Add Power Shell script to be called after installation

* fix-msi: Copy power shell script to installer.

* fix-msi: Add option to do download and move PS file

* fix-msi: First try run at the end

* iis_ad: Fix wrong execution order and call to powershell

* fix-msi: Add messages and test to Power Shell script

---------

Co-authored-by: thiagoftsm <[email protected]>
(cherry picked from commit e80fdfd)
@stelfrag stelfrag mentioned this pull request Mar 28, 2025
stelfrag pushed a commit to stelfrag/netdata that referenced this pull request Mar 28, 2025
* allow users to install any version; stop netdata before installing the update

* fix compatibility with wix 4

* fix-msi: Remove conflict between options (https://docs.firegiant.com/wix3/xsd/wix/majorupgrade/)

* fix-msi: Comment not allowed code. We cannot have more than one primary key listed (FileInUse)

* fix-msi: Add Power Shell script to be called after installation

* fix-msi: Copy power shell script to installer.

* fix-msi: Add option to do download and move PS file

* fix-msi: First try run at the end

* iis_ad: Fix wrong execution order and call to powershell

* fix-msi: Add messages and test to Power Shell script

---------

Co-authored-by: thiagoftsm <[email protected]>
(cherry picked from commit e80fdfd)
Ferroin pushed a commit that referenced this pull request Apr 2, 2025
* allow users to install any version; stop netdata before installing the update

* fix compatibility with wix 4

* fix-msi: Remove conflict between options (https://docs.firegiant.com/wix3/xsd/wix/majorupgrade/)

* fix-msi: Comment not allowed code. We cannot have more than one primary key listed (FileInUse)

* fix-msi: Add Power Shell script to be called after installation

* fix-msi: Copy power shell script to installer.

* fix-msi: Add option to do download and move PS file

* fix-msi: First try run at the end

* iis_ad: Fix wrong execution order and call to powershell

* fix-msi: Add messages and test to Power Shell script

---------

Co-authored-by: thiagoftsm <[email protected]>
(cherry picked from commit e80fdfd)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/packaging Packaging and operating systems support
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants