core-image-publisher

Merge lp:~cprov/core-image-publisher/fixing-namespace-conflict into lp:core-image-publisher

  1. fixing-namespace-conflict
  2. Merge into trunk
Proposed by Celso Providelo
Status: Merged
Approved by: Thomi Richards
Approved revision: 9
Merged at revision: 9
Proposed branch: lp:~cprov/core-image-publisher/fixing-namespace-conflict
Merge into: lp:core-image-publisher
Diff against target: 16 lines (+2/-2)
1 file modified
core_image_publisher/__init__.py (+2/-2)
To merge this branch: bzr merge lp:~cprov/core-image-publisher/fixing-namespace-conflict
Reviewer Review Type Date Requested Status
Thomi Richards (community) Approve
Review via email: mp+254338@code.launchpad.net

Commit message

Fixing namespace conflict in main(), preventing main script to run in production.

Description of the change

Fixing namespace conflict with local 'worker' variable in main and the core_image_publisher.worker module.

To post a comment you must log in.
Revision history for this message
Thomi Richards (thomir-deactivatedaccount) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'core_image_publisher/__init__.py'
2--- core_image_publisher/__init__.py 2015-03-26 21:02:50 +0000
3+++ core_image_publisher/__init__.py 2015-03-27 03:12:45 +0000
4@@ -84,10 +84,10 @@
5 connection = create_connection_from_config(config)
6 try:
7 with connection:
8- worker = CoreImagePublisherQueueMonitor(
9+ monitor = CoreImagePublisherQueueMonitor(
10 connection,
11 worker.logging_worker
12 )
13- worker.run()
14+ monitor.run()
15 except KeyboardInterrupt:
16 print("Bye!")

Subscribers

People subscribed via source and target branches