Project

Profile

Help

Issue #2111

pulp streamer keep proxy for all repos

Added by behi_1370 over 4 years ago. Updated over 1 year ago.

Status:
CLOSED - CURRENTRELEASE
Priority:
High
Start date:
Due date:
Estimated time:
Severity:
1. Low
Version - Nectar:
Platform Release:
2.10.1
Target Release - Nectar:
1.5.3
OS:
CentOS 7
Triaged:
Yes
Groomed:
No
Sprint Candidate:
Yes
Tags:
Pulp 2
Sprint:
Sprint 7
Quarter:

Description

after clients downloads rpms from a repository which has "on demand" download policy and also has "proxy" settings, all downloads from other repositories that don't have any proxy setting also go through proxy server until restarting pulp_streamer service!!

Associated revisions

Revision 4fe7327c View on GitHub
Added by Jeremy Cline over 4 years ago

Change how Nectar configures requests to be thread-safe

Previously, Nectar set the configuration at a session level. This didn't cause problems because Nectar is only ever used to download with one set of settings. However, the pulp_streamer has to change the configuration on a request-by-request basis and it needs to use a session to pool connections. This leads to trouble when attempting to update the configuration since Nectar changes it for all requests rather than a specific one.

This patch instead adds a method to translate a Nectar configuration to a set of requests kwargs that work with the requests.request API. This allows the streamer to stash its session instance between instances of a Nectar HTTPThreadedDownloader without corruption.

fixes #2111

Revision 5e979c7a View on GitHub
Added by ipanova@redhat.com about 4 years ago

Update python-nectar external_deps.

History

#1 Updated by amacdona@redhat.com over 4 years ago

  • Priority changed from Normal to High
  • Severity changed from 2. Medium to 1. Low
  • Triaged changed from No to Yes

#2 Updated by mhrivnak over 4 years ago

  • Sprint Candidate changed from No to Yes

#3 Updated by jcline@redhat.com over 4 years ago

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

#6 Updated by mhrivnak over 4 years ago

  • Sprint/Milestone set to 24

#8 Updated by mhrivnak over 4 years ago

  • Sprint/Milestone changed from 24 to 25

#9 Updated by jcline@redhat.com over 4 years ago

  • Status changed from ASSIGNED to POST

#10 Updated by Anonymous over 4 years ago

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

#11 Updated by jcline@redhat.com over 4 years ago

  • Project changed from Pulp to Nectar

#12 Updated by jcline@redhat.com over 4 years ago

I can't find how to add a new target release, but I think the next release could be 1.5.3. I did add a new public method to the HTTPThreadedDownloader, but the behavior should be the same (sans bug).

#13 Updated by jcline@redhat.com over 4 years ago

  • Target Release - Nectar set to 1.5.3

#14 Updated by semyers about 4 years ago

External deps need to be updated to include this in a future pulp release.

#15 Updated by jcline@redhat.com about 4 years ago

Ina kindly took care of adding the new build to the deps yesterday, I've associated the commit

#16 Updated by semyers about 4 years ago

  • Platform Release set to 2.10.1

#17 Updated by semyers about 4 years ago

  • Status changed from MODIFIED to 5

#18 Updated by semyers about 4 years ago

  • Status changed from 5 to CLOSED - CURRENTRELEASE

#19 Updated by bmbouter over 2 years ago

  • Sprint set to Sprint 7

#20 Updated by bmbouter over 2 years ago

  • Sprint/Milestone deleted (25)

#21 Updated by bmbouter over 1 year ago

  • Tags Pulp 2 added

Please register to edit this issue

Also available in: Atom PDF