From 336409091d644a0d27de5d870b5c87acb1e4e66d Mon Sep 17 00:00:00 2001 From: Josh Durgin Date: Tue, 12 Jan 2021 22:33:38 -0500 Subject: [PATCH] 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 --- teuthology/exceptions.py | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) 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 }}', ] -- 2.39.5