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

mng/vm aliases and chain upgrades #110

Merged

Conversation

ngurmen
Copy link
Contributor

@ngurmen ngurmen commented Dec 17, 2023

Linked issues

None

Dependencies

None

Changes

  • added task to add vm aliases.json using template
  • added task to add upgrade.json for chains if defined

Breaking changes

None

Additional comments

The folowing directory/file structure needs to be created for upgrade.json files at the same directory where the playbook is called.

- inventories
- files
   - {{ network_id }} i.e. fuji
      - vms
         - {{ chain_id }}
            - upgrade.json

Also, a new variable for the files_path is needed in the user configuration file as follows:

files_path: /path/to/files  i.e. /home/ubuntu/my_ansible/files

@ngurmen ngurmen changed the title Mng/vm aliases and chain upgrades mng/vm aliases and chain upgrades Dec 17, 2023
@Nuttymoon Nuttymoon self-requested a review December 18, 2023 12:53
roles/node/tasks/install-vms.yml Outdated Show resolved Hide resolved
roles/node/tasks/config-chains.yml Outdated Show resolved Hide resolved
Copy link
Contributor

@Nuttymoon Nuttymoon left a comment

Choose a reason for hiding this comment

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

Almost there!

I just noticed that the right naming is chain upgrades and not updates. Would you mind renaming the variables for clarity?

roles/node/defaults/main.yml Outdated Show resolved Hide resolved
roles/node/defaults/main.yml Outdated Show resolved Hide resolved
roles/node/tasks/config-chains.yml Show resolved Hide resolved
ngurmen and others added 3 commits December 20, 2023 09:35
Co-authored-by: Gauthier Leonard <gauthier_leonard@msn.com>
Co-authored-by: Gauthier Leonard <gauthier_leonard@msn.com>
@Nuttymoon Nuttymoon merged commit 922aace into AshAvalanche:main Dec 21, 2023
@ngurmen ngurmen deleted the mng/vm_aliases_and_chain_upgrades branch December 21, 2023 15:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants