Skip to content

Commit

Permalink
skip expired images when preparing build matrix
Browse files Browse the repository at this point in the history
  • Loading branch information
linostar committed Sep 9, 2024
1 parent e9d675c commit 26a4849
Show file tree
Hide file tree
Showing 2 changed files with 13 additions and 5 deletions.
16 changes: 12 additions & 4 deletions src/image/prepare_single_image_build_matrix.py
Original file line number Diff line number Diff line change
@@ -1,6 +1,7 @@
#!/usr/bin/env python3

import argparse
from datetime import datetime, timezone
import glob
import json
import os
Expand Down Expand Up @@ -103,10 +104,17 @@ def validate_image_trigger(data: dict) -> None:

# set an output as a marker for later knowing if we need to release
if "release" in builds[img_number]:
release_to = "true"
# the workflow GH matrix has a problem parsing nested JSON dicts
# so let's remove this field since we don't need it for the builds
builds[img_number]["release"] = "true"
min_eol = datetime.strptime(min(
v["end-of-life"] for v in builds[img_number]["release"].values()
), "%Y-%m-%dT%H:%M:%SZ").replace(tzinfo=timezone.utc)
if min_eol < datetime.now(timezone.utc):
print(f"Track {img_number} skipped because it reached its end of life")
builds[img_number]["release"] = ""
else:
release_to = "true"
# the workflow GH matrix has a problem parsing nested JSON dicts
# so let's remove this field since we don't need it for the builds
builds[img_number]["release"] = "true"
else:
builds[img_number]["release"] = ""

Expand Down
2 changes: 1 addition & 1 deletion src/image/release.py
Original file line number Diff line number Diff line change
Expand Up @@ -79,7 +79,7 @@
expired = False
for risk, value in risks.dict(exclude_none=True).items():
if risk in ["end-of-life", "end_of_life"]:
if all_releases[track][risk] < datetime.now(timezone.utc):
if all_releases[track]["end-of-life"] < datetime.now(timezone.utc):
# skip tracks that have expired end_of_life values
expired = True
break
Expand Down

0 comments on commit 26a4849

Please sign in to comment.