lp:~matsubara/charms/trusty/jenkins-slave/trusty-jenkins-slave

Branch merges

Propose for merging

1 branch proposed for merging into this one.

No branches dependent on this one.

Merged into lp:~juju-qa/charms/trusty/jenkins-slave/trunk at revision 21

Curtis Hovey (community): Approve (code) on 2015-09-23

Diff: 70 lines (+18/-8)

5 files modified

hooks/slave-relation-changed (+13/-1)
hooks/slave-relation-joined (+1/-1)
hooks/start (+0/-3)
hooks/stop (+0/-3)
metadata.yaml (+4/-0)

api

Related source package recipes

No recipes using this branch. (?)

Create packaging recipe

Related snap packages

Related bugs

Related blueprints

27. By Diogo Matsubara on 2015-09-23

forgot a space in if conditional

26. By Diogo Matsubara on 2015-09-23

review comments: fix credentials to cope with old slave versions that doesn't need credentials

25. By Diogo Matsubara on 2015-09-03

remove start/stop hooks since jenkins slaves depend on a slave-relation-[joined,changed,departed] to be started or stopped

24. By Diogo Matsubara on 2015-09-03

increase verbosity of hooks

23. By Diogo Matsubara on 2015-09-03

add extension interface

22. By Diogo Matsubara on 2015-08-28

check if credentials have been set

21. By Diogo Matsubara on 2015-08-28

add credentials to jenkins-slave.conf

20. By Curtis Hovey on 2014-05-15

Always install deps first.

19. By Curtis Hovey on 2014-05-15

Added the missing ! to ensure the slave is added.

18. By Curtis Hovey on 2014-05-15

Revise executable check to avoid non-exec files from
triggering a exit of 1.