Skip to content

Commit

Permalink
remove meta dependency of ansible-role-pam and document it instead
Browse files Browse the repository at this point in the history
 - galaxy complains about this with something like
"Error parsing dependency Expecting dependency name format to match
'username.role_name', instead got ansible-role-pam"
 - also change categories to galaxy_tags
  • Loading branch information
martbhell committed Nov 10, 2016
1 parent 4ab0539 commit b996a62
Show file tree
Hide file tree
Showing 2 changed files with 19 additions and 17 deletions.
15 changes: 15 additions & 0 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -56,6 +56,21 @@ fgci_slurmrepo_version: "fgcislurm1605"

A playbook that uses this role: https://github.com/CSC-IT-Center-for-Science/fgci-ansible

Or you can check out the [tests/test.yml](tests/test.yml) in this repo.

Example Playbook
----------------

Including an example of how to use your role (for instance, with variables passed in as parameters) is always nice for users too:

<pre>
- hosts: servers
roles:
- { role: ansible-role-pam }
- { role: ansible-role-nhc }
- { role: ansible-role-slurm }
</pre>

### Known Issues

- This role used to be able to build slurm rpms, distribute them and install them. The last tag/release that had this feature was v1.5.0
Expand Down
21 changes: 4 additions & 17 deletions meta/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -117,24 +117,11 @@ galaxy_info:
# Below are all categories currently available. Just as with
# the platforms above, uncomment those that apply to your role.
#
categories:
#- cloud
#- cloud:ec2
#- cloud:gce
#- cloud:rax
- clustering
#- database
#- database:nosql
#- database:sql
#- development
#- monitoring
#- networking
#- packaging
- system
#- web
dependencies:
galaxy_tags:
- clustering
- system
dependencies: []
# List your role dependencies here, one per line.
# Be sure to remove the '[]' above if you add dependencies
# to this list.
- { role: ansible-role-pam }

0 comments on commit b996a62

Please sign in to comment.