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

Add support to the updater to toggle auto-updates on and off. #12202

Merged
merged 3 commits into from
Mar 2, 2022

Conversation

Ferroin
Copy link
Member

@Ferroin Ferroin commented Feb 21, 2022

Summary

This restores a subset of the changes that were reverted in #12182 that are generally known to be clean and working correctly.

Specifically, it re-adds the code in the updater script for toggling automatic updates on and off, the changes in the packaging code that include the updater script in our native packages, and the associated documentation changes for both parts.

The kickstart script integration will be handled in a separate PR at some point later this week.

Test Plan

Testing of the new --enable-auto-updates and --disable-auto-updates options can be done trivially on any existing static or locally built install by downloading the copy of the script from this PR and simply running it with the appropriate option.

This also adds the updater script to our native packages.
@github-actions github-actions bot added area/docs area/packaging Packaging and operating systems support labels Feb 21, 2022
@Ferroin Ferroin marked this pull request as ready for review February 22, 2022 12:55
maneamarius
maneamarius previously approved these changes Feb 28, 2022
Copy link
Contributor

@maneamarius maneamarius left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@kickoke kickoke left a comment

Choose a reason for hiding this comment

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

Docs: It seems that you are committing changes that are already online. Could you rebase to make sure you're committing on top of the newest version?

@Ferroin
Copy link
Member Author

Ferroin commented Feb 28, 2022

Docs: It seems that you are committing changes that are already online. Could you rebase to make sure you're committing on top of the newest version?

It is up to date, the documentation changes got caught in #12182 as collateral damage, and this PR is re-adding them.

@Ferroin Ferroin requested a review from kickoke February 28, 2022 18:35
@Ferroin Ferroin requested a review from ilyam8 March 1, 2022 13:16
Copy link
Contributor

@kickoke kickoke left a comment

Choose a reason for hiding this comment

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

Docs: Looks good to me

@Ferroin Ferroin merged commit 3dc3144 into netdata:master Mar 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/docs area/packaging Packaging and operating systems support
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants