##// END OF EJS Templates
svn: use more detailed logs/errors so exception_tracker can show it with details.
marcink -
r521:c9d42283 stable
parent child Browse files
Show More
@@ -356,7 +356,6 b' class SvnRemote(object):'
356 raise Exception(
356 raise Exception(
357 "Path %s is not a valid Subversion repository." % repo_path)
357 "Path %s is not a valid Subversion repository." % repo_path)
358
358
359 # TODO: johbo: URL checks ?
360 import subprocess
359 import subprocess
361 rdump = subprocess.Popen(
360 rdump = subprocess.Popen(
362 ['svnrdump', 'dump', '--non-interactive', src_url],
361 ['svnrdump', 'dump', '--non-interactive', src_url],
@@ -370,6 +369,7 b' class SvnRemote(object):'
370 rdump.wait()
369 rdump.wait()
371 load.wait()
370 load.wait()
372
371
372 log.debug('Return process ended with code: %s', rdump.returncode)
373 if rdump.returncode != 0:
373 if rdump.returncode != 0:
374 errors = rdump.stderr.read()
374 errors = rdump.stderr.read()
375 log.error('svnrdump dump failed: statuscode %s: message: %s',
375 log.error('svnrdump dump failed: statuscode %s: message: %s',
@@ -377,9 +377,12 b' class SvnRemote(object):'
377 reason = 'UNKNOWN'
377 reason = 'UNKNOWN'
378 if 'svnrdump: E230001:' in errors:
378 if 'svnrdump: E230001:' in errors:
379 reason = 'INVALID_CERTIFICATE'
379 reason = 'INVALID_CERTIFICATE'
380
381 if reason == 'UNKNOWN':
382 reason = 'UNKNOWN:{}'.format(errors)
380 raise Exception(
383 raise Exception(
381 'Failed to dump the remote repository from %s.' % src_url,
384 'Failed to dump the remote repository from %s. Reason:%s' % (
382 reason)
385 src_url, reason))
383 if load.returncode != 0:
386 if load.returncode != 0:
384 raise Exception(
387 raise Exception(
385 'Failed to load the dump of remote repository from %s.' %
388 'Failed to load the dump of remote repository from %s.' %
General Comments 0
You need to be logged in to leave comments. Login now