Stouts / Stouts.jenkins

Ansible role which manage Jenkins CI
MIT License
41 stars 35 forks source link

Several fixes, it was not working on Ubuntu, fix also for latest apache ... #7

Closed AlbanAndrieu closed 10 years ago

AlbanAndrieu commented 10 years ago

...and for missing prefix. Jenkins war is now using the LTS instead of the latest for stability reason (it can be override now)

AlbanAndrieu commented 10 years ago

It seems that travis failed because of the Idempotence test. I indeed change jenkins to the LTS and configuration file. Is that an issue?

farridav commented 10 years ago

*Bump.. to get this working for me ive had to add

- name: enable required Apache proxy modules
  apache2_module: state=present name={{ item }}
  with_items:
    - proxy
    - proxy_http

Is this PR on the way to repair and merge or would you like me to make a new one with just the apache proxy modules?

AlbanAndrieu commented 10 years ago

Thanks, I have added your requirement into the PR. I have also added the creation of the JENKINS_HOME directory in case it does not exist (It is a none blocking task)

klen commented 10 years ago

Thank you. The request has been merged.

I fixed the Travis tests.

AlbanAndrieu commented 10 years ago

Super duper. Have a nice evening