FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

[ ON HOLD] Uninstall dappnode on error by pablomendezroyo · Pull Request #342 · dappnode/DAppNode · GitHub

[ ON HOLD] Uninstall dappnode on error - #342

Draft
pablomendezroyo wants to merge 1 commit into
masterfrom
pablo/uninstall-on-error
Draft

[ ON HOLD] Uninstall dappnode on error#342
pablomendezroyo wants to merge 1 commit into
masterfrom
pablo/uninstall-on-error

Conversation

pablomendezroyo commented Aug 25, 2021
edited by sync-by-unito Bot
Loading

Copy link
Copy Markdown
Contributor
  • Catch exit signal
  • call dappnode uninstaller on exit

┆Issue is synchronized with this Basecamp todo by Unito

pablomendezroyo self-assigned this Aug 25, 2021
pablomendezroyo added the installer This issue is related to the installer label Aug 25, 2021
pablomendezroyo linked an issue Aug 25, 2021 that may be closed by this pull request
4 tasks

Copy link
Copy Markdown
Contributor

@eduadiez @3alpha Could this have un-intended consequences? What is the uninstall script removing?

Copy link
Copy Markdown
Contributor Author

@eduadiez @3alpha Could this have un-intended consequences? What is the uninstall script removing?

maybe would be safer to just uninstall dappnode on error if the installation was done from script method, since this method is commonly used on personal hardware not purchased from dappnode

pablomendezroyo changed the title Uninstall dappnode on error [ ON HOLD] Uninstall dappnode on error May 11, 2023
pablomendezroyo marked this pull request as draft May 11, 2023 08:27
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

installer This issue is related to the installer

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Uninstall dappnode and Exit if dappnode_install.sh fails

3 participants


Back | FazBrowse Home | New Git URL