diff options
author | Paul Eggleton <paul.eggleton@linux.intel.com> | 2014-04-30 13:32:04 +0100 |
---|---|---|
committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2014-04-30 21:52:33 +0100 |
commit | 3a4bb1aa604189618cc16c702efb9647fbe86108 (patch) | |
tree | 2f8976d41de48e19ff06d77ac71b4ada6d3bc2cf /meta/lib/oeqa/controllers/masterimage.py | |
parent | 9e7b0ca3832ce7ea9826e5d90ec9f10a7772099e (diff) | |
download | poky-3a4bb1aa604189618cc16c702efb9647fbe86108.tar.gz |
oeqa: add proper handling for command errors where needed
For use outside of tests themselves, we want a better error than
AssertionError, so create one and allow us to request it when calling
runCmd(). This enables us to avoid tracebacks during master image
operations if the power control command fails.
(From OE-Core rev: 89868383685091b0d3723fb8f29590f3f6610078)
Signed-off-by: Paul Eggleton <paul.eggleton@linux.intel.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Diffstat (limited to 'meta/lib/oeqa/controllers/masterimage.py')
-rw-r--r-- | meta/lib/oeqa/controllers/masterimage.py | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/meta/lib/oeqa/controllers/masterimage.py b/meta/lib/oeqa/controllers/masterimage.py index d151e24bd7..f2585d4860 100644 --- a/meta/lib/oeqa/controllers/masterimage.py +++ b/meta/lib/oeqa/controllers/masterimage.py | |||
@@ -20,6 +20,7 @@ import subprocess | |||
20 | import oeqa.targetcontrol | 20 | import oeqa.targetcontrol |
21 | import oeqa.utils.sshcontrol as sshcontrol | 21 | import oeqa.utils.sshcontrol as sshcontrol |
22 | import oeqa.utils.commands as commands | 22 | import oeqa.utils.commands as commands |
23 | from oeqa.utils import CommandError | ||
23 | 24 | ||
24 | from abc import ABCMeta, abstractmethod | 25 | from abc import ABCMeta, abstractmethod |
25 | 26 | ||
@@ -94,7 +95,10 @@ class MasterImageHardwareTarget(oeqa.targetcontrol.BaseTarget): | |||
94 | def power_ctl(self, msg): | 95 | def power_ctl(self, msg): |
95 | if self.powercontrol_cmd: | 96 | if self.powercontrol_cmd: |
96 | cmd = "%s %s" % (self.powercontrol_cmd, msg) | 97 | cmd = "%s %s" % (self.powercontrol_cmd, msg) |
97 | commands.runCmd(cmd, preexec_fn=os.setsid, env=self.origenv) | 98 | try: |
99 | commands.runCmd(cmd, assert_error=False, preexec_fn=os.setsid, env=self.origenv) | ||
100 | except CommandError as e: | ||
101 | bb.fatal(str(e)) | ||
98 | 102 | ||
99 | def power_cycle(self, conn): | 103 | def power_cycle(self, conn): |
100 | if self.powercontrol_cmd: | 104 | if self.powercontrol_cmd: |