1879657 - Probe expiry alerts fail if other bugs use metric-expiry-alert tag

Status: RESOLVED FIXED

(Data Platform and Tools :: General, defect)

Description

[Ben Wu [:benwu]]

Issue is described in this issue https://github.com/mozilla/probe-scraper/issues/651

We just [now had CI issues](https://app.circleci.com/pipelines/github/mozilla/probe-scraper/1708/workflows/3c1a3b3a-dfeb-4035-af12-20ba7066f1ce/jobs/3874) due to two bugs filed in bugzilla with the `metric-expiry-alert` whiteboard tag:

* https://bugzilla.mozilla.org/show_bug.cgi?id=1862765
* https://bugzilla.mozilla.org/show_bug.cgi?id=1862764

(see also [my comment](https://github.com/mozilla/probe-scraper/pull/649#issuecomment-1792138608))

The test failed with:

        probes_with_bugs = {}
        for bug in found_bugs:
            if (
>               re.search(r"release: \[?version (\d+)", bug["description"]).group(1)
                != version
            ):
E           AttributeError: 'NoneType' object has no attribute 'group'

because the bug description did not have `version 120` in there.
It might be enough to make that check more robust and check for the none case.

Creating a bug here for testing.

Comment 1

[Jan-Erik Rediger [:janerik]]

badboy merged PR [mozilla/probe-scraper]: [Bug 1879657] Skip bugs with invalid descriptions in expiry alerts (#686) in 4ab0e85.