Bug 680520 - missing sync id's
Summary: missing sync id's
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Pulp
Classification: Retired
Component: z_other
Version: unspecified
Hardware: Unspecified
OS: Unspecified
medium
medium
Target Milestone: ---
: Sprint 22
Assignee: Jason Connor
QA Contact: Preethi Thomas
URL:
Whiteboard:
Depends On: 680503
Blocks: verified-to-close 683939 684061
TreeView+ depends on / blocked
 
Reported: 2011-02-25 19:05 UTC by Shannon Hughes
Modified: 2011-08-16 12:21 UTC (History)
2 users (show)

Fixed In Version:
Clone Of: 680503
Environment:
Last Closed: 2011-08-16 12:09:14 UTC
Embargoed:


Attachments (Terms of Use)

Description Shannon Hughes 2011-02-25 19:05:44 UTC
calling the sync api and getting back a sync id then immediately using it in a sync status call and receiving error message that the sync id does not exist:

Started GET "/sync_management/status/?_=1298656756138&repo_id=kgems-1234777&sync_id=ed50d051-4108-11e0-89b7-00219b3b5088" for 10.11.11.1 at Fri Feb 25 12:59:03 -0500 2011
  Processing by SyncManagementController#status as JSON
  Parameters: {"repo_id"=>"kgems-1234777", "sync_id"=>"ed50d051-4108-11e0-89b7-00219b3b5088", "_"=>"1298656756138"}
Resource GET request: /pulp/api/repositories/kgems-1234777/sync/ed50d051-4108-11e0-89b7-00219b3b5088/
Error parsing the body: 737: unexpected token at '"No sync with id ed50d051-4108-11e0-89b7-00219b3b5088 found"'

also, would be nice to have error message return in JSON format

Comment 1 Mike McCune 2011-03-09 17:06:19 UTC
FYI:

This is set to depend on https://bugzilla.redhat.com/show_bug.cgi?id=680503

Comment 2 Jason Connor 2011-04-13 16:24:28 UTC
I changed the underlying locking around the task queue storage, I believe that it has addressed the bug, but I wasn't able to reproduce it before or after the fix, so I'm not really sure. Marking as modified anyway and crossing my fingers...

Comment 3 Jay Dobies 2011-04-13 19:22:52 UTC
Fixed in build 0.163.

Comment 4 Preethi Thomas 2011-04-14 17:59:35 UTC
moving to verified

[root@preethi ~]# rpm -q pulp
pulp-0.0.163-1.fc14.noarch

[root@preethi ~]# curl -k -u admin:admin --request POST --data '{}' --header 'accept: application/json' --header 'content-type: application/json' https://localhost/pulp/api/repositories/f14-original/sync//
{"scheduled_time": {"$date": 1302790191909}, "exception": null, "status_path": "/pulp/api/repositories/f14-original/sync/64a13cee-66c2-11e0-b525-002564a85a58/", "finish_time": null, "start_time": "1302804591", "traceback": null, "method_name": "_sync", "state": "running", "result": null, "progress": null, "id": "64a13cee-66c2-11e0-b525-002564a85a58"}[root@preethi ~]# 
[root@preethi ~]# 
[root@preethi ~]# curl -k -u admin:admin https://localhost/pulp/api/repositories/f14-original/sync/64a13cee-66c2-11e0-b525-002564a85a58/
{"scheduled_time": {"$date": 1302790191909}, "exception": null, "status_path": "/pulp/api/repositories/f14-original/sync/64a13cee-66c2-11e0-b525-002564a85a58/", "finish_time": null, "start_time": "1302804591", "traceback": null, "method_name": "_sync", "state": "running", "result": null, "progress": {"status": "FINISHED", "num_success": 2773, "size_total": 3273644559, "num_download": 1, "item_name": null, "items_left": 0, "items_total": 2773, "step": "Running Createrepo", "size_left": 0, "details": {"tree_file": {"num_success": 7, "total_count": 7, "items_left": 0, "size_left": 0, "total_size_bytes": 0, "num_error": 0}, "rpm": {"num_success": 2766, "total_count": 2766, "items_left": 0, "size_left": 0, "total_size_bytes": 3273644559, "num_error": 0}}, "error_details": [], "num_error": 0}, "id": "64a13cee-66c2-11e0-b525-002564a85a58"}[root@preethi ~]#

Comment 5 Preethi Thomas 2011-08-16 12:09:14 UTC
Closing with Community Release 15

pulp-0.0.223-4.

Comment 6 Preethi Thomas 2011-08-16 12:21:52 UTC
Closing with Community Release 15

pulp-0.0.223-4.


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