-
Notifications
You must be signed in to change notification settings - Fork 127
Lustre mount via Ansible for SMHP Slurm LCS #682
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
base: main
Are you sure you want to change the base?
Changes from 3 commits
8a73b80
15cc3a8
715c6e4
b6ca6d5
51cd4e9
4918505
7a37354
97885b9
0d2efa4
0a1aeab
1003b6b
357d413
a2f0a8b
00045a2
5217c0b
73146bc
269efa6
72fad2b
36498ea
ce3022b
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -10,9 +10,6 @@ FSX_OPENZFS_DNS_NAME="$1" | |
OPENZFS_MOUNT_POINT="$2" | ||
NFS_VERSION=4.2 | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Does this need to be hardcoded? Can we add a comment why Version 4.2 is used There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. We need to pin versions. For everything. |
||
|
||
# Ansible Version | ||
ANSIBLE_VERSION="10.7.0" | ||
|
||
# Function for error handling | ||
handle_error() | ||
{ | ||
|
@@ -33,16 +30,6 @@ verify_parameters() | |
fi | ||
} | ||
|
||
# Install Ansible and collections: Move to higher LCS once others start using Ansible too. | ||
install_ansible() | ||
{ | ||
apt-get update | ||
# apt-get install -y ansible=$ANSIBLE_VERSION | ||
apt-get install -y python3-pip | ||
python3 -m pip install "ansible==${ANSIBLE_VERSION}" | ||
ansible-galaxy collection install ansible.posix | ||
} | ||
|
||
# Install NFS Client based on OS | ||
install_nfs_client() | ||
{ | ||
|
@@ -66,7 +53,6 @@ main() | |
echo "Mount_fsx_openzfs called with fsx_openzfs_dns_name: $FSX_OPENZFS_DNS_NAME" | ||
echo "Using openzfs_mount_point: $OPENZFS_MOUNT_POINT" | ||
verify_parameters | ||
install_ansible | ||
install_nfs_client | ||
mount_fs | ||
echo "FSx OpenZFS mounted successfully to $OPENZFS_MOUNT_POINT" | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,28 @@ | ||
#!/bin/bash | ||
|
||
set -ex | ||
|
||
# Ansible Version | ||
ANSIBLE_VERSION="10.7.0" | ||
|
||
# Install Ansible and collections: Move to higher LCS once others start using Ansible too. | ||
install_ansible() | ||
{ | ||
apt-get update | ||
# apt-get install -y ansible=$ANSIBLE_VERSION | ||
apt-get install -y python3-pip | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. do we need retry on the network call, similar to what we have with other calls that was due to issues observed with u22.04 AMI. How many nodes has this script been tested on? |
||
python3 -m pip install "ansible==${ANSIBLE_VERSION}" | ||
ansible-galaxy collection install ansible.posix | ||
|
||
# Verify ansible installation | ||
echo "Ansible version:" | ||
ansible --version | ||
} | ||
|
||
main() | ||
{ | ||
echo "Installing Ansible..." | ||
install_ansible | ||
} | ||
|
||
main |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Where is this getting installed? need architecture diagram and readme
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please clarify, does ansible get installed on every node?
Uh oh!
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Architecture diagram for what? Yes, ansible, the package, is installed on every node (hence localhost to mount in inventory)