Project

Profile

Help

Issue #2111

closed

pulp streamer keep proxy for all repos

Added by behi_1370 over 7 years ago. Updated almost 5 years 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!!

Actions #1

Updated by amacdona@redhat.com over 7 years ago

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

Updated by mhrivnak over 7 years ago

  • Sprint Candidate changed from No to Yes
Actions #3

Updated by jcline@redhat.com over 7 years ago

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

Updated by mhrivnak over 7 years ago

  • Sprint/Milestone set to 24
Actions #8

Updated by mhrivnak over 7 years ago

  • Sprint/Milestone changed from 24 to 25
Actions #9

Updated by jcline@redhat.com over 7 years ago

  • Status changed from ASSIGNED to POST

Added by Jeremy Cline over 7 years ago

Revision 4fe7327c | View on GitHub

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

Actions #10

Updated by Anonymous over 7 years ago

  • Status changed from POST to MODIFIED
  • % Done changed from 0 to 100
Actions #11

Updated by jcline@redhat.com over 7 years ago

  • Project changed from Pulp to Nectar
Actions #12

Updated by jcline@redhat.com over 7 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).

Actions #13

Updated by jcline@redhat.com over 7 years ago

  • Target Release - Nectar set to 1.5.3
Actions #14

Updated by semyers over 7 years ago

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

Added by ipanova@redhat.com over 7 years ago

Revision 5e979c7a | View on GitHub

Update python-nectar external_deps.

Actions #15

Updated by jcline@redhat.com over 7 years ago

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

Actions #16

Updated by semyers over 7 years ago

  • Platform Release set to 2.10.1
Actions #17

Updated by semyers over 7 years ago

  • Status changed from MODIFIED to 5
Actions #18

Updated by semyers over 7 years ago

  • Status changed from 5 to CLOSED - CURRENTRELEASE
Actions #19

Updated by bmbouter about 6 years ago

  • Sprint set to Sprint 7
Actions #20

Updated by bmbouter about 6 years ago

  • Sprint/Milestone deleted (25)
Actions #21

Updated by bmbouter almost 5 years ago

  • Tags Pulp 2 added

Also available in: Atom PDF