Skip to content

Conversation

@rerpha
Copy link
Contributor

@rerpha rerpha commented Jan 23, 2026

…ore deploys

## Preparatory Steps for Client and Server

Before starting any manual deployments, run the [`instrument_deploy.yaml` ansible playbook](https://github.com/ISISComputingGroup/ansible-playbooks/tree/main?tab=readme-ov-file#instrument_deployyaml) which currently installs the JDK on instruments.

Copy link
Member

Choose a reason for hiding this comment

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

where do you run this and if ndx do you need to install ansible first?

Copy link
Member

Choose a reason for hiding this comment

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

do you run this on your own computer just for the ndx you intend to target?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

You run it in your own WSL instance which has SSH key-based access to the instruments - i can add some notes to https://github.com/ISISComputingGroup/ansible-playbooks?tab=readme-ov-file#ansible-ndx to make this clearer

Copy link
Member

Choose a reason for hiding this comment

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

i think adding an example shownig use of --limit for a deploy would be helpful so we don't have multiple people accidentally deploying to e.g. all of ts1 at the same time

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Copy link
Contributor Author

Choose a reason for hiding this comment

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

i think adding an example shownig use of --limit for a deploy would be helpful so we don't have multiple people accidentally deploying to e.g. all of ts1 at the same time

it prompts for this when you run it so you don't accidentally run it everywhere

Copy link
Contributor Author

Choose a reason for hiding this comment

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

have also added note in a0a32d0

@rerpha rerpha requested a review from FreddieAkeroyd January 23, 2026 15:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Flash Review

Development

Successfully merging this pull request may close these issues.

3 participants