ipaserver: Run custodia setup only once

The custodia setup is executed twice. At first in
ipaserver_setup_custodia and then additionally in ipaserver_setup_ca.

The custodia setup code in ipaserver_setup_ca.py has been adapted to fit
the code in ipaserver_setup_custodia.py.

The extra Setup custodia step in the server roles has been removed
together with ipaserver_setup_custodia.py.

roles/ipaserver/library/ipaserver_setup_ca.py

@@ -354,7 +354,7 @@ def main():
                       options.no_hbac_allow, options._dirsrv_pkcs12_info,
                       options.no_pkinit)
 
-    # setup CA ##############################################################
+    # setup custodia ########################################################
 
     if hasattr(custodiainstance, "get_custodia_instance"):
         if hasattr(custodiainstance.CustodiaModes, "FIRST_MASTER"):
@@ -362,9 +362,14 @@ def main():
         else:
             mode = custodiainstance.CustodiaModes.MASTER_PEER
         custodia = custodiainstance.get_custodia_instance(options, mode)
-        custodia.set_output(ansible_log)
-        with redirect_stdout(ansible_log):
-            custodia.create_instance()
+    else:
+        custodia = custodiainstance.CustodiaInstance(options.host_name,
+                                                     options.realm_name)
+    custodia.set_output(ansible_log)
+    with redirect_stdout(ansible_log):
+        custodia.create_instance()
+
+    # setup CA ##############################################################
 
     if options.setup_ca:
         if not options.external_cert_files and options.external_ca:

roles/ipaserver/library/ipaserver_setup_custodia.py

@@ -1,118 +0,0 @@
-# -*- coding: utf-8 -*-
-
-# Authors:
-#   Thomas Woerner <twoerner@redhat.com>
-#
-# Based on ipa-client-install code
-#
-# Copyright (C) 2017-2022  Red Hat
-# see file 'COPYING' for use and warranty information
-#
-# This program is free software; you can redistribute it and/or modify
-# it under the terms of the GNU General Public License as published by
-# the Free Software Foundation, either version 3 of the License, or
-# (at your option) any later version.
-#
-# This program is distributed in the hope that it will be useful,
-# but WITHOUT ANY WARRANTY; without even the implied warranty of
-# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-# GNU General Public License for more details.
-#
-# You should have received a copy of the GNU General Public License
-# along with this program.  If not, see <http://www.gnu.org/licenses/>.
-
-from __future__ import (absolute_import, division, print_function)
-
-__metaclass__ = type
-
-ANSIBLE_METADATA = {
-    'metadata_version': '1.0',
-    'supported_by': 'community',
-    'status': ['preview'],
-}
-
-DOCUMENTATION = '''
----
-module: ipaserver_setup_custodia
-short_description: Setup custodia
-description: Setup custodia
-options:
-  realm:
-    description: Kerberos realm name of the IPA deployment
-    type: str
-    required: yes
-  hostname:
-    description: Fully qualified name of this host
-    type: str
-    required: no
-  setup_ca:
-    description: Configure a dogtag CA
-    type: bool
-    default: no
-    required: no
-author:
-    - Thomas Woerner (@t-woerner)
-'''
-
-EXAMPLES = '''
-'''
-
-RETURN = '''
-'''
-
-from ansible.module_utils.basic import AnsibleModule
-from ansible.module_utils.ansible_ipa_server import (
-    check_imports, setup_logging, AnsibleModuleLog, options,
-    api_Backend_ldap2,
-    custodiainstance, redirect_stdout
-)
-
-
-def main():
-    ansible_module = AnsibleModule(
-        argument_spec=dict(
-            # basic
-            realm=dict(required=True, type='str'),
-            hostname=dict(required=False, type='str'),
-            setup_ca=dict(required=False, type='bool', default=False),
-        ),
-    )
-
-    ansible_module._ansible_debug = True
-    check_imports(ansible_module)
-    setup_logging()
-    ansible_log = AnsibleModuleLog(ansible_module)
-
-    # set values ############################################################
-
-    options.realm_name = ansible_module.params.get('realm')
-    options.host_name = ansible_module.params.get('hostname')
-    options.setup_ca = ansible_module.params.get('setup_ca')
-    options.promote = False
-
-    # init ##################################################################
-
-    api_Backend_ldap2(options.host_name, options.setup_ca, connect=True)
-
-    # setup custodia ########################################################
-
-    if hasattr(custodiainstance, "get_custodia_instance"):
-        if hasattr(custodiainstance.CustodiaModes, "FIRST_MASTER"):
-            mode = custodiainstance.CustodiaModes.FIRST_MASTER
-        else:
-            mode = custodiainstance.CustodiaModes.MASTER_PEER
-        custodia = custodiainstance.get_custodia_instance(options, mode)
-    else:
-        custodia = custodiainstance.CustodiaInstance(options.host_name,
-                                                     options.realm_name)
-    custodia.set_output(ansible_log)
-    with redirect_stdout(ansible_log):
-        custodia.create_instance()
-
-    # done ##################################################################
-
-    ansible_module.exit_json(changed=True)
-
-
-if __name__ == '__main__':
-    main()

roles/ipaserver/tasks/install.yml

@@ -267,12 +267,6 @@
       idmax: "{{ result_ipaserver_test.idmax }}"
       _pkinit_pkcs12_info: "{{ result_ipaserver_test._pkinit_pkcs12_info if result_ipaserver_test._pkinit_pkcs12_info != None else omit }}"
 
-  - name: Install - Setup custodia
-    ipaserver_setup_custodia:
-      realm: "{{ result_ipaserver_test.realm }}"
-      hostname: "{{ result_ipaserver_test.hostname }}"
-      setup_ca: "{{ result_ipaserver_test.setup_ca }}"
-
   - name: Install - Setup CA
     ipaserver_setup_ca:
       dm_password: "{{ ipadm_password }}"