lp:~matsubara/tarmac/bundle-lander

Branch merges

Merged into lp:~launchpad/tarmac/lp-tarmac at revision 384

Gary Poster (community): Approve on 2010-12-15

Diff: 834 lines (+661/-35)

7 files modified

tarmac/bin/commands.py (+20/-0)
tarmac/pidfile.py (+77/-0)
tarmac/plugins/bundlecommand.py (+215/-30)
tarmac/plugins/tests/test_bundlecommand.py (+280/-0)
tarmac/tests/mock.py (+17/-0)
tarmac/tests/test_commands.py (+21/-5)
tarmac/tests/test_pidfile.py (+31/-0)

api

Propose for merging

Related source package recipes

No recipes using this branch. (?)

Create packaging recipe

Related snap packages

Related bugs

Related blueprints

398. By Diogo Matsubara on 2010-12-15

merge from lp-tarmac and fix 1 conflict

397. By Diogo Matsubara on 2010-12-15

add tests to functions in the pidfile.py module.

396. By Diogo Matsubara on 2010-12-15

fix XXX in test_commmands.py and add test for running bundle command with lock file.

395. By Diogo Matsubara on 2010-12-15

add allowed_lock_period option to the config. make tests pass again.

394. By Diogo Matsubara on 2010-12-14

also log the lock file age and pid when the lock is younger than the allowed lock period

393. By Diogo Matsubara on 2010-12-14

implement lock file mechanism as suggested by Gary: lock file is a pid file, it allows a configuration option to be set so we know when tarmac is taking too long to end its run, and it shows the age of the pid file. Includes a modified version of the pidfile.py file from lib/canonical/lazr/pidfile.py

392. By Diogo Matsubara on 2010-12-14

update test_run_failure to show that emails were sent and the merge proposals were updated.

391. By Diogo Matsubara on 2010-12-14

fix format_result to include notification of attached log file only when sending email, not when commenting in MP.

390. By Diogo Matsubara on 2010-12-10

remove unnecessary XXX

389. By Diogo Matsubara on 2010-12-10

email and formatting is now done and tested