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

prometheus / client_ruby / 519
84%
master: 100%

Build:
Build:
LAST BUILD BRANCH: fix_with_labels
DEFAULT BRANCH: master
Ran 14 Oct 2019 11:48AM UTC
Jobs 4
Files 96
Run time 4min
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

pending completion
519

Pull #161

travis-ci

web-flow
Fix the "subtle" bug introduced in the previous commit

So, again, I don't propose we actually do this, this is just illustrative
of what we need to do, and didn't want it to get lost.

The problem with the previous commit is that for each metric, we open a
new FileMappedDict. All of them sharing the same file is "fine", actually.
The problem comes from the fact that each instance of FileMappedDict has
their own version of `@used`. When one metric adds a new entry to the file,
it moves its own `@used` pointed, but the other instances don't notice this.

As soon as a second instance that had already been loaded adds a new labelset,
it corrupts the file.

This only shows up when running the benchmarks, with histograms. Counters
(in our benchmark) don't reproduce it, because each metric in our benchmark
only has one labelset. They all get added initially to the file, and an
already loded Dict doesn't add new ones. The reason Histograms reproduce it
is that new observations may report a bucket that wasn't seen before, which
is effectively a new labelset.

The solution is for all MetricStore instances to share one single FileMappedDict.
This is not particularly hard, but the current MetricStore is not written
in a way that allows that without doing this crap.

We should find a better way. I'm just leaving this here as documentation
of this problem, for the next brave soul attempting this
Pull Request #161: One file per process instead of per metric/process for DirectFileStore [DOESNT WORK]

53 of 53 new or added lines in 2 files covered. (100.0%)

3855 of 4587 relevant lines covered (84.04%)

33.18 hits per line

Jobs
ID Job ID Ran Files Coverage
1 519.1 (2.3.8) 14 Oct 2019 11:48AM UTC 0
84.01
Travis Job 519.1
2 519.2 (2.4.5) 14 Oct 2019 11:48AM UTC 0
84.01
Travis Job 519.2
3 519.3 (2.5.3) 14 Oct 2019 11:48AM UTC 0
84.01
Travis Job 519.3
5 519.5 (jruby-9.1.9.0) 14 Oct 2019 11:52AM UTC 0
0.0
Travis Job 519.5
Source Files on build 519
Detailed source file information is not available for this build.
  • Back to Repo
  • Travis Build #519
  • Pull Request #161
  • PR Base - master (#515)
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

© 2025 Coveralls, Inc