Project

Profile

Help

Issue #711

closed

python-kombu does not close() connections properly

Added by bmbouter about 9 years ago. Updated about 5 years ago.

Status:
CLOSED - CURRENTRELEASE
Priority:
Normal
Assignee:
Category:
-
Sprint/Milestone:
-
Start date:
Due date:
Estimated time:
Severity:
1. Low
Version:
Master
Platform Release:
2.7.0
OS:
Triaged:
Yes
Groomed:
No
Sprint Candidate:
No
Tags:
Pulp 2
Sprint:
Quarter:

Description

Eventually, opening and closing the connection will result in the system running out of file handles.

This has been filed in upstream kombu, but this BZ is designed to track the inclusion of that patch into the python-kombu that we carry until the fix is released.

+ This bug was cloned from Bugzilla Bug #1195361 +

QE note: the test for this is simply to perform a sync of a large repo like RHEL server. If Pulp does not run out of file handles and crash then we are in good shape and this bug is no longer present:)

Actions #1

Updated by bmbouter about 9 years ago

Here is the upstream issue: https://github.com/celery/kombu/issues/455

+ This comment was cloned from Bugzilla #1195361 comment 1 +

Actions #2

Updated by bmbouter about 9 years ago

Downstream backport PR available at: https://github.com/pulp/pulp/pull/1672

+ This comment was cloned from Bugzilla #1195361 comment 2 +

Actions #3

Updated by bmbouter about 9 years ago

Merged to master and rebuilt in koji as python-kombu-3.0.24-6.pulp

http://koji.katello.org/koji/packageinfo?packageID=665

+ This comment was cloned from Bugzilla #1195361 comment 3 +

Actions #4

Updated by bmbouter about 9 years ago

  • Severity set to 1. Low
  • Platform Release set to master
Actions #5

Updated by cduryee about 9 years ago

  • Description updated (diff)
Actions #6

Updated by bmbouter about 9 years ago

  • Platform Release changed from master to 2.7.0

Added by bmbouter almost 9 years ago

Revision 972264db | View on GitHub

Fixes 917 and 1006 by stopping an thread that should have exited

https://pulp.plan.io/issues/917 https://pulp.plan.io/issues/1006

fixes #917 fixes #1006 re #711

Added by bmbouter almost 9 years ago

Revision 972264db | View on GitHub

Fixes 917 and 1006 by stopping an thread that should have exited

https://pulp.plan.io/issues/917 https://pulp.plan.io/issues/1006

fixes #917 fixes #1006 re #711

Actions #7

Updated by dkliban@redhat.com almost 9 years ago

  • Status changed from MODIFIED to 5
Actions #8

Updated by pthomas@redhat.com over 8 years ago

  • Status changed from 5 to 6

Moving to verified

Synced multiple large repos (RHEL, CentOS) without any issues.

Actions #9

Updated by amacdona@redhat.com over 8 years ago

  • Status changed from 6 to CLOSED - CURRENTRELEASE
Actions #11

Updated by bmbouter about 5 years ago

  • Tags Pulp 2 added

Also available in: Atom PDF