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

zalando / skipper / 26047726937
80%

Build:
DEFAULT BRANCH: master
Ran 18 May 2026 05:13PM UTC
Jobs 1
Files 302
Run time 1min
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

18 May 2026 04:54PM UTC coverage: 79.985% (+0.02%) from 79.97%
26047726937

push

github

web-flow
Add `decision_log.dropped` counter for async decision logging (#4007)

## What & Why?

Follow-up to #4001, which introduced a bounded async channel in front of
`logDecision` so that slow log plugins (e.g. `eopa_dl` writing to S3)
don't add latency to HTTP responses. When the channel is full, events
are silently dropped.

Currently, those drops are only visible as warn log lines. This PR
surfaces the same event as a Prometheus counter so we can:

- **Alert** when drops exceed a threshold (should we decide on one based
on operational experience)
- **Tune `buffer_size_limit_events`** based on observed drop rate.

The counter is emitted via skipper's standard `metrics.Default`:

```
skipper_custom_total{key="decision_log.dropped.<bundle>"}
```

to match the existing OPA counter convention (`decision.allow.<bundle>`,
`decision.deny.<bundle>`, `decision.err.<bundle>`).

_Given naming is hard, please feel free to suggest a better name for the
metric._

Signed-off-by: Santhosh Kannan <29794224+x418@users.noreply.github.com>

1 of 1 new or added line in 1 file covered. (100.0%)

9 existing lines in 4 files now uncovered.

27047 of 33815 relevant lines covered (79.99%)

87967.09 hits per line

Coverage Regressions

Lines Coverage ∆ File
3
66.89
-0.25% skipper.go
2
93.04
-1.74% eskipfile/watch.go
2
98.66
-0.67% filters/apiusagemonitoring/spec.go
2
93.02
-0.56% filters/diag/diag.go
Jobs
ID Job ID Ran Files Coverage
1 26047726937.1 18 May 2026 05:13PM UTC 302
79.99
GitHub Action Run
Source Files on build 26047726937
  • Tree
  • List 302
  • Changed 7
  • Source Changed 1
  • Coverage Changed 7
Coverage ∆ File Lines Relevant Covered Missed Hits/Line
  • Back to Repo
  • Github Actions Build #26047726937
  • fe1759c6 on github
  • Prev Build on master (#26025786671)
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