Added platform check #783

Kwaizer · 2024-04-09T09:40:45Z

github-actions · 2024-04-09T09:41:59Z

pylint output

************* Module alws.crud.errata
alws/crud/errata.py:1:0: C0302: Too many lines in module (1668/1000) (too-many-lines)
alws/crud/errata.py:69:5: W0511: FIXME: ovallib dependency should stay optional (fixme)
alws/crud/errata.py:167:5: W0511: TODO: (fixme)
alws/crud/errata.py:226:29: W0511: TODO: Add test mapping here (fixme)
alws/crud/errata.py:1317:1: W0511: TODO: Check db_record (fixme)
alws/crud/errata.py:635:29: W1401: Anomalous backslash in string: '\d'. String constant might be missing an r prefix. (anomalous-backslash-in-string)
alws/crud/errata.py:635:31: W1401: Anomalous backslash in string: '\w'. String constant might be missing an r prefix. (anomalous-backslash-in-string)
alws/crud/errata.py:635:33: W1401: Anomalous backslash in string: '\-'. String constant might be missing an r prefix. (anomalous-backslash-in-string)
alws/crud/errata.py:635:35: W1401: Anomalous backslash in string: '\_'. String constant might be missing an r prefix. (anomalous-backslash-in-string)
alws/crud/errata.py:635:41: W1401: Anomalous backslash in string: '\d'. String constant might be missing an r prefix. (anomalous-backslash-in-string)
alws/crud/errata.py:635:43: W1401: Anomalous backslash in string: '\.'. String constant might be missing an r prefix. (anomalous-backslash-in-string)
alws/crud/errata.py:635:45: W1401: Anomalous backslash in string: '\w'. String constant might be missing an r prefix. (anomalous-backslash-in-string)
alws/crud/errata.py:1:0: C0114: Missing module docstring (missing-module-docstring)
alws/crud/errata.py:19:0: E0401: Unable to import 'createrepo_c' (import-error)
alws/crud/errata.py:95:0: C0115: Missing class docstring (missing-class-docstring)
alws/crud/errata.py:100:4: C0116: Missing function or method docstring (missing-function-docstring)
alws/crud/errata.py:95:0: R0903: Too few public methods (1/2) (too-few-public-methods)
alws/crud/errata.py:125:0: C0116: Missing function or method docstring (missing-function-docstring)
alws/crud/errata.py:158:0: C0116: Missing function or method docstring (missing-function-docstring)
alws/crud/errata.py:158:0: R0914: Too many local variables (33/15) (too-many-locals)
alws/crud/errata.py:178:4: R1702: Too many nested blocks (7/5) (too-many-nested-blocks)
alws/crud/errata.py:158:0: R0912: Too many branches (47/12) (too-many-branches)
alws/crud/errata.py:158:0: R0915: Too many statements (118/50) (too-many-statements)
alws/crud/errata.py:410:0: C0116: Missing function or method docstring (missing-function-docstring)
alws/crud/errata.py:423:0: C0116: Missing function or method docstring (missing-function-docstring)
alws/crud/errata.py:423:0: R0912: Too many branches (13/12) (too-many-branches)
alws/crud/errata.py:480:0: C0116: Missing function or method docstring (missing-function-docstring)
alws/crud/errata.py:508:0: C0116: Missing function or method docstring (missing-function-docstring)
alws/crud/errata.py:508:0: R0914: Too many local variables (22/15) (too-many-locals)
alws/crud/errata.py:615:0: C0116: Missing function or method docstring (missing-function-docstring)
alws/crud/errata.py:615:0: R0914: Too many local variables (24/15) (too-many-locals)
alws/crud/errata.py:778:11: W0718: Catching too general exception Exception (broad-exception-caught)
alws/crud/errata.py:786:0: C0116: Missing function or method docstring (missing-function-docstring)
alws/crud/errata.py:812:0: C0116: Missing function or method docstring (missing-function-docstring)
alws/crud/errata.py:812:0: R0913: Too many arguments (9/5) (too-many-arguments)
alws/crud/errata.py:842:23: E1102: func.count is not callable (not-callable)
alws/crud/errata.py:884:0: C0116: Missing function or method docstring (missing-function-docstring)
alws/crud/errata.py:925:0: C0116: Missing function or method docstring (missing-function-docstring)
alws/crud/errata.py:925:0: R0913: Too many arguments (7/5) (too-many-arguments)
alws/crud/errata.py:925:0: R0914: Too many local variables (25/15) (too-many-locals)
alws/crud/errata.py:1053:0: C0116: Missing function or method docstring (missing-function-docstring)
alws/crud/errata.py:1119:0: C0116: Missing function or method docstring (missing-function-docstring)
alws/crud/errata.py:1185:0: C0116: Missing function or method docstring (missing-function-docstring)
alws/crud/errata.py:1185:0: R0914: Too many local variables (19/15) (too-many-locals)
alws/crud/errata.py:1238:0: C0116: Missing function or method docstring (missing-function-docstring)
alws/crud/errata.py:1299:0: C0116: Missing function or method docstring (missing-function-docstring)
alws/crud/errata.py:1318:0: C0116: Missing function or method docstring (missing-function-docstring)
alws/crud/errata.py:1318:0: R0913: Too many arguments (7/5) (too-many-arguments)
alws/crud/errata.py:1368:0: C0116: Missing function or method docstring (missing-function-docstring)
alws/crud/errata.py:1400:15: W0718: Catching too general exception Exception (broad-exception-caught)
alws/crud/errata.py:1428:19: W0718: Catching too general exception Exception (broad-exception-caught)
alws/crud/errata.py:1436:0: C0116: Missing function or method docstring (missing-function-docstring)
alws/crud/errata.py:1486:19: W0718: Catching too general exception Exception (broad-exception-caught)
alws/crud/errata.py:1512:23: W0718: Catching too general exception Exception (broad-exception-caught)
alws/crud/errata.py:1531:0: C0116: Missing function or method docstring (missing-function-docstring)
alws/crud/errata.py:1531:0: R0912: Too many branches (13/12) (too-many-branches)
alws/crud/errata.py:1621:0: C0116: Missing function or method docstring (missing-function-docstring)
alws/crud/errata.py:28:0: W0611: Unused GitHubIssueStatus imported from alws.constants (unused-import)
************* Module alws.routers.errata
alws/routers/errata.py:103:5: W0511: TODO: Retrieve updateinfo based on record_id AND platform_id (fixme)
alws/routers/errata.py:127:1: W0511: TODO: Update this endpoint to include platform_id. (fixme)
alws/routers/errata.py:188:1: W0511: TODO: Update endpoint to take into account platform_id, see (fixme)
alws/routers/errata.py:1:0: C0114: Missing module docstring (missing-module-docstring)
alws/routers/errata.py:28:0: C0116: Missing function or method docstring (missing-function-docstring)
alws/routers/errata.py:40:0: C0116: Missing function or method docstring (missing-function-docstring)
alws/routers/errata.py:59:0: C0116: Missing function or method docstring (missing-function-docstring)
alws/routers/errata.py:74:0: C0116: Missing function or method docstring (missing-function-docstring)
alws/routers/errata.py:75:4: C0103: Argument name "pageNumber" doesn't conform to snake_case naming style (invalid-name)
alws/routers/errata.py:79:4: C0103: Argument name "platformId" doesn't conform to snake_case naming style (invalid-name)
alws/routers/errata.py:80:4: C0103: Argument name "cveId" doesn't conform to snake_case naming style (invalid-name)
alws/routers/errata.py:74:0: R0913: Too many arguments (8/5) (too-many-arguments)
alws/routers/errata.py:76:4: W0622: Redefining built-in 'id' (redefined-builtin)
alws/routers/errata.py:81:4: W0621: Redefining name 'status' from outer scope (line 3) (redefined-outer-name)
alws/routers/errata.py:100:0: C0116: Missing function or method docstring (missing-function-docstring)
alws/routers/errata.py:120:0: C0116: Missing function or method docstring (missing-function-docstring)
alws/routers/errata.py:131:0: C0116: Missing function or method docstring (missing-function-docstring)
alws/routers/errata.py:148:0: C0116: Missing function or method docstring (missing-function-docstring)
alws/routers/errata.py:157:38: E1101: Instance of 'ValueError' has no 'message' member (no-member)
alws/routers/errata.py:164:0: C0116: Missing function or method docstring (missing-function-docstring)
alws/routers/errata.py:191:0: C0116: Missing function or method docstring (missing-function-docstring)
alws/routers/errata.py:202:0: C0116: Missing function or method docstring (missing-function-docstring)
alws/routers/errata.py:8:0: W0611: Unused settings imported from alws.config (unused-import)

