Issue #5560
closed
PULP_SETTINGS environment variable does not work
Added by dkliban@redhat.com about 5 years ago. Updated almost 5 years ago.
Description
Settings file in /etc/pulp/settings.py is the only place where settings can be defined. Users used to be able to set PULP_SETTINGS environment variable to point to a different location.
Updated by bmbouter about 5 years ago
I think this would be simpler if we didn't expect settings in /etc/pulp/settings.py, but instead recommend users use PULP_SETTINGS itself. Users would get more capabilities by for instance being able to natively provide their settings with .toml instead.
The Ansible installer should continue to use /etc/pulp/settings.py, only it needs to also specify PULP_SETTINGS in its systemd files, along with the other environment vars it sets.
Updated by rochacbruno about 5 years ago
Updated by rochacbruno about 5 years ago
Updated by bmbouter about 5 years ago
I think we should make this a 3.0 blocker, and also treat this as a docs change mostly to undocument the /etc/pulp/settings.py expectation, and add the PULP_SETTINGS callout to specify that file.
Updated by bmbouter about 5 years ago
- Status changed from NEW to ASSIGNED
- Assignee set to bmbouter
- Sprint set to Sprint 60
Added by Mike DePaulo about 5 years ago
Updated by bmbouter about 5 years ago
- Status changed from ASSIGNED to POST
Added by Mike DePaulo about 5 years ago
Revision 6e3ad089 | View on GitHub
Adds the PULP_SETTINGS Env Var
Sets it in:
- systemd units
- pulp-devel role bash script
- Any task/handler that calls django-admin
Added by Mike DePaulo about 5 years ago
Revision 6e3ad089 | View on GitHub
Adds the PULP_SETTINGS Env Var
Sets it in:
- systemd units
- pulp-devel role bash script
- Any task/handler that calls django-admin
Added by bmbouter about 5 years ago
Revision 513952bd | View on GitHub
Remove hard coded /etc/pulp/settings.py file
This changes Pulp to not assume any settings file is needed or looked
for at all. It encourages the user to set one via the PULP_SETTINGS
Environment Variable. The settings documentation was updated to reflect
this.
The Installer will continue working as is. The PR below causes the
installer to set PULP_SETTINGS correctly allowing settings to work
the same way in those environments.
Required PR: https://github.com/pulp/ansible-pulp/pull/182
Updated by Anonymous about 5 years ago
- Status changed from POST to MODIFIED
Applied in changeset ansible-pulp3|6e3ad089cc5309f94103a9b1cfe2736d137dace8.
Updated by bmbouter about 5 years ago
- Status changed from MODIFIED to POST
Bringing back to POST for one more PR to merge.
Updated by bmbouter about 5 years ago
- Status changed from POST to MODIFIED
Applied in changeset pulpcore|513952bde418e8370471a21814c1bcaffd36fef1.
Added by bmbouter about 5 years ago
Revision df2f3528 | View on GitHub
Set PULP_SETTINGS
pulpcore recently got a change where /etc/pulp/settings.py was no longer hard coded. This causes the installer to set that same path in various places.
Updated by bmbouter about 5 years ago
- Status changed from MODIFIED to POST
PR available at: https://github.com/pulp/pulplift/pull/51
Updated by bmbouter about 5 years ago
- Status changed from POST to MODIFIED
Applied in changeset pulplift|df2f35282acb41d4c246a833cd5dc5b4b28c91d6.
Added by Mike DePaulo about 5 years ago
Revision 072b830e | View on GitHub
Problem: CI with old plugin-template fails
due to PULP_SETTINGS being unset for multiple commands like django-admin.
Solution: Set PULP_SETTINGS in .travis.yml
re: #5560 PULP_SETTINGS environment variable does not work https://pulp.plan.io/issues/5560
[noissue]
Updated by bmbouter almost 5 years ago
- Status changed from MODIFIED to CLOSED - CURRENTRELEASE
.
Containers: Adds the PULP_SETTINGS Env Var
Please merge before PR: https://github.com/pulp/pulpcore/pull/342
re: #5560 PULP_SETTINGS environment variable does not work https://pulp.plan.io/issues/5560
[noissue]