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 over 9 years ago

  • Tags Sprint Candidate added
Actions #2

Updated by cduryee over 9 years ago

Good catch, thanks for remembering to remove this:)

Actions #3

Updated by mhrivnak over 9 years ago

  • Priority changed from Normal to Low
Actions #4

Updated by bmbouter over 9 years ago

  • Tags Groomed added
Actions #5

Updated by rbarlow over 9 years ago

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

Updated by rbarlow over 9 years ago

  • Platform Release set to master

Added by rbarlow over 9 years ago

Revision d83d78f2 | View on GitHub

Remove an unnecessary import.

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

closes #114

Added by rbarlow over 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 over 9 years ago

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

Updated by rbarlow over 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 about 9 years ago

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

Updated by bmbouter about 9 years ago

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

Updated by dkliban@redhat.com about 9 years ago

  • Status changed from MODIFIED to 5
Actions #13

Updated by Skullman about 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 over 6 years ago

  • Sprint set to March 2015
Actions #18

Updated by bmbouter over 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