ICTO / ansible-jenkins

Ansible playbook: Jenkins
MIT License
282 stars 171 forks source link

Add authentication information to the jenkins cli call #37

Closed clarete closed 8 years ago

clarete commented 8 years ago

The jenkins setup I'm dealing with has all the security options removed for anonymous users. This patch introduces two new variables to the role: jenkins_admin_user and jenkins_admin_password. When BOTH are defined, it will extend the string that contains the CLI command with the following snippet:

--username {admin_user} --password {admin_password}

This will address issue #21.

clarete commented 8 years ago

I just noticed this closed PR https://github.com/ICTO/ansible-jenkins/pull/22, I can change this patch to get rid of the two variable style and move to the single one with auth_params just like the other patch, since it also accepts -i key parameters. Let me know what's preferable and I can update the PR if necessary.