Bug 645580 - repo sync cancel fails with exception in the log for some repos
Summary: repo sync cancel fails with exception in the log for some repos
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Pulp
Classification: Retired
Component: z_other
Version: unspecified
Hardware: Unspecified
OS: Unspecified
medium
medium
Target Milestone: ---
: ---
Assignee: Jay Dobies
QA Contact: Jason Connor
URL:
Whiteboard:
Depends On:
Blocks: verified-to-close
TreeView+ depends on / blocked
 
Reported: 2010-10-21 20:53 UTC by Preethi Thomas
Modified: 2014-03-31 01:38 UTC (History)
2 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2011-08-16 14:20:25 UTC
Embargoed:


Attachments (Terms of Use)

Description Preethi Thomas 2010-10-21 20:53:38 UTC
Description of problem:

I have a repo with the feed http://download.fedora.redhat.com/pub/fedora/linux/updates/13/x86_64/

Label              	f13_x86_64_update        
Name               	f13-repo--arch=x86_64    
Feed               	{u'url': u'http://download.fedora.redhat.com/pub/fedora/linux/updates/13/x86_64/', u'type': u'yum', u'supported_types': [u'yum', u'local', u'rhn']}
Arch               	noarch                   
Sync Schedule      	None                     
Packages           	7237                     
Files              	0         

running cancel_sync fails on the repo


[root@preethi ~]# pulp-admin repo cancel_sync --id=f13_x86_64_update 
Sync for repository f13_x86_64_update canceled



I see this in the pulp.log

2010-10-21 16:53:08,865 [ERROR][Thread-1] deliver_exception() @ thread.py:229 - Failed to deliver exception CancelException to thread[140270139725584]: Thread is not active

and 

[root@preethi ~]# pulp-admin repo status --id=f13_x86_64_update
+------------------------------------------+
        Status for f13_x86_64_update
+------------------------------------------+
Repository: f13_x86_64_update
Number of Packages: 7237
Last Sync: 2010-10-20 18:02:42.587000
Currently syncing: progress unknown

Comment 1 Jay Dobies 2011-01-07 14:07:06 UTC
commit	2b39885aa2f17f00418d9922a94ca79c9176b1e0
tree	e4c5e078c76d03c0912c206fc2c4f1c4aae0a0aa


645580 - Reworked thread logic to take into account the three state
possibilities.

See in code comments for more information.


src/pulp/server/tasking/queue/fifo.py
src/pulp/server/tasking/queue/thread.py
src/pulp/server/tasking/task.py

Comment 2 Jay Dobies 2011-01-07 19:21:57 UTC
Fixed in build 0.122.

Comment 3 Preethi Thomas 2011-01-10 18:37:56 UTC
verified
[root@preethi ~]# rpm -q pulp
pulp-0.0.122-1.fc14.noarch

[root@preethi ~]# pulp-admin -u admin -p admin repo sync  --id=f14-updates -F
Sync for repository f14-updates started
You can safely CTRL+C this current command and it will continue
[=                                                 ] 2% (312 of 9935 pkgs)
Sync: Canceled
Warning: 11 errors occurred



[root@preethi ~]#  pulp-admin -u admin -p admin repo cancel_sync --id=f14-updates 
Sync for repository f14-updates canceled

[root@preethi ~]#  pulp-admin -u admin -p admin repo status --id=f14-updates+------------------------------------------+
           Status for f14-updates
+------------------------------------------+
Repository: f14-updates
Number of Packages: 5280
Last Sync: 2011-01-10 12:34:58.657000

Comment 4 Preethi Thomas 2011-08-16 14:20:25 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.