diff --git a/README-group.md b/README-group.md index f845ef0582e33add6953b10f41bde443fcac0d6e..662fcd01bb3a2a09e238c61c8bc4e779b3ce20b0 100644 --- a/README-group.md +++ b/README-group.md @@ -154,6 +154,7 @@ Variable | Description | Required -------- | ----------- | -------- `ipaadmin_principal` | The admin principal is a string and defaults to `admin` | no `ipaadmin_password` | The admin password is a string and is required if there is no admin ticket available on the node | no +`ipaapi_context` | The context in which the module will execute. Executing in a server context is preferred. If not provided context will be determined by the execution environment. Valid values are `server` and `client`. | no `name` \| `cn` | The list of group name strings. | no `description` | The group description string. | no `gid` \| `gidnumber` | The GID integer. | no diff --git a/tests/group/test_group.yml b/tests/group/test_group.yml index 855443bd4d64c369f83c3a8b9293ddda03a36d7c..74b170c5bcbefcd159a5ade037c2cce9aa9b53b7 100644 --- a/tests/group/test_group.yml +++ b/tests/group/test_group.yml @@ -1,6 +1,6 @@ --- - name: Test group - hosts: ipaserver + hosts: "{{ ipa_test_host | default('ipaserver') }}" become: true gather_facts: false @@ -8,18 +8,21 @@ - name: Ensure users user1, user2 and user3 are absent ipauser: ipaadmin_password: SomeADMINpassword + ipaapi_context: "{{ ipa_context | default(omit) }}" name: user1,user2,user3 state: absent - name: Ensure group group3, group2 and group1 are absent ipagroup: ipaadmin_password: SomeADMINpassword + ipaapi_context: "{{ ipa_context | default(omit) }}" name: group3,group2,group1 state: absent - name: Ensure users user1..user3 are present ipauser: ipaadmin_password: SomeADMINpassword + ipaapi_context: "{{ ipa_context | default(omit) }}" users: - name: user1 first: user1 @@ -36,6 +39,7 @@ - name: Ensure group1 is present ipagroup: ipaadmin_password: SomeADMINpassword + ipaapi_context: "{{ ipa_context | default(omit) }}" name: group1 register: result failed_when: not result.changed or result.failed @@ -43,6 +47,7 @@ - name: Ensure group1 is present again ipagroup: ipaadmin_password: SomeADMINpassword + ipaapi_context: "{{ ipa_context | default(omit) }}" name: group1 register: result failed_when: result.changed or result.failed @@ -50,6 +55,7 @@ - name: Ensure group2 is present ipagroup: ipaadmin_password: SomeADMINpassword + ipaapi_context: "{{ ipa_context | default(omit) }}" name: group2 register: result failed_when: not result.changed or result.failed @@ -57,6 +63,7 @@ - name: Ensure group2 is present again ipagroup: ipaadmin_password: SomeADMINpassword + ipaapi_context: "{{ ipa_context | default(omit) }}" name: group2 register: result failed_when: result.changed or result.failed @@ -64,6 +71,7 @@ - name: Ensure group3 is present ipagroup: ipaadmin_password: SomeADMINpassword + ipaapi_context: "{{ ipa_context | default(omit) }}" name: group3 register: result failed_when: not result.changed or result.failed @@ -71,6 +79,7 @@ - name: Ensure group3 is present again ipagroup: ipaadmin_password: SomeADMINpassword + ipaapi_context: "{{ ipa_context | default(omit) }}" name: group3 register: result failed_when: result.changed or result.failed @@ -78,6 +87,7 @@ - name: Ensure groups group2 and group3 are present in group group1 ipagroup: ipaadmin_password: SomeADMINpassword + ipaapi_context: "{{ ipa_context | default(omit) }}" name: group1 group: - group2 @@ -89,6 +99,7 @@ - name: Ensure groups group2 and group3 are present in group group1 again ipagroup: ipaadmin_password: SomeADMINpassword + ipaapi_context: "{{ ipa_context | default(omit) }}" name: group1 group: - group2 @@ -100,6 +111,7 @@ - name: Ensure group3 ia present in group group1 ipagroup: ipaadmin_password: SomeADMINpassword + ipaapi_context: "{{ ipa_context | default(omit) }}" name: group1 group: - group3 @@ -110,6 +122,7 @@ - name: Ensure users user1, user2 and user3 are present in group group1 ipagroup: ipaadmin_password: SomeADMINpassword + ipaapi_context: "{{ ipa_context | default(omit) }}" name: group1 user: - user1 @@ -122,6 +135,7 @@ - name: Ensure users user1, user2 and user3 are present in group group1 again ipagroup: ipaadmin_password: SomeADMINpassword + ipaapi_context: "{{ ipa_context | default(omit) }}" name: group1 user: - user1 @@ -133,6 +147,7 @@ #- ipagroup: # ipaadmin_password: SomeADMINpassword + # ipaapi_context: "{{ ipa_context | default(omit) }}" # name: group1 # user: # - user7 @@ -141,6 +156,7 @@ - name: Ensure user user7 is absent in group group1 ipagroup: ipaadmin_password: SomeADMINpassword + ipaapi_context: "{{ ipa_context | default(omit) }}" name: group1 user: - user7 @@ -152,6 +168,7 @@ - name: Ensure group group4 is absent ipagroup: ipaadmin_password: SomeADMINpassword + ipaapi_context: "{{ ipa_context | default(omit) }}" name: group4 state: absent register: result @@ -160,6 +177,7 @@ - name: Ensure group group3, group2 and group1 are absent ipagroup: ipaadmin_password: SomeADMINpassword + ipaapi_context: "{{ ipa_context | default(omit) }}" name: group3,group2,group1 state: absent register: result @@ -168,8 +186,8 @@ - name: Ensure users user1, user2 and user3 are absent ipauser: ipaadmin_password: SomeADMINpassword + ipaapi_context: "{{ ipa_context | default(omit) }}" name: user1,user2,user3 state: absent register: result failed_when: not result.changed or result.failed - diff --git a/tests/group/test_group_client_context.yml b/tests/group/test_group_client_context.yml new file mode 100644 index 0000000000000000000000000000000000000000..5e58cbfdcefe70194d98f69e125c63bb3c62e979 --- /dev/null +++ b/tests/group/test_group_client_context.yml @@ -0,0 +1,37 @@ +--- +- name: Test group + hosts: ipaclients, ipaserver + become: no + gather_facts: no + + tasks: + - name: Include FreeIPA facts. + include_tasks: ../env_freeipa_facts.yml + + # Test will only be executed if host is not a server. + - name: Execute with server context in the client. + ipagroup: + ipaadmin_password: SomeADMINpassword + ipaapi_context: server + name: ThisShouldNotWork + register: result + failed_when: not (result.failed and result.msg is regex("No module named '*ipaserver'*")) + when: ipa_host_is_client + +# Import basic module tests, and execute with ipa_context set to 'client'. +# If ipaclients is set, it will be executed using the client, if not, +# ipaserver will be used. +# +# With this setup, tests can be executed against an IPA client, against +# an IPA server using "client" context, and ensure that tests are executed +# in upstream CI. + +- name: Test group using client context, in client host. + import_playbook: test_group.yml + when: groups['ipaclients'] + vars: + ipa_test_host: ipaclients + +- name: Test group using client context, in server host. + import_playbook: test_group.yml + when: groups['ipaclients'] is not defined or not groups['ipaclients']