Merge pull request #13 from hellofresh/feature/id-limits

Add min_id max_id configuration options

Author: antonu17

README.md

@@ -35,6 +35,8 @@ feel free to create pull requests. You can find available options in `defaults/m
 Some default values for domain specific configuration options are:
 
     sssd_domain_defaults:
+      min_id: 1
+      max_id: 0
       id_provider: ldap
       auth_provider: ldap
       enumerate: 'false'
@@ -51,6 +53,8 @@ Role supports configuring multiple domains using following syntax:
 
     sssd_domains:
       - name: domain_name
+        min_id:
+        max_id:
         id_provider:
         auth_provider:
         ldap_uri:

defaults/main.yml

@@ -11,6 +11,8 @@ sssd_nss:
   filter_groups: root
 
 sssd_domain_defaults:
+  min_id: 1
+  max_id: 0
   id_provider: ldap
   auth_provider: ldap
   enumerate: 'false'
@@ -24,6 +26,8 @@ sssd_domain_defaults:
 
 sssd_domains:
   - name: default
+    min_id: "{{ sssd_domain_defaults.min_id }}"
+    max_id: "{{ sssd_domain_defaults.max_id }}"
     id_provider: "{{ sssd_domain_defaults.id_provider }}"
     auth_provider: "{{ sssd_domain_defaults.auth_provider }}"
     ldap_uri: "{{ sssd_domain_defaults.ldap_uri }}"

requirements.txt

@@ -1,3 +1,3 @@
-ansible==2.3.2.0
+ansible==2.4.6.0
 molecule
 docker-py

templates/sssd.conf.j2

@@ -11,6 +11,12 @@ filter_groups = {{ sssd_nss.filter_users }}
 {% for domain in sssd_domains %}
 [domain/{{ domain.name }}]
 {# providers #}
+min_id = {{ domain.min_id
+    if domain.min_id is defined
+    else sssd_domain_defaults.min_id }}
+max_id = {{ domain.max_id
+    if domain.max_id is defined
+    else sssd_domain_defaults.max_id }}
 id_provider = {{ domain.id_provider
     if domain.id_provider is defined
     else sssd_domain_defaults.id_provider }}