-----------------------------------
Your code has been rated at 8.73/10

github-actions · 2024-04-09T09:42:01Z

Coverage report for changed files •

File	Stmts	Miss	Cover	Missing
crud
errata.py	682	511	25%	79–85, 98–99, 102–110, 115, 118, 122–123, 129, 139, 142, 144–145, 147, 149–150, 155–156, 160–161, 167, 172, 176–184, 189–191, 194–215, 218–226, 229, 237–238, 242–244, 247–254, 256–257, 324–332, 336, 339–343, 346–359, 362–371, 374, 377–382, 385–396, 399–400, 404–408, 411, 436, 440, 458–465, 472–482, 490, 495–497, 499–501, 504–506, 508–511, 538, 547–551, 567–568, 592–595, 602, 606–607, 616–621, 623–625, 629–630, 635–636, 710–712, 722–723, 762, 818, 827, 836, 850–852, 858, 870–876, 879–882, 888–889, 892–893, 896–897, 900–902, 904, 917–919, 933–940, 944, 948–951, 963–964, 968, 970–987, 1001–1006, 1015–1018, 1025–1026, 1036, 1040, 1043, 1080, 1085–1087, 1099–1101, 1118–1119, 1133, 1146–1150, 1153, 1168–1174, 1183–1187, 1191–1193, 1201–1203, 1217, 1227–1231, 1235–1241, 1245–1250, 1254–1256, 1258–1265, 1270–1272, 1282–1290, 1296–1298, 1306–1308, 1313, 1324–1325, 1328–1333, 1337, 1351, 1364, 1368, 1375, 1379–1383, 1385, 1390, 1392–1393, 1395, 1405, 1409, 1414, 1416–1419, 1422–1424, 1426–1429, 1434–1435, 1440–1445, 1447, 1453, 1455, 1464–1468, 1472–1473, 1478–1479, 1483, 1487, 1492–1495, 1503, 1505, 1507, 1510–1512, 1516, 1518, 1523–1526, 1531–1532, 1536–1540, 1542, 1549–1550, 1559–1562, 1566–1567, 1572–1573, 1577–1585, 1588, 1603, 1628, 1669, 1674, 1682, 1701–1705, 1711, 1718–1719, 1727–1728
routers
errata.py	71	32	54%	36, 45, 50–51, 55, 64–66, 70, 84, 112, 124, 134–135, 152–153, 156–157, 170, 175–183, 192–193, 206–207
TOTAL	9998	4455	55%

Tests	Skipped	Failures	Errors	Time
82	13 💤	0 ❌	0 🔥	35.813s ⏱️

Kwaizer · 2024-04-09T10:05:36Z

Do I need to fix syntax in the same pull request?

javihernandez · 2024-04-09T10:44:20Z

alws/crud/errata.py

@@ -140,6 +140,10 @@ async def get_oval_xml(
        select(models.Platform).where(models.Platform.name == platform_name)
    )
    platform: models.Platform = platform.scalars().first()
+
+    if platform is None:
+        return None


if not platform: return

javihernandez · 2024-04-09T10:44:26Z

alws/routers/errata.py

@@ -61,7 +61,13 @@ async def get_oval_xml(
    only_released: bool = False,
    db: AsyncSession = Depends(get_db),
 ):
-    return await errata_crud.get_oval_xml(db, platform_name, only_released)
+    records = await errata_crud.get_oval_xml(db, platform_name, only_released)
+    if records is None:


here, we can also have if not records

maccelf · 2024-04-09T12:43:27Z

Do I need to fix syntax in the same pull request?

Yes

github-actions · 2024-04-17T09:55:32Z

Code Coverage Summary

Package	Line Rate
alws	76%
alws.auth	77%
alws.auth.oauth	100%
alws.crud	39%
alws.dramatiq	60%
alws.middlewares	93%
alws.perms	86%
alws.routers	56%
alws.schemas	80%
alws.utils	42%
Summary	56% (5543 / 9820)

Linter reports

Pylint report

