Skip to content

chore(cirrus): use server-side glean#13882

Merged
relud merged 1 commit into
mainfrom
relud-cirrus-server-side-glean
Nov 17, 2025
Merged

chore(cirrus): use server-side glean#13882
relud merged 1 commit into
mainfrom
relud-cirrus-server-side-glean

Conversation

@relud

@relud relud commented Nov 4, 2025

Copy link
Copy Markdown
Contributor

Because

  • We want to limit our glean pings to what comes from our gcpv2 environments

This commit

  • Swaps out python glean sdk for glean-parser generated server_events.py

Fixes #13857

@relud relud force-pushed the relud-cirrus-server-side-glean branch from cc123e0 to 70812c2 Compare November 4, 2025 04:39

@jaredlockhart jaredlockhart 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.

God much nicer, I much prefer this, thank you @relud 🙏

After this deploys can we just do a sanity check to make sure the telemetry's landing where we expect 🙏

@relud

relud commented Nov 4, 2025

Copy link
Copy Markdown
Contributor Author

holding off on merging this while I debug an issue in experimenter's server-side glean that I think will also impact cirrus

@relud

relud commented Nov 10, 2025

Copy link
Copy Markdown
Contributor Author

This is blocked on the release of mozilla/glean_parser#820, no other issues found in experimenter are relevant here

@relud relud force-pushed the relud-cirrus-server-side-glean branch from 70812c2 to 51998f3 Compare November 17, 2025 17:14
@relud relud enabled auto-merge November 17, 2025 17:14
@relud

relud commented Nov 17, 2025

Copy link
Copy Markdown
Contributor Author

updated to fixed glean version 18.1.1, merging

@relud relud added this pull request to the merge queue Nov 17, 2025
Merged via the queue into main with commit b3cafd5 Nov 17, 2025
16 checks passed
@relud relud deleted the relud-cirrus-server-side-glean branch November 17, 2025 18:20
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.

Use server-side glean in cirrus

2 participants