cobbler 2.0.1 fix for reposync --only

[Alexander, Ian]

This is in reference to https://fedorahosted.org/cobbler/ticket/491.  It
looks like an indentation error in the remote file.  Once you've patched
remote.py, you will have to restart cobblerd.

 

root@cobbler:/usr/lib/python2.4/site-packages/cobbler#diff -u
remote.py.orig  remote.py

--- remote.py.orig      2009-10-02 15:53:16.000000000 -0500

+++ remote.py   2009-10-18 01:07:18.000000000 -0500

@@ -230,11 +230,11 @@

             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)

+            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)

[Devan Goodwin]

On Sun, Oct 18, 2009 at 3:12 AM, Alexander, Ian
<Ian.Alexander(a)travelocity.com&gt; wrote:
>  This is in reference to https://fedorahosted.org/cobbler/ticket/491.
>  It
>  looks like an indentation error in the remote file.  Once you’ve patched
>  remote.py, you will have to restart cobblerd.
>  
Thanks for the patch Ian. I'm getting set up to test it now, will try
to have it applied asap.

Cheers,

Devan

-- 
Devan Goodwin <dgoodwin(a)rm-rf.ca&gt;
http://rm-rf.ca

[Devan Goodwin]

The actual issue in 491 appears to have already been fixed in 2.0.1,
but you're right that indentation looks off as does the compare. I've
applied your fixes just the same, things seem to work fine.

Cheers,

Devan

On Sat, Oct 24, 2009 at 6:27 PM, Devan Goodwin <dgoodwin(a)rm-rf.ca&gt; wrote:
>  On Sun, Oct 18, 2009 at 3:12 AM, Alexander, Ian
>  <Ian.Alexander(a)travelocity.com&gt; wrote:
> > This is in reference to https://fedorahosted.org/cobbler/ticket/491.  It
> > looks like an indentation error in the remote file.  Once you’ve patched
> > remote.py, you will have to restart cobblerd.
> >
>
>  Thanks for the patch Ian. I'm getting set up to test it now, will try
>  to have it applied asap.
>
>  Cheers,
>
>  Devan
>
>  --
>  Devan Goodwin <dgoodwin(a)rm-rf.ca&gt;
>  http://rm-rf.ca
>  


-- 
Devan Goodwin <dgoodwin(a)rm-rf.ca&gt;
http://rm-rf.ca