kojira: make 'ignore_other_volumes' a config only option.

* pull 'DEFAULT' repo handling out of the loop.
* Add 'ignore_other_volumes' to kojira.conf
This commit is contained in:
Joe Simmons-Talbott 2021-09-22 10:55:23 -04:00 committed by Tomas Kopecek
parent b5bf4f9a3f
commit 61645c6cd9
2 changed files with 8 additions and 5 deletions

View file

@ -643,12 +643,14 @@ class RepoManager(object):
session.logout()
def pruneLocalRepos(self):
volname = 'DEFAULT'
volumedir = pathinfo.volumedir(volname)
repodir = "%s/repos" % volumedir
self._pruneLocalRepos(repodir, self.options.deleted_repo_lifetime)
for volinfo in self.session.listVolumes():
volname = volinfo['name']
volumedir = pathinfo.volumedir(volname)
if volname == 'DEFAULT': # currently the only path for non-dist repos.
repodir = "%s/repos" % volumedir
self._pruneLocalRepos(repodir, self.options.deleted_repo_lifetime)
distrepodir = "%s/repos-dist" % volumedir
self._pruneLocalRepos(distrepodir, self.options.dist_repo_lifetime)
@ -1209,8 +1211,6 @@ def get_options():
parser.add_option("--logfile", help="Specify logfile")
parser.add_option("--queue-file",
help="If specified, queue is dumped to separate status file each cycle")
parser.add_option("--ignore-other-volumes", action="store_true",
help="Ignore repos on other volumes")
(options, args) = parser.parse_args()
config = koji.read_config_files(options.configFile)

View file

@ -46,3 +46,6 @@ logfile=/var/log/kojira.log
; as otherwise you can end with weird behaviour. For details see
; https://pagure.io/koji/issue/2159
; check_external_repos = false
; don't attempt to remove repos on non-default volumes
; ignore_other_volumes = false