Merge pull request #753 from mozilla/philimon/beta_split_failsafe

Philimon/add fail safe for beta split reporting

Author: philimon-reset

choose_l10n_beta_split.py

@@ -24,6 +24,10 @@ def distribute_mappings_evenly(mappings):
         min_idx = loads.index(min(loads))
         balanced_splits[min_idx][site] = list(regions)
         loads[min_idx] += len(regions)
+    for i in range(len(balanced_splits)):
+        logging.info(f"Beta split: {i}")
+        current_split = [f"{site} {' '.join(regions)}" for site, regions in balanced_splits[i].items()]
+        logging.info(f"{'\n'.join(current_split)}")
     return balanced_splits
 
 

choose_l10n_ci_set.py

@@ -30,7 +30,15 @@ def valid_l10n_mappings():
                 region_file = json.load(f)
                 if region_file.get("sites"):
                     for site in region_file.get("sites"):
-                        mapping[site].add(region)
+                        report_mapping = True
+                        if site != "demo" and os.path.exists(f"./l10n_CM/constants/{site}/{region}"):
+                            with open(f"./l10n_CM/constants/{site}/{region}/{site}_ad.json", "r+") as ad_file, open(f"./l10n_CM/constants/{site}/{region}/{site}_cc.json", "r+") as cc_file:
+                                ad = json.load(ad_file)
+                                cc = json.load(cc_file)
+                                if ad.get("skip") and cc.get("skip"):
+                                    report_mapping = False
+                        if report_mapping:
+                            mapping[site].add(region)
     return mapping