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

Update uninstaller documentation. #13627

Merged
merged 3 commits into from Sep 21, 2022

Conversation

Ferroin
Copy link
Member

@Ferroin Ferroin commented Sep 5, 2022

Summary

This properly documents the preferred uninstallation method, as well as tidying up the existing documentation as a fallback method.

Test Plan

n/a

@github-actions github-actions bot added area/docs area/packaging Packaging and operating systems support labels Sep 5, 2022
thiagoftsm
thiagoftsm previously approved these changes Sep 5, 2022
kickstart script (internally, the kickstart script also uses this uninstaller script, it just handles the process
outlined below for you).

This uninstaller script is self-contained other than requiring a `.environment` file that was generated ruing
Copy link
Contributor

Choose a reason for hiding this comment

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

@Ferroin Did you mean to say "was generated during installation" here?

DShreve2
DShreve2 previously approved these changes Sep 6, 2022
Copy link
Contributor

@DShreve2 DShreve2 left a comment

Choose a reason for hiding this comment

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

Two comments for consistency; approved!

packaging/installer/UNINSTALL.md Outdated Show resolved Hide resolved
packaging/installer/UNINSTALL.md Outdated Show resolved Hide resolved
Co-authored-by: DShreve2 <david@netdata.cloud>
@Ferroin Ferroin dismissed stale reviews from DShreve2 and thiagoftsm via 8b5a6fa September 7, 2022 11:23
@Ferroin Ferroin merged commit c66854b into netdata:master Sep 21, 2022
@Ferroin Ferroin deleted the uninstaller-doc-update branch September 21, 2022 16:24
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