Project

Profile

Help

Issue #711

python-kombu does not close() connections properly

Added by bmbouter over 6 years ago. Updated over 2 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:)

Associated revisions

Revision 972264db View on GitHub
Added by bmbouter over 6 years ago

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

Revision 972264db View on GitHub
Added by bmbouter over 6 years ago

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

History

#1 Updated by bmbouter over 6 years ago

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

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

#2 Updated by bmbouter over 6 years ago

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

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

#3 Updated by bmbouter over 6 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 +

#4 Updated by bmbouter over 6 years ago

  • Severity set to 1. Low
  • Platform Release set to master

#5 Updated by cduryee over 6 years ago

  • Description updated (diff)

#6 Updated by bmbouter over 6 years ago

  • Platform Release changed from master to 2.7.0

#7 Updated by dkliban@redhat.com over 6 years ago

  • Status changed from MODIFIED to 5

#8 Updated by pthomas@redhat.com over 6 years ago

  • Status changed from 5 to 6

Moving to verified

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

#9 Updated by amacdona@redhat.com almost 6 years ago

  • Status changed from 6 to CLOSED - CURRENTRELEASE

#11 Updated by bmbouter over 2 years ago

  • Tags Pulp 2 added

Please register to edit this issue

Also available in: Atom PDF