************* Module alws.crud.errata
alws/crud/errata.py:1:0: C0302: Too many lines in module (1702/1000) (too-many-lines)
alws/crud/errata.py:69:5: W0511: FIXME: ovallib dependency should stay optional (fixme)
alws/crud/errata.py:167:5: W0511: TODO: (fixme)
alws/crud/errata.py:226:29: W0511: TODO: Add test mapping here (fixme)
alws/crud/errata.py:1317:1: W0511: TODO: Check db_record (fixme)
alws/crud/errata.py:635:29: W1401: Anomalous backslash in string: '\d'. String constant might be missing an r prefix. (anomalous-backslash-in-string)
alws/crud/errata.py:635:31: W1401: Anomalous backslash in string: '\w'. String constant might be missing an r prefix. (anomalous-backslash-in-string)
alws/crud/errata.py:635:33: W1401: Anomalous backslash in string: '\-'. String constant might be missing an r prefix. (anomalous-backslash-in-string)
alws/crud/errata.py:635:35: W1401: Anomalous backslash in string: '\_'. String constant might be missing an r prefix. (anomalous-backslash-in-string)
alws/crud/errata.py:635:41: W1401: Anomalous backslash in string: '\d'. String constant might be missing an r prefix. (anomalous-backslash-in-string)
alws/crud/errata.py:635:43: W1401: Anomalous backslash in string: '\.'. String constant might be missing an r prefix. (anomalous-backslash-in-string)
alws/crud/errata.py:635:45: W1401: Anomalous backslash in string: '\w'. String constant might be missing an r prefix. (anomalous-backslash-in-string)
alws/crud/errata.py:95:0: R0903: Too few public methods (1/2) (too-few-public-methods)
alws/crud/errata.py:158:0: C0116: Missing function or method docstring (missing-function-docstring)
alws/crud/errata.py:158:0: R0914: Too many local variables (33/15) (too-many-locals)
alws/crud/errata.py:178:4: R1702: Too many nested blocks (7/5) (too-many-nested-blocks)
alws/crud/errata.py:158:0: R0912: Too many branches (47/12) (too-many-branches)
alws/crud/errata.py:158:0: R0915: Too many statements (118/50) (too-many-statements)
alws/crud/errata.py:423:0: C0116: Missing function or method docstring (missing-function-docstring)
alws/crud/errata.py:423:0: R0912: Too many branches (13/12) (too-many-branches)
alws/crud/errata.py:508:0: C0116: Missing function or method docstring (missing-function-docstring)
alws/crud/errata.py:508:0: R0914: Too many local variables (22/15) (too-many-locals)
alws/crud/errata.py:615:0: C0116: Missing function or method docstring (missing-function-docstring)
alws/crud/errata.py:615:0: R0914: Too many local variables (24/15) (too-many-locals)
alws/crud/errata.py:778:11: W0718: Catching too general exception Exception (broad-exception-caught)
alws/crud/errata.py:812:0: C0116: Missing function or method docstring (missing-function-docstring)
alws/crud/errata.py:842:23: E1102: func.count is not callable (not-callable)
alws/crud/errata.py:925:0: C0116: Missing function or method docstring (missing-function-docstring)
alws/crud/errata.py:925:0: R0914: Too many local variables (25/15) (too-many-locals)
alws/crud/errata.py:1053:0: C0116: Missing function or method docstring (missing-function-docstring)
alws/crud/errata.py:1119:0: C0116: Missing function or method docstring (missing-function-docstring)
alws/crud/errata.py:1185:0: C0116: Missing function or method docstring (missing-function-docstring)
alws/crud/errata.py:1185:0: R0914: Too many local variables (19/15) (too-many-locals)
alws/crud/errata.py:1238:0: C0116: Missing function or method docstring (missing-function-docstring)
alws/crud/errata.py:1368:0: C0116: Missing function or method docstring (missing-function-docstring)
alws/crud/errata.py:1400:15: W0718: Catching too general exception Exception (broad-exception-caught)
alws/crud/errata.py:1428:19: W0718: Catching too general exception Exception (broad-exception-caught)
alws/crud/errata.py:1436:0: C0116: Missing function or method docstring (missing-function-docstring)
alws/crud/errata.py:1486:19: W0718: Catching too general exception Exception (broad-exception-caught)
alws/crud/errata.py:1512:23: W0718: Catching too general exception Exception (broad-exception-caught)
alws/crud/errata.py:1531:0: C0116: Missing function or method docstring (missing-function-docstring)
alws/crud/errata.py:1531:0: R0914: Too many local variables (19/15) (too-many-locals)
alws/crud/errata.py:1571:16: E1205: Too many arguments for logging format string (logging-too-many-args)
alws/crud/errata.py:1574:24: E0602: Undefined variable 'exc' (undefined-variable)
alws/crud/errata.py:1646:34: E1101: Module 'createrepo_c' has no 'SHA256' member (no-member)
alws/crud/errata.py:1531:0: R0912: Too many branches (18/12) (too-many-branches)
alws/crud/errata.py:28:0: W0611: Unused GitHubIssueStatus imported from alws.constants (unused-import)
************* Module alws.routers.errata
alws/routers/errata.py:127:1: W0511: TODO: Update this endpoint to include platform_id. (fixme)
alws/routers/errata.py:75:4: C0103: Argument name "pageNumber" doesn't conform to snake_case naming style (invalid-name)
alws/routers/errata.py:79:4: C0103: Argument name "platformId" doesn't conform to snake_case naming style (invalid-name)
alws/routers/errata.py:80:4: C0103: Argument name "cveId" doesn't conform to snake_case naming style (invalid-name)
alws/routers/errata.py:76:4: W0622: Redefining built-in 'id' (redefined-builtin)
alws/routers/errata.py:81:4: W0621: Redefining name 'status' from outer scope (line 3) (redefined-outer-name)
alws/routers/errata.py:157:38: E1101: Instance of 'ValueError' has no 'message' member (no-member)
alws/routers/errata.py:8:0: W0611: Unused settings imported from alws.config (unused-import)

-----------------------------------
Your code has been rated at 8.99/10

Black report

--- alws/routers/errata.py	2024-04-17 09:53:49.711843+00:00
+++ alws/routers/errata.py	2024-04-17 09:55:04.443300+00:00
@@ -100,20 +100,15 @@
 async def get_updateinfo_xml(
     record_id: str,
     platform_id: Optional[int] = None,
     db: AsyncSession = Depends(get_db),
 ):
-    updateinfo_xml = await errata_crud.get_updateinfo_xml_from_pulp(
-        db, record_id, platform_id
-    )
+    updateinfo_xml = await errata_crud.get_updateinfo_xml_from_pulp(db, record_id, platform_id)
     if updateinfo_xml is None:
         raise HTTPException(
             status_code=status.HTTP_404_NOT_FOUND,
-            detail=(
-                f"Unable to find errata records with {record_id=} and "
-                f"{platform_id=} in pulp"
-            ),
+            detail=(f"Unable to find errata records with {record_id=} and " f"{platform_id=} in pulp"),
         )
     return updateinfo_xml
 
 
 @router.post("/update/", response_model=errata_schema.ErrataRecord)
@@ -148,13 +143,11 @@
 async def update_package_status(
     packages: List[errata_schema.ChangeErrataPackageStatusRequest],
     db: AsyncSession = Depends(get_db),
 ):
     try:
