diff options
author | Ian Alexander <Ian.Alexander@travelocity.com> | 2009-10-24 19:10:10 -0300 |
---|---|---|
committer | Devan Goodwin <dgoodwin@rm-rf.ca> | 2009-10-24 19:59:45 -0300 |
commit | 80b2c8f1c37912535fe986f182cc7edb310ffcfe (patch) | |
tree | 38de408e9e05a2e7220b55e3f4861d2efcac7f1c | |
parent | 2c6a7eb1f08e3c306fb0e87263860d0e56168a85 (diff) | |
download | cobbler-80b2c8f1c37912535fe986f182cc7edb310ffcfe.tar.gz cobbler-80b2c8f1c37912535fe986f182cc7edb310ffcfe.tar.xz cobbler-80b2c8f1c37912535fe986f182cc7edb310ffcfe.zip |
Correct indentation on background reposync and check for empty list.
-rw-r--r-- | cobbler/remote.py | 12 |
1 files changed, 8 insertions, 4 deletions
diff --git a/cobbler/remote.py b/cobbler/remote.py index a6a92e3f..a227d10f 100644 --- a/cobbler/remote.py +++ b/cobbler/remote.py @@ -226,15 +226,19 @@ class CobblerXMLRPCInterface: def background_reposync(self, options, token): def runner(self): + # NOTE: WebUI passes in repos here, CLI passes only: repos = options.get("repos", []) only = options.get("only", None) if only is not None: repos = [ only ] - if repos != "": + + if len(repos) > 0: for name in repos: - self.remote.api.reposync(tries=self.options.get("tries",3), name=name, nofail=True, logger=self.logger) - else: - self.remote.api.reposync(tries=self.options.get("tries",3), name=None, nofail=False, logger=self.logger) + self.remote.api.reposync(tries=self.options.get("tries", + 3), name=name, nofail=True, logger=self.logger) + else: + self.remote.api.reposync(tries=self.options.get("tries",3), + name=None, nofail=False, logger=self.logger) return True return self.__start_task(runner, token, "reposync", "Reposync", options) |