Skip to content

insights: remove frozen pings from critical count

Warren Gifford requested to merge insights/critical-frozen-pings into main

Created by: coury-clark

Closes https://github.com/sourcegraph/sourcegraph/issues/32191

Removed frozen insights from the critical telemetry count.

I also noticed that our pings were failing on errors, which probably isn't ideal. I included a small refactor to try every ping emitter at least once, logging errors that may be thrown.

Test plan

Tested locally

"codeInsightsCriticalTelemetry": {
        "TotalInsights": 0
    }

"InsightTotalCounts": {
            "ViewCounts": [
                {
                    "ViewType": "LINE",
                    "TotalCount": 1
                }
            ],

Merge request reports