-
Notifications
You must be signed in to change notification settings - Fork 98
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
Renamed macOS Login Item #2207
Renamed macOS Login Item #2207
Conversation
Signed-off-by: Miguel Verdaguer Velázquez <verdx@riseup.net>
… postinstall.sh) Signed-off-by: Miguel Verdaguer Velázquez <verdx@riseup.net>
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.
Please, continue with the testing with the following cases:
- Upgrade from previous versions (screenshots)
- Reboot macOS and check if the service is running (the agent is running and connected after the reboot)
- Uninstall removes successfully all files (no traces)
Further testingThe testing asked for in #2207 (review) has been completed without error. Installation of 4.4.1
Upgrade to 4.4.2
Uninstall according to documentation
Run the Startup Item independently
|
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.
The macOS sources installation should be modified as this script (darwin-init.sh) is still used: https://github.com/search?q=repo%3Awazuh%2Fwazuh%20darwin-init.sh&type=code
Also, these variables do not match:
LAUNCHER_SCRIPT=/Library/StartupItems/WAZUH/launcher.sh
STARTUP_SCRIPT=/Library/StartupItems/WAZUH/WAZUH
The macOS sources installation guide should be reviewed as it is referencing an old /Library/StartupItems/OSSEC
file
https://documentation.wazuh.com/current/deployment-options/wazuh-from-sources/wazuh-agent/index.html
…in-init.sh Signed-off-by: Miguel Verdaguer Velázquez <verdx@riseup.net>
Signed-off-by: Miguel Verdaguer Velázquez <verdx@riseup.net>
UpdateAfter this commentary, #2207 (review), and speaking with @rauldpm and @DFolchA, the approach to this problem has changed, instead of removing the use of TestingThe testing is on hold for the creation of the packages, which is in the queue in the pipeline https://ci.wazuh.info/job/Packages_builder_macos/ Documentation fixA new PR has been created for the documentation, to change the error mentioned in this comment, |
…the package Signed-off-by: Miguel Verdaguer Velázquez <verdx@riseup.net>
Signed-off-by: Miguel Verdaguer Velázquez <verdx@riseup.net>
Signed-off-by: Miguel Verdaguer Velázquez <verdx@riseup.net>
Signed-off-by: Miguel Verdaguer Velázquez <verdx@riseup.net>
Signed-off-by: Miguel Verdaguer Velázquez <verdx@riseup.net>
05fa878
to
3f6fcb5
Compare
Signed-off-by: Miguel Verdaguer Velázquez <verdx@riseup.net>
Signed-off-by: Miguel Verdaguer Velázquez <verdx@riseup.net>
Signed-off-by: Miguel Verdaguer Velázquez <verdx@riseup.net>
UpdateThe solution chosen has brought a lot of problems when creating the package, with an error appearing saying the StartupItem script Error in the package building
TestsUsing the new solution, the package has been created correctly(https://ci.wazuh.info/job/Packages_builder_macos/9943) and tested on macOS Ventura: Installation of 4.4.2
Upgrade to 4.4.3
Uninstall according to documentation
|
Description
The name of the script used as a Login Item is changed, thus changing the name of the Login Item appearing on
Settings -> General -> Login Items
. The call todarwin-init.sh
during the installation has been removed, as it only repeats what is done in thepreinstall.sh
script.Tests
Create the package: https://ci.wazuh.info/job/Packages_builder_macos/9828
Install it on MacOS Ventura
Check Login Item