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

Provide necessary permission for the kickstart to run the netdata-updater script #15132

Merged
merged 4 commits into from Jun 2, 2023

Conversation

tkatsoulas
Copy link
Contributor

@tkatsoulas tkatsoulas commented Jun 1, 2023

Summary

At the moment the updater script on the native packages is installed with ownership root netdata . Kickstart need to have root access to verify the existence of the netdata-updater script and run it.

Test Plan

Download kickstart script from this PR.

  1. Make an installation with any kind of native packages
  2. Verify that the automatic updates are enabled.

Further testing.

Enforce other auto update types
--auto-update-type Specify a particular scheduling type for auto-updates (valid types: systemd, interval, crontab)

Additional Information
For users: How does this change affect me?

@github-actions github-actions bot added the area/packaging Packaging and operating systems support label Jun 1, 2023
@tkatsoulas
Copy link
Contributor Author

AFA I can tell the debian packages are not affected by this issue.

Signed-off-by: Tasos Katsoulas <tasos@netdata.cloud>
@tkatsoulas tkatsoulas marked this pull request as ready for review June 1, 2023 13:15
@tkatsoulas tkatsoulas requested a review from Ferroin as a code owner June 1, 2023 13:15
@tkatsoulas
Copy link
Contributor Author

This Enforce other auto update types --auto-update-type Specify a particular scheduling type for auto-updates (valid types: systemd, interval, crontab) option is probably broken. We will investigate in different issue

packaging/installer/kickstart.sh Outdated Show resolved Hide resolved
packaging/installer/kickstart.sh Outdated Show resolved Hide resolved
tkatsoulas and others added 3 commits June 1, 2023 17:27
Co-authored-by: Austin S. Hemmelgarn <austin@netdata.cloud>
Signed-off-by: Tasos Katsoulas <tasos@netdata.cloud>
Signed-off-by: Tasos Katsoulas <tasos@netdata.cloud>
@Ferroin
Copy link
Member

Ferroin commented Jun 1, 2023

Static build failures are unrelated to this PR and should be fixed by netdata/helper-images#210.

@ilyam8
Copy link
Member

ilyam8 commented Jun 1, 2023

Make an installation with any kind of native packages

No problem on Debian. /usr/libexec/netdata/netdata-updater.sh is 755

@tkatsoulas tkatsoulas changed the title Fix installer/uninstaller and updater scripts ownership permission Provide necessary permission for the kickstart to run the netdata-updater script Jun 1, 2023
@tkatsoulas tkatsoulas merged commit e2063ca into master Jun 2, 2023
125 checks passed
@ilyam8 ilyam8 deleted the patch-auto-updates-fix branch June 12, 2023 11:57
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.

None yet

3 participants