Created attachment 518324 [details]
Patch to fix the described bug

Description of problem:

In the pulp.server.api.depsolver.DepSolver constructor, a yum.repos.RepoStorage object is instantiated as such:

self._repostore = RepoStorage(self)

But the RepoStorage constructor isn't expecting a Pulp DepSolver object; it's expecting a yum.YumBase object.  This doesn't become evident until DepSolver.setup() is called, whereupon the stack trace listed below obtains.


Version-Release number of selected component (if applicable):

0.0.223

How reproducible:

Always

Steps to Reproduce:

1. Attempt to resolve the dependencies of a package:

from pulp.client.api import server
from pulp.client.api.service import ServiceAPI
from pulp.client.consumer.config import ConsumerConfig
from pulp.client.admin.credentials import Login as LoginBundle

config = ConsumerConfig()
pulpserver = server.PulpServer(config.server['host'],
                               int(config.server['port']),
                               config.server['scheme'],
                               config.server['path'])
bundle = LoginBundle()
pulpserver.set_ssl_credentials(bundle.crtpath())
server.set_active_server(pulpserver)

sapi = ServiceAPI()
sapi.dependencies(["perl"], ["centos-5-x86_64-os"], recursive=1)
  
Actual results:

Ugly borkage:

AttributeError: DepSolver instance has no attribute '_override_sigchecks'
Traceback (most recent call last):
  File "/usr/lib/python2.6/site-packages/pulp/server/webservices/controllers/decorators.py", line 48, in report_error
    return method(self, *args, **kwargs)
  File "/usr/lib/python2.6/site-packages/pulp/server/webservices/controllers/decorators.py", line 127, in _auth_decorator
    value = method(self, *args, **kwargs)
  File "/usr/lib/python2.6/site-packages/pulp/server/webservices/controllers/services.py", line 72, in POST
    return self.ok(papi.package_dependency(data[\'pkgnames\'], data[\'repoids\'], recursive=recursive))
  File "/usr/lib/python2.6/site-packages/pulp/server/api/package.py", line 241, in package_dependency
    dsolve = DepSolver(repos, pkgnames)
  File "/usr/lib/python2.6/site-packages/pulp/server/api/depsolver.py", line 38, in __init__
    self.setup()
  File "/usr/lib/python2.6/site-packages/pulp/server/api/depsolver.py", line 49, in setup
    self._repostore.add(self.yrepo)
  File "/usr/lib/python2.6/site-packages/yum/repos.py", line 113, in add
    repoobj._override_sigchecks = self.ayum._override_sigchecks

Expected results:

Ugly dependency list.

Additional info:

Patch attached.  This at least appears to fix this exact bug, although I've been getting sporadic stack traces of a different nature with it.  That might be user error, though.