[owncloud] fix linting

(cherry picked from commit 601f9ff) (cherry picked from commit 219a2fa)
debops · Aug 23, 2021 · 4174614 · 4174614
1 parent c2a0151
commit 4174614
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 2 deletions.
diff --git a/CHANGELOG.rst b/CHANGELOG.rst
@@ -154,6 +154,12 @@ General
 - Do not remove the whole PKI hook directory when the :command:`nginx` hook
   script is removed by the role.
 
+:ref:`debops.owncloud` role
+'''''''''''''''''''''''''''
+
+- Correctly handle the exit code of the :command:`occ ldap:show-config` command
+  when the LDAP configuration is not yet available.
+
 :ref:`debops.pki` role
 ''''''''''''''''''''''
 

diff --git a/ansible/roles/owncloud/tasks/ldap.yml b/ansible/roles/owncloud/tasks/ldap.yml
@@ -14,8 +14,7 @@
   register: owncloud__register_ldap_default_config_exits
   become: True
   become_user: '{{ owncloud__app_user }}'
-  #ignore_errors: True
-  failed_when: (owncloud__register_ldap_default_config_exits.rc != 0 and 
+  failed_when: (owncloud__register_ldap_default_config_exits.rc != 0 and
                 'Invalid configID' not in owncloud__register_ldap_default_config_exits.stdout)
 
 - name: Create empty LDAP configuration