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

Creating a more robust nvm-purge. #75

Merged
merged 2 commits into from
Jan 3, 2024
Merged

Creating a more robust nvm-purge. #75

merged 2 commits into from
Jan 3, 2024

Conversation

TechDufus
Copy link
Owner

This is to prep for future functionality to upgrade nvm/node versions in the future, or upgrade when running FluxNodeInstall against a previously non-ansible managed FluxNode (node started with multitoolbox only).

This pull request introduces changes to the roles and tasks related to the daemon and nvm. The most important changes include the addition of new tasks for starting and stopping the daemon service, modifications to the nvm installation task to install a specific version of Node, and the addition of tasks to stop Flux via PM2 and stop the Daemon.

Summary of changes:

  • roles/daemon/tasks/main.yml: Added two new tasks, "Start Daemon Service" and "Stop Daemon Service", which include tasks from separate files.
  • roles/daemon/tasks/daemon-start.yml: Added a new task named "Ensure Scripts Exist" which includes tasks from the daemon-scripts.yml file.
  • roles/daemon/tasks/daemon-stop.yml: Added a new task named "Ensure Scripts Exist" which includes tasks from the daemon-scripts.yml file.
  • roles/nvm/tasks/nvm-install.yml: Modified the task to install a specific version of Node using nvm.
  • roles/nvm/tasks/nvm-purge.yml: Added two new tasks to stop Flux via PM2 and stop the Daemon.

@TechDufus TechDufus added the enhancement New feature or request label Dec 30, 2023
@TechDufus TechDufus requested a review from JKTUNING January 3, 2024 02:29
@TechDufus
Copy link
Owner Author

Merging because i want to test more on stream. Sorry @JKTUNING

@TechDufus TechDufus merged commit 3b17ded into main Jan 3, 2024
1 check passed
@TechDufus TechDufus deleted the nvm/robust_purge branch January 12, 2024 15:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants