Description of problem:
There are 2 commits in spacewalk master for reposync/cdnsync downloader:
- fixing maximal retry count if there is more mirrors than default retries - 3
- catching KeyboardException preventing some ugly traceback during downloading
These commits have incorrectly BZ1446198 in their message but they are not mentioned in BZ1446198.
Switching to MODIFIED because these commits are already in spacewalk.
Tested on spacewalk-backend-2.5.3-149. As there is no real bug or reproducer present, I did just a bit of manual testing based on info from Jan. First commit just fixes an obvious mistake in the code, the other one improves interrupt handling in the package downloading phase of cdn-sync. When Ctrl+C'ed, cdn-sync now finishes all remaining transfers before exiting. In case of network connectivity loss (simulated with "iptables -A OUTPUT -p tcp --destination-port 443 -j DROP") it is still possible to interrupt the process, but it takes two Ctrl+C's.
Re-verified with spacewalk-backend-2.5.3-151.
Tested on cdn-sync and spacewalk-repo-sync:
1) Sanity - the tools work as expected, checked by automation
2) CTRL+C now doesn't produce ugly traceback and waits for open connections to close - valid for "download" phases, but for example after "<time> Repo URL: <URL>" output, CTRL+C results in
^CTraceback (most recent call last):
File "/usr/lib/python2.6/site-packages/urlgrabber/grabber.py", line 1603, in _progress_update
def _progress_update(self, download_total, downloaded, upload_total, uploaded):
Process has been interrupted.
... which is not subject of this BZ.
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.
For information on the advisory, and where to find the updated
files, follow the link below.
If the solution does not work for you, open a new bug report.