From 4f59d31cacf3e92423fef8dd11cb272cd675eecc Mon Sep 17 00:00:00 2001
From: Igor Raits <igor.raits@gmail.com>
Date: Jul 20 2021 15:40:39 +0000
Subject: kojira: Do not ever clean up repositories where 'latest' points to


We don't want to leave symlink broken… Ideally we would find out stuff
where we should even remove "latest" but it is not straightforward and I
believe this simple check is good enough for the time being.

Fixes: https://pagure.io/koji/issue/2832
Signed-off-by: Igor Raits <igor.raits@gmail.com>

---

diff --git a/util/kojira b/util/kojira
index 3f6ed4d..eec1a01 100755
--- a/util/kojira
+++ b/util/kojira
@@ -661,9 +661,17 @@ class RepoManager(object):
             if not os.path.isdir(tagdir):
                 self.logger.debug("%s is not a directory, skipping", tagdir)
                 continue
+
+            # Remember where latest symlink points to (if exists)
+            repo_latest_path = "%s/latest" % tagdir
+            repo_latest_id = (
+                os.readlink(repo_latest_path)
+                if os.path.isdir(repo_latest_path)
+                else None
+            )
             for repo_id in os.listdir(tagdir):
-                if repo_id == 'latest':
-                    # ignore latest symlinks
+                if repo_id == 'latest' or repo_id == repo_latest_id:
+                    # ignore latest symlinks or repo where one points to
                     continue
                 try:
                     repo_id = int(repo_id)
@@ -931,6 +939,9 @@ class RepoManager(object):
             if tag_id not in build_tags and tag_id not in ignored_build_tags:
                 # repos for these tags are no longer required
                 for repo in repolist:
+                    if repo.dist:
+                        # Dist repos should expire only basd on the time
+                        continue
                     if repo.ready():
                         repo.expire()