Project

Profile

Help

Task #5629

closed

Introduce CONTENT_ORIGIN as a required setting

Added by bmbouter almost 5 years ago. Updated almost 5 years ago.

Status:
CLOSED - CURRENTRELEASE
Priority:
Normal
Assignee:
Category:
-
Sprint/Milestone:
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.


Sub-issues 3 (0 open3 closed)

Story #5648: Add CONTENT_ORIGIN to the installerCLOSED - CURRENTRELEASE

Actions
Task #5649: Remove CONTENT_HOST from pulpcoreCLOSED - CURRENTRELEASE

Actions
Task #5686: Update places in Travis that call 'django-admin runserver'CLOSED - CURRENTRELEASEdaviddavis

Actions
Actions #1

Updated by daviddavis almost 5 years ago

  • Sprint set to Sprint 61

This is a 3.0 GA blocker. Adding to sprint.

Actions #2

Updated by bmbouter almost 5 years ago

  • Status changed from NEW to ASSIGNED
  • Assignee set to bmbouter
Actions #3

Updated by fao89 almost 5 years ago

  • Tracker changed from Issue to Task
  • % Done set to 0
Actions #4

Updated by bmbouter almost 5 years ago

  • Subject changed from Rename CONTENT_HOST to CONTENT_ORIGIN to Introduce CONTENT_ORIGIN as a required setting
  • Description updated (diff)
Actions #5

Updated by bmbouter almost 5 years ago

  • Status changed from ASSIGNED to NEW
  • Assignee deleted (bmbouter)
Actions #6

Updated by bmbouter almost 5 years ago

  • Subject changed from Introduce CONTENT_ORIGIN as a required setting to [Epic] Introduce CONTENT_ORIGIN as a required setting

Added by bmbouter almost 5 years ago

Revision 0ddeaf33 | View on GitHub

Set content_origin automatically

The plugin_template sets content_origin as http://$(hostname):24816`. This aligns with the current way services are tested on Travis

https://pulp.plan.io/issues/5629 re #5629

Added by Mike DePaulo almost 5 years ago

Revision c7ee477c | View on GitHub

Set content_origin to the 1st node's 1st address.

If not set by the user.

Limited scope (single-node clusters), but sufficient for minikube/k3s/our Travis CI. fixes: #5138 In pulp-operator, the content_origin should be set to the actual accessible service URL https://pulp.plan.io/issues/5138

re: #5629 [Epic] Introduce CONTENT_ORIGIN as a required setting https://pulp.plan.io/issues/5629

Actions #7

Updated by bmbouter almost 5 years ago

  • Subject changed from [Epic] Introduce CONTENT_ORIGIN as a required setting to Introduce CONTENT_ORIGIN as a required setting
Actions #8

Updated by daviddavis almost 5 years ago

  • Status changed from NEW to ASSIGNED
  • Assignee set to daviddavis

Added by bmbouter almost 5 years ago

Revision e027cd3f | View on GitHub

Introduce CONTENT_ORIGIN setting

This also adds a check that the user specified CONTENT_ORIGIN at startup. Pulp refuses to start if it is not set with an error message for the user.

Required PR: https://github.com/PulpQE/pulp-smash/pull/1227 Required PR: https://github.com/pulp/ansible-pulp/pull/185 Required PR: https://github.com/pulp/pulp_file/pull/303

https://pulp.plan.io/issues/5629 re #5629

Added by bmbouter almost 5 years ago

Revision 14e5cc31 | View on GitHub

Set CONTENT_ORIGIN settings

The Travis environment needs content_origin set as a setting.

This also updates the tests to expect Pulp's Distribution.base_url to provide absolute urls.

Required PR: https://github.com/pulp/pulpcore/pull/358 Required PR: https://github.com/PulpQE/pulp-smash/pull/1227 Required PR: https://github.com/pulp/ansible-pulp/pull/185

https://pulp.plan.io/issues/5629 re #5629

Added by daviddavis almost 5 years ago

Revision 3c35434f | View on GitHub

Removing the content_origin setting since pulp_operator sets it

ref #5629

Required PR: https://github.com/pulp/pulpcore/pull/358 Required PR: https://github.com/PulpQE/pulp-smash/pull/1227

Added by bmbouter almost 5 years ago

Revision e8a9176c | View on GitHub

Use CONTENT_ORIGIN instead of CONTENT_HOST

pulpcore changed the setting name from CONTENT_HOST to CONTENT_ORIGIN.

Required PR: https://github.com/pulp/pulpcore/pull/370 Required PR: https://github.com/PulpQE/pulp-smash/pull/1227

https://pulp.plan.io/issues/5629 re #5629

Actions #9

Updated by daviddavis almost 5 years ago

  • Status changed from ASSIGNED to MODIFIED

Added by bmbouter almost 5 years ago

Revision 74acaf3c | View on GitHub

Fix performance tests for CONTENT_ORIGIN

The performance tests now expect an absolute url which is provided thanks to the CONTENT_ORIGIN change.

https://pulp.plan.io/issues/5629 closes #5629

Actions #10

Updated by bmbouter almost 5 years ago

  • Status changed from MODIFIED to POST

bringing back to POST for one more PR: https://github.com/pulp/pulp_file/pull/310

Actions #11

Updated by bmbouter almost 5 years ago

  • Status changed from POST to MODIFIED
Actions #12

Updated by daviddavis almost 5 years ago

  • Status changed from MODIFIED to POST
Actions #13

Updated by daviddavis almost 5 years ago

  • Status changed from POST to MODIFIED
Actions #14

Updated by bmbouter almost 5 years ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE

Also available in: Atom PDF