Remove superfluous code.

twiggy.Logger.trace() returns a clone with adjusted options.
Not using the return value simply does nothing.

Author: felixfontein

src/antsibull_core/logging.py

@@ -271,10 +271,6 @@ def error(self, format_spec: str, *args, **kwargs) -> None:
     def critical(self, format_spec: str, *args, **kwargs) -> None:
         pass
 
-    @abc.abstractmethod
-    def trace(self) -> None:
-        pass
-
     @abc.abstractmethod
     def fields(self, **kwargs) -> Logger:
         pass
@@ -302,9 +298,6 @@ def error(self, format_spec: str, *args, **kwargs) -> None:
     def critical(self, format_spec: str, *args, **kwargs) -> None:
         self.logger.critical(format_spec, *args, **kwargs)
 
-    def trace(self) -> None:
-        self.logger.trace()
-
     def fields(self, **kwargs) -> Logger:
         return TwiggyLogger(self.logger.fields(**kwargs))
 

src/antsibull_core/utils/http.py

@@ -73,11 +73,9 @@ async def __aenter__(self) -> aiohttp.ClientResponse:
                 status = str(status_code)
                 response.close()
             except _AsyncIoTimeoutError:
-                flog.trace()
                 status = "timeout"
                 wait_factor = 0.5
             except Exception as error:  # pylint:disable=broad-except
-                flog.trace()
                 status = str(error)
 
             errors.append(status)