[Pulp-list] Unit Tests and Temp Directories

[Jason Dobies]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Same comment applies to cert generation too:

cert_generator:36
cmd = 'openssl x509 -req -sha1 -CA /etc/pki/pulp/ca.crt -CAkey
/etc/pki/pulp/ca.key -set_serial %s -days 3650' % serial

/etc/pki/pulp isn't writeable by my user so this command is going to bomb.

I'd rather make these configuration values and override them in tests
than have our pulp-dev script do even more running around my machine
loosening permissions. That approach would ultimately mess us up since
it would bring our dev environments further from what real deployments
would look like by having weak permissions.


On 08/19/2010 09:33 AM, Jason Dobies wrote:
> I'm getting this when I run the unit tests:
> 
> ======================================================================
> ERROR: test_errata_repo_sync_rhel (test_errata.TestErrata)
> ----------------------------------------------------------------------
> Traceback (most recent call last):
>   File "/home/jdob/vault/code/pulp/test/unit/test_errata.py", line 373,
> in test_errata_repo_sync_rhel
>     self.rapi.sync(r['id'])
>   File
> "/home/jdob/vault/code/pulp/test/unit/../../src/pulp/auditing.py", line
> 186, in _audit
>     result = method(*args, **kwargs)
>   File
> "/home/jdob/vault/code/pulp/test/unit/../../src/pulp/api/repo.py", line
> 568, in sync
>     added_packages, added_errataids = repo_sync.sync(repo, repo_source)
>   File
> "/home/jdob/vault/code/pulp/test/unit/../../src/pulp/api/repo_sync.py",
> line 57, in sync
>     repo_dir = synchronizer.sync(repo, repo_source)
>   File
> "/home/jdob/vault/code/pulp/test/unit/../../src/pulp/api/repo_sync.py",
> line 296, in sync
>     os.makedirs(repo_dir)
>   File "/usr/lib64/python2.6/os.py", line 157, in makedirs
>     mkdir(name, mode)
> OSError: [Errno 13] Permission denied: '/var/lib/pulp/test_errata_repo_sync'
> 
> 
> /var/lib/pulp belongs to apache so I can actually run and use pulp. Can
> we make sure to use a conf value for where things are stored and
> override it in the tests to something in /tmp? We were doing this in the
> past on the API tests to get around this issue, I'm not sure if it was
> removed when the new config subsystem was introduced or just not used.
> 

_______________________________________________
Pulp-list mailing list
Pulp-list at redhat.com
https://www.redhat.com/mailman/listinfo/pulp-list


- -- 
Jason Dobies
RHCE# 805008743336126
Freenode: jdob
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2.0.14 (GNU/Linux)
Comment: Using GnuPG with Fedora - http://enigmail.mozdev.org/

iQEcBAEBAgAGBQJMbTNoAAoJEOMmcTqOSQHC5RUH/35ez43/mvxA51htebq/4qi5
8w+gh/oocjTsEH6GfHIaIxTHJb3sgp6PVuW3rpIJe+MY77JrHs192sQGNjFD2QI7
rss/iagpIASxZBrjXw1abFzifl+cdinSpjshCLT09pdV6TUwBXH5Hpn+v92Xum2V
kDkIUE6hBji3rCp24Ovv6s+veKkBxNN58Ytc1xRKBJlsv0EKqWY/oLlCVYKDdHU9
vA11LN6JkjMj6LKnpg5OZ3QyzW4Rbt0w4oDyGplF/J9L0typcFfqa/U+cr/FWZjE
eHQMhbuRZZeNITcCShUp5DVEaMatT7fpdQqKBKKX9clQE8JYYyj81UgNvaixZVg=
=qBtW
-----END PGP SIGNATURE-----