Glean: Add a way to set branch/active experiment from ExperimentsDebugActivity
Categories
(Data Platform and Tools :: Glean: SDK, task, P1)
Tracking
(Not tracked)
People
(Reporter: travis_, Assigned: travis_)
References
Details
(Whiteboard: [telemetry:mobilesdk:m8])
Attachments
(1 file)
|
(deleted),
text/x-github-pull-request
|
Details |
This is follow up work for 1543969.
Need to add a way to override the branch and active experiment via the ExperimentsDebugActivity in service-experiments.
|
Assignee | |
Updated•6 years ago
|
|
|
Assignee | |
Updated•6 years ago
|
|
|
Assignee | |
Comment 1•6 years ago
|
||
I've hit a bit of a snag on this, so I'll push my Draft PR so you can look if needed. My first question is, what sort of validation do we need to do (if any) on what is supplied to the ExperimentsDebugActivity for the experiment id and branch name? My next question is, do we need to expose the functionality to be able to clear overrides through the ExperimentsDebugActivity?
|
|
Assignee | |
Comment 2•6 years ago
|
||
|
||
Comment 3•5 years ago
|
||
(In reply to Travis Long from comment #1)
My first question is, what sort of validation do we need to do (if any) on what is supplied to the ExperimentsDebugActivity for the experiment id and branch name?
The id and branch name are just used locally for matching, so i don't think we need to apply any validation to the string.
My next question is, do we need to expose the functionality to be able to clear overrides through the ExperimentsDebugActivity?
Yes, that is a good point. If we don't provide it we can't turn the overrides off.
How about we just add one command that clears all possible overrides?
|
||
Updated•5 years ago
|
|
|
Assignee | |
Updated•5 years ago
|
Description
•