Bug#773642: jenkins-job-builder: deleting jobs is completly broken

2014-12-21 Thread Holger Levsen
package: jenkins-job-builder
severity: important
tags: patch upstream
version: 0.9.0-0.1

Hi,

trying to delete a job results in this:

root@hestia:/srv/jenkins/job-cfg# chroot /root/jessie/ su jenkins -c jenkins-
jobs delete kali_build_rolling_armel
INFO:root:Deleting jobs in [kali_build_rolling_armel]
Traceback (most recent call last):
  File /usr/bin/jenkins-jobs, line 10, in module
sys.exit(main())
  File /usr/share/jenkins-job-builder/jenkins_jobs/cmd.py, line 119, in main
execute(options, config)
  File /usr/share/jenkins-job-builder/jenkins_jobs/cmd.py, line 204, in 
execute
builder.delete_job(job, options.path)
  File /usr/share/jenkins-job-builder/jenkins_jobs/builder.py, line 611, in 
delete_job
self.load_files(fn)
  File /usr/share/jenkins-job-builder/jenkins_jobs/builder.py, line 576, in 
load_files
if os.path.isdir(path):
  File /usr/lib/python2.7/genericpath.py, line 41, in isdir
st = os.stat(s)
TypeError: coercing to Unicode: need string or buffer, NoneType found

The upstream commit https://git.openstack.org/cgit/openstack-infra/jenkins-
job-builder/commit/?id=b7ff37ca5dfa1e7387c636b8b0108404a0bf has the fix.

The workaround is easy (rm -r $job_directory), so I've only filed this with 
severity important, but I do think this should be fixed for jessie.


cheers,
Holger


signature.asc
Description: This is a digitally signed message part.


Bug#773642: jenkins-job-builder: deleting jobs is completly broken

2014-12-21 Thread Holger Levsen
On Sonntag, 21. Dezember 2014, Holger Levsen wrote:
 The workaround is easy (rm -r $job_directory), so I've only filed this with
 severity important, but I do think this should be fixed for jessie.

the workaround is actually slightly more complicated: rm -r $job_cfg.xml 
$job_directory  then a jenkins restart

still easy, but more annoying and less trivial. And a jenkins restart is super 
annoying, so I'm even more tempted to raise severity to serious...




signature.asc
Description: This is a digitally signed message part.