1067402 - allow alias' for rules

Status: RESOLVED FIXED

(Release Engineering Graveyard :: Applications: Balrog (backend), defect)

Description

[[e-mail] (:bhearsum)]

I was going through all of the Balrog rules today to make sure that we don't have anything overlapping between betas/releases/esrs. In doing so I need to update a bunch of rule ids in release configs. Right now, these are just numbers that correspond to the rule_id column in the balrog database - they really mean nothing. It might be nice if we supported alias' on them so we could do things like:
releaseConfig['testChannelRuleIds'] = ['beta-cdntest', 'beta-localtest']

...which have much more meaning, and would hopefully prevent typo errors (eg, typing 62 instead of 63).

Comment 1

[[e-mail] (:bhearsum)]

Got to thinking about this a bit today...here's a couple of random thoughts:
* We need to be careful not to allow collisions between rule ids and alias. Eg: if we have a rule id of 100 and someone sets an alias of 100 to another rule, there's no way for the server to resolve it.
** One way of enforcing this is to require at least one non-numeric character in the alias.
* We should think about whether we may want multiple alias' for a single rule. Multiple vs. single alias is likely to have an effect on the database schema. My gut says that we're opening the door to a lot of confusion if we have multiple alias' for a single rule, but we should think about it a bit more.

Comment 2

[[e-mail] (:bhearsum)]

Another thing to consider is how alias might work with the history API. Should you be able to query by alias there? Eg: /rules/<rule_id_or_alias>/revisions. What would you return for that? The revisions of the rule as far back as that alias was on it? Or return the full history of the rule that currently matches that alias?

Comment 3

[[e-mail] (:bhearsum)]

Attachment: add alias support to backend

Comment 4

[[e-mail] (:bhearsum)]

Attachment: add alias support to UI

Comment 5

[[github robot]]

Commit pushed to master at https://github.com/mozilla/balrog

https://github.com/mozilla/balrog/commit/b808ecd839182e48dfb250f7243a51639341ce92
Merge pull request #40 from bhearsum/rule-alias

bug 1067402: Allow rules to have alias'. r=catlee

Comment 6

[[e-mail] (:bhearsum)]

Comment on attachment 8703155 [details] [review]
add alias support to backend

Got r+ in PR. Will do final verification on aus4-dev before pushing to production.

Comment 7

[[e-mail] (:bhearsum)]

Verified in dev this morning, and pushed to prod.