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.

Sign up for GitHub

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

Add a general ATS machine called docker #162

Merged
merged 2 commits into from
Nov 7, 2024
Merged

Add a general ATS machine called docker #162

Changes from 1 commit
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
00c59ac
Added a docker ats machine for launching tests in a more general manner
ldowen Oct 31, 2024
f6049e9
Cosmetic changes to docker.py
ldowen Nov 6, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Cosmetic changes to docker.py
  • Loading branch information
@ldowen
ldowen committed Nov 6, 2024
commit f6049e9d7f63d9b67a0e30379fb80e509ab924b2
5 changes: 2 additions & 3 deletions ats/atsMachines/docker.py
View file
Original file line number Diff line number Diff line change
@@ -5,7 +5,7 @@
from ats.atsut import RUNNING, TIMEDOUT

class Docker (machines.Machine):
"Typical Linux machine."
"Typical Linux machine. To use, set MACHINE_TYPE environment variable to docker"
def init (self) :
self.npMax = 1

@@ -48,12 +48,11 @@ def examineOptions(self, options):
# machines.py sets this
self.npMax = self.numberTestsRunningMax
self.mpiexe = options.mpiexe
self.mpiexe = options.mpiexe
self.nompi = options.nompi
if (self.npMax < 2 and self.mpiexe):
terminal("WARNING: npMax should be > 1 to use mpiexe")
if (self.mpiexe and self.nompi):
terminal("WARNING: Setting --nompi and --mpiexe contradicts")
terminal("WARNING: --mpiexe will be ignored when --nompi")

def calculateCommandList(self, test):
"""Prepare for run of executable using a suitable command. First we get the plain command
Loading
Oops, something went wrong.