]> git.apps.os.sepia.ceph.com Git - ceph-cm-ansible.git/commitdiff
all: Using tests as filters is deprecated
authorDavid Galloway <dgallowa@redhat.com>
Mon, 4 Nov 2019 21:32:18 +0000 (16:32 -0500)
committerDavid Galloway <dgallowa@redhat.com>
Mon, 11 Nov 2019 18:34:42 +0000 (13:34 -0500)
Signed-off-by: David Galloway <dgallowa@redhat.com>
29 files changed:
roles/ansible-managed/tasks/main.yml
roles/cobbler/tasks/main.yml
roles/cobbler/tasks/settings.yml
roles/cobbler_profile/tasks/download_image.yml
roles/cobbler_profile/tasks/import_distro_image.yml
roles/cobbler_profile/tasks/import_distro_iso.yml
roles/common/tasks/apt_systems.yml
roles/common/tasks/beta_repos.yml
roles/common/tasks/epel.yml
roles/common/tasks/kerberos.yml
roles/common/tasks/rhel-entitlements.yml
roles/dhcp-server/tasks/main.yml
roles/nsupdate_web/tasks/main.yml
roles/ntp-server/tasks/main.yml
roles/paddles/tasks/setup_db.yml
roles/paddles/tasks/setup_service.yml
roles/pcp/tasks/collector.yml
roles/pcp/tasks/manager.yml
roles/pcp/tasks/web.yml
roles/testnode/tasks/apt_systems.yml
roles/testnode/tasks/resolvconf.yml
roles/testnode/tasks/secondary_nic.yml
roles/testnode/tasks/ssh.yml
roles/testnode/tasks/user.yml
roles/testnode/tasks/yum/repos.yml
roles/users/tasks/update_keys.yml
roles/vmhost/tasks/networking.yml
tools/set-bmc-static.yml
tools/update-nvme-firmware.yml

index cf5f23e845c04b2cf199a8292c3906f8c525b1fc..a30c12769a346f12d690538ba05863d1c4a45efe 100644 (file)
@@ -20,7 +20,7 @@
 
 - name: Delete the ansible users password.
   command: "passwd -d {{ cm_user }}"
-  when: user_created is defined and user_created|changed
+  when: user_created is defined and user_created is changed
   tags:
    - user
 
index 82520919b26ed3b038b86263adfbf8ac0e83cbcd..c2aa71f39b96ab29b112a237d882207f3f944f8a 100644 (file)
@@ -59,7 +59,7 @@
     name: xinetd
     state: reloaded
     enabled: yes
-  when: tftp_enabled is defined and tftp_enabled|changed
+  when: tftp_enabled is defined and tftp_enabled is changed
   tags:
     - tftp
 
index e9037f03334fd957bf1f7423fe2b16381a5906dd..0456e3b8d8601f59f4945d71996d795431553263 100644 (file)
@@ -25,7 +25,7 @@
   register: server_value
 
 - import_tasks: restart.yml
-  when: users_digest|changed or dynamic_settings|changed or server_value|changed
+  when: users_digest is changed or dynamic_settings is changed or server_value is changed
 
 - name: Update settings
   command: cobbler setting edit --name={{ item.name }} --value={{ item.value}}
index 33f356259027803cfa723c0165c71067c2e3d18f..a5acfd44f5ca6bd2f2df18ae3e8969f2739c6c18 100644 (file)
@@ -25,4 +25,4 @@
 
 - name: Set files_exist if the required files are in place
   set_fact:
-    files_exist: "{{ ( kernel_stat.stat.exists or download_kernel|changed) and ( initrd_stat.stat.exists or download_initrd|changed ) }}"
+    files_exist: "{{ ( kernel_stat.stat.exists or download_kernel is changed) and ( initrd_stat.stat.exists or download_initrd is changed ) }}"
index 01535c29c4283af4e224b8790aa507418c878528..29d8e1e148cd55ba717fefbe5f4fb6f909858eb5 100644 (file)
 - name: Set arch
   set_fact:
       arch: "{{ distro.arch|default('x86_64') }}"
-  when: download_kernel is defined and download_kernel|success
+  when: download_kernel is defined and download_kernel is success
 
 - name: Add the distro to cobbler
   command: cobbler distro add --kernel {{ kernel_path }} --initrd {{ initrd_path }} --name {{ distro_name }}
