Merge ~adam-collard/maas/+git/maas-release-tools:wsjfify-fix into ~maas-committers/maas/+git/maas-release-tools:main

[Adam Collard]

Blacklist customfield_10348 for story points name collision

[MAAS Lander (maas-lander)]

UNIT TESTS
-b wsjfify-fix lp:~adam-collard/maas/+git/maas-release-tools into -b main lp:~maas-committers/maas/+git/maas-release-tools

STATUS: SUCCESS
COMMIT: a09849f478ce689a6a8c93de9e612fcdf7f6db93

review: Approve

diff --git a/maas_release_tools/scripts/wsjfify.py b/maas_release_tools/scripts/wsjfify.py
index d9bcbdd..267c2b1 100644
--- a/maas_release_tools/scripts/wsjfify.py
+++ b/maas_release_tools/scripts/wsjfify.py
@@ -1,6 +1,5 @@
 """Recalculate WSJF for tickets with Cost of Delay and Story Points."""
 
-
 import argparse
 import json
 
@@ -48,9 +47,9 @@ def find_estimated_issues(jira, project, component):
 
 
 def calculate_wsjf(issue):
-    return round(
-        issue.fields["cost_of_delay"] / issue.fields["story_points"], 3
-    )
+    cod = issue.fields["cost_of_delay"]
+    story_points = issue.fields["story_points"]
+    return round(cod / story_points, 3)
 
 
 def pythonise_propertyholder(fields_by_name):
@@ -79,16 +78,24 @@ def main():
     def slugify(name):
         return name.lower().replace(" ", "_").replace("-", "_")
 
+    # slugify can cause fields to be ambiguous, and can end up using
+    # the wrong one. Blacklist some known failures.
+    blacklist_ids = {"customfield_10348"}  # story points
     fields_by_name = {
-        slugify(field["name"]): field["id"] for field in all_fields
+        slugify(field["name"]): field["id"]
+        for field in all_fields
+        if field["id"] not in blacklist_ids
     }
-
     # Make the fields dict-like and friendly
     pythonise_propertyholder(fields_by_name)
 
     for issue in find_estimated_issues(jira, project="PF", component="MAAS"):
         existing_wsjf = issue.fields["wsjf"]
-        wsjf = calculate_wsjf(issue)
+        try:
+            wsjf = calculate_wsjf(issue)
+        except KeyError as ke:
+            print(f"Failed calculating WSJF for {issue.key} {ke}")
+            raise
         if wsjf != existing_wsjf:
             print(
                 f"[{issue.key}] setting WSJF to {wsjf} (was {existing_wsjf})"