search

cloudmesh / cloudmesh-slurm

Other
0 stars 0 forks source link

1: use now the pi nfs command #6

Closed jpfleischer closed 2 years ago

jpfleischer commented 2 years ago

right now nfs is manual but we need to integrate the new pi nfs

jpfleischer commented 2 years ago

this has been integrated. i want to comment that most of the step 2 within the slurm installation is now out-commented. i am not sure if i should leave that entire section within the script while it is out-commented, or just delete it.

jpfleischer commented 2 years ago

here is the script. https://github.com/cloudmesh/cloudmesh-slurm/blob/main/cloudmesh/slurm/slurm.py

laszewsk commented 2 years ago

please see comments in script. You may like the comments. especially my proposal for the script_executor, making this prg more readable ;-)

laszewsk commented 2 years ago

can you send me url to pi nfs command i reviewed slurm.py and made some suggestions

jpfleischer commented 2 years ago

pi nfs command resides here: https://github.com/cloudmesh/cloudmesh-pi-cluster/blob/main/cloudmesh/pi/command/pi.py