lp://staging/~vila/charms/precise/tarmac/cleanup-tests

Branch merges

Propose for merging

No branches dependent on this one.

Merged into lp://staging/~canonical-ci-engineering/charms/precise/tarmac/trunk at revision 58

Evan (community): Approve on 2014-11-14

Diff: 131 lines (+25/-25)

3 files modified

hooks/hooks.py (+1/-1)
templates/cron.tmpl (+1/-0)
unit_tests/test_hooks.py (+23/-24)

api

Related source package recipes

No recipes using this branch. (?)

Create packaging recipe

Related snap packages

Related bugs

Related blueprints

62. By Vincent Ladeuil on 2014-11-14

Hairy bootstrapping issue: when tarmac run the tests it needs USER to be set which is done in the cron template. So we need that fix to be available on tarmac to be able to land this fix.

61. By Vincent Ladeuil on 2014-11-14

Add ssh_dir to the common mocks or adding a new ssh test will target ~/.ssh and confuse testers ;)

60. By Vincent Ladeuil on 2014-11-14

Cleanup removing '%' usage in faour of .format(). Cleanup 'tmpl_data' in favour of 'expanded' as we care about the template expansion result.

59. By Vincent Ladeuil on 2014-11-14

Fix existing test failure: split() is better than split(' ').

58. By Vincent Ladeuil on 2014-11-10

Remove unused imports.

57. By Vincent Ladeuil on 2014-11-10

Don't hash hosts for ssh or we lose track of which ones are there.

56. By Evan on 2014-07-03

Cron is unhappy if it doesn't have an empty line at the end of the file.

55. By Evan on 2014-07-01

This greatly confuses juju.

54. By Evan on 2014-06-30

There's no need for env when we have an environment variables section.

53. By Evan on 2014-06-27

Typo.