• Home
  • Features
  • Pricing
  • Docs
  • Announcements
  • Sign In

gnosis / dex-services / 5473 / 3
31%
master: 31%

Build:
DEFAULT BRANCH: master
Ran 12 Nov 2020 01:16PM UTC
Files 108
Run time 17s
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

12 Nov 2020 01:05PM UTC coverage: 28.721% (-0.009%) from 28.73%
OPEN_SOLVER_VERSION=v0.1.1 PRIVATE_SOLVER_VERSION=v0.8.6

push

travis-pro

web-flow
Reduce gas estimator error logs (#1525)

The gas now estimator from time to time fails single requests for which we get an alert because this is an error log. Since we use multiple estimators if previous one fails this is not a problem and I don't see value in having this be an alert.

The easiest way to solve this would be to turn the error log into a warning, which I find acceptable too.
This PR instead starts off logging warnings but after an estimator failed N times in a row starts logging an error.

This is has the advantage that we would notice an estimator going down permanently or changing their api in a backwards incompatible way. On the other hand it could still be annoying to get an alert when an estimator goes down for 10 minutes when this hasn't actually impacted our service because of the fallback estimators. I am not sure what others prefer so let's discuss that in this PR.

### Test Plan
Temporarily add a test that with always failing estimator and observe it's errors (also change log to println).

14063 of 56233 branches covered (25.01%)

Branch coverage included in aggregate %.

3782 of 5900 relevant lines covered (64.1%)

40130.82 hits per line

Source Files on job 5473.3 (OPEN_SOLVER_VERSION=v0.1.1 PRIVATE_SOLVER_VERSION=v0.8.6)
  • Tree
  • List 0
  • Changed 28
  • Source Changed 1
  • Coverage Changed 28
Coverage ∆ File Lines Relevant Covered Missed Hits/Line Branch Hits Branch Misses
  • Back to Build 1877
  • Travis Job 5473.3
  • f2311ff4 on github
  • Prev Job for OPEN_SOLVER_VERSION=v0.1.1 PRIVATE_SOLVER_VERSION=v0.8.6 on master (#5471.3)
  • Next Job for OPEN_SOLVER_VERSION=v0.1.1 PRIVATE_SOLVER_VERSION=v0.8.6 on master (#5477.3)
STATUS · Troubleshooting · Open an Issue · Sales · Support · CAREERS · ENTERPRISE · START FREE · SCHEDULE DEMO
ANNOUNCEMENTS · TWITTER · TOS & SLA · Supported CI Services · What's a CI service? · Automated Testing

© 2026 Coveralls, Inc