diff --git a/roles/ipaclient/files/py3test.py b/roles/ipaclient/files/py3test.py deleted file mode 100644 index 6bf70ab5..00000000 --- a/roles/ipaclient/files/py3test.py +++ /dev/null @@ -1,9 +0,0 @@ -#!/usr/bin/python3 - -# Test ipaclient python3 binding -from ipaclient.install.client import SECURE_PATH # noqa: F401 - -# Check ipapython version to be >= 4.6 -from ipapython.version import NUM_VERSION, VERSION -if NUM_VERSION < 40600: - raise Exception("ipa %s not usable with python3" % VERSION) diff --git a/roles/ipaclient/tasks/install.yml b/roles/ipaclient/tasks/install.yml index 97bf7949..fccc72e0 100644 --- a/roles/ipaclient/tasks/install.yml +++ b/roles/ipaclient/tasks/install.yml @@ -7,9 +7,6 @@ state: present when: ipaclient_install_packages | bool -#- name: Install - Include Python2/3 import test -# import_tasks: "{{ role_path }}/tasks/python_2_3_test.yml" - - name: Install - Set ipaclient_servers set_fact: ipaclient_servers: "{{ groups['ipaservers'] | list }}" @@ -113,10 +110,6 @@ fail: msg="Keytab or password is required for getting otp" when: ipaadmin_keytab is undefined and ipaadmin_password is undefined - #- name: Install - Include Python2/3 import test - # import_tasks: "{{ role_path }}/tasks/python_2_3_test.yml" - # delegate_to: "{{ result_ipaclient_test.servers[0] }}" - - name: Install - Get One-Time Password for client enrollment no_log: yes ipaclient_get_otp: diff --git a/roles/ipaclient/tasks/python_2_3_test.yml b/roles/ipaclient/tasks/python_2_3_test.yml deleted file mode 100644 index 511dea46..00000000 --- a/roles/ipaclient/tasks/python_2_3_test.yml +++ /dev/null @@ -1,18 +0,0 @@ ---- -- block: - - name: Verify Python3 import - script: py3test.py - register: result_py3test - failed_when: False - changed_when: False - check_mode: no - - - name: Set python interpreter to 3 - set_fact: - ansible_python_interpreter: "/usr/bin/python3" - when: result_py3test.rc == 0 - - - name: Set python interpreter to 2 - set_fact: - ansible_python_interpreter: "/usr/bin/python2" - when: result_py3test.failed or result_py3test.rc != 0 diff --git a/roles/ipaclient/tasks/uninstall.yml b/roles/ipaclient/tasks/uninstall.yml index 095099a3..90078542 100644 --- a/roles/ipaclient/tasks/uninstall.yml +++ b/roles/ipaclient/tasks/uninstall.yml @@ -1,9 +1,6 @@ --- # tasks to uninstall IPA client -# - name: Uninstall - Include Python2/3 import test -# import_tasks: "{{ role_path }}/tasks/python_2_3_test.yml" - - name: Uninstall - Uninstall IPA client command: > /usr/sbin/ipa-client-install diff --git a/roles/ipareplica/files/py3test.py b/roles/ipareplica/files/py3test.py deleted file mode 100644 index ffb009cd..00000000 --- a/roles/ipareplica/files/py3test.py +++ /dev/null @@ -1,16 +0,0 @@ -#!/usr/bin/python3 - -# Test ipaerver python3 binding -try: - from ipaserver.install.server.replicainstall import ( # noqa: F401 - install_check, - ) -except ImportError: - from ipaserver.install.server.replicainstall import ( # noqa: F401 - promote_check, - ) - -# Check ipapython version to be >= 4.6 -from ipapython.version import NUM_VERSION, VERSION -if NUM_VERSION < 40590: - raise Exception("ipa %s not usable with python3" % VERSION) diff --git a/roles/ipareplica/tasks/install.yml b/roles/ipareplica/tasks/install.yml index ddb3f85f..24e47657 100644 --- a/roles/ipareplica/tasks/install.yml +++ b/roles/ipareplica/tasks/install.yml @@ -35,9 +35,6 @@ when: ipareplica_install_packages | bool -#- name: Install - Include Python2/3 import test -# import_tasks: "{{ role_path }}/tasks/python_2_3_test.yml" - - name: Install - Set ipareplica_servers set_fact: ipareplica_servers: "{{ groups['ipaservers'] | list }}" diff --git a/roles/ipareplica/tasks/python_2_3_test.yml b/roles/ipareplica/tasks/python_2_3_test.yml deleted file mode 100644 index d407932b..00000000 --- a/roles/ipareplica/tasks/python_2_3_test.yml +++ /dev/null @@ -1,23 +0,0 @@ ---- -- block: - - name: Verify Python3 import - script: py3test.py - register: result_py3test - failed_when: False - changed_when: False - check_mode: no - - - name: Set python interpreter to 3 - set_fact: - ansible_python_interpreter: "/usr/bin/python3" - when: result_py3test.rc == 0 - - - name: Fail for IPA 4.5.90 - fail: msg="You need to install python2 bindings for ipa server usage" - when: result_py3test.rc != 0 and "not usable with python3" in - result_py3test.stdout - - - name: Set python interpreter to 2 - set_fact: - ansible_python_interpreter: "/usr/bin/python2" - when: result_py3test.failed or result_py3test.rc != 0 diff --git a/roles/ipareplica/tasks/uninstall.yml b/roles/ipareplica/tasks/uninstall.yml index 65068e6f..5df73e93 100644 --- a/roles/ipareplica/tasks/uninstall.yml +++ b/roles/ipareplica/tasks/uninstall.yml @@ -1,9 +1,6 @@ --- # tasks to uninstall IPA replica -# - name: Uninstall - Include Python2/3 import test -# import_tasks: "{{ role_path }}/tasks/python_2_3_test.yml" - - name: Uninstall - Uninstall IPA replica command: > /usr/sbin/ipa-server-install diff --git a/roles/ipaserver/files/py3test.py b/roles/ipaserver/files/py3test.py deleted file mode 100644 index 701e3424..00000000 --- a/roles/ipaserver/files/py3test.py +++ /dev/null @@ -1,9 +0,0 @@ -#!/usr/bin/python3 - -# Test ipaerver python3 binding -from ipaserver.install.server.install import install_check # noqa: F401 - -# Check ipapython version to be >= 4.6 -from ipapython.version import NUM_VERSION, VERSION -if NUM_VERSION < 40590: - raise Exception("ipa %s not usable with python3" % VERSION) diff --git a/roles/ipaserver/tasks/install.yml b/roles/ipaserver/tasks/install.yml index 687f72db..77060c33 100644 --- a/roles/ipaserver/tasks/install.yml +++ b/roles/ipaserver/tasks/install.yml @@ -34,9 +34,6 @@ when: ipaserver_install_packages | bool -#- name: Install - Include Python2/3 import test -# import_tasks: "{{ role_path }}/tasks/python_2_3_test.yml" - - include_tasks: "{{ role_path }}/tasks/copy_external_cert.yml" with_items: "{{ ipaserver_external_cert_files_from_controller }}" when: ipaserver_external_cert_files_from_controller is defined and diff --git a/roles/ipaserver/tasks/python_2_3_test.yml b/roles/ipaserver/tasks/python_2_3_test.yml deleted file mode 100644 index c8b2009a..00000000 --- a/roles/ipaserver/tasks/python_2_3_test.yml +++ /dev/null @@ -1,23 +0,0 @@ ---- -- block: - - name: Verify Python3 import - script: py3test.py - register: result_py3test - failed_when: False - changed_when: False - check_mode: no - - - name: Set python interpreter to 3 - set_fact: - ansible_python_interpreter: "/usr/bin/python3" - when: result_py3test.rc == 0 - - - name: Fail for IPA 4.5.90 - fail: msg="You need to install python2 bindings for ipa server usage" - when: result_py3test.rc != 0 and "not usable with python3" - in result_py3test.stdout - - - name: Set python interpreter to 2 - set_fact: - ansible_python_interpreter: "/usr/bin/python2" - when: result_py3test.failed or result_py3test.rc != 0 diff --git a/roles/ipaserver/tasks/uninstall.yml b/roles/ipaserver/tasks/uninstall.yml index d018429f..71de9faf 100644 --- a/roles/ipaserver/tasks/uninstall.yml +++ b/roles/ipaserver/tasks/uninstall.yml @@ -1,9 +1,6 @@ --- # tasks to uninstall IPA server -# - name: Uninstall - Include Python2/3 import test -# import: "{{ role_path }}/tasks/python_2_3_test.yml" - - name: Uninstall - Uninstall IPA server command: > /usr/sbin/ipa-server-install diff --git a/utils/ansible-freeipa.spec.in b/utils/ansible-freeipa.spec.in index 554059d5..2761f6dd 100644 --- a/utils/ansible-freeipa.spec.in +++ b/utils/ansible-freeipa.spec.in @@ -37,8 +37,6 @@ for i in roles/ipa*/library/*.py roles/ipa*/module_utils/*.py plugins/*/*.py; do sed -i '1{/\/usr\/bin\/python*/d;}' $i chmod a-x $i done -# Add execute flag to py3test.py scripts -chmod a+x roles/ipa*/files/py3test.py %build