Project

Profile

Help

Task #413

closed

Our unit tests require read permissions on server.conf

Added by rbarlow about 9 years ago. Updated almost 5 years ago.

Status:
CLOSED - CURRENTRELEASE
Priority:
Normal
Assignee:
Category:
-
Sprint/Milestone:
-
Start date:
Due date:
% Done:

0%

Estimated time:
Platform Release:
2.8.0
Groomed:
No
Sprint Candidate:
No
Tags:
Pulp 2
Sprint:
Quarter:

Description

I attempted to fix our unit tests to not require reading server.conf, but it turns out that it is more effort than I want to do today to refactor how our configuration loading happens during unit tests.

Refactoring how we do configuration loading will be a very good thing to do in general, especially if we want to support virtualenv development.

For now, pulp-dev.py installs server.conf world-readable. When we find time to fix this bug, we should install server.conf as root:apache 640, just like the RPM. This way the development environment will more closely resemble the production environment, which will be a good thing.

+ This bug was cloned from Bugzilla Bug #1084708 +


Related issues

Has duplicate Pulp - Issue #1230: Ability to run unit tests on a system without pulp-server installedCLOSED - DUPLICATEActions
Blocked by Pulp - Issue #607: server/config.py reads server.conf when the module is loaded.CLOSED - CURRENTRELEASEsemyersActions

Also available in: Atom PDF