Project

Profile

Help

Refactor #114

closed

pulp/server/pulp/server/initialization.py unnecessarily imports kombu.transport.qpid

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

Status:
CLOSED - CURRENTRELEASE
Priority:
Low
Assignee:
Category:
-
Sprint/Milestone:
-
Start date:
Due date:
% Done:

100%

Estimated time:
Platform Release:
2.7.0
Groomed:
Yes
Sprint Candidate:
Yes
Tags:
Pulp 2
Sprint:
March 2015
Quarter:

Description

TODO:
1. Remove the comment and import line here [0].
2. Run tests and fix any test failures.

Pulp used to have a workaround whereby kombu.transport.qpid would have to be imported early on in the Pulp initialization process. This was required so that monkey patches were applied to class objects before execution actually began. That is no longer required, and this should be removed.

[0]: https://github.com/pulp/pulp/blob/cc8ec0cae37a38793d7ebc784e83bd969e6e585a/server/pulp/server/initialization.py#L31-33

Actions #1

Updated by bmbouter about 9 years ago

  • Tags Sprint Candidate added
Actions #2

Updated by cduryee about 9 years ago

Good catch, thanks for remembering to remove this:)

Actions #3

Updated by mhrivnak about 9 years ago

  • Priority changed from Normal to Low
Actions #4

Updated by bmbouter about 9 years ago

  • Tags Groomed added
Actions #5

Updated by rbarlow about 9 years ago

  • Status changed from NEW to ASSIGNED
  • Assignee set to rbarlow
Actions #6

Updated by rbarlow about 9 years ago

  • Platform Release set to master

Added by rbarlow about 9 years ago

Revision d83d78f2 | View on GitHub

Remove an unnecessary import.

https://pulp.plan.io/issues/114

closes #114

Added by rbarlow about 9 years ago

Revision d83d78f2 | View on GitHub

Remove an unnecessary import.

https://pulp.plan.io/issues/114

closes #114

Actions #7

Updated by rbarlow about 9 years ago

  • Status changed from ASSIGNED to POST
  • Sprint/Milestone set to 14
Actions #8

Updated by rbarlow about 9 years ago

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

Updated by rbarlow about 9 years ago

  • Platform Release changed from master to 2.7.0
Actions #10

Updated by bmbouter almost 9 years ago

  • Groomed set to Yes
  • Tags deleted (Groomed)
Actions #11

Updated by bmbouter almost 9 years ago

  • Sprint Candidate set to Yes
  • Tags deleted (Sprint Candidate)
Actions #12

Updated by dkliban@redhat.com almost 9 years ago

  • Status changed from MODIFIED to 5
Actions #13

Updated by Skullman almost 9 years ago

  • Status changed from 5 to 6
Actions #15

Updated by rbarlow over 8 years ago

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

Updated by bmbouter about 6 years ago

  • Sprint set to March 2015
Actions #18

Updated by bmbouter about 6 years ago

  • Sprint/Milestone deleted (14)
Actions #19

Updated by bmbouter about 5 years ago

  • Tags Pulp 2 added

Also available in: Atom PDF