Open
Conversation
adskyiproger
commented
Feb 6, 2026
| if (fs.existsSync(dest)) { | ||
| warn(` ⚠️ Skipping existing file: ${dest}`); | ||
| return; | ||
| } |
Collaborator
Author
There was a problem hiding this comment.
Initially we supposed only create helm chart values and ansible inventory files.
It is safe to update as well
Always overwrite generated configuration file, otherwise updates will not work
adskyiproger
commented
Feb 6, 2026
| if (fs.existsSync(outputPath)) { | ||
| warn(` ⚠️ Skipping ${templatePath}, file already exists at ${outputPath}`); | ||
| return; | ||
| } |
Collaborator
Author
There was a problem hiding this comment.
Initially we supposed only create helm chart values and ansible inventory files.
It is safe to update as well
Always overwrite generated configuration file, otherwise updates will not work
adskyiproger
commented
Feb 6, 2026
| } | ||
| } | ||
|
|
||
| export function getUsers(data: any): any { |
Collaborator
Author
There was a problem hiding this comment.
Makes sense to move this kind of functions to external file (library), If we add more methods in the future
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
1 participant
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.
Testing
Integration with
environment:initOption no selected
Go to next step
Option Yes selected: For new environment
Display existing users and enter edit mode
Save without adding any users: No confirmation message
Exit without saving: No confirmation message
Option Yes selected: For existing environment
Display existing users and enter edit mode
Script features
Add new user
NOTE: by default all new users are added present
Add new user without ssh keys
Summary
Edit existing user
Select user to edit:
Edit user summary:
Edit user change state:
Edit user after editing:
Back to main screen after editing
Save changes after editing user: script back to normal execution
Verify updated file
Remove user
NOTE: Script has support for multiple users removal
View all after user removal
Confirmation dialog
Success save and exist
User was removed from inventory file