-  when: download|changed or (files_exist and
+  when: download is changed or (files_exist and
         profile is defined and profile.stdout == '')
   register: import
 
index 8cc382e308e367b0718e0e44916c855121476f67..76a1bd6d53a755d179e92415c3034d5d891a77d8 100644 (file)
     opts: "loop"
     fstype: "iso9660"
     state: mounted
-  when: download|changed or (iso_stat.stat.exists and
+  when: download is changed or (iso_stat.stat.exists and
         profile is defined and profile.stdout == '')
   register: mount
 
 - name: Set arch
   set_fact:
       arch: "{{ distro.arch|default('x86_64') }}"
-  when: mount is defined and mount|changed
+  when: mount is defined and mount is changed
 
 - name: Import the distro (also creates the profile)
   command: cobbler import --path={{ iso_mount }} --name={{ distro_name }} --arch={{ arch }}
   register: import
-  when: mount is defined and mount|changed
+  when: mount is defined and mount is changed
 
 - name: Unmount ISO
   mount:
@@ -48,4 +48,4 @@
     src: "{{ iso_path }}"
     fstype: "iso9660"
     state: unmounted
-  when: mount is defined and mount|changed
+  when: mount is defined and mount is changed
index 619be91dc1ef0129587e87af6127ce22761de578..e5fe259a3370a9088494a6f9605043743450f178 100644 (file)
@@ -21,6 +21,6 @@
 
 - name: Inform the OS of the new timezone
   command: dpkg-reconfigure --frontend noninteractive tzdata
-  when: etc_timezone|changed
+  when: etc_timezone is changed
   tags:
     - timezone
index f5e91741ad382575efbb0bd1456d9ec5fbe82c23..fec85741673b1ffbde6b210f28c20a622e64b9bb 100644 (file)
@@ -12,4 +12,4 @@
 
 - name: Clean yum cache
   shell: yum clean all
-  when: beta_repo is defined and beta_repo|changed
+  when: beta_repo is defined and beta_repo is changed
index 0cbded29c288d1e465270b9970ec488247c1625a..f3befb8d317f4983c3c596890ffe6be5404f8b23 100644 (file)
@@ -29,4 +29,4 @@
 
 - name: Clean yum cache
   shell: yum clean all
-  when: epel_repo is defined and epel_repo|changed
+  when: epel_repo is defined and epel_repo is changed
index 6f1c04ca344e3fa0e18818854aeabb566ec12071..d50fa6328ca435ec1f4dfc1a653108ff7b945700 100644 (file)
@@ -17,7 +17,7 @@
     update_cache: yes
   # Register and retry to work around transient http issues
   register: apt_cache_update
-  until: apt_cache_update|success
+  until: apt_cache_update is success
   # try for 2 minutes before failing
   retries: 24
   delay: 5
index 6a4033bd09e7ae6f5236ba45a5d7a575d719252d..0b016ce24f1bf7b0e59da5032aeb21ce9f955de7 100644 (file)
@@ -44,7 +44,7 @@
   no_log: true
   when: rhsm_registered == false and have_entitlements == true
   register: entitled
-  until: entitled|success
+  until: entitled is success
   retries: 5
   delay: 10
   failed_when:
@@ -54,7 +54,7 @@
 - name: Set rhsm_registered if we just registered
   set_fact:
     rhsm_registered: true
-  when: entitled|success
+  when: entitled is success
 
 # Output of this command is, for example:
 # 7.1
@@ -75,7 +75,7 @@
   command: subscription-manager unregister
   when: ansible_distribution_version not in rhsm_release_list.stdout_lines
   register: unregistered_beta_distro
-  until: unregistered_beta_distro|success
+  until: unregistered_beta_distro is success
   retries: 5
   delay: 10
 
@@ -84,7 +84,7 @@
 - name: Set rhsm_registered to false if Beta/Alpha/RC release
   set_fact:
     rhsm_registered: false
-  when: not unregistered_beta_distro|skipped
+  when: unregistered_beta_distro is not skipped
 
 - name: Run beta_repos.yml playbook for Beta/Alpha/RC release
   import_tasks: beta_repos.yml
 - name: Store list of enabled repos
   set_fact:
     repo_list: "{{ repo_list_cmd.stdout.split('\n') }}"
-  when: repo_list_cmd is defined and not repo_list_cmd|skipped
+  when: repo_list_cmd is defined and repo_list_cmd is not skipped
 
 - name: Set replace_repos false if entitlements are missing or if we unregistered
   set_fact:
     replace_repos: false
-  when: have_entitlements == false or unregistered_beta_distro|changed
+  when: have_entitlements == false or unregistered_beta_distro is changed
 
 - name: Set replace_repos true if rhsm_repos differs from repo_list
   set_fact:
   set_fact:
     replace_repos: true
   when: rhsm_registered == true and
-        (entitled|changed and entitled.rc == 0)
+        (entitled is changed and entitled.rc == 0)
 
 - name: Disable all rhsm repos
   command: subscription-manager repos --disable '*'
   file:
     path: /etc/yum.repos.d/cd.repo
     state: absent
-  when: entitled|success
+  when: entitled is success
index d68e59964834edd4eb3bdec39389a68721fcee68..38e0c6e9b8c9fe509a0924de798a80a9c2183d3a 100644 (file)
 - name: Test new config
   command: dhcpd -t -cf /etc/dhcp/dhcpd.conf
   register: dhcpd_config_test_result
-  when: dhcp_global_config|changed or dhcp_subnet_config|changed
+  when: dhcp_global_config is changed or dhcp_subnet_config is changed
 
 - name: Restart dhcpd
   service:
     name: dhcpd
     state: restarted
   when:
-    - (dhcp_global_config|changed or dhcp_subnet_config|changed)
+    - (dhcp_global_config is changed or dhcp_subnet_config is changed)
     - dhcpd_config_test_result is defined
     - dhcpd_config_test_result.rc == 0
index 11f32a22e9a464cb697e766e41f2adfb80ab4785..3d29452ac574f63ac50602b30e511eded302ca6e 100644 (file)
@@ -67,7 +67,7 @@
     daemon_reload: true
     enabled: true
     state: restarted
-  when: ship_service|changed
+  when: ship_service is changed
 
 - name: Ship nginx configuration
   template:
index 3dfc2c636260e1131604d22b264e04b17a495359..dd6fba4f3fe2489d54043194fbc3b393c02d3be3 100644 (file)
@@ -87,7 +87,7 @@
     name: ntpd
     state: restarted
   when:
-    - conf_written|changed
+    - conf_written is changed
     - use_ntp == true
 
 - name: Restart chronyd service when conf changed
@@ -95,7 +95,7 @@
     name: chronyd
     state: restarted
   when:
-    - conf_written|changed
+    - conf_written is changed
     - use_chrony == true
 
 - name: Check for firewalld
index 347b7a8f737af9719cf22ca8c1f30616a0945be7..0380f2e44c16ed9edcf0459ba9ee3ba3ce334c63 100644 (file)
     name: "{{ paddles_user }}"
     password: "{{ db_pass }}"
   become_user: postgres
-  when: create_db|changed
+  when: create_db is changed
 
 - name: Run pecan populate
   command: ./virtualenv/bin/pecan populate prod.py
   args:
     chdir: "{{ paddles_repo_path }}"
   become_user: "{{ paddles_user }}"
-  when: create_db|changed
+  when: create_db is changed
 
 - name: Copy alembic config template to alembic.ini
   command: cp ./alembic.ini.in alembic.ini
@@ -38,5 +38,5 @@
   shell: ./virtualenv/bin/alembic stamp head
   args:
     chdir: "{{ paddles_repo_path }}"
-  when: alembic_ini|changed
+  when: alembic_ini is changed
   become_user: "{{ paddles_user }}"
index 91d33ca2bb6fdaff96429699cedbd00a2888cf87..057e89b7dbec351a9aed756dfcf10a6e87aa9f96 100644 (file)
@@ -15,7 +15,7 @@
 
 - name: Read supervisord config
   command: supervisorctl update
-  when: supervisor_conf|changed
+  when: supervisor_conf is changed
 
 - name: Ensure paddles is running
   supervisorctl:
@@ -26,7 +26,7 @@
   supervisorctl:
     name: paddles
     state: restarted
-  when: prod_conf is defined and prod_conf|changed
+  when: prod_conf is defined and prod_conf is changed
   tags:
     - config
 
index 55051f05f364008f20655e10d6d7b86ccd7daa43..8c01e5db315b59cdd6048b4265370870a0ffd9e2 100644 (file)
@@ -31,9 +31,9 @@
     state: restarted
     enabled: yes
   when:
-    install_pcp_apt|changed or
-    install_pcp_yum|changed or
-    install_pcp_dnf|changed
+    install_pcp_apt is changed or
+    install_pcp_yum is changed or
+    install_pcp_dnf is changed
 
 - name: Restart pmlogger
   service:
@@ -41,6 +41,6 @@
     state: restarted
     enabled: yes
   when:
-    install_pcp_apt|changed or
-    install_pcp_yum|changed or
-    install_pcp_dnf|changed
+    install_pcp_apt is changed or
+    install_pcp_yum is changed or
+    install_pcp_dnf is changed
index b6e450bc352e6801afe3a437f84566240357cb77..81f34292c76dd00fe7299ee7b9384f8e2bdf26b9 100644 (file)
     state: restarted
     enabled: yes
   when:
-    install_pmmgr_apt|changed or
-    install_pmmgr_yum|changed or
-    target_host|changed or
-    target_discovery|changed or
-    pmmgr_options|changed or
-    update_pmlogmerge_retain|changed or
-    update_pmlogmerge_reduce|changed
+    install_pmmgr_apt is changed or
+    install_pmmgr_yum is changed or
+    target_host is changed or
+    target_discovery is changed or
+    pmmgr_options is changed or
+    update_pmlogmerge_retain is changed or
+    update_pmlogmerge_reduce is changed
index 37c3393b05222d4d60620102a4e90eca29ba01d6..15082b67ec7b5e6ce6d06d0bf74d274f3e2ba357 100644 (file)
@@ -49,6 +49,6 @@
     name: "{{ pmmgr_service }}"
     state: restarted
   when:
-    install_pcp_webapi|changed or
-    install_pcp_webjs|changed or
-    enable_pmwebd|changed
+    install_pcp_webapi is changed or
+    install_pcp_webjs is changed or
+    enable_pmwebd is changed
index 0a9d2294e25dde16483fcbc0934da38680612fa6..8bf81a2bf9378c685d4068da9f799c57f09f5abc 100644 (file)
@@ -44,7 +44,7 @@
   command:
     mount -o remount,user_xattr /
   when: add_user_xattr is defined and
-        add_user_xattr|changed
+        add_user_xattr is changed
 
 - name: Ensure fuse group exists.
   group:
index 92b15f1d60b2639fbe9f5afc75f13a05d5bfeb41..f435af29a7fcd5bfa501bb8b5e037477871b69d5 100644 (file)
@@ -36,7 +36,7 @@
 
 - name: Set bounce_interface if we just enabled dhcp
   set_fact:
-    bounce_interface: "{{ dhcp_enabled|changed }}"
+    bounce_interface: "{{ dhcp_enabled is changed }}"
 
 - name: ifdown and ifup
   shell: "ifdown {{ interface }} && ifup {{ interface }}"
index 08415081d90dc983b6d309732d99035822ca3a4b..84168621ecd9739045fb55b1d764420e3b3c9e95 100644 (file)
@@ -51,7 +51,7 @@
 - name: "Bounce {{ nic_to_configure }}"
   shell: "ifdown {{ nic_to_configure }} && ifup {{ nic_to_configure }}"
   when:
-    - wrote_network_config|changed
+    - wrote_network_config is changed
     - ansible_os_family == 'Debian'
 
 - name: "Write RHEL/CentOS network config for {{ nic_to_configure }}"
@@ -76,5 +76,5 @@
 - name: "Bounce {{ nic_to_configure }}"
   shell: "ifdown {{ nic_to_configure }} && ifup {{ nic_to_configure }}"
   when:
-    - wrote_network_config|changed
+    - wrote_network_config is changed
     - ansible_os_family == 'RedHat'
index 81b7d2dd51aefe364b97a415a891fbaa89318e3c..a0f14550584aaaccf53365e75397a7f42d3600c8 100644 (file)
@@ -23,7 +23,7 @@
     key=https://raw.githubusercontent.com/ceph/keys/autogenerated/ssh/@all.pub
   # Register and retry to work around transient githubusercontent.com issues
   register: ssh_key_update
-  until: ssh_key_update|success
+  until: ssh_key_update is success
   # try for 2 minutes to retrieve the key before failing
   retries: 24
   delay: 5
index cc127e8078a9bc321785bbac2d24b8bba0567081..ec83b9ae434db6ff9b8e199d0765acff4f14d986 100644 (file)
@@ -28,7 +28,7 @@
 - name: Delete the teuthology users password.
   command: "passwd -d {{ teuthology_user }}"
   when: teuthology_user_existence is defined and
-        teuthology_user_existence|changed
+        teuthology_user_existence is changed
 
 - name: Add a user for xfstests to test user quotas.
   user:
index 3e9a25b644f7ff9698a92363fb1cc75199619465..9ca8b3ce2d133f39b99e9b64612075c93bef81b9 100644 (file)
@@ -28,6 +28,6 @@
 
 - name: Clean yum cache
   shell: yum clean all
-  when: (repo_file is defined and repo_file|changed) or
-        (gpg_keys is defined and gpg_keys|changed) or
-        (version_repo_file is defined and version_repo_file|changed)
+  when: (repo_file is defined and repo_file is changed) or
+        (gpg_keys is defined and gpg_keys is changed) or
+        (version_repo_file is defined and version_repo_file is changed)
index b93cdb972d7fcccc138a453950f3d188f48a6947..8b16fd6ca35c8f94c933f04871db5c6b15bd1f19 100644 (file)
@@ -17,7 +17,7 @@
   connection: local
   run_once: true
   register: clone_keys
-  until: clone_keys|success
+  until: clone_keys is success
   retries: 5
   delay: 10
 
index db9baa1464ac9a2e4845a27babcb833e8469aa26..a04b8b6709497a6a6536263dc60acb399947f2d3 100644 (file)
@@ -14,4 +14,4 @@
 
 - name: Activate new network config
   shell: /sbin/ifdown -a; /sbin/ifup -a
-  when: interface_install|changed
+  when: interface_install is changed
index ec22341411d6710bb4ed1aee0d0259671dd5040d..8919c3e8933e78e4ed75d229847cc7bf55d3601e 100644 (file)
     shell: "ipmitool user set name {{ power_uid }} {{ power_user }}"
     when:
       - setup_user
-      - (set_username_locally is defined and set_username_locally|failed)
+      - (set_username_locally is defined and set_username_locally is failed)
 
   - name: Initial setup of permissions
     shell: "ipmitool channel setaccess {{ ipmi_channel_id }} {{ power_uid }} privilege=4"
     when:
       - setup_user
-      - (set_permissions_locally is defined and set_permissions_locally|failed)
+      - (set_permissions_locally is defined and set_permissions_locally is failed)
 
   - name: Initial setup of password
     shell: "ipmitool user set password {{ power_uid }} {{ power_pass }}"
     register: set_password_locally
     when:
       - setup_user
-      - (set_password_locally is defined and set_password_locally|failed)
+      - (set_password_locally is defined and set_password_locally is failed)
     ignore_errors: true
 
   - name: Set BMC to use DHCP via SSH
     register: set_to_dhcp_remotely
     when:
       - use_dhcp
-      - set_to_dhcp_locally|failed
+      - set_to_dhcp_locally is failed
 
   - name: Check existing network settings via SSH
     shell: "ipmitool lan print {{ ipmi_channel_id }} | grep 'IP Address Source\\|IP Address\\|Subnet Mask\\|Default Gateway IP\\|VLAN ID' | cut -d ':' -f2 | sed 's/^ //g'"
     shell: "ipmitool -I lanplus -U {{ power_user }} -P {{ power_pass }} -H {{ inventory_hostname_short }}.{{ ipmi_domain }} mc reset cold"
     delegate_to: localhost
     when: force_mc_reset or
-          (set_to_dhcp_locally is defined and set_to_dhcp_locally|changed) or
-          (set_to_dhcp_remotely is defined and set_to_dhcp_remotely|changed) or
-          (network_settings_change_required is defined and network_settings_change_required and not set_to_static|failed)
+          (set_to_dhcp_locally is defined and set_to_dhcp_locally is changed) or
+          (set_to_dhcp_remotely is defined and set_to_dhcp_remotely is changed) or
+          (network_settings_change_required is defined and network_settings_change_required and not set_to_static is failed)
index 3ea205df34013c111a3c4e9918010d50201cd3b4..28480b0e1ffa37546764a0c2aafb5f90719ed452 100644 (file)
@@ -17,4 +17,4 @@
 
   - name: Update firmware
     command: "isdct load -f -intelssd 0"
-    when: installed|changed
+    when: installed is changed