EOSIO / eos

An open source smart contract platform
https://developers.eos.io/manuals/eos
MIT License
11.29k stars 3.6k forks source link

Report errors only after repeated failures in integration tests - 2.1.x #10991

Closed nickjjzhao closed 2 years ago

nickjjzhao commented 2 years ago

Change Description

This PR is to make sure any exceptions occurred in command get transaction_trace only be reported after repeated failures in integration tests. Suppressing intermediate errors will reduce confusion in the log.

An example of the exceptions in get transaction_trace looks like:

[2021-12-28T09:44:10Z] 175: 2021-12-28T09:44:07.571755            ERROR: Exception during "get transaction_trace". Exception message: Error 3200005: http request fail
--
  | [2021-12-28T09:44:10Z] 175: Error Details:
  | [2021-12-28T09:44:10Z] 175: Error code 404
  | [2021-12-28T09:44:10Z] 175: : {"code":404,"message":"Trace API: transaction id missing in the transaction id log files","error":{"code":0,"name":"","what":"","details":[]}}
  | [2021-12-28T09:44:10Z] 175:
  | [2021-12-28T09:44:10Z] 175: .  cmd Duration=0.011 sec. Context: (transaction id=129dd3500efa582c96d064c216b6dcc20a038a59e5ef98f027da0faad56e1982)

Change Type

Select ONE:

Testing Changes

Select ANY that apply:

Consensus Changes

API Changes

Documentation Additions