Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add a cgit role for Gerrit #3

Closed
wants to merge 4 commits into from
Closed

Add a cgit role for Gerrit #3

wants to merge 4 commits into from

Conversation

nigelbabu
Copy link
Contributor

No description provided.

- template: mode=0644 src=cgitrepos dest=/etc/cgitrepos

- copy: mode=0644 src=cgit.httpd.conf dest=/etc/httpd/conf.d/cgit.conf
notify: restart httpd
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would be better to use the handler to check the config and restart

@mscherer
Copy link
Contributor

So I merged it (after resolving conflict, since we did edit the same file). I am going to see if it work fine or not and fix.

@mscherer mscherer closed this Sep 14, 2016
@mscherer mscherer deleted the cgit-for-gerrit branch September 14, 2016 11:12
@mscherer
Copy link
Contributor

fatal: [dev.gluster.org]: FAILED! => {"changed": false, "failed": true, "msg": "AnsibleError: template error while templating string: no test named 'True'. String: # {{ ansible_managed }}\n{% if git_repositories is defined and scan_path is not True %}\n{% for repo in git_repositories %}\n{% if not repo.private is defined or not repo.private %}\nrepo.url={% if repo.url is defined %}\n{{ repo.url }}\n{% else %}\n{{ repo.name }}\n{% endif %}\nrepo.path={{ git_repositories_dir}}/{{ repo.name }}\nrepo.desc={{ repo.desc }}\nrepo.owner={{ repo.owner }}\n{% endif %}\n{% endfor %}\n{% elif scan_path is True %}\nscan-dir={{ git_repositories_dir }}/\n{% endif %}\n"}

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants