Bug 1195434 - Event Listeners API: repo.publish.finish triggered before related task has finished executing
Summary: Event Listeners API: repo.publish.finish triggered before related task has fi...
Keywords:
Status: CLOSED UPSTREAM
Alias: None
Product: Pulp
Classification: Retired
Component: API/integration
Version: 2.6 Beta
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
: ---
Assignee: pulp-bugs
QA Contact: pulp-qe-list
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2015-02-23 20:48 UTC by mkovacik
Modified: 2015-02-28 23:25 UTC (History)
0 users

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2015-02-28 23:25:49 UTC
Embargoed:


Attachments (Terms of Use)
Test case screen log (28.25 KB, text/plain)
2015-02-23 20:48 UTC, mkovacik
no flags Details


Links
System ID Private Priority Status Summary Last Updated
Pulp Redmine 713 0 None None None Never

Description mkovacik 2015-02-23 20:48:00 UTC
Created attachment 994495 [details]
Test case screen log

Description of problem:
The repo.publish.finish http event listener gets triggered before the repo finishes publishing.
The related publish task.finish_time is always later than the request.time posted to requestb.in.

Version-Release number of selected component (if applicable):
pulp-server-2.6.0-0.7.beta.fc21.noarch


How reproducible:
Always

Steps to Reproduce:
0. set up an rpm repo with feed
1. create an http event listener:
  - reports to a bin (http://requestb.in/<bin_id>)
  - types: ['repo.sync.finish', 'repo.publish.finish']
2. update the event listener to types: ['repo.publish.finish']
3. publish repo yum_distributor
4. wait for spawned tasks to finish
5. collect requests posted to the bin

Actual results:
request.time < publish_task.finish_time

Expected results:
request.time >= publish_task.finish_time

Additional info:
Automated test case: https://github.com/RedHatQE/pulp-automation/blob/master/tests/general_tests/test_23_event_listener.py ; EventListenerTest.test_05_repo_publish_finish

Comment 1 Brian Bouterse 2015-02-28 23:25:49 UTC
Moved to https://pulp.plan.io/issues/713


Note You need to log in before you can comment on or make changes to this bug.