1816463 - The CBH Nimbus feature configuration does not accept integers from the implemented range

Status: VERIFIED FIXED

(Core :: Privacy: Anti-Tracking, defect, P2)

Description

[Patriciu Pop [:ppop], Ecosystem QA]

Attachment: CBH_experimenter_error.png

[Prerequisites]:

• Have access to the Nimbus Experimenter page(LDAP).

[Steps to reproduce]:

1. Open the browser with the profile from prerequisites.
2. Navigate to the Branches page of a Nimbus Experiment using the CBH feature. (https://stage.experimenter.nonprod.dataops.mozgcp.net/nimbus/ppop-cbh-handling-test/edit/branches)
3. Observe the "Value" field of the branches.

[Expected result]:

• The JSON is validated and no errors are displayed.

[Actual result]:

• The "1 is not one of ['0', '1', '2']" error is displayed and the JSON is invalid.

[Notes]:

• Attached a screenshot of the issue on the Experimenter page, as well as screenshots of the feature configs of both Searchfox and the Nimbus Admin:

Comment 1

[Patriciu Pop [:ppop], Ecosystem QA]

Attachment: CBH_nimbus_admin_featureconfig.png

Comment 2

[Patriciu Pop [:ppop], Ecosystem QA]

Attachment: CBH_searchfox_featureconfig.png

Comment 3

[Jared Hirsch [:jhirsch] (he/him) (Needinfo please)]

it looks like enums don't support integers, so the right fix here is probably to have string names for the variants. planning to fix this along with the fix for bug 1816464.

Comment 4

[Jared Lockhart [:jlockhart]]

This is a bug in Experimenter, filed here:

https://github.com/mozilla/experimenter/issues/8308

Comment 5

[Jared Hirsch [:jhirsch] (he/him) (Needinfo please)]

OK, and looks like this was already fixed. Thanks :jlockhart! I can't dupe this to a github bug, so I'll assign it to you and mark it as fixed.

Comment 6

[Ciprian Muresan [:cmuresan], Ecosystem QA]

Marking this as verified as the feature configuration is no longer available and the issue is no longer reproducible due to this.