-
Notifications
You must be signed in to change notification settings - Fork 25
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
Move test resources to jfrog #243
Conversation
@@ -9,12 +9,21 @@ | |||
import reframe.utility.sanity as sn | |||
|
|||
|
|||
class fetch_lampps_resources(rfm.RunOnlyRegressionTest): | |||
executable = 'wget https://jfrog.svc.cscs.ch/artifactory/cscs-reframe-tests/LAMMPS.tar.gz && tar -xvf LAMMPS.tar.gz' |
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.
executable = 'wget https://jfrog.svc.cscs.ch/artifactory/cscs-reframe-tests/LAMMPS.tar.gz && tar -xvf LAMMPS.tar.gz' | |
jfrog = 'https://jfrog.svc.cscs.ch/artifactory/cscs-reframe-tests' | |
executable = f'wget {jfrog}/LAMMPS.tar.gz && tar -xvf LAMMPS.tar.gz' |
- maybe jfrog should be set in the config file(s) ?
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.
Not sure it's that useful because we have many tests pulling from jfrog in different directories. Do you think we will need to move them to a different directory eventually?
@@ -9,6 +9,14 @@ | |||
import reframe.utility.sanity as sn | |||
|
|||
|
|||
class fetch_namd_resources(rfm.RunOnlyRegressionTest): | |||
executable = 'wget https://jfrog.svc.cscs.ch/artifactory/cscs-reframe-tests/NAMD.tar.gz && tar -xvf LAMMPS.tar.gz' |
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.
same
cscs-ci run alps-todi-cpe |
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.
@ekouts do you think we should make the jfrog repo more general in the configuration or as a utility?
cscs-ci run alps-todi-cpe |
I would leave it as it is, but if you both agree, then I can change it. |
cscs-ci run alps-todi-cpe |
cscs-ci run alps-eiger-cpe |
2 similar comments
cscs-ci run alps-eiger-cpe |
cscs-ci run alps-eiger-cpe |
No description provided.