Bug 669435 - Setting a repo sync timeout in milliseconds or microseconds doesn't work
Summary: Setting a repo sync timeout in milliseconds or microseconds doesn't work
Keywords:
Status: CLOSED NOTABUG
Alias: None
Product: Pulp
Classification: Retired
Component: z_other
Version: unspecified
Hardware: Unspecified
OS: Unspecified
low
medium
Target Milestone: ---
: ---
Assignee: Pradeep Kilambi
QA Contact: Preethi Thomas
URL:
Whiteboard:
Depends On:
Blocks: 673053
TreeView+ depends on / blocked
 
Reported: 2011-01-13 16:55 UTC by Jay Dobies
Modified: 2011-07-15 19:30 UTC (History)
2 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2011-03-25 06:39:21 UTC
Embargoed:


Attachments (Terms of Use)

Description Jay Dobies 2011-01-13 16:55:55 UTC
I'm not sure we could even kill a thread that quickly given the way our tasking framework works. So while we support from an interface perspective the ability to specify a timeout in milliseconds or microseconds, I don't think we could ever enforce that.

For example, the following doesn't indicate a timeout took place. But I can't see how it'd be possible that the sync normally completed in 20 milliseconds.

$ pulp-admin repo sync --id pulp-f14-64 --timeout milliseconds:20
Sync for repository pulp-f14-64 started
Use "repo status" to check on the progress

$ pulp-admin repo status --id pulp-f14-64 
+------------------------------------------+
           Status for pulp-f14-64
+------------------------------------------+
Repository: pulp-f14-64
Number of Packages: 10
Last Sync: 2011-01-13 11:51:53.243000

Comment 1 Sayli Karmarkar 2011-03-25 06:39:21 UTC
$ sudo pulp-admin repo sync --id=test1 --timeout=seconds:-22 --help
Usage: pulp-admin <options> repo sync <options>

Options:
  -h, --help         show this help message and exit
  --id=ID            repository id (required)
  --timeout=TIMEOUT  sync timeout in <units>:<value> format (e.g. hours:2
                     valid units: seconds, minutes, hours, days, weeks
  --no-packages      skip packages from the sync process
  --no-errata        skip errata from the sync process
  --no-distribution  skip distributions from the sync process
  -F, --foreground   synchronize repository in the foreground

As milliseconds and microseconds are removed from valid units in UG and help text for repo sync, this bug no longer exist.


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