Skip to content

chore(experimenter): fix server-side glean#13914

Merged
relud merged 1 commit intomainfrom
relud-experimenter-fix-glean
Nov 6, 2025
Merged

chore(experimenter): fix server-side glean#13914
relud merged 1 commit intomainfrom
relud-experimenter-fix-glean

Conversation

@relud
Copy link
Contributor

@relud relud commented Nov 6, 2025

Because

  • settings.APP_VERSION is None, and glean app_display_version must not be None
  • glean application_id must use a - separator not .
  • nimbus enrollments must be a list

This commit

  • Changes from settings.APP_VERSION to experimenter.base.app_version()
  • Changes glean app id from experimenter.backend to experimenter-backend
  • Changes nimbus enrollments fallback value from None to []

Fixes #13858

@relud
Copy link
Contributor Author

relud commented Nov 6, 2025

these changes have been verified in stage with the help of @akkomar to resolve the issue of glean server-side pings being rejected at schema validation

@relud relud force-pushed the relud-experimenter-fix-glean branch from c596b11 to 6f75a36 Compare November 6, 2025 16:47
@relud relud enabled auto-merge November 6, 2025 16:57
@relud relud added this pull request to the merge queue Nov 6, 2025
Merged via the queue into main with commit 45952f2 Nov 6, 2025
17 checks passed
@relud relud deleted the relud-experimenter-fix-glean branch November 6, 2025 17:35
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 experimenter

2 participants