From: Josh Durgin Date: Wed, 13 Jan 2021 03:33:38 +0000 (-0500) Subject: exceptions: only use one of label or command for fingerprint X-Git-Tag: 1.1.0~21^2 X-Git-Url: http://git.apps.os.sepia.ceph.com/?a=commitdiff_plain;h=336409091d644a0d27de5d870b5c87acb1e4e66d;p=teuthology.git exceptions: only use one of label or command for fingerprint Commands like those running workunits include the ceph sha1 being tested, so they're not useful for grouping. This also lets us group together other tests if we like, for example to map tests with small differences in configuration to the same fingerprint for sentry. Also use the plain command, it's already a string at this point so there's no reason to add spaces between its characters. Signed-off-by: Josh Durgin --- diff --git a/teuthology/exceptions.py b/teuthology/exceptions.py index 71f33e7622..a33cec0415 100644 --- a/teuthology/exceptions.py +++ b/teuthology/exceptions.py @@ -74,9 +74,8 @@ class CommandFailedError(Exception): Used by sentry instead of grouping by backtrace. """ return [ - ' '.join(self.command), + self.label or self.command, 'exit status {}'.format(self.exitstatus), - self.label, '{{ type }}', ]