Conversation
Owner
Author
|
I marked as draft, because if merged as-is, this will require a full node and nvm purge in order to update node (which will need to reinstall nvm). If we break out the node install steps into it's own role I will work on breaking the current |
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
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
TLDR: Create an update / upgrade path for nvm/node versions without borking nodes. This will allow upgrades / downgrades to the
nvm.node.versionprovided in thevars.ymlfile.This pull request includes changes to the roles and variables related to NVM, Node, Logrotate, PM2, and Watchdog. The most important changes include adding tasks to check and install NVM and Node, installing Logrotate, PM2, and Watchdog, and updating the Node version in the
vars.ymlfile.Summary of changes:
roles/nvm/tasks/nvm-install.yml: Added tasks to check if NVM and Node are installed, get the installed Node version, determine if a Node update is needed, install Logrotate, PM2, and Watchdog, and start PM2 after installation. [1] [2]roles/nvm/tasks/nvm-purge.yml: Added tasks to purge Logrotate, PM2, and Watchdog with specific tags and elevated privileges.roles/pm2/tasks/pm2-stop.yml: Added theignore_errorsoption to theansible.builtin.shelltask to allow it to continue executing even if it encounters an error.vars.yml: Updated the Node version from16.19.1to20.9.0to specify the desired version of Node to be installed.