Project

Profile

Help

Issue #1635

closed

Pulp fails to reconnect to qpid after a disconnect

Added by dkliban@redhat.com about 8 years ago. Updated about 5 years ago.

Status:
CLOSED - CURRENTRELEASE
Priority:
High
Assignee:
Category:
-
Sprint/Milestone:
-
Start date:
Due date:
Estimated time:
Severity:
2. Medium
Version:
Platform Release:
2.8.0
OS:
Triaged:
Yes
Groomed:
No
Sprint Candidate:
No
Tags:
Pulp 2
Sprint:
Quarter:

Description

After python-kombu 3.0.33-1.pulp.fc23 got added to the deps, the reconnect support in Pulp broke.

Actions #1

Updated by pthomas@redhat.com about 8 years ago

  • Subject changed from Pulp fails to reconnect to qpid after a diconnect to Pulp fails to reconnect to qpid after a disconnect
Actions #2

Updated by jortel@redhat.com about 8 years ago

  • Triaged changed from No to Yes
Actions #3

Updated by bmbouter about 8 years ago

This is filed against upstream Qpid: https://issues.apache.org/jira/browse/QPID-7053

Actions #4

Updated by bmbouter about 8 years ago

  • Status changed from NEW to ASSIGNED
Actions #5

Updated by bmbouter about 8 years ago

This is also being tracked in python-kombu here: https://github.com/celery/kombu/issues/568

Actions #7

Updated by bmbouter about 8 years ago

I've submitted patches to upstream Qpid and upstream Kombu. These patches work together to resolve the problem. In my testing with these patches applied reconnect support works. Also pulp-smash agrees!

The Kombu patch[0] needs to be added into our python-kombu-3.0.33 and rebuilt in Koji.

[0]: https://patch-diff.githubusercontent.com/raw/celery/kombu/pull/569.patch

Actions #8

Updated by bmbouter about 8 years ago

Added by pcreech about 8 years ago

Revision fbeaa505 | View on GitHub

Add kombu patch for reconnect issue

Adding kombu patch for reconnect issue.

Patch is from https://github.com/celery/kombu/pull/569/

re #1635 https://pulp.plan.io/issues/1635

Added by pcreech about 8 years ago

Revision fbeaa505 | View on GitHub

Add kombu patch for reconnect issue

Adding kombu patch for reconnect issue.

Patch is from https://github.com/celery/kombu/pull/569/

re #1635 https://pulp.plan.io/issues/1635

Actions #9

Updated by bmbouter about 8 years ago

Built in MRG as python-qpid-0.30-9.

Also changes merged in upstream kombu to work with the newly published python-qpid library. That patch has been added to python-kombu we carry python-kombu-3.0.33-4.pulp. That has been built in koji.katello.org. Thanks pcreech.

Actions #10

Updated by bmbouter about 8 years ago

Added by bmbouter about 8 years ago

Revision 4bef5bb3 | View on GitHub

python-kombu-3.0.33-4.pulp is now included via external_deps.json

closes #1635 https://pulp.plan.io/issues/1635

Added by bmbouter about 8 years ago

Revision 4bef5bb3 | View on GitHub

python-kombu-3.0.33-4.pulp is now included via external_deps.json

closes #1635 https://pulp.plan.io/issues/1635

Actions #11

Updated by bmbouter about 8 years ago

PR available at: https://github.com/pulp/pulp/pull/2456

This PR cannot be merged until the necessary packages are available in

Actions #12

Updated by bmbouter about 8 years ago

  • Status changed from ASSIGNED to POST
Actions #13

Updated by bmbouter about 8 years ago

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

Updated by bmbouter about 8 years ago

Actions #15

Updated by dkliban@redhat.com about 8 years ago

  • Status changed from MODIFIED to 5
Actions #16

Updated by dkliban@redhat.com about 8 years ago

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

Updated by bmbouter about 5 years ago

  • Tags Pulp 2 added

Also available in: Atom PDF