1771132 - Drop the "Has Regression Range" field as no longer necessary

Status: RESOLVED FIXED

(bugzilla.mozilla.org :: Administration, task)

Description

[Marco Castelluccio [:marco]]

Since we introduced the "Regressed By" field, "Has Regression Range" is not as useful anymore.
There are still cases where we have a regression range and no clear regressor, but is there any benefit in knowing that there is a regression range directly from a field?

Comment 1

[David Lawrence [:dkl]]

This is a list of products we currently have the field enabled:

  qr/^cf_has_regression_range$/ => {
    "Core Graveyard"                      => [],
    "Core"                                => [],
    "DevTools Graveyard"                  => [],
    "DevTools"                            => [],
    "External Software Affecting Firefox" => [],
    "Firefox Build System"                => [],
    "Firefox for Android Graveyard"       => [],
    "Firefox for Android"                 => [],
    "Firefox for iOS"                     => [],
    "Firefox Graveyard"                   => [],
    "Firefox"                             => [],
    "GeckoView"                           => [],
    "NSS"                                 => [],
    "Tech Evangelism Graveyard"           => [],
    "Tech Evangelism"                     => [],
    "Toolkit Graveyard"                   => [],
    "Toolkit"                             => [],
    "WebExtensions"                       => [],
  },

We would need to make sure that all products are file with turning it off completely or we can narrow down the list. The current values for the field would still be in the DB but the field would just no longer be visible.

Comment 2

[Marco Castelluccio [:marco]]

There was no disagreement in dropping the field, https://groups.google.com/a/mozilla.org/g/dev-platform/c/BSrur7jYfN8, so I went ahead and updated the autonag bot not to set nor use the field anymore in https://github.com/mozilla/relman-auto-nag/pull/1495.

I think we can now disable the field.

Comment 3

[David Lawrence [:dkl]]

Field has been disabled. I will also remove the code that dictates visibility and create a PR for the change.

Comment 4

[David Lawrence [:dkl]]

Attachment: GitHub Pull Request

Comment 5

[David Lawrence [:dkl]]

Merged.
https://github.com/mozilla-bteam/bmo/commit/e8889a3c006d2026c0339bf142f9f1c1659cab4a