Skip to content

feat: expose proposal false-positive ratio#1139

Open
DarshanCode2005 wants to merge 4 commits into
NixOS:mainfrom
DarshanCode2005:false-positive-query
Open

feat: expose proposal false-positive ratio#1139
DarshanCode2005 wants to merge 4 commits into
NixOS:mainfrom
DarshanCode2005:false-positive-query

Conversation

@DarshanCode2005

@DarshanCode2005 DarshanCode2005 commented Jun 21, 2026

Copy link
Copy Markdown
Collaborator

Towards #1002

Adds a false_positive_ratio SQL exporter metric: triaged proposal rejection rate (rejected / (accepted + rejected), with published counted as accepted). Users can see matching triage quality without running manual SQL.

Comment thread infra/README.md

Moreover, a [Postgres Exporter](https://github.com/prometheus-community/postgres_exporter) and a [SQL exporter](https://github.com/justwatchcom/sql_exporter) run in the host exposing Postgres and application specific metrics. A dashboard for these metrics is [here](https://grafana.nixos.org/d/beo2uotj65lvkb/nix-security-tracker?orgId=1&from=now-6h&to=now&timezone=browser&var-Instance=tracker.security.nixos.org:9237).

### Application metrics

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If you want to point people to the source of just the matching quality metrics, consider extracting just those into a separate file and point to that file. This is of course conditional on that these metrics are somewhat special. If they're not, it's enough to just point people to the general metrics file (and maybe rename it accordingly so it can be found more easily just by browsing the source).

@fricklerhandwerk fricklerhandwerk left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks alright, I need to fix the local Grafana setup to be able to test it though.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants