-
Notifications
You must be signed in to change notification settings - Fork 5.8k
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
Switch to using netdata-updater.sh to toggle auto updates on and off when installing. #12296
Switch to using netdata-updater.sh to toggle auto updates on and off when installing. #12296
Conversation
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.
Docs: Minor suggestions and clarification needed.
395fb75
to
b2de617
Compare
Rebased to pick up changes in master, and fixed ambiguous heading in docs. |
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.
Docs: LGTM
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.
I'd add a shellcheck exemption regarding TPUT_RED warning
Agreed, but that’s orthogonal to this PR (changes in this PR are not what has caused this warning, so the warning should not be considered as blocking this PR). |
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.
LGTM
Co-authored-by: Tina Luedtke <kickoke@users.noreply.github.com>
b2de617
to
6b6930a
Compare
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.
Docs: LGTM
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.
LGTM. Not that I really tested the change, though
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.
looks good, except for that shellcheck which we may want to fix at some point
Summary
This will allow us to handle control of auto-updates in a consistent manner no matter which install method was used.
Test Plan
Full testing will require waiting until on or after 2022-03-03, as prior to that nightly package builds will not include the updater script.
General testing is a simple case of verifying that auto-updates are enabled (or disabled) correctly when using the copy of the kickstart script from this PR.
Additional Information
This is the second half of #12202.