Project

Profile

Help

Task #3696

Story #3693: Lazy for Pulp3

Add the streamer's settings to settings.yaml in the content app area

Added by dkliban@redhat.com about 2 years ago. Updated 7 months ago.

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

100%

Estimated time:
Platform Release:
Groomed:
Yes
Sprint Candidate:
Yes
Tags:
Sprint:
Sprint 41

Description

These settings would be added to the settings.py and server.yaml file with defaults.

redirect_host: The hostname to use when redirecting the client. This is required when using lazy.
redirect_port: The port to use when redirecting the client. This is required when using lazy.
redirect_prefix_path: The prefix path to prepend the to the URL when redirecting. This allows you to reroot squid. Defaults to None.
https_retrieval: If true, use https as the protocol the client should use, otherwise http. Defaults to True.


Related issues

Blocks Pulp - Task #3697: Have Pulp do a simple redirect (no signing) to squid when the content app needs to serve a RemoteArtifactCLOSED - CURRENTRELEASE

<a title="Actions" class="icon-only icon-actions js-contextmenu" href="#">Actions</a>

Associated revisions

Revision c35bba60 View on GitHub
Added by jortel@redhat.com almost 2 years ago

Support redirect in the content application. closes #3696, #3697

Revision c35bba60 View on GitHub
Added by jortel@redhat.com almost 2 years ago

Support redirect in the content application. closes #3696, #3697

History

#1 Updated by bmbouter about 2 years ago

  • Subject changed from Add the squid settings to settings.yaml in the content app area to Add the streamer's settings to settings.yaml in the content app area

squid's config is read by squid.conf and is in another story.

#2 Updated by bmbouter about 2 years ago

What exactly are the settings we need? I think this ticket still needs that level of planning.

#3 Updated by dkliban@redhat.com about 2 years ago

The following settings from pulp 2 are applicable to pulp 3.

[lazy]

  1. redirect_host:
  2. redirect_port:
  3. redirect_path:
  4. https_retrieval: true

#4 Updated by bmbouter about 2 years ago

  • Description updated (diff)

I agree those are the right settings. I'm adding them to the description along with some descriptions.

I renamed redirect_path to redirect_prefix_path because I thought it more clearly called out that it's only the prefix. I believe that's correct.

I want to retell what I think https_retrieval does for the users. It allows them to force https or http to match how they've configured their squid. It's a global setting which is kind of unfortunate because if you have some clients that can't support https you have to disable https globally. At some point in the future we may consider an option that would allow this setting at a distribution level.

Also for https_retrieval, we can't rely on "use the same protocol the client used to contact Pulp with" because when deploying Pulp behind a reverse proxy that provides TLS termination (for example) would always appear to pulp as http when the client is actually requesting https.

#5 Updated by bmbouter almost 2 years ago

  • Description updated (diff)
  • Groomed changed from No to Yes
  • Sprint Candidate changed from No to Yes

Added a default for https_retrieval.

#6 Updated by dkliban@redhat.com almost 2 years ago

  • Sprint set to Sprint 41

#7 Updated by dkliban@redhat.com almost 2 years ago

  • Blocks Task #3697: Have Pulp do a simple redirect (no signing) to squid when the content app needs to serve a RemoteArtifact added

#8 Updated by jortel@redhat.com almost 2 years ago

  • Status changed from NEW to ASSIGNED
  • Assignee set to jortel@redhat.com

The redirect settings pertain to the content app and should be added under the existing CONTENT section.

#9 Updated by jortel@redhat.com almost 2 years ago

  • Status changed from ASSIGNED to POST

#10 Updated by jortel@redhat.com almost 2 years ago

  • Status changed from POST to MODIFIED
  • % Done changed from 0 to 100

#11 Updated by daviddavis about 1 year ago

  • Sprint/Milestone set to 3.0.0

#12 Updated by bmbouter about 1 year ago

  • Tags deleted (Pulp 3)

#13 Updated by bmbouter 7 months ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE

Please register to edit this issue

Also available in: Atom PDF