Actions
Task #5629
closed
Introduce CONTENT_ORIGIN as a required setting
Start date:
Due date:
% Done:
100%
Estimated time:
(Total: 0:00 h)
Platform Release:
Groomed:
No
Sprint Candidate:
No
Tags:
Sprint:
Sprint 61
Quarter:
Description
As discussed on the mailing list we need a new required setting because:
1. plugins like pulp_docker require this CONTENT_HOST to be set, but others may not, so this setting could become a dividing line for plugins that want it set versus those that don't.
2. Clients are semantically unsure how to handle responses for one Pulp server versus another. This setting changes the client's responsibilities (relative vs absolute URL handling).
3. Relative urls are not as usable as absolute urls.
Solution¶
Add a setting named CONTENT_ORIGIN that is required.
Actions
.
Set content_origin automatically
The plugin_template sets
content_originas http://$(hostname):24816`. This aligns with the current way services are tested on Travishttps://pulp.plan.io/issues/5629 re #5629