Closed Bug 2025587 Opened 2 months ago Closed 2 months ago

Fenix v149 manifest: gecko-nimbus-validation.test-preference has both gecko-pref and default (now mutually exclusive)

Categories

(Firefox :: Nimbus Desktop Client, defect)

defect

Tracking

()

RESOLVED FIXED

People

(Reporter: jlockhart, Unassigned)

References

(Regression)

Details

(Keywords: regression)

Attachments

(1 file)

The Fenix v149.0.0 manifest defines gecko-nimbus-validation.test-preference with both default: '' and gecko-pref:

gecko-nimbus-validation:
  description: Feature for validating Nimbus Gecko functionality
  variables:
    test-preference:
      description: A test gecko preference
      type: String
      default: ''
      gecko-pref:
        pref: gecko.nimbus.test
        branch: default

As of application-services build 150.20260321050323 (March 21, 2026), the FML now enforces that gecko-pref and default are mutually exclusive on feature variables (bug 2020683). When gecko-pref is set, the default comes from the Gecko preference system, so specifying default in the manifest is ambiguous.

This causes FmlClient to raise a ValidationError when parsing the v149.0.0 Fenix manifests, which currently crashes Experimenter with a 500 error when validating experiments targeting Fenix v149.

The fix is to remove default: '' from the test-preference variable definition in the Fenix manifest (mobile/android/fenix/app/nimbus.fml.yaml), since the default is provided by the gecko.nimbus.test preference.

This was found via the failing CI on https://github.com/mozilla/experimenter/pull/15002 (application-services version bump).

This was fixed as an uplift in bug 2012763

Status: NEW → RESOLVED
Closed: 2 months ago
Keywords: regression
Regressed by: 2020683
Resolution: --- → FIXED

Well it turns out we need a temporary fix in A-S to allow us to ingest manifests until the uplift lands.

Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Status: REOPENED → RESOLVED
Closed: 2 months ago2 months ago
Flags: qe-verify+
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: