search

stackhpc / ansible-role-openhpc

Ansible role for OpenHPC
Apache License 2.0
46 stars 15 forks source link

Issue51 #57

Closed sjpb closed 3 years ago

sjpb commented 3 years ago

Depends on PR https://github.com/stackhpc/ansible-role-openhpc/pull/56

Closes #51 by raising an error like this if a group has no hosts in the play:

AnsibleFilterError: Group "testohpc_compute" contains no hosts in this play - was --limit used?
sjpb commented 3 years ago

Marked as draft as I want to rebase before review

jovial commented 3 years ago

This looks like it expands upon https://github.com/stackhpc/ansible-role-openhpc/pull/58. Could we merge that so I can just see the changes related to this change?

sjpb commented 3 years ago

This looks like it expands upon #58. Could we merge that so I can just see the changes related to this change?

Done.

sjpb commented 3 years ago

I had to change the base and change back again to master to make it update the commits.