-        return {
-            "ok": bool(await errata_crud.update_package_status(db, packages))
-        }
+        return {"ok": bool(await errata_crud.update_package_status(db, packages))}
     except ValueError as e:
         return {"ok": False, "error": e.message}
 
 
 @router.post(
@@ -178,24 +171,17 @@
         return {"message": f"Record {record_id} already in progress"}
     db_record.release_status = ErrataReleaseStatus.IN_PROGRESS
     db_record.last_release_log = None
     await session.commit()
     release_errata.send(record_id, platform_id, force)
-    return {
-        "message": f"Release updateinfo record {record_id} has been started"
-    }
+    return {"message": f"Release updateinfo record {record_id} has been started"}
 
 
 @router.post("/bulk_release_records/")
 async def bulk_release_errata_records(records_ids: List[str]):
     bulk_errata_release.send(records_ids)
-    return {
-        "message": (
-            "Following records scheduled for release:"
-            f" {', '.join(records_ids)}"
-        )
-    }
+    return {"message": ("Following records scheduled for release:" f" {', '.join(records_ids)}")}
 
 
 @router.post('/reset-matched-packages')
 async def reset_matched_packages(
     record_id: str,
--- alws/crud/errata.py	2024-04-17 09:53:49.707843+00:00
+++ alws/crud/errata.py	2024-04-17 09:55:05.026166+00:00
@@ -104,54 +104,36 @@
             is_empty = criteria_node.simplify()
             if is_empty:
                 to_remove.append(criteria)
         for criteria in to_remove:
             self.criteria["criteria"].remove(criteria)
-        if (
-            self.parent is not None
-            and len(self.criteria["criterion"]) == 1
-            and len(self.criteria["criteria"]) == 0
-        ):
-            self.parent.criteria["criterion"].append(
-                self.criteria["criterion"].pop()
-            )
-        if (
-            len(self.criteria["criteria"]) == 0
-            and len(self.criteria["criterion"]) == 0
-        ):
+        if self.parent is not None and len(self.criteria["criterion"]) == 1 and len(self.criteria["criteria"]) == 0:
+            self.parent.criteria["criterion"].append(self.criteria["criterion"].pop())
+        if len(self.criteria["criteria"]) == 0 and len(self.criteria["criterion"]) == 0:
             return True
         return False
 
 
-async def get_oval_xml(
-    db: AsyncSession, platform_name: str, only_released: bool = False
-):
+async def get_oval_xml(db: AsyncSession, platform_name: str, only_released: bool = False):
     query = select(models.NewErrataRecord).options(
         selectinload(models.NewErrataRecord.packages)
         .selectinload(models.NewErrataPackage.albs_packages)
         .selectinload(models.NewErrataToALBSPackage.build_artifact)
         .selectinload(models.BuildTaskArtifact.build_task),
-        selectinload(models.NewErrataRecord.references).selectinload(
-            models.NewErrataReference.cve
-        ),
-    )
-
-    platform = await db.execute(
-        select(models.Platform).where(models.Platform.name == platform_name)
-    )
+        selectinload(models.NewErrataRecord.references).selectinload(models.NewErrataReference.cve),
+    )
+
+    platform = await db.execute(select(models.Platform).where(models.Platform.name == platform_name))
     platform: models.Platform = platform.scalars().first()
 
     if not platform:
         return
 
     query = query.filter(models.NewErrataRecord.platform_id == platform.id)
 
     if only_released:
-        query = query.filter(
-            models.NewErrataRecord.release_status
-            == ErrataReleaseStatus.RELEASED
-        )
+        query = query.filter(models.NewErrataRecord.release_status == ErrataReleaseStatus.RELEASED)
 
     records = (await db.execute(query)).scalars().all()
     return errata_records_to_oval(records)
 
 
@@ -178,20 +160,14 @@
     for record in records:
         is_freezed = record.freezed
         rhel_evra_mapping = collections.defaultdict(dict)
         rhel_name_mapping = collections.defaultdict(set)
         for pkg in record.packages:
-            albs_pkgs = [
-                albs_pkg
-                for albs_pkg in pkg.albs_packages
-                if albs_pkg.status == ErrataPackageStatus.released
-            ]
+            albs_pkgs = [albs_pkg for albs_pkg in pkg.albs_packages if albs_pkg.status == ErrataPackageStatus.released]
             for albs_pkg in albs_pkgs:
                 rhel_evra = f"{pkg.epoch}:{pkg.version}-{pkg.release}"
-                albs_evra = (
-                    f"{albs_pkg.epoch}:{albs_pkg.version}-{albs_pkg.release}"
-                )
+                albs_evra = f"{albs_pkg.epoch}:{albs_pkg.version}-{albs_pkg.release}"
                 arch = albs_pkg.arch
                 if arch == "noarch":
                     arch = pkg.arch
                 rhel_evra_mapping[rhel_evra][arch] = albs_evra
                 rhel_name_mapping[rhel_evra].add(albs_pkg.name)
@@ -209,13 +185,11 @@
                     if criterion["ref"] in criterion_refs:
                         continue
                     criterion_refs.add(criterion["ref"])
                     if criterion["comment"] == "Red Hat CoreOS 4 is installed":
                         continue
-                    criterion["comment"] = debrand_comment(
-                        criterion["comment"], record.platform.distr_version
-                    )
+                    criterion["comment"] = debrand_comment(criterion["comment"], record.platform.distr_version)
                     if not is_freezed:
                         evra = evra_regex.search(criterion["comment"])
                         if evra:
                             evra = evra.group()
                             if evra not in rhel_evra_mapping.keys():
@@ -223,28 +197,22 @@
                             package_name = criterion["comment"].split()[0]
                             if package_name not in rhel_name_mapping[evra]:
                                 continue
                             # TODO: Add test mapping here
                             #       test_id: rhel_evra
-                            criterion["comment"] = criterion[
-                                "comment"
-                            ].replace(
+                            criterion["comment"] = criterion["comment"].replace(
                                 evra,
-                                rhel_evra_mapping[evra][
-                                    next(iter(rhel_evra_mapping[evra].keys()))
-                                ],
+                                rhel_evra_mapping[evra][next(iter(rhel_evra_mapping[evra].keys()))],
                             )
                     criterion_list.append(criterion)
                 if len(criterion_list) == 1 and re.search(
                     r"is signed with AlmaLinux OS",
                     criterion_list[0]["comment"],
                 ):
                     criterion_list = []
                 criteria["criterion"] = criterion_list
-                links_tracking.update(
-                    criterion["ref"] for criterion in criterion_list
-                )
+                links_tracking.update(criterion["ref"] for criterion in criterion_list)
             criteria_list = new_criteria_list
         for criteria in record.original_criteria:
             criteria_node = CriteriaNode(criteria, None)
             criteria_node.simplify()
         if record.oval_title:
@@ -257,24 +225,18 @@
             "id": debrand_id(record.definition_id),
             "version": record.definition_version,
             "class": record.definition_class,
             "metadata": {
                 "title": title,
-                "description": (
-                    record.description
-                    if record.description
-                    else record.original_description
-                ),
+                "description": (record.description if record.description else record.original_description),
                 "advisory": {
                     "from": record.contact_mail,
                     "severity": record.severity,
                     "rights": record.rights,
                     "issued_date": record.issued_date,
                     "updated_date": record.updated_date,
-                    "affected_cpe_list": debrand_affected_cpe_list(
-                        record.affected_cpe, record.platform.distr_version
-                    ),
+                    "affected_cpe_list": debrand_affected_cpe_list(record.affected_cpe, record.platform.distr_version),
                     "bugzilla": [
                         {
                             "id": ref.ref_id,
                             "href": ref.href,
                             "title": ref.title,
@@ -327,20 +289,16 @@
             objects.add(test["id"])
             if get_test_cls_by_tag(test["type"]) in (
                 RpminfoTest,
                 RpmverifyfileTest,
             ):
-                test["comment"] = debrand_comment(
-                    test["comment"], record.platform.distr_version
-                )
+                test["comment"] = debrand_comment(test["comment"], record.platform.distr_version)
             test["object_ref"] = debrand_id(test["object_ref"])
             if test.get("state_ref"):
                 test["state_ref"] = debrand_id(test["state_ref"])
             links_tracking.update([test["object_ref"], test["state_ref"]])
-            oval.append_object(
-                get_test_cls_by_tag(test["type"]).from_dict(test)
-            )
+            oval.append_object(get_test_cls_by_tag(test["type"]).from_dict(test))
         for obj in record.original_objects:
             obj["id"] = debrand_id(obj["id"])
             if obj["id"] in objects:
                 continue
             if obj["id"] not in links_tracking:
@@ -350,36 +308,28 @@
                 obj["instance_var_ref"] = debrand_id(obj["instance_var_ref"])
                 links_tracking.add(obj["instance_var_ref"])
             if get_object_cls_by_tag(obj["type"]) == RpmverifyfileObject:
                 if obj["filepath"] == "/etc/redhat-release":
                     obj["filepath"] = "/etc/almalinux-release"
-            oval.append_object(
-                get_object_cls_by_tag(obj["type"]).from_dict(obj)
-            )
+            oval.append_object(get_object_cls_by_tag(obj["type"]).from_dict(obj))
         for state in record.original_states:
             state["id"] = debrand_id(state["id"])
             if state["id"] in objects:
                 continue
             if state["id"] not in links_tracking:
                 continue
             if state.get("evr"):
                 if state["evr"] in rhel_evra_mapping:
                     if state["arch"]:
-                        state["arch"] = "|".join(
-                            rhel_evra_mapping[state["evr"]].keys()
-                        )
-                    state["evr"] = rhel_evra_mapping[state["evr"]][
-                        next(iter(rhel_evra_mapping[state["evr"]].keys()))
-                    ]
+                        state["arch"] = "|".join(rhel_evra_mapping[state["evr"]].keys())
+                    state["evr"] = rhel_evra_mapping[state["evr"]][next(iter(rhel_evra_mapping[state["evr"]].keys()))]
             objects.add(state["id"])
             state_cls = get_state_cls_by_tag(state["type"])
             if state_cls == RpminfoState:
                 if not is_freezed:
                     if state["signature_keyid"]:
-                        state["signature_keyid"] = gpg_keys[
-                            record.platform.distr_version
-                        ].lower()
+                        state["signature_keyid"] = gpg_keys[record.platform.distr_version].lower()
             elif state_cls == RpmverifyfileState:
                 if state["name"] == "^redhat-release":
                     state["name"] = "^almalinux-release"
             oval.append_object(state_cls.from_dict(state))
         for var in record.original_variables:
@@ -387,25 +337,18 @@
             if var["id"] in objects:
                 continue
             if var["id"] not in links_tracking:
                 continue
             objects.add(var["id"])
-            oval.append_object(
-                get_variable_cls_by_tag(var["type"]).from_dict(var)
-            )
+            oval.append_object(get_variable_cls_by_tag(var["type"]).from_dict(var))
             for obj in record.original_objects:
-                if (
-                    obj["id"]
-                    != var["arithmetic"]["object_component"]["object_ref"]
-                ):
+                if obj["id"] != var["arithmetic"]["object_component"]["object_ref"]:
                     continue
                 if obj["id"] in objects:
                     continue
                 objects.add(obj["id"])
-                oval.append_object(
-                    get_object_cls_by_tag(obj["type"]).from_dict(obj)
-                )
+                oval.append_object(get_object_cls_by_tag(obj["type"]).from_dict(obj))
     return oval.dump_to_string()
 
 
 def prepare_search_params(
     errata_record: Union[models.NewErrataRecord, BaseErrataRecord],
@@ -490,13 +433,11 @@
         if update_record.title == record.original_title:
             record.title = None
         else:
             record.title = update_record.title
         if record.title:
-            record.oval_title = get_oval_title(
-                record.title, record.id, record.severity
-            )
+            record.oval_title = get_oval_title(record.title, record.id, record.severity)
     if update_record.description is not None:
         if update_record.description == record.original_description:
             record.description = None
         else:
             record.description = update_record.description
@@ -522,13 +463,11 @@
         errata_package.version,
         clean_release(errata_package.release),
     ))
     # We add ErrataToALBSPackage if we find a matching package already
     # in production repositories.
-    for prod_package in prod_repos_cache.get(clean_package_name, {}).get(
-        errata_package.arch, []
-    ):
+    for prod_package in prod_repos_cache.get(clean_package_name, {}).get(errata_package.arch, []):
         mapping = models.NewErrataToALBSPackage(
             pulp_href=prod_package.pulp_href,
             status=ErrataPackageStatus.released,
             name=prod_package.name,
             version=prod_package.version,
@@ -648,21 +587,17 @@
         issued_date=errata.issued_date,
         updated_date=errata.updated_date,
         description=None,
         original_description=errata.description,
         title=None,
-        oval_title=get_oval_title(
-            errata.title, alma_errata_id, errata.severity
-        ),
+        oval_title=get_oval_title(errata.title, alma_errata_id, errata.severity),
         original_title=get_verbose_errata_title(errata.title, errata.severity),
         contact_mail=platform.contact_mail,
         status=errata.status,
         version=errata.version,
         severity=errata.severity,
-        rights=jinja2.Template(platform.copyright).render(
-            year=datetime.datetime.utcnow().year
-        ),
+        rights=jinja2.Template(platform.copyright).render(year=datetime.datetime.utcnow().year),
         definition_id=errata.definition_id,
         definition_version=errata.definition_version,
         definition_class=errata.definition_class,
         affected_cpe=errata.affected_cpe,
         criteria=None,
@@ -681,15 +616,11 @@
     # References
     self_ref_exists = False
     for ref in errata.references:
         db_cve = None
         if ref.cve:
-            db_cve = await db.execute(
-                select(models.ErrataCVE).where(
-                    models.ErrataCVE.id == ref.cve.id
-                )
-            )
+            db_cve = await db.execute(select(models.ErrataCVE).where(models.ErrataCVE.id == ref.cve.id))
             db_cve = db_cve.scalars().first()
             if db_cve is None:
                 db_cve = models.ErrataCVE(
                     id=ref.cve.id,
                     cvss3=ref.cve.cvss3,
@@ -716,14 +647,11 @@
         db_errata.references.append(db_reference)
         items_to_insert.append(db_reference)
     if not self_ref_exists:
         html_id = db_errata.id.replace(":", "-")
         self_ref = models.NewErrataReference(
-            href=(
-                "https://errata.almalinux.org/"
-                f"{platform.distr_version}/{html_id}.html"
-            ),
+            href=("https://errata.almalinux.org/" f"{platform.distr_version}/{html_id}.html"),
             ref_id=db_errata.id,
             ref_type=ErrataReferenceType.self_ref,
             title=db_errata.id,
         )
         db_errata.references.append(self_ref)
@@ -749,15 +677,11 @@
             reboot_suggested=False,
         )
         db_errata.packages.append(db_package)
         items_to_insert.append(db_package)
         # Create ErrataToAlbsPackages
-        items_to_insert.extend(
-            await get_matching_albs_packages(
-                db, db_package, prod_repos_cache, db_errata.module
-            )
-        )
+        items_to_insert.extend(await get_matching_albs_packages(db, db_package, prod_repos_cache, db_errata.module))
 
     db.add_all(items_to_insert)
     await db.commit()
     await db.refresh(db_errata)
     if not settings.github_integration_enabled:
@@ -791,13 +715,11 @@
     options = [
         selectinload(models.NewErrataRecord.packages)
         .selectinload(models.NewErrataPackage.albs_packages)
         .selectinload(models.NewErrataToALBSPackage.build_artifact)
         .selectinload(models.BuildTaskArtifact.build_task),
-        selectinload(models.NewErrataRecord.references).selectinload(
-            models.NewErrataReference.cve
-        ),
+        selectinload(models.NewErrataRecord.references).selectinload(models.NewErrataReference.cve),
     ]
     query = (
         select(models.NewErrataRecord)
         .options(*options)
         .order_by(models.NewErrataRecord.updated_date.desc())
@@ -820,64 +742,48 @@
     cve_id: Optional[str] = None,
     status: Optional[ErrataReleaseStatus] = None,
 ):
     options = []
     if compact:
-        options.append(
-            load_only(
-                models.NewErrataRecord.id, models.NewErrataRecord.updated_date
-            )
-        )
+        options.append(load_only(models.NewErrataRecord.id, models.NewErrataRecord.updated_date))
     else:
         options.extend([
             selectinload(models.NewErrataRecord.packages)
             .selectinload(models.NewErrataPackage.albs_packages)
             .selectinload(models.NewErrataToALBSPackage.build_artifact)
             .selectinload(models.BuildTaskArtifact.build_task),
-            selectinload(models.NewErrataRecord.references).selectinload(
-                models.NewErrataReference.cve
-            ),
+            selectinload(models.NewErrataRecord.references).selectinload(models.NewErrataReference.cve),
         ])
 
     def generate_query(count=False):
         query = select(func.count(models.NewErrataRecord.id))
         if not count:
             query = select(models.NewErrataRecord).options(*options)
             query = query.order_by(models.NewErrataRecord.id.desc())
         if errata_id:
-            query = query.filter(
-                models.NewErrataRecord.id.like(f"%{errata_id}%")
-            )
+            query = query.filter(models.NewErrataRecord.id.like(f"%{errata_id}%"))
         if errata_ids:
             query = query.filter(models.NewErrataRecord.id.in_(errata_ids))
         if title:
             query = query.filter(
                 or_(
                     models.NewErrataRecord.title.like(f"%{title}%"),
                     models.NewErrataRecord.original_title.like(f"%{title}%"),
                 )
             )
         if platform:
-            query = query.filter(
-                models.NewErrataRecord.platform_id == platform
-            )
+            query = query.filter(models.NewErrataRecord.platform_id == platform)
         if cve_id:
-            query = query.filter(
-                models.NewErrataRecord.cves.like(f"%{cve_id}%")
-            )
+            query = query.filter(models.NewErrataRecord.cves.like(f"%{cve_id}%"))
         if status:
-            query = query.filter(
-                models.NewErrataRecord.release_status == status
-            )
+            query = query.filter(models.NewErrataRecord.release_status == status)
         if page and not count:
             query = query.slice(10 * page - 10, 10 * page)
         return query
 
     return {
-        "total_records": (
-            await db.execute(generate_query(count=True))
-        ).scalar(),
+        "total_records": (await db.execute(generate_query(count=True))).scalar(),
         "records": (await db.execute(generate_query())).scalars().all(),
         "current_page": page,
     }
 
 
@@ -889,12 +795,11 @@
         for record in request:
             errata_record = await db.execute(
                 select(models.NewErrataRecord)
                 .where(
                     models.NewErrataRecord.id == record.errata_record_id,
-                    models.NewErrataRecord.platform_id
-                    == record.errata_platform_id,
+                    models.NewErrataRecord.platform_id == record.errata_platform_id,
                 )
                 .options(
                     selectinload(models.NewErrataRecord.packages)
                     .selectinload(models.NewErrataPackage.albs_packages)
                     .selectinload(models.NewErrataToALBSPackage.build_artifact)
@@ -906,18 +811,12 @@
             for errata_pkg in errata_record.packages:
                 if errata_pkg.source_srpm != record.source:
                     continue
                 for albs_pkg in errata_pkg.albs_packages:
                     if albs_pkg.status == ErrataPackageStatus.released:
-                        raise ValueError(
-                            "There is already released package "
-                            f"with same source: {albs_pkg}"
-                        )
-                    if (
-                        albs_pkg.build_id != record.build_id
-                        and record_approved
-                    ):
+                        raise ValueError("There is already released package " f"with same source: {albs_pkg}")
+                    if albs_pkg.build_id != record.build_id and record_approved:
                         albs_pkg.status = ErrataPackageStatus.skipped
                     if albs_pkg.build_id == record.build_id:
                         albs_pkg.status = record.status
     return True
 
@@ -973,22 +872,16 @@
         if errata_pkg.albs_artifact_id is not None:
             query = models.BuildTaskArtifact.id == errata_pkg.albs_artifact_id
         db_pkg = await session.execute(
             select(models.BuildTaskArtifact)
             .where(query)
-            .options(
-                selectinload(models.BuildTaskArtifact.build_task).selectinload(
-                    models.BuildTask.rpm_modules
-                )
-            )
+            .options(selectinload(models.BuildTaskArtifact.build_task).selectinload(models.BuildTask.rpm_modules))
         )
         db_pkg = db_pkg.scalars().first()
         if not db_pkg:
             continue
-        db_module = next((
-            i for i in db_pkg.build_task.rpm_modules if '-devel' not in i.name
-        ))
+        db_module = next((i for i in db_pkg.build_task.rpm_modules if '-devel' not in i.name))
         if db_module is not None:
             rpm_module = {
                 "name": db_module.name,
                 "stream": db_module.stream,
                 "version": int(db_module.version),
@@ -996,22 +889,15 @@
                 # we use here repository arch, because looking by
                 # noarch package (that can be referred to other arch)
                 # we can get module with different arch
                 "arch": arch,
             }
-    collection_name = (
-        f"{platform.name.lower()}-for-{arch}-{repo_stage}-"
-        f"rpms__{platform_version}_default"
-    )
-    default_summary = clean_errata_title(
-        record.get_title(), severity=record.severity
-    )
+    collection_name = f"{platform.name.lower()}-for-{arch}-{repo_stage}-" f"rpms__{platform_version}_default"
+    default_summary = clean_errata_title(record.get_title(), severity=record.severity)
     pulp_record = {
         "id": record.id,
-        "updated_date": datetime.datetime.utcnow().strftime(
-            "%Y-%m-%d %H:%M:%S"
-        ),
+        "updated_date": datetime.datetime.utcnow().strftime("%Y-%m-%d %H:%M:%S"),
         "issued_date": record.issued_date.strftime("%Y-%m-%d %H:%M:%S"),
         "description": record.get_description(),
         "fromstr": record.contact_mail,
         "status": record.status,
         "title": record.get_title(),
@@ -1067,13 +953,11 @@
                     select(models.BuildTaskArtifact)
                     .where(
                         models.BuildTaskArtifact.href == pkg_href,
                     )
                     .options(
-                        selectinload(
-                            models.BuildTaskArtifact.build_task
-                        ).selectinload(models.BuildTask.rpm_modules)
+                        selectinload(models.BuildTaskArtifact.build_task).selectinload(models.BuildTask.rpm_modules)
                     )
                 )
             )
             .scalars()
             .all()
@@ -1081,12 +965,11 @@
         for db_pkg in db_pkg_list:
             errata_pkgs = await db.execute(
                 select(models.NewErrataToALBSPackage)
                 .where(
                     or_(
-                        models.NewErrataToALBSPackage.albs_artifact_id
-                        == db_pkg.id,
+                        models.NewErrataToALBSPackage.albs_artifact_id == db_pkg.id,
                         models.NewErrataToALBSPackage.pulp_href == pkg_href,
                     )
                 )
                 .options(
                     selectinload(models.NewErrataToALBSPackage.errata_package),
@@ -1119,15 +1002,11 @@
 def append_update_packages_in_update_records(
     pulp_db: Session,
     errata_records: List[Dict[str, Any]],
     updateinfo_mapping: DefaultDict[
         str,
-        List[
-            Tuple[
-                models.BuildTaskArtifact, dict, models.NewErrataToALBSPackage
-            ]
-        ],
+        List[Tuple[models.BuildTaskArtifact, dict, models.NewErrataToALBSPackage]],
     ],
 ):
     with pulp_db.begin():
         for record in errata_records:
             record_uuid = uuid.UUID(record["pulp_href"].split("/")[-2])
@@ -1136,13 +1015,11 @@
                 continue
             pulp_record = pulp_db.execute(
                 select(UpdateRecord)
                 .where(UpdateRecord.content_ptr_id == record_uuid)
                 .options(
-                    selectinload(UpdateRecord.collections).selectinload(
-                        UpdateCollection.packages
-                    ),
+                    selectinload(UpdateRecord.collections).selectinload(UpdateCollection.packages),
                 )
             )
             pulp_record: UpdateRecord = pulp_record.scalars().first()
             for _, pulp_pkg, pkg in packages:
                 already_released = False
@@ -1152,15 +1029,11 @@
                     collection.name,
                 ).group()
                 if pulp_pkg["arch"] not in (collection_arch, "noarch"):
                     continue
                 already_released = next(
-                    (
-                        package
-                        for package in collection.packages
-                        if package.filename == pulp_pkg["location_href"]
-                    ),
+                    (package for package in collection.packages if package.filename == pulp_pkg["location_href"]),
                     None,
                 )
                 if already_released:
                     continue
                 collection.packages.append(
@@ -1175,13 +1048,11 @@
                         src=pulp_pkg["rpm_sourcerpm"],
                         sum=pulp_pkg["sha256"],
                         sum_type=cr.checksum_type("sha256"),
                     )
                 )
-                pulp_record.updated_date = datetime.datetime.utcnow().strftime(
-                    "%Y-%m-%d %H:%M:%S"
-                )
+                pulp_record.updated_date = datetime.datetime.utcnow().strftime("%Y-%m-%d %H:%M:%S")
 
 
 def get_albs_packages_from_record(
     record: models.NewErrataRecord,
     pulp_packages: Dict[str, Any],
@@ -1225,12 +1096,11 @@
             full_albs_name = pkg_names_mapping["albs"].get(missing_pkg)
             full_name = full_albs_name if full_albs_name else full_name
             missing_pkg_names.append(full_name)
         msg = (
             "Cannot release updateinfo record, the following packages "
-            "are missing from platform repositories or have wrong status: "
-            + ", ".join(missing_pkg_names)
+            "are missing from platform repositories or have wrong status: " + ", ".join(missing_pkg_names)
         )
         if not force:
             raise ValueError(msg)
     return repo_mapping, missing_pkg_names
 
@@ -1257,15 +1127,11 @@
             blacklist_updateinfo=[],
         )
         latest_repo_version = await pulp.get_repo_latest_version(repo_href)
         if latest_repo_version:
             errata_records = await pulp.list_updateinfo_records(
-                id__in=[
-                    record_id
-                    for record_id in updateinfo_mapping.keys()
-                    if record_id == db_record.id
-                ],
+                id__in=[record_id for record_id in updateinfo_mapping.keys() if record_id == db_record.id],
                 repository_version=latest_repo_version,
             )
             logging.info("Appending packages to existing errata records")
             with get_pulp_db() as pulp_db:
                 append_update_packages_in_update_records(
@@ -1283,13 +1149,11 @@
                 repo_href,
                 publish=False,
             )
         )
         if publish:
-            publish_tasks.append(
-                pulp.create_rpm_publication(repo_href, sleep_time=30.0)
-            )
+            publish_tasks.append(pulp.create_rpm_publication(repo_href, sleep_time=30.0))
     if not publish:
         return release_tasks
     logging.info("Releasing errata packages in async tasks")
     await asyncio.gather(*release_tasks)
     logging.info("Publicating repositories in async tasks")
@@ -1303,13 +1167,11 @@
         .options(
             selectinload(models.NewErrataRecord.references),
             selectinload(models.NewErrataRecord.packages)
             .selectinload(models.NewErrataPackage.albs_packages)
             .selectinload(models.NewErrataToALBSPackage.build_artifact),
-            selectinload(models.NewErrataRecord.platform).selectinload(
-                models.Platform.repos
-            ),
+            selectinload(models.NewErrataRecord.platform).selectinload(models.Platform.repos),
         )
         .with_for_update()
     )
     return query
 
@@ -1322,14 +1184,11 @@
     repo_mapping: dict,
     db_record: list,
     force_flag: bool,
     missing_pkg_names: List[str],
 ) -> str:
-    release_log = [
-        f"Record {record_id} successfully released at"
-        f" {datetime.datetime.utcnow()}\n"
-    ]
+    release_log = [f"Record {record_id} successfully released at" f" {datetime.datetime.utcnow()}\n"]
     pkg_fields = [
         RpmPackage.name,
         RpmPackage.epoch,
         RpmPackage.version,
         RpmPackage.release,
@@ -1374,13 +1233,11 @@
     async with asynccontextmanager(get_db)() as session:
         session: AsyncSession
         query = generate_query_for_release([record_id])
         query = query.filter(models.NewErrataRecord.platform_id == platform_id)
         db_record = await session.execute(query)
-        db_record: Optional[models.NewErrataRecord] = (
-            db_record.scalars().first()
-        )
+        db_record: Optional[models.NewErrataRecord] = db_record.scalars().first()
         if not db_record:
             logging.info("Record with %s id doesn't exists", record_id)
             return
 
         logging.info("Record release %s has been started", record_id)
@@ -1464,12 +1321,11 @@
                 records_ids,
             )
             return
 
         logging.info(
-            "Starting bulk errata release, the following records are"
-            " locked: %s",
+            "Starting bulk errata release, the following records are" " locked: %s",
             [rec.id for rec in db_records],
         )
         for db_record in db_records:
             logging.info("Preparing data for %s", db_record.id)
             search_params = prepare_search_params(db_record)
@@ -1520,13 +1376,11 @@
             release_tasks.extend(tasks)
         await session.commit()
     logging.info("Executing release tasks")
     await asyncio.gather(*release_tasks)
     logging.info("Executing publication tasks")
-    await asyncio.gather(
-        *(pulp.create_rpm_publication(href) for href in set(repos_to_publish))
-    )
+    await asyncio.gather(*(pulp.create_rpm_publication(href) for href in set(repos_to_publish)))
     logging.info("Bulk errata release is finished")
 
 
 async def get_updateinfo_xml_from_pulp(
     db: AsyncSession,
@@ -1546,15 +1400,11 @@
 
     platform_name = None
     if platform_id:
         async with db.begin():
             platform_name = (
-                await db.execute(
-                    select(models.Platform.name).where(
-                        models.Platform.id == platform_id
-                    )
-                )
+                await db.execute(select(models.Platform.name).where(models.Platform.id == platform_id))
             ).scalar()
             if not platform_name:
                 return
 
     cr_upd = cr.UpdateInfo()
@@ -1657,16 +1507,12 @@
         (
             await session.execute(
                 select(models.NewErrataRecord)
                 .where(models.NewErrataRecord.id == record_id)
                 .options(
-                    selectinload(models.NewErrataRecord.platform).selectinload(
-                        models.Platform.repos
-                    ),
-                    selectinload(models.NewErrataRecord.packages).selectinload(
-                        models.NewErrataPackage.albs_packages
-                    ),
+                    selectinload(models.NewErrataRecord.platform).selectinload(models.Platform.repos),
+                    selectinload(models.NewErrataRecord.packages).selectinload(models.NewErrataPackage.albs_packages),
                 )
                 .with_for_update()
             )
         )
         .scalars()
@@ -1682,13 +1528,11 @@
         for_release=False,
         module=record.module,
     )
     await session.execute(
         delete(models.NewErrataToALBSPackage).where(
-            models.NewErrataToALBSPackage.errata_package_id.in_(
-                (pkg.id for pkg in record.packages)
-            )
+            models.NewErrataToALBSPackage.errata_package_id.in_((pkg.id for pkg in record.packages))
         )
     )
     for package in record.packages:
         items_to_insert.extend(
             await get_matching_albs_packages(

Bandit report

Run started:2024-04-17 09:55:06.422575

Test results:
	No issues identified.

Code scanned:
	Total lines of code: 1754
	Total lines skipped (#nosec): 0
	Total potential issues skipped due to specifically being disabled (e.g., #nosec BXXX): 0

Run metrics:
	Total issues (by severity):
		Undefined: 0
		Low: 0
		Medium: 0
		High: 0
	Total issues (by confidence):
		Undefined: 0
		Low: 0
		Medium: 0
		High: 0
Files skipped (0):

View full reports on the Job Summary page.

javihernandez reviewed Apr 9, 2024

View reviewed changes

Kwaizer force-pushed the platf_check branch 4 times, most recently from 1a6e098 to 0185d45 Compare April 10, 2024 08:55

Added platform check

7d27f34

Kwaizer force-pushed the platf_check branch from 0185d45 to 7d27f34 Compare April 17, 2024 09:53

Korulag approved these changes Apr 17, 2024

View reviewed changes

Korulag merged commit 38ccc79 into AlmaLinux:master Apr 17, 2024
2 checks passed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Added platform check #783

Added platform check #783

Kwaizer commented Apr 9, 2024

github-actions bot commented Apr 9, 2024 •

edited

Loading

github-actions bot commented Apr 9, 2024

Kwaizer commented Apr 9, 2024

javihernandez Apr 9, 2024

javihernandez Apr 9, 2024

maccelf commented Apr 9, 2024

github-actions bot commented Apr 17, 2024

Added platform check #783

Added platform check #783

Conversation

Kwaizer commented Apr 9, 2024

github-actions bot commented Apr 9, 2024 • edited Loading

github-actions bot commented Apr 9, 2024

Kwaizer commented Apr 9, 2024

javihernandez Apr 9, 2024

Choose a reason for hiding this comment

javihernandez Apr 9, 2024

Choose a reason for hiding this comment

maccelf commented Apr 9, 2024

github-actions bot commented Apr 17, 2024

Code Coverage Summary

Linter reports

github-actions bot commented Apr 9, 2024 •

edited

Loading