Skip to content

Add `counts.saml_group_links` metric

Bogdan Denkovych requested to merge add-counts-saml_group_links-metric into master

What does this MR do and why?

Closes Add Instrumentation for Self-Managed Group SAML (#364137 - closed)

In SAML Group Sync - Self-Managed SAML Implementation (#285150 - closed) we released group SAML sync for self-managed. We would like to understand its adoption. We want to create a chart on https://app.periscopedata.com/ that would show us the amount of unique self-managed instances with at least one SAML group link configured over time. Since this feature is added to %15.1 it is important for us to add this metric to see traction on this feature from the start.

DB Query Plans

explain SELECT COUNT("saml_group_links"."id") FROM "saml_group_links"

width=8) (actual time=94.760..94.762 rows=1 loops=1)
   Buffers: shared hit=189 read=52 dirtied=10
   I/O Timings: read=62.896 write=0.000
   ->  Index Only Scan using saml_group_links_pkey on public.saml_group_links  (cost=0.29..255.96 rows=11251 width=8) (actual time=10.018..90.458 rows=11252 loops=1)
         Heap Fetches: 649
         Buffers: shared hit=189 read=52 dirtied=10
         I/O Timings: read=62.896 write=0.000

Screenshots or screen recordings

These are strongly recommended to assist reviewers and reduce the time to merge your change.

How to set up and validate locally

Numbered steps to set up and validate the change are strongly suggested.

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Edited by Bogdan Denkovych

Merge request reports