Skip to content
  1. Sep 02, 2021
    • Rafael Guterres Jeffman's avatar
      dnszone: Use IPAAnsibleModule class. · 21a48d59
      Rafael Guterres Jeffman authored
      Use IPAAnsibleModule methods and ipamodule_base_docs ducument fragment.
      21a48d59
    • Rafael Guterres Jeffman's avatar
      Make IPAAnsibleModule base class of FreeIPABaseModule. · 0c430d0a
      Rafael Guterres Jeffman authored
      By making IPAAnsibleModule the base class of FreeIPABaseModule, instead
      of AnsibleModule, some methods on FreeIPABaseModule can be removed and
      suport for commom parameters in modules using the older class can use
      the same commom parameters (ipaadmin_principal and ipaadmin_password)
      as the other parameters. This will also allow easier deprecation of
      FreeIPABaseModule, which is hard to maintain.
      
      To be able to use IPAAnsibleModule as the base class, it was moved
      within the file, to position before FreeIPABaseModule declaration.
      
      This patch also modifies IPAAnsibleModule by:
      
          * removing usage of `self` in methods not requiring it, turning
            the methods into @statimethod;
      
          * adding comments to all the methods in IPAAnsibleModule, which
            makes it easier to understand what the individual methods do,
            and what their parameters represent.
      0c430d0a
    • Thomas Woerner's avatar
      host: Use IPAAnsibleModule class · d356fa42
      Thomas Woerner authored
      ipaadmin_variables are handled by IPAAnsibleModule,
      ansible_module.params_get is used to get the parameters and
      ansible_module.ipa_connect is used to simplify the module.
      d356fa42
    • Thomas Woerner's avatar
      user: Use IPAAnsibleModule class · c2df7be2
      Thomas Woerner authored
      ipaadmin_variables are handled by IPAAnsibleModule,
      ansible_module.params_get is used to get the parameters and
      ansible_module.ipa_connect is used to simplify the module.
      c2df7be2
    • Thomas Woerner's avatar
      trust: Use IPAAnsibleModule class · c4fbd0d5
      Thomas Woerner authored
      ipaadmin_variables are handled by IPAAnsibleModule,
      ansible_module.params_get is used to get the parameters and
      ansible_module.ipa_connect is used to simplify the module.
      c4fbd0d5
    • Thomas Woerner's avatar
      sudorule: Use IPAAnsibleModule class · cb48d4b8
      Thomas Woerner authored
      ipaadmin_variables are handled by IPAAnsibleModule,
      ansible_module.params_get is used to get the parameters and
      ansible_module.ipa_connect is used to simplify the module.
      cb48d4b8
    • Thomas Woerner's avatar
      sudocmd: Use IPAAnsibleModule class · 7864adff
      Thomas Woerner authored
      ipaadmin_variables are handled by IPAAnsibleModule,
      ansible_module.params_get is used to get the parameters and
      ansible_module.ipa_connect is used to simplify the module.
      7864adff
    • Thomas Woerner's avatar
      sudocmdgroup: Use IPAAnsibleModule class · 09264767
      Thomas Woerner authored
      ipaadmin_variables are handled by IPAAnsibleModule,
      ansible_module.params_get is used to get the parameters and
      ansible_module.ipa_connect is used to simplify the module.
      09264767
    • Thomas Woerner's avatar
      service: Use IPAAnsibleModule class · 748f05d3
      Thomas Woerner authored
      ipaadmin_variables are handled by IPAAnsibleModule,
      ansible_module.params_get is used to get the parameters and
      ansible_module.ipa_connect is used to simplify the module.
      748f05d3
    • Thomas Woerner's avatar
      server: Use IPAAnsibleModule class · ab6097d1
      Thomas Woerner authored
      ipaadmin_variables are handled by IPAAnsibleModule,
      ansible_module.params_get is used to get the parameters and
      ansible_module.ipa_connect is used to simplify the module.
      ab6097d1
    • Thomas Woerner's avatar
      selfservice: Use IPAAnsibleModule class · 325ffb7b
      Thomas Woerner authored
      ipaadmin_variables are handled by IPAAnsibleModule,
      ansible_module.params_get is used to get the parameters and
      ansible_module.ipa_connect is used to simplify the module.
      325ffb7b
    • Thomas Woerner's avatar
      role: Use IPAAnsibleModule class · 954ae982
      Thomas Woerner authored
      ipaadmin_variables are handled by IPAAnsibleModule,
      ansible_module.params_get is used to get the parameters and
      ansible_module.ipa_connect is used to simplify the module.
      954ae982
    • Thomas Woerner's avatar
      privilege: Use IPAAnsibleModule class · ba72bd02
      Thomas Woerner authored
      ipaadmin_variables are handled by IPAAnsibleModule,
      ansible_module.params_get is used to get the parameters and
      ansible_module.ipa_connect is used to simplify the module.
      ba72bd02
    • Thomas Woerner's avatar
      permission: Use IPAAnsibleModule class · ce00f32d
      Thomas Woerner authored
      ipaadmin_variables are handled by IPAAnsibleModule,
      ansible_module.params_get is used to get the parameters and
      ansible_module.ipa_connect is used to simplify the module.
      ce00f32d
    • Thomas Woerner's avatar
      hostgroup: Use IPAAnsibleModule class · 91d8f5a6
      Thomas Woerner authored
      ipaadmin_variables are handled by IPAAnsibleModule,
      ansible_module.params_get is used to get the parameters and
      ansible_module.ipa_connect is used to simplify the module.
      91d8f5a6
    • Thomas Woerner's avatar
      hbacsvcgroup: Use IPAAnsibleModule class · 9d4572be
      Thomas Woerner authored
      ipaadmin_variables are handled by IPAAnsibleModule,
      ansible_module.params_get is used to get the parameters and
      ansible_module.ipa_connect is used to simplify the module.
      9d4572be
    • Thomas Woerner's avatar
      hbacsvc: Use IPAAnsibleModule class · 7d2bdd71
      Thomas Woerner authored
      ipaadmin_variables are handled by IPAAnsibleModule,
      ansible_module.params_get is used to get the parameters and
      ansible_module.ipa_connect is used to simplify the module.
      7d2bdd71
    • Thomas Woerner's avatar
      hbacrule: Use IPAAnsibleModule class · f87520d9
      Thomas Woerner authored
      ipaadmin_variables are handled by IPAAnsibleModule,
      ansible_module.params_get is used to get the parameters and
      ansible_module.ipa_connect is used to simplify the module.
      f87520d9
    • Thomas Woerner's avatar
      group: Use IPAAnsibleModule class · 31ee4f9b
      Thomas Woerner authored
      ipaadmin_variables are handled by IPAAnsibleModule,
      ansible_module.params_get is used to get the parameters and
      ansible_module.ipa_connect is used to simplify the module.
      31ee4f9b
    • Thomas Woerner's avatar
      dnsrecord: Use IPAAnsibleModule class · 7318302f
      Thomas Woerner authored
      ipaadmin_variables are handled by IPAAnsibleModule,
      ansible_module.params_get is used to get the parameters and
      ansible_module.ipa_connect is used to simplify the module.
      7318302f
    • Thomas Woerner's avatar
      forwardzone: Use IPAAnsibleModule class · a8045551
      Thomas Woerner authored
      ipaadmin_variables are handled by IPAAnsibleModule,
      ansible_module.params_get is used to get the parameters and
      ansible_module.ipa_connect is used to simplify the module.
      a8045551
    • Thomas Woerner's avatar
      dnsconfig: Use IPAAnsibleModule class · 97b47cc8
      Thomas Woerner authored
      ipaadmin_variables are handled by IPAAnsibleModule,
      ansible_module.params_get is used to get the parameters and
      ansible_module.ipa_connect is used to simplify the module.
      97b47cc8
    • Thomas Woerner's avatar
      delegation: Use IPAAnsibleModule class · c634cfd5
      Thomas Woerner authored
      ipaadmin_variables are handled by IPAAnsibleModule,
      ansible_module.params_get is used to get the parameters and
      ansible_module.ipa_connect is used to simplify the module.
      c634cfd5
    • Thomas Woerner's avatar
      vault: Use IPAAnsibleModule class · fc218878
      Thomas Woerner authored
      ipaadmin_variables are handled by IPAAnsibleModule,
      ansible_module.params_get is used to get the parameters and
      ansible_module.ipa_connect is used to simplify the module.
      fc218878
  2. Sep 01, 2021
  3. Aug 31, 2021
    • Rafael Guterres Jeffman's avatar
      ipaserver: Modify vars loading in ansible-freeipa roles. · e70763b0
      Rafael Guterres Jeffman authored
      When loading variables in all ansible-freeipa roles, it is expected
      that a file with these variables is present for each supported Linux
      distribution, and then, based on the information about the distribution
      provided by Ansible, the correct file is loaded.
      
      Previously, only the facts `distribution` and dinstribution version
      related facts were used, which required specific files, or links to
      files for distributions in the same "family", which will probably have
      the same variables set.
      
      This change adds searching for files based on the `os_family` fact,
      allowing distributions that follow the same family rules to be
      supported, without any changes to the codebase. It is still possible
      that a specific distribution configuration overrides the default
      behavior, as `os_family` has lower priority than `distribution`.
      
      For example, distributions on the `RedHat` family, like Oracle Linux,
      Alma Linux, and Rocky Linux, work withoutadding new files, or links to
      files, to fill the `vars`.
      
      Fix issue #573. Fix issue #523.
      e70763b0
    • Rafael Guterres Jeffman's avatar
      ipareplica: Modify vars loading in ansible-freeipa tests. · f63eaf46
      Rafael Guterres Jeffman authored
      When loading variables in all ansible-freeipa roles, it is expected
      that a file with these variables is present for each supported Linux
      distribution, and then, based on the information about the distribution
      provided by Ansible, the correct file is loaded.
      
      Previously, only the facts `distribution` and dinstribution version
      related facts were used, which required specific files, or links to
      files for distributions in the same "family", which will probably have
      the same variables set.
      
      This change adds searching for files based on the `os_family` fact,
      allowing distributions that follow the same family rules to be
      supported, without any changes to the codebase. It is still possible
      that a specific distribution configuration overrides the default
      behavior, as `os_family` has lower priority than `distribution`.
      
      For example, distributions on the `RedHat` family, like Oracle Linux,
      Alma Linux, and Rocky Linux, work withoutadding new files, or links to
      files, to fill the `vars`.
      
      Fix issue #573. Fix issue #523.
      f63eaf46
    • Rafael Guterres Jeffman's avatar
      ipaclient: Modify vars loading in ansible-freeipa roles. · 8259eb1e
      Rafael Guterres Jeffman authored
      When loading variables in all ansible-freeipa roles, it is expected
      that a file with these variables is present for each supported Linux
      distribution, and then, based on the information about the distribution
      provided by Ansible, the correct file is loaded.
      
      Previously, only the facts `distribution` and dinstribution version
      related facts were used, which required specific files, or links to
      files for distributions in the same "family", which will probably have
      the same variables set.
      
      This change adds searching for files based on the `os_family` fact,
      allowing distributions that follow the same family rules to be
      supported, without any changes to the codebase. It is still possible
      that a specific distribution configuration overrides the default
      behavior, as `os_family` has lower priority than `distribution`.
      
      For example, distributions on the `RedHat` family, like Oracle Linux,
      Alma Linux, and Rocky Linux, work withoutadding new files, or links to
      files, to fill the `vars`.
      
      Fix issue #573. Fix issue #523.
      8259eb1e