]> git.apps.os.sepia.ceph.com Git - teuthology.git/commitdiff
exceptions: only use one of label or command for fingerprint 1597/head
authorJosh Durgin <jdurgin@redhat.com>
Wed, 13 Jan 2021 03:33:38 +0000 (22:33 -0500)
committerJosh Durgin <jdurgin@redhat.com>
Wed, 13 Jan 2021 03:48:01 +0000 (22:48 -0500)
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 <jdurgin@redhat.com>
teuthology/exceptions.py

index 71f33e7622e19b3018af2415f6cbc08151a39547..a33cec041583b55e6a6f4dff815f63966180cc97 100644 (file)
@@ -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 }}',
         ]