diff --git a/common/deploy-steps.j2 b/common/deploy-steps.j2 index 41ccdc19b2..ca91887e8f 100644 --- a/common/deploy-steps.j2 +++ b/common/deploy-steps.j2 @@ -21,8 +21,6 @@ {% set upgrade_steps_max = 6 -%} {% set external_upgrade_steps_max = 3 -%} {% set post_upgrade_steps_max = 4 -%} -{% set fast_forward_upgrade_steps_max = 9 -%} -{% set fast_forward_upgrade_prep_steps_max = 3 -%} {% set post_update_steps_max = 4 -%} {% set scale_steps_max = 1 -%} @@ -92,10 +90,6 @@ parameters: type: boolean default: false description: This flag is being used conditionally run a set of task needed to upgrade three releases within the deployment tasks. - FastForwardUpgradeReleases: - type: comma_delimited_list - default: ['rocky', 'stein', 'train'] - description: List of releases to fast forward through during upgrade. Last release in list is used for post steps. ContainerLogStdoutPath: type: string description: Absolute path for container stdout output (Podman only) @@ -161,7 +155,7 @@ parameters: Type of update, to differentiate between UPGRADE and UPDATE cases when StackAction is UPDATE (both are the same stack action). constraints: - - allowed_values: ['', 'UPGRADE', 'FASTFORWARDUPGRADE'] + - allowed_values: ['', 'UPGRADE'] default: '' ContainerCli: type: string @@ -1184,95 +1178,6 @@ outputs: when: ansible_hostname is not defined tags: - scale - fast_forward_upgrade_playbook: -{{ self.deploy_steps_str_replace_params() }} - template: | - - hosts: DEPLOY_TARGET_HOST - any_errors_fatal: yes - vars: - enable_paunch: ENABLE_PAUNCH - tasks: - - set_fact: - releases: {get_param: [FastForwardUpgradeReleases]} -{% raw %} - - set_fact: - ffu_releases: "{{ releases | difference( releases | last )}}" - - include_tasks: fast_forward_upgrade_release_tasks.yaml - loop_control: - loop_var: release - with_items: '{{ ffu_releases }}' - - set_fact: - release: "{{ releases | last }}" - ffu_packages_apply: True -{% endraw %} - - include_tasks: fast_forward_upgrade_post_role_tasks.yaml - fast_forward_upgrade_release_tasks: | - - include_tasks: fast_forward_upgrade_prep_tasks.yaml - - include_tasks: fast_forward_upgrade_bootstrap_tasks.yaml - fast_forward_upgrade_prep_tasks: | -{%- for role in roles %} - - shell: | - #!/bin/bash - if [ ! -f /root/.ffu_workaround ]; then - touch /root/.ffu_workaround - os-apply-config -m /var/lib/os-collect-config/{{role.deprecated_server_resource_name|default(role.name)}}Deployment.json - systemctl stop os-collect-config - rm -r /var/lib/os-collect-config/* - rm -f /usr/libexec/os-refresh-config/configure.d/40-hiera-datafiles - rm -f /usr/libexec/os-apply-config/templates/etc/puppet/hiera.yaml - rm -f /usr/libexec/os-refresh-config/configure.d/10-hiera-disable - fi - when: tripleo_role_name == '{{role.name}}' - name: Run Fast Forward Upgrade Prep Workarounds for {{role.name}} -{%- endfor %} - - name: Create /var/lib/container-puppet - file: path=/var/lib/container-puppet state=directory setype=svirt_sandbox_file_t selevel=s0 recurse=true - - name: Write container-puppet.py if Paunch is enabled - no_log: True - copy: src=docker_puppet_script.yaml dest=/var/lib/container-puppet/container-puppet.py force=yes mode=0600 - when: - - enable_paunch|bool - - name: Write container-puppet.sh - no_log: True - copy: src=container_puppet_script.yaml dest=/var/lib/container-puppet/container-puppet.sh force=yes mode=0755 setype=container_file_t - - include_tasks: fast_forward_upgrade_prep_role_tasks.yaml - with_sequence: start=0 end={{fast_forward_upgrade_prep_steps_max}} - loop_control: - loop_var: step - fast_forward_upgrade_post_role_tasks: | -{%- for role in roles %} - - include_tasks: {{role.name}}/fast_forward_post_upgrade_tasks.yaml - when: tripleo_role_name == '{{role.name}}' - tags: - - always -{%- endfor %} - - name: Openstack Heat Agents package update - package: name=openstack-heat-agents state=latest - - name: Update os-collect-config - package: name=os-collect-config state=latest - - name: Start os-collect-config back up - service: name=os-collect-config state=started enabled=yes - fast_forward_upgrade_prep_role_tasks: | -{%- for role in roles %} - - include_tasks: {{role.name}}/fast_forward_upgrade_tasks.yaml - when: tripleo_role_name == '{{role.name}}' - tags: - - always -{%- endfor %} - fast_forward_upgrade_bootstrap_tasks: | - - include_tasks: fast_forward_upgrade_bootstrap_role_tasks.yaml - with_sequence: start={{fast_forward_upgrade_prep_steps_max+1}} end={{fast_forward_upgrade_steps_max}} - loop_control: - loop_var: step - fast_forward_upgrade_bootstrap_role_tasks: | -{%- for role in roles %} - - include_tasks: {{role.name}}/fast_forward_upgrade_tasks.yaml - when: - - tripleo_role_name == '{{role.name}}' - - is_bootstrap_node|bool - tags: - - always -{%- endfor %} post_update_steps_tasks: | {%- for role in roles %} - include_tasks: {{role.name}}/post_update_tasks.yaml diff --git a/common/services/role.role.j2.yaml b/common/services/role.role.j2.yaml index c63b475959..8566a751eb 100644 --- a/common/services/role.role.j2.yaml +++ b/common/services/role.role.j2.yaml @@ -212,24 +212,6 @@ resources: expression: coalesce($.data, []).where($ != null).select($.get('external_upgrade_tasks')).where($ != null).flatten().distinct() data: {get_attr: [ServiceChain, role_data]} - FastForwardUpgradeTasks: - type: OS::Heat::Value - properties: - type: comma_delimited_list - value: - yaql: - expression: coalesce($.data, []).where($ != null).select($.get('fast_forward_upgrade_tasks')).where($ != null).flatten().distinct() - data: {get_attr: [ServiceChain, role_data]} - - FastForwardPostUpgradeTasks: - type: OS::Heat::Value - properties: - type: comma_delimited_list - value: - yaql: - expression: coalesce($.data, []).where($ != null).select($.get('fast_forward_post_upgrade_tasks')).where($ != null).flatten().distinct() - data: {get_attr: [ServiceChain, role_data]} - PreUpgradeRollingTasks: type: OS::Heat::Value properties: @@ -371,8 +353,6 @@ outputs: external_update_tasks: {get_attr: [ExternalUpdateTasks, value]} external_upgrade_tasks: {get_attr: [ExternalUpgradeTasks, value]} scale_tasks: {get_attr: [ScaleTasks, value]} - fast_forward_upgrade_tasks: {get_attr: [FastForwardUpgradeTasks, value]} - fast_forward_post_upgrade_tasks: {get_attr: [FastForwardPostUpgradeTasks, value]} pre_upgrade_rolling_tasks: {get_attr: [PreUpgradeRollingTasks, value]} upgrade_tasks: {get_attr: [UpgradeTasks, value]} post_upgrade_tasks: {get_attr: [PostUpgradeTasks, value]} diff --git a/deployment/README.rst b/deployment/README.rst index 14c63a0b72..f30525bfa3 100644 --- a/deployment/README.rst +++ b/deployment/README.rst @@ -176,36 +176,6 @@ all containerized services and the updated configuration if any. Note: as pacemaker is not containerized, the points 1 and 4 happen in deployment/pacemaker/pacemaker-baremetal-puppet.yaml. -Fast-forward Upgrade Steps --------------------------- - -Each service template may optionally define a `fast_forward_upgrade_tasks` key, -which is a list of Ansible tasks to be performed during the fast-forward -upgrade process. As with Upgrade steps each task is associated to a particular -step provided as a variable and used along with a release variable by a basic -conditional that determines when the task should run. - -Steps are broken down into two categories, prep tasks executed across all hosts -and bootstrap tasks executed on a single host for a given role. - -The individual steps then correspond to the following tasks during the upgrade: - -Prep steps: - -- Step=0: Check running services -- Step=1: Stop the service -- Step=2: Stop the cluster -- Step=3: Update repos - -Bootstrap steps: - -- Step=4: DB backups -- Step=5: Pre package update commands -- Step=6: Package updates -- Step=7: Post package update commands -- Step=8: DB syncs -- Step=9: Verification - Input Parameters ---------------- diff --git a/deployment/aodh/aodh-api-container-puppet.yaml b/deployment/aodh/aodh-api-container-puppet.yaml index 5b0e1add6d..5ff88c42c4 100644 --- a/deployment/aodh/aodh-api-container-puppet.yaml +++ b/deployment/aodh/aodh-api-container-puppet.yaml @@ -273,49 +273,3 @@ outputs: tripleo_containers_to_stop: - aodh_api tripleo_delegate_to: "{{ groups['aodh_api'] | default([]) }}" - fast_forward_upgrade_tasks: - - when: - - step|int == 0 - - release == 'rocky' - block: - - name: Check for aodh running under apache - tags: common - shell: |- - set -o pipefail - httpd -t -D DUMP_VHOSTS | grep -q aodh_wsgi - register: aodh_httpd_enabled_result - - name: Set fact aodh_httpd_enabled - set_fact: - aodh_httpd_enabled: "{{ aodh_httpd_enabled_result.rc == 0 }}" - - name: Check if httpd is running - failed_when: false - command: systemctl is-active --quiet httpd - register: httpd_running_result - when: - - httpd_running is undefined - - name: Set fact httpd_running if undefined - set_fact: - httpd_running: "{{ httpd_running_result.rc == 0 }}" - when: - - httpd_running is undefined - - name: Stop and disable aodh (under httpd) - service: name=httpd state=stopped enabled=no - when: - - step|int == 1 - - release == 'rocky' - - aodh_httpd_enabled|bool - - httpd_running|bool - - name: Aodh package update - package: - name: 'openstack-aodh*' - state: latest - when: - - step|int == 6 - - is_bootstrap_node|bool - - aodh_httpd_enabled|bool - - name: aodh db sync - command: aodh-dbsync - when: - - step|int == 8 - - is_bootstrap_node|bool - - aodh_httpd_enabled|bool diff --git a/deployment/aodh/aodh-evaluator-container-puppet.yaml b/deployment/aodh/aodh-evaluator-container-puppet.yaml index 1b57137625..07085bc9dd 100644 --- a/deployment/aodh/aodh-evaluator-container-puppet.yaml +++ b/deployment/aodh/aodh-evaluator-container-puppet.yaml @@ -131,21 +131,3 @@ outputs: tripleo_containers_to_stop: - aodh_evaluator tripleo_delegate_to: "{{ groups['aodh_evaluator'] | default([]) }}" - fast_forward_upgrade_tasks: - - when: - - step|int == 0 - - release == 'rocky' - block: - - name: FFU check if openstack-aodh-evaluator is deployed - command: systemctl is-enabled --quiet openstack-aodh-evaluator - failed_when: false - register: aodh_evaluator_enabled_result - - name: Set fact aodh_evaluator_enabled - set_fact: - aodh_evaluator_enabled: "{{ aodh_evaluator_enabled_result.rc == 0 }}" - - name: FFU stop and disable openstack-aodh-evaluator service - service: name=openstack-aodh-evaluator state=stopped enabled=no - when: - - step|int == 1 - - release == 'rocky' - - aodh_evaluator_enabled|bool diff --git a/deployment/aodh/aodh-listener-container-puppet.yaml b/deployment/aodh/aodh-listener-container-puppet.yaml index a592fcc445..4940d3d9fa 100644 --- a/deployment/aodh/aodh-listener-container-puppet.yaml +++ b/deployment/aodh/aodh-listener-container-puppet.yaml @@ -131,21 +131,3 @@ outputs: tripleo_containers_to_stop: - aodh_listener tripleo_delegate_to: "{{ groups['aodh_listener'] | default([]) }}" - fast_forward_upgrade_tasks: - - when: - - step|int == 0 - - release == 'rocky' - block: - - name: FFU check if openstack-aodh-listener is deployed - command: systemctl is-enabled --quiet openstack-aodh-listener - failed_when: false - register: aodh_listener_enabled_result - - name: Set fact aodh_listener_enabled - set_fact: - aodh_listener_enabled: "{{ aodh_listener_enabled_result.rc == 0 }}" - - name: FFU stop and disable openstack-aodh-listener service - service: name=openstack-aodh-listener state=stopped enabled=no - when: - - step|int == 1 - - release == 'rocky' - - aodh_listener_enabled|bool diff --git a/deployment/aodh/aodh-notifier-container-puppet.yaml b/deployment/aodh/aodh-notifier-container-puppet.yaml index 945cbb7eed..c48e0364a7 100644 --- a/deployment/aodh/aodh-notifier-container-puppet.yaml +++ b/deployment/aodh/aodh-notifier-container-puppet.yaml @@ -132,21 +132,3 @@ outputs: tripleo_containers_to_stop: - aodh_notifier tripleo_delegate_to: "{{ groups['aodh_notifier'] | default([]) }}" - fast_forward_upgrade_tasks: - - when: - - step|int == 0 - - release == 'rocky' - block: - - name: FFU check if openstack-aodh-notifier is deployed - command: systemctl is-enabled --quiet openstack-aodh-notifier - failed_when: false - register: aodh_notifier_enabled_result - - name: Set fact aodh_notifier_enabled - set_fact: - aodh_notifier_enabled: "{{ aodh_notifier_enabled_result.rc == 0 }}" - - name: FFU stop and disable openstack-aodh-notifier service - service: name=openstack-aodh-notifier state=stopped enabled=no - when: - - step|int == 1 - - release == 'rocky' - - aodh_notifier_enabled|bool diff --git a/deployment/ceilometer/ceilometer-agent-central-container-puppet.yaml b/deployment/ceilometer/ceilometer-agent-central-container-puppet.yaml index c2342c96f3..709f868370 100644 --- a/deployment/ceilometer/ceilometer-agent-central-container-puppet.yaml +++ b/deployment/ceilometer/ceilometer-agent-central-container-puppet.yaml @@ -187,21 +187,3 @@ outputs: tripleo_containers_to_stop: - ceilometer_agent_central tripleo_delegate_to: "{{ groups['ceilometer_agent_central'] | default([]) }}" - fast_forward_upgrade_tasks: - - when: - - step|int == 0 - - release == 'rocky' - block: - - name: FFU check if openstack-ceilometer-central is deployed - command: systemctl is-enabled --quiet openstack-ceilometer-central - failed_when: false - register: ceilometer_agent_central_enabled_result - - name: Set fact ceilometer_agent_central_enabled - set_fact: - ceilometer_agent_central_enabled: "{{ ceilometer_agent_central_enabled_result.rc == 0 }}" - - name: FFU stop and disable openstack-ceilometer-central service - service: name=openstack-ceilometer-central state=stopped enabled=no - when: - - step|int == 1 - - release == 'rocky' - - ceilometer_agent_central_enabled|bool diff --git a/deployment/ceilometer/ceilometer-agent-compute-container-puppet.yaml b/deployment/ceilometer/ceilometer-agent-compute-container-puppet.yaml index ecb94dca8a..ebd7012a91 100644 --- a/deployment/ceilometer/ceilometer-agent-compute-container-puppet.yaml +++ b/deployment/ceilometer/ceilometer-agent-compute-container-puppet.yaml @@ -126,21 +126,3 @@ outputs: name: virt_sandbox_use_netlink persistent: yes state: yes - fast_forward_upgrade_tasks: - - when: - - step|int == 0 - - release == 'rocky' - block: - - name: FFU check if openstack-ceilometer-compute is deployed - command: systemctl is-enabled openstack-ceilometer-compute - failed_when: false - register: ceilometer_agent_compute_enabled_result - - name: Set fact ceilometer_agent_compute_enabled - set_fact: - ceilometer_agent_compute_enabled: "{{ ceilometer_agent_compute_enabled_result.rc == 0 }}" - - name: FFU stop and disable openstack-ceilometer-compute service - when: - - step|int == 1 - - release == 'rocky' - - ceilometer_agent_compute_enabled|bool - service: name=openstack-ceilometer-compute state=stopped enabled=no diff --git a/deployment/ceilometer/ceilometer-agent-ipmi-container-puppet.yaml b/deployment/ceilometer/ceilometer-agent-ipmi-container-puppet.yaml index c0de9f2904..8680acf6b6 100644 --- a/deployment/ceilometer/ceilometer-agent-ipmi-container-puppet.yaml +++ b/deployment/ceilometer/ceilometer-agent-ipmi-container-puppet.yaml @@ -139,21 +139,3 @@ outputs: mode: "{{ item.mode }}" with_items: - { 'path': /var/log/containers/ceilometer, 'setype': svirt_sandbox_file_t, 'mode': '0750' } - fast_forward_upgrade_tasks: - - when: - - step|int == 0 - - release == 'rocky' - block: - - name: FFU check if openstack-ceilometer-ipmi is enabled - command: systemctl is-enabled openstack-ceilometer-ipmi - failed_when: false - register: ceilometer_ipmi_enabled_result - - name: Set fact ceilometer_ipmi_enabled - set_fact: - ceilometer_ipmi_enabled: "{{ ceilometer_ipmi_enabled_result.rc == 0 }}" - - name: FFU stop and disable openstack-ceilometer-ipmi service - when: - - step|int == 1 - - release == 'rocky' - - ceilometer_ipmi_enabled|bool - service: name=openstack-ceilometer-ipmi state=stopped enabled=no diff --git a/deployment/ceilometer/ceilometer-agent-notification-container-puppet.yaml b/deployment/ceilometer/ceilometer-agent-notification-container-puppet.yaml index 5819c8038f..a88d9ff653 100644 --- a/deployment/ceilometer/ceilometer-agent-notification-container-puppet.yaml +++ b/deployment/ceilometer/ceilometer-agent-notification-container-puppet.yaml @@ -250,21 +250,3 @@ outputs: tripleo_containers_to_stop: - ceilometer_agent_notification tripleo_delegate_to: "{{ groups['ceilometer_agent_notification'] | default([]) }}" - fast_forward_upgrade_tasks: - - when: - - step|int == 0 - - release == 'rocky' - block: - - name: FFU check if openstack-ceilometer-notification is deployed - command: systemctl is-enabled openstack-ceilometer-notification - failed_when: false - register: ceilometer_agent_notification_enabled_result - - name: Set fact ceilometer_agent_notification_enabled - set_fact: - ceilometer_agent_notification_enabled: "{{ ceilometer_agent_notification_enabled_result.rc == 0 }}" - - name: FFU stop and diable openstack-ceilometer-notification service - when: - - step|int == 1 - - release == 'rocky' - - ceilometer_agent_notification_enabled|bool - service: name=openstack-ceilometer-notification state=stopped enabled=no diff --git a/deployment/ceph-ansible/ceph-base.yaml b/deployment/ceph-ansible/ceph-base.yaml index e79a7dab2d..1a044b130b 100644 --- a/deployment/ceph-ansible/ceph-base.yaml +++ b/deployment/ceph-ansible/ceph-base.yaml @@ -44,7 +44,7 @@ parameters: Type of update, to differentiate between UPGRADE and UPDATE cases when StackAction is UPDATE (both are the same stack action). constraints: - - allowed_values: ['', 'UPGRADE', 'FASTFORWARDUPGRADE'] + - allowed_values: ['', 'UPGRADE'] default: '' NodeDataLookup: type: json diff --git a/deployment/cinder/cinder-api-container-puppet.yaml b/deployment/cinder/cinder-api-container-puppet.yaml index 75304dbea0..6e570bbf82 100644 --- a/deployment/cinder/cinder-api-container-puppet.yaml +++ b/deployment/cinder/cinder-api-container-puppet.yaml @@ -461,52 +461,6 @@ outputs: - cinder_api - cinder_api_cron tripleo_delegate_to: "{{ groups['cinder_api'] | default([]) }}" - fast_forward_upgrade_tasks: - - when: - - step|int == 0 - - release == 'rocky' - block: - - name: Check is cinder_api is deployed - command: systemctl is-enabled --quiet openstack-cinder-api - failed_when: false - register: cinder_api_enabled_result - - name: Set fact cinder_api_enabled - set_fact: - cinder_api_enabled: "{{ cinder_api_enabled_result.rc == 0 }}" - - name: Stop openstack-cinder-api - service: name=openstack-cinder-api state=stopped - when: - - step|int == 1 - - release == 'rocky' - - cinder_api_enabled|bool - - name: Extra removal of services for cinder - shell: | - cinder-manage service list |\ - grep -v Binary | tr '@' ' ' |\ - awk '{print $1 " " $2}' |\ - while read i ; do cinder-manage service remove $i ; done - when: - - step|int == 5 - - release == 'stein' - - is_bootstrap_node|bool - - name: Extra migration for cinder - command: cinder-manage db online_data_migrations - when: - - step|int == 5 - - release == 'stein' - - is_bootstrap_node|bool - - name: Cinder package update - package: - name: 'openstack-cinder*' - state: latest - when: - - step|int == 6 - - is_bootstrap_node|bool - - name: Cinder db sync - command: cinder-manage db sync - when: - - step|int == 8 - - is_bootstrap_node|bool external_deploy_tasks: - name: Manage Cinder Volume Type become: true diff --git a/deployment/cinder/cinder-backup-pacemaker-puppet.yaml b/deployment/cinder/cinder-backup-pacemaker-puppet.yaml index 8055918806..1919908ba5 100644 --- a/deployment/cinder/cinder-backup-pacemaker-puppet.yaml +++ b/deployment/cinder/cinder-backup-pacemaker-puppet.yaml @@ -393,33 +393,3 @@ outputs: when: - step|int == 3 block: *cinder_backup_fetch_retag_container_tasks - - fast_forward_upgrade_tasks: - - when: - - step|int == 0 - - release == 'rocky' - - is_bootstrap_node|bool - block: - - name: Check cluster resource status - pacemaker_resource: - resource: openstack-cinder-backup - state: show - check_mode: false - failed_when: false - register: cinder_backup_res_result - - name: Set fact cinder_backup_res - set_fact: - cinder_backup_res: "{{ cinder_backup_res_result.rc == 0 }}" - - name: Disable the openstack-cinder-backup cluster resource - pacemaker_resource: - resource: openstack-cinder-backup - state: disable - wait_for_resource: true - register: cinder_backup_output - retries: 5 - until: cinder_backup_output.rc == 0 - when: - - step|int == 2 - - release == 'rocky' - - is_bootstrap_node|bool - - cinder_backup_res|bool diff --git a/deployment/cinder/cinder-scheduler-container-puppet.yaml b/deployment/cinder/cinder-scheduler-container-puppet.yaml index c9d618efe2..d2bc561390 100644 --- a/deployment/cinder/cinder-scheduler-container-puppet.yaml +++ b/deployment/cinder/cinder-scheduler-container-puppet.yaml @@ -158,21 +158,3 @@ outputs: tripleo_containers_to_stop: - cinder_scheduler tripleo_delegate_to: "{{ groups['cinder_scheduler'] | default([]) }}" - fast_forward_upgrade_tasks: - - when: - - step|int == 0 - - release == 'rocky' - block: - - name: Check if cinder_scheduler is deployed - command: systemctl is-enabled --quiet openstack-cinder-scheduler - failed_when: false - register: cinder_scheduler_enabled_result - - name: Set fact cinder_scheduler_enabled - set_fact: - cinder_scheduler_enabled: "{{ cinder_scheduler_enabled_result.rc == 0 }}" - - name: Stop openstack-cinder-scheduler - service: name=openstack-cinder-scheduler state=stopped enabled=no - when: - - step|int == 1 - - release == 'rocky' - - cinder_scheduler_enabled|bool diff --git a/deployment/cinder/cinder-volume-container-puppet.yaml b/deployment/cinder/cinder-volume-container-puppet.yaml index f87840fbca..b9cea65799 100644 --- a/deployment/cinder/cinder-volume-container-puppet.yaml +++ b/deployment/cinder/cinder-volume-container-puppet.yaml @@ -362,21 +362,3 @@ outputs: when: - cvol_active_active_tls_enabled|bool - step|int == 3 - fast_forward_upgrade_tasks: - - when: - - step|int == 0 - - release == 'rocky' - block: - - name: Check if cinder_volume is deployed - command: systemctl is-enabled --quiet openstack-cinder-volume - failed_when: false - register: cinder_volume_enabled_result - - name: Set fact cinder_volume_enabled - set_fact: - cinder_volume_enabled: "{{ cinder_volume_enabled_result.rc == 0 }}" - - name: Stop openstack-cinder-volume - service: name=openstack-cinder-volume state=stopped enabled=no - when: - - step|int == 1 - - release == 'rocky' - - cinder_volume_enabled|bool diff --git a/deployment/cinder/cinder-volume-pacemaker-puppet.yaml b/deployment/cinder/cinder-volume-pacemaker-puppet.yaml index 6b8b032e6e..6ca8ebca61 100644 --- a/deployment/cinder/cinder-volume-pacemaker-puppet.yaml +++ b/deployment/cinder/cinder-volume-pacemaker-puppet.yaml @@ -386,33 +386,3 @@ outputs: pacemaker_resource: resource: openstack-cinder-volume state: enable - - fast_forward_upgrade_tasks: - - when: - - step|int == 0 - - release == 'rocky' - - is_bootstrap_node|bool - block: - - name: Check cluster resource status - pacemaker_resource: - resource: openstack-cinder-volume - state: show - check_mode: false - failed_when: false - register: cinder_volume_res_result - - name: Set fact cinder_volume_res - set_fact: - cinder_volume_res: "{{ cinder_volume_res_result.rc == 0 }}" - - name: Disable the openstack-cinder-volume cluster resource - pacemaker_resource: - resource: openstack-cinder-volume - state: disable - wait_for_resource: true - register: cinder_volume_output - retries: 5 - until: cinder_volume_output.rc == 0 - when: - - step|int == 2 - - release == 'rocky' - - is_bootstrap_node|bool - - cinder_volume_res|bool diff --git a/deployment/database/mysql-container-puppet.yaml b/deployment/database/mysql-container-puppet.yaml index d15eda82a3..27064fa1a0 100644 --- a/deployment/database/mysql-container-puppet.yaml +++ b/deployment/database/mysql-container-puppet.yaml @@ -338,26 +338,3 @@ outputs: - name: Check and upgrade Mysql database after major version upgrade command: "{{ container_cli }} exec -u root mysql mysql_upgrade" when: step|int == 2 - fast_forward_upgrade_tasks: - - when: - - step|int == 6 - - release == 'rocky' - - is_bootstrap_node|bool - block: - - name: Remove whitespaces in .my.cnf - replace: - path: /root/.my.cnf - regexp: '^ +' - replace: '' - - name: Create cell0 db - mysql_db: - name: nova_cell0 - state: present - login_unix_socket: '/var/lib/mysql/mysql.sock' - - name: Grant access to cell0 db - mysql_user: - name: nova - host_all: yes - state: present - priv: '*.*:ALL' - login_unix_socket: '/var/lib/mysql/mysql.sock' diff --git a/deployment/database/mysql-pacemaker-puppet.yaml b/deployment/database/mysql-pacemaker-puppet.yaml index 9aa45ddfad..d82bc436f0 100644 --- a/deployment/database/mysql-pacemaker-puppet.yaml +++ b/deployment/database/mysql-pacemaker-puppet.yaml @@ -626,27 +626,3 @@ outputs: tripleo_transfer_dest_dir: /var/lib/mysql tripleo_transfer_dest_host: "{{hostvars[groups['overcloud'][0]]['mysql_short_bootstrap_node_name']}}" tripleo_transfer_flag_file: /var/lib/tripleo/transfer-flags/var-lib-mysql - - fast_forward_upgrade_tasks: - - when: - - step|int == 6 - - release == 'rocky' - - is_bootstrap_node|bool - block: - - name: Remove whitespaces in .my.cnf - replace: - path: /root/.my.cnf - regexp: '^ +' - replace: '' - - name: Create cell0 db - mysql_db: - name: nova_cell0 - state: present - login_unix_socket: '/var/lib/mysql/mysql.sock' - - name: Grant access to cell0 db - mysql_user: - name: nova - host_all: yes - state: present - priv: '*.*:ALL' - login_unix_socket: '/var/lib/mysql/mysql.sock' diff --git a/deployment/database/redis-pacemaker-puppet.yaml b/deployment/database/redis-pacemaker-puppet.yaml index 744ae6bd7a..70373685ab 100644 --- a/deployment/database/redis-pacemaker-puppet.yaml +++ b/deployment/database/redis-pacemaker-puppet.yaml @@ -462,33 +462,3 @@ outputs: when: - step|int == 3 block: *redis_fetch_retag_container_tasks - - fast_forward_upgrade_tasks: - - when: - - step|int == 0 - - release == 'rocky' - - is_bootstrap_node|bool - block: - - name: Check cluster resource status of redis - pacemaker_resource: - resource: {get_attr: [RedisBase, role_data, service_name]} - state: show - check_mode: false - failed_when: false - register: redis_res_result - - name: Set fact redis_res - set_fact: - redis_res: "{{ redis_res_result.rc == 0 }}" - - name: Disable the redis cluster resource - pacemaker_resource: - resource: {get_attr: [RedisBase, role_data, service_name]} - state: disable - wait_for_resource: true - register: redis_output - retries: 5 - until: redis_output.rc == 0 - when: - - step|int == 2 - - release == 'rocky' - - is_bootstrap_node|bool - - redis_res|bool diff --git a/deployment/glance/glance-api-container-puppet.yaml b/deployment/glance/glance-api-container-puppet.yaml index db1ada0c59..5416c3261b 100644 --- a/deployment/glance/glance-api-container-puppet.yaml +++ b/deployment/glance/glance-api-container-puppet.yaml @@ -743,31 +743,3 @@ outputs: tripleo_containers_to_stop: - glance_api tripleo_delegate_to: "{{ groups['glance_api'] | default([]) }}" - fast_forward_upgrade_tasks: - - when: - - step|int == 0 - - release == 'rocky' - block: - - name: Check if glance_api is deployed - command: systemctl is-enabled --quiet openstack-glance-api - failed_when: false - register: glance_api_enabled_result - - name: Set fact glance_api_enabled - set_fact: - glance_api_enabled: "{{ glance_api_enabled_result.rc == 0 }}" - - name: Stop openstack-glance-api - service: name=openstack-glance-api state=stopped enabled=no - when: - - step|int == 1 - - release == 'rocky' - - glance_api_enabled|bool - - name: glance package update - package: name=openstack-glance state=latest - when: - - step|int == 6 - - is_bootstrap_node|bool - - name: glance db sync - command: glance-manage db_sync - when: - - step|int == 8 - - is_bootstrap_node|bool diff --git a/deployment/gnocchi/gnocchi-api-container-puppet.yaml b/deployment/gnocchi/gnocchi-api-container-puppet.yaml index 7826246a6b..78574a069a 100644 --- a/deployment/gnocchi/gnocchi-api-container-puppet.yaml +++ b/deployment/gnocchi/gnocchi-api-container-puppet.yaml @@ -391,67 +391,3 @@ outputs: tripleo_containers_to_stop: - gnocchi_api tripleo_delegate_to: "{{ groups['gnocchi_api'] | default([]) }}" - fast_forward_upgrade_tasks: - - when: - - step|int == 0 - - release == 'rocky' - block: - - name: Check if httpd service is running - command: systemctl is-active --quiet httpd - tags: common - failed_when: false - register: httpd_running_result - when: - - httpd_running is undefined - - name: Set fact httpd_running if unset - set_fact: - httpd_running: "{{ httpd_running_result.rc == 0 }}" - when: - - httpd_running is undefined - - name: Check if gnocchi_api is deployed - command: systemctl is-enabled --quiet openstack-gnocchi-api - tags: common - failed_when: false - register: gnocchi_api_enabled_result - - name: Set fact gnocchi_api_enabled - set_fact: - gnocchi_api_enabled: "{{ gnocchi_api_enabled_result.rc == 0 }}" - - name: Check for gnocchi_api running under apache - tags: common - shell: |- - set -o pipefail - httpd -t -D DUMP_VHOSTS | grep -q gnocchi - register: gnocchi_httpd_enabled_result - - name: Set fact gnocchi_httpd_enabled - set_fact: - gnocchi_httpd_enabled: "{{ gnocchi_httpd_enabled_result.rc == 0 }}" - - name: Stop and disable gnocchi_api service - service: name=openstack-gnocchi-api state=stopped enabled=no - when: - - step|int == 1 - - release == 'rocky' - - gnocchi_api_enabled|bool - - name: Stop and disable httpd service - when: - - step|int == 1 - - release == 'rocky' - - gnocchi_httpd_enabled|bool - - httpd_running|bool - service: name=httpd state=stopped enabled=no - - name: Update gnocchi packages - package: - name: - - openstack-gnocchi* - - numpy - state: latest - # (pradk): We have to explicitly update numpy as its obsoleted - # by python2-numpy. - when: - - step|int == 6 - - is_bootstrap_node|bool - - name: Sync gnocchi DB - command: gnocchi-upgrade --skip-storage - when: - - step|int == 8 - - is_bootstrap_node|bool - - gnocchi_api_enabled|bool diff --git a/deployment/gnocchi/gnocchi-metricd-container-puppet.yaml b/deployment/gnocchi/gnocchi-metricd-container-puppet.yaml index 950d00824e..4096e65d00 100644 --- a/deployment/gnocchi/gnocchi-metricd-container-puppet.yaml +++ b/deployment/gnocchi/gnocchi-metricd-container-puppet.yaml @@ -186,21 +186,3 @@ outputs: tripleo_containers_to_stop: - gnocchi_metricd tripleo_delegate_to: "{{ groups['gnocchi_metricd'] | default([]) }}" - fast_forward_upgrade_tasks: - - when: - - step|int == 0 - - release == 'rocky' - block: - - name: FFU check if openstack-gnocchi-metricd is deployed - command: systemctl is-enabled --quiet openstack-gnocchi-metricd - failed_when: false - register: gnocchi_metricd_enabled_result - - name: Set fact gnocchi_metricd_enabled - set_fact: - gnocchi_metricd_enabled: "{{ gnocchi_metricd_enabled_result.rc == 0 }}" - - name: FFU stop and disable openstack-gnocchi-metricd service - service: name=openstack-gnocchi-metricd state=stopped enabled=no - when: - - step|int == 1 - - release == 'rocky' - - gnocchi_metricd_enabled|bool diff --git a/deployment/gnocchi/gnocchi-statsd-container-puppet.yaml b/deployment/gnocchi/gnocchi-statsd-container-puppet.yaml index eb2930aa08..3d5cfe28b3 100644 --- a/deployment/gnocchi/gnocchi-statsd-container-puppet.yaml +++ b/deployment/gnocchi/gnocchi-statsd-container-puppet.yaml @@ -180,23 +180,3 @@ outputs: tripleo_containers_to_stop: - gnocchi_statsd tripleo_delegate_to: "{{ groups['gnocchi_statsd'] | default([]) }}" - fast_forward_upgrade_tasks: - - name: FFU check if openstack-gnocchi-statsd is deployed - command: systemctl is-enabled --quiet openstack-gnocchi-statsd - failed_when: false - register: gnocchi_statsd_enabled_result - when: - - step|int == 0 - - release == 'rocky' - - name: Set fact gnocchi_statsd_enabled - set_fact: - gnocchi_statsd_enabled: "{{ gnocchi_statsd_enabled_result.rc == 0 }}" - when: - - step|int == 0 - - release == 'rocky' - - name: FFU stop and disable openstack-gnocchi-statsd service - service: name=openstack-gnocchi-statsd state=stopped enabled=no - when: - - step|int == 2 - - release == 'rocky' - - gnocchi_statsd_enabled|bool diff --git a/deployment/heat/heat-api-cfn-container-puppet.yaml b/deployment/heat/heat-api-cfn-container-puppet.yaml index d4b8cc712e..edafe9b46d 100644 --- a/deployment/heat/heat-api-cfn-container-puppet.yaml +++ b/deployment/heat/heat-api-cfn-container-puppet.yaml @@ -236,21 +236,3 @@ outputs: tripleo_containers_to_stop: - heat_api_cfn tripleo_delegate_to: "{{ groups['heat_api_cfn'] | default([]) }}" - fast_forward_upgrade_tasks: - - when: - - step|int == 0 - - release == 'rocky' - block: - - name: FFU check if openstack-heat-api-cfn service is enabled - command: systemctl is-enabled openstack-heat-api-cfn - failed_when: false - register: heat_api_cfn_enabled_result - - name: Set fact heat_api_cfn_enabled - set_fact: - heat_api_cfn_enabled: "{{ heat_api_cfn_enabled_result.rc == 0 }}" - - name: FFU stop and disable openstack-heat-api-cfn service - when: - - step|int == 1 - - release == 'rocky' - - heat_api_cfn_enabled|bool - service: name=openstack-heat-api-cfn state=stopped enabled=no diff --git a/deployment/heat/heat-api-cloudwatch-disabled-puppet.yaml b/deployment/heat/heat-api-cloudwatch-disabled-puppet.yaml index b3fd79b588..ce9232d957 100644 --- a/deployment/heat/heat-api-cloudwatch-disabled-puppet.yaml +++ b/deployment/heat/heat-api-cloudwatch-disabled-puppet.yaml @@ -37,27 +37,3 @@ outputs: value: service_name: heat_api_cloudwatch_disabled upgrade_tasks: [] - fast_forward_upgrade_tasks: - - when: - - step|int == 0 - - release == 'rocky' - block: - - name: FFU check if heat_api_cloudwatch is deployed - command: systemctl is-enabled openstack-heat-api-cloudwatch - failed_when: false - register: heat_api_cloudwatch_enabled_result - - name: Set fact heat_api_cloudwatch_enabled - set_fact: - heat_api_cloudwatch_enabled: "{{ heat_api_cloudwatch_enabled_result.rc == 0 }}" - - name: FFU stop and disable the heat-api-cloudwatch service. - service: name=openstack-heat-api-cloudwatch state=stopped enabled=no - when: - - step|int == 1 - - release == 'rocky' - - heat_api_cloudwatch_enabled|bool - - name: Remove heat_api_cloudwatch package - package: name=openstack-heat-api-cloudwatch state=removed - failed_when: false - when: - - step|int == 2 - - release == 'rocky' diff --git a/deployment/heat/heat-api-container-puppet.yaml b/deployment/heat/heat-api-container-puppet.yaml index 7797256e41..f016d207a6 100644 --- a/deployment/heat/heat-api-container-puppet.yaml +++ b/deployment/heat/heat-api-container-puppet.yaml @@ -298,33 +298,3 @@ outputs: - heat_api - heat_api_cron tripleo_delegate_to: "{{ groups['heat_api'] | default([]) }}" - fast_forward_upgrade_tasks: - - when: - - step|int == 0 - - release == 'rocky' - block: - - name: FFU check openstack-heat-api is enabled - command: systemctl is-enabled openstack-heat-api - failed_when: false - register: heat_api_enabled_result - - name: Set fact heat_api_enabled - set_fact: - heat_api_enabled: "{{ heat_api_enabled_result.rc == 0 }}" - - name: FFU stop and disable openstack-heat-api - service: name=openstack-heat-api state=stopped enabled=no - when: - - step|int == 1 - - release == 'rocky' - - heat_api_enabled|bool - - name: FFU Heat package update - package: - name: 'openstack-heat*' - state: latest - when: - - step|int == 6 - - is_bootstrap_node|bool - - name: FFU Heat db-sync - command: heat-manage db_sync - when: - - step|int == 8 - - is_bootstrap_node|bool diff --git a/deployment/heat/heat-engine-container-puppet.yaml b/deployment/heat/heat-engine-container-puppet.yaml index 60f08c7476..fc42d27441 100644 --- a/deployment/heat/heat-engine-container-puppet.yaml +++ b/deployment/heat/heat-engine-container-puppet.yaml @@ -310,21 +310,3 @@ outputs: tripleo_containers_to_stop: - heat_engine tripleo_delegate_to: "{{ groups['heat_engine'] | default([]) }}" - fast_forward_upgrade_tasks: - - when: - - step|int == 0 - - release == 'rocky' - block: - - name: FFU check if openstack-heat-engine is enabled - command: systemctl is-enabled --quiet openstack-heat-engine - failed_when: false - register: heat_engine_enabled_result - - name: Set fact heat_engine_enabled - set_fact: - heat_engine_enabled: "{{ heat_engine_enabled_result.rc == 0 }}" - - name: FFU stop and disable openstack-heat-engine service - when: - - step|int == 1 - - release == 'rocky' - - heat_engine_enabled|bool - service: name=openstack-heat-engine state=stopped enabled=no diff --git a/deployment/ironic/ironic-api-container-puppet.yaml b/deployment/ironic/ironic-api-container-puppet.yaml index d2f4a28502..8b9ce7c01a 100644 --- a/deployment/ironic/ironic-api-container-puppet.yaml +++ b/deployment/ironic/ironic-api-container-puppet.yaml @@ -330,39 +330,3 @@ outputs: tags: - online_upgrade - online_upgrade_ironic - fast_forward_upgrade_tasks: - - when: - - step|int == 0 - - release == 'rocky' - block: - - name: Check if ironic_api is deployed - command: systemctl is-enabled --quiet openstack-ironic-api - failed_when: false - register: ironic_api_enabled_result - - name: Set fact ironic_api_enabled - set_fact: - ironic_api_enabled: "{{ ironic_api_enabled_result.rc == 0 }}" - - name: Stop openstack-ironic-api - service: name=openstack-ironic-api state=stopped enabled=no - when: - - step|int == 1 - - release == 'rocky' - - ironic_api_enabled|bool - - name: Ironic package update - package: - name: 'openstack-ironic*' - state: latest - when: - - step|int == 6 - - is_bootstrap_node|bool - - name: Synchronize the database - command: ironic-dbsync --config-file /etc/ironic/ironic.conf upgrade - when: - - step|int == 8 - - is_bootstrap_node|bool - - name: Do online data migration - command: ironic-dbsync --config-file /etc/ironic/ironic.conf online_data_migrations - when: - - step|int == 8 - - is_bootstrap_node|bool - - release == 'stein' diff --git a/deployment/ironic/ironic-conductor-container-puppet.yaml b/deployment/ironic/ironic-conductor-container-puppet.yaml index a2775aca5d..ea24f7072e 100644 --- a/deployment/ironic/ironic-conductor-container-puppet.yaml +++ b/deployment/ironic/ironic-conductor-container-puppet.yaml @@ -611,28 +611,3 @@ outputs: - step|int == 0 tags: common block: *ironic_config_data_cleanup - fast_forward_upgrade_tasks: - - when: - - step|int == 0 - - release == 'rocky' - block: - - name: Check if ironic_conductor is deployed - command: systemctl is-enabled --quiet openstack-ironic-conductor - failed_when: false - register: ironic_conductor_enabled_result - - name: Set fact ironic_conductor_enabled - set_fact: - ironic_conductor_enabled: "{{ ironic_conductor_enabled_result.rc == 0 }}" - - name: Stop openstack-ironic-conductor - service: name=openstack-ironic-conductor state=stopped enabled=no - when: - - step|int == 1 - - release == 'rocky' - - ironic_conductor_enabled|bool - - name: Ironic packages update - package: - name: 'openstack-ironic*' - state: latest - when: - - step|int == 6 - - is_bootstrap_node|bool diff --git a/deployment/keystone/keystone-container-puppet.yaml b/deployment/keystone/keystone-container-puppet.yaml index ab32e6cd23..bfbec77033 100644 --- a/deployment/keystone/keystone-container-puppet.yaml +++ b/deployment/keystone/keystone-container-puppet.yaml @@ -830,46 +830,3 @@ outputs: - keystone - keystone_cron tripleo_delegate_to: "{{ groups['keystone'] | default([]) }}" - fast_forward_upgrade_tasks: - - when: - - step|int == 0 - - release == 'rocky' - block: - - name: Check for keystone running under apache - tags: common - shell: "httpd -t -D DUMP_VHOSTS | grep -q keystone_wsgi" - failed_when: false - register: keystone_httpd_enabled_result - - name: Set fact keystone_httpd_enabled - set_fact: - keystone_httpd_enabled: "{{ keystone_httpd_enabled_result.rc == 0 }}" - - name: Check if httpd is running - failed_when: false - command: systemctl is-active --quiet httpd - register: httpd_running_result - when: - - httpd_running is undefined - - name: Set fact httpd_running if undefined - set_fact: - httpd_running: "{{ httpd_running_result.rc == 0 }}" - when: - - httpd_running is undefined - - name: Stop and disable keystone (under httpd) - service: name=httpd state=stopped enabled=no - when: - - step|int == 1 - - release == 'rocky' - - keystone_httpd_enabled|bool - - httpd_running|bool - - name: Keystone package update - package: - name: 'openstack-keystone*' - state: latest - when: - - step|int == 6 - - is_bootstrap_node|bool - - name: keystone db sync - command: keystone-manage db_sync - when: - - step|int == 8 - - is_bootstrap_node|bool diff --git a/deployment/manila/manila-api-container-puppet.yaml b/deployment/manila/manila-api-container-puppet.yaml index 1e99d48a27..392d5c38e2 100644 --- a/deployment/manila/manila-api-container-puppet.yaml +++ b/deployment/manila/manila-api-container-puppet.yaml @@ -303,35 +303,3 @@ outputs: - { 'path': /var/log/containers/manila, 'setype': svirt_sandbox_file_t, 'mode': '0750' } - { 'path': /var/log/containers/httpd/manila-api, 'setype': svirt_sandbox_file_t, 'mode': '0750' } upgrade_tasks: [] - fast_forward_upgrade_tasks: - - name: Check if manila_api is deployed - command: systemctl is-enabled --quiet openstack-manila-api - failed_when: false - register: manila_api_enabled_result - when: - - step|int == 0 - - release == 'rocky' - - name: Set fact manila_api_enabled - set_fact: - manila_api_enabled: "{{ manila_api_enabled_result.rc == 0 }}" - when: - - step|int == 0 - - release == 'rocky' - - name: Stop openstack-manila-api - service: name=openstack-manila-api state=stopped - when: - - step|int == 1 - - release == 'rocky' - - manila_api_enabled|bool - - name: manila package update - package: - name: 'openstack-manila*' - state: latest - when: - - step|int == 6 - - is_bootstrap_node|bool - - name: Manila db sync - command: manila-manage db sync - when: - - step|int == 8 - - is_bootstrap_node|bool diff --git a/deployment/manila/manila-scheduler-container-puppet.yaml b/deployment/manila/manila-scheduler-container-puppet.yaml index 71160eb341..4368a7f99b 100644 --- a/deployment/manila/manila-scheduler-container-puppet.yaml +++ b/deployment/manila/manila-scheduler-container-puppet.yaml @@ -117,23 +117,3 @@ outputs: persistent: yes state: yes upgrade_tasks: [] - fast_forward_upgrade_tasks: - - name: Check if manila_scheduler is deployed - command: systemctl is-enabled --quiet openstack-manila-scheduler - failed_when: false - register: manila_scheduler_enabled_result - when: - - step|int == 0 - - release == 'rocky' - - name: Set fact manila_scheduler_enabled - set_fact: - manila_scheduler_enabled: "{{ manila_scheduler_enabled_result.rc == 0 }}" - when: - - step|int == 0 - - release == 'rocky' - - name: Stop openstack-manila-scheduler - service: name=openstack-manila-scheduler state=stopped enabled=no - when: - - step|int == 1 - - release == 'rocky' - - manila_scheduler_enabled|bool diff --git a/deployment/manila/manila-share-container-puppet.yaml b/deployment/manila/manila-share-container-puppet.yaml index 841fc8bc32..af63730576 100644 --- a/deployment/manila/manila-share-container-puppet.yaml +++ b/deployment/manila/manila-share-container-puppet.yaml @@ -172,23 +172,3 @@ outputs: path: /etc/ceph state: directory upgrade_tasks: [] - fast_forward_upgrade_tasks: - - name: Check if manila_share is deployed - command: systemctl is-enabled --quiet openstack-manila-share - failed_when: false - register: manila_share_enabled_result - when: - - step|int == 0 - - release == 'rocky' - - name: Set fact manila_share_enabled - set_fact: - manila_share_enabled: "{{ manila_share_enabled_result.rc == 0 }}" - when: - - step|int == 0 - - release == 'rocky' - - name: Stop openstack-manila-share - service: name=openstack-manila-share state=stopped enabled=no - when: - - step|int == 1 - - release == 'rocky' - - manila_share_enabled|bool diff --git a/deployment/manila/manila-share-pacemaker-puppet.yaml b/deployment/manila/manila-share-pacemaker-puppet.yaml index fb123be7aa..ba73a72edd 100644 --- a/deployment/manila/manila-share-pacemaker-puppet.yaml +++ b/deployment/manila/manila-share-pacemaker-puppet.yaml @@ -374,36 +374,3 @@ outputs: when: - step|int == 3 block: *manila_share_fetch_retag_container_tasks - - fast_forward_upgrade_tasks: - - name: Check cluster resource status - pacemaker_resource: - resource: openstack-manila-share - state: show - check_mode: false - failed_when: false - register: manila_share_res_result - when: - - step|int == 0 - - release == 'rocky' - - is_bootstrap_node|bool - - name: Set fact manila_share_res - set_fact: - manila_share_res: "{{ manila_share_res_result.rc == 0 }}" - when: - - step|int == 0 - - release == 'rocky' - - is_bootstrap_node|bool - - name: Disable the openstack-manila-share cluster resource - pacemaker_resource: - resource: openstack-manila-share - state: disable - wait_for_resource: true - register: manila_share_output - retries: 5 - until: manila_share_output.rc == 0 - when: - - step|int == 2 - - release == 'rocky' - - is_bootstrap_node|bool - - manila_share_res|bool diff --git a/deployment/memcached/memcached-container-puppet.yaml b/deployment/memcached/memcached-container-puppet.yaml index 4433991f87..9b375c279a 100644 --- a/deployment/memcached/memcached-container-puppet.yaml +++ b/deployment/memcached/memcached-container-puppet.yaml @@ -210,22 +210,3 @@ outputs: tripleo_containers_to_stop: - memcached tripleo_delegate_to: "{{ groups['memcached'] | default([]) }}" - fast_forward_upgrade_tasks: - - when: - - step|int == 0 - - release == 'rocky' - block: - - name: Check if memcached is deployed - command: systemctl is-enabled --quiet memcached - tags: common - failed_when: false - register: memcached_enabled_result - - name: memcached_enabled - set_fact: - memcached_enabled: "{{ memcached_enabled_result.rc == 0 }}" - - name: Stop and disable memcached service - when: - - step|int == 2 - - release == 'rocky' - - memcached_enabled|bool - service: name=memcached state=stopped enabled=no diff --git a/deployment/metrics/collectd-container-puppet.yaml b/deployment/metrics/collectd-container-puppet.yaml index ef3f8c9443..99b9398092 100644 --- a/deployment/metrics/collectd-container-puppet.yaml +++ b/deployment/metrics/collectd-container-puppet.yaml @@ -714,21 +714,3 @@ outputs: import_role: name: tripleo_provision_mcelog when: {get_param: CollectdEnableMcelog} - fast_forward_upgrade_tasks: - - when: - - step|int == 0 - - release == 'rocky' - block: - - name: Check if collectd.service is deployed - command: systemctl is-enabled --quiet collectd.service - failed_when: false - register: collectd_enabled_result - - name: Set fact collectd_enabled - set_fact: - collectd_enabled: "{{ collectd_enabled_result.rc == 0 }}" - - name: Stop collectd service - service: name=collectd.service state=stopped enabled=no - when: - - step|int == 1 - - release == 'rocky' - - collectd_enabled|bool diff --git a/deployment/neutron/neutron-api-container-puppet.yaml b/deployment/neutron/neutron-api-container-puppet.yaml index 4c9e219bf3..3f51a03faf 100644 --- a/deployment/neutron/neutron-api-container-puppet.yaml +++ b/deployment/neutron/neutron-api-container-puppet.yaml @@ -581,40 +581,3 @@ outputs: tripleo_containers_to_stop: - neutron_api tripleo_delegate_to: "{{ groups['neutron_api'] | default([]) }}" - fast_forward_upgrade_tasks: - - when: - - step|int == 0 - - release == 'rocky' - block: - - name: Check if neutron_server is deployed - command: systemctl is-enabled --quiet neutron-server - failed_when: false - register: neutron_server_enabled_result - - name: Set fact neutron_server_enabled - set_fact: - neutron_server_enabled: "{{ neutron_server_enabled_result.rc == 0 }}" - - name: Stop neutron_server - service: name=neutron-server state=stopped enabled=no - when: - - step|int == 1 - - release == 'rocky' - - neutron_server_enabled|bool - - when: - - step|int == 6 - - is_bootstrap_node|bool - block: - - name: Neutron package update - package: - name: 'openstack-neutron*' - state: latest - - name: Neutron package update workaround - package: name=python-networking-odl state=latest - # package python-networking-cisco may or may not be present - - name: Networking cisco db sync workaround - failed_when: false - package: name=python-networking-cisco state=latest - - name: Neutron db sync - command: neutron-db-manage upgrade head - when: - - step|int == 8 - - is_bootstrap_node|bool diff --git a/deployment/neutron/neutron-dhcp-container-puppet.yaml b/deployment/neutron/neutron-dhcp-container-puppet.yaml index bba4603285..e94cc32fda 100644 --- a/deployment/neutron/neutron-dhcp-container-puppet.yaml +++ b/deployment/neutron/neutron-dhcp-container-puppet.yaml @@ -439,24 +439,6 @@ outputs: mode: 0755 content: {get_file: ./kill-script} upgrade_tasks: [] - fast_forward_upgrade_tasks: - - when: - - step|int == 0 - - release == 'rocky' - block: - - name: Check if neutron_dhcp_agent is deployed - command: systemctl is-enabled --quiet neutron-dhcp-agent - failed_when: false - register: neutron_dhcp_agent_enabled_result - - name: Set fact neutron_dhcp_agent_enabled - set_fact: - neutron_dhcp_agent_enabled: "{{ neutron_dhcp_agent_enabled_result.rc == 0 }}" - - name: Stop neutron_dhcp_agent - service: name=neutron-dhcp-agent state=stopped enabled=no - when: - - step|int == 2 - - release == 'rocky' - - neutron_dhcp_agent_enabled|bool post_upgrade_tasks: - name: Check for neutron user getent: diff --git a/deployment/neutron/neutron-l3-container-puppet.yaml b/deployment/neutron/neutron-l3-container-puppet.yaml index 52570957b9..9dac721d5c 100644 --- a/deployment/neutron/neutron-l3-container-puppet.yaml +++ b/deployment/neutron/neutron-l3-container-puppet.yaml @@ -394,24 +394,6 @@ outputs: mode: 0755 content: {get_file: ./kill-script} upgrade_tasks: [] - fast_forward_upgrade_tasks: - - when: - - step|int == 0 - - release == 'rocky' - block: - - name: Check if neutron_l3_agent is deployed - command: systemctl is-enabled --quiet neutron-l3-agent - failed_when: false - register: neutron_l3_agent_enabled_result - - name: Set fact neutron_l3_agent_enabled - set_fact: - neutron_l3_agent_enabled: "{{ neutron_l3_agent_enabled_result.rc == 0 }}" - - name: Stop neutron_l3_agent - service: name=neutron-l3-agent state=stopped enabled=no - when: - - step|int == 1 - - release == 'rocky' - - neutron_l3_agent_enabled|bool post_upgrade_tasks: - name: Check for neutron user getent: diff --git a/deployment/neutron/neutron-metadata-container-puppet.yaml b/deployment/neutron/neutron-metadata-container-puppet.yaml index 4a823fb608..9166c90543 100644 --- a/deployment/neutron/neutron-metadata-container-puppet.yaml +++ b/deployment/neutron/neutron-metadata-container-puppet.yaml @@ -210,24 +210,6 @@ outputs: persistent: yes state: yes upgrade_tasks: [] - fast_forward_upgrade_tasks: - - when: - - step|int == 0 - - release == 'rocky' - block: - - name: Check if neutron_metadata_agent is deployed - command: systemctl is-enabled --quiet neutron-metadata-agent - failed_when: false - register: neutron_metadata_agent_enabled_result - - name: Set fact neutron_metadata_agent_enabled - set_fact: - neutron_metadata_agent_enabled: "{{ neutron_metadata_agent_enabled_result.rc == 0 }}" - - name: Stop neutron_metadata_agent - service: name=neutron-metadata-agent state=stopped enabled=no - when: - - step|int == 1 - - release == 'rocky' - - neutron_metadata_agent_enabled|bool post_upgrade_tasks: - name: Check for neutron user getent: diff --git a/deployment/neutron/neutron-ovs-agent-container-puppet.yaml b/deployment/neutron/neutron-ovs-agent-container-puppet.yaml index 26dff5a96d..4c78a4963f 100644 --- a/deployment/neutron/neutron-ovs-agent-container-puppet.yaml +++ b/deployment/neutron/neutron-ovs-agent-container-puppet.yaml @@ -406,21 +406,3 @@ outputs: regexp=".*neutron-" state=absent when: step|int == 5 - fast_forward_upgrade_tasks: - - when: - - step|int == 0 - - release == 'rocky' - block: - - name: Check if neutron_ovs_agent is deployed - command: systemctl is-enabled --quiet neutron-openvswitch-agent - failed_when: false - register: neutron_ovs_agent_enabled_result - - name: Set fact neutron_ovs_agent_enabled - set_fact: - neutron_ovs_agent_enabled: "{{ neutron_ovs_agent_enabled_result.rc == 0 }}" - - name: Stop neutron_openvswitch_agent - service: name=neutron-openvswitch-agent state=stopped enabled=no - when: - - step|int == 1 - - release == 'rocky' - - neutron_ovs_agent_enabled|bool diff --git a/deployment/neutron/neutron-ovs-dpdk-agent-container-puppet.yaml b/deployment/neutron/neutron-ovs-dpdk-agent-container-puppet.yaml index 2a803d4f5c..730be3d6d9 100644 --- a/deployment/neutron/neutron-ovs-dpdk-agent-container-puppet.yaml +++ b/deployment/neutron/neutron-ovs-dpdk-agent-container-puppet.yaml @@ -161,5 +161,3 @@ outputs: get_attr: [NeutronOvsAgent, role_data, upgrade_tasks] update_tasks: get_attr: [NeutronOvsAgent, role_data, update_tasks] - fast_forward_upgrade_tasks: - get_attr: [NeutronOvsAgent, role_data, fast_forward_upgrade_tasks] diff --git a/deployment/nova/nova-api-container-puppet.yaml b/deployment/nova/nova-api-container-puppet.yaml index 3603d6f290..a6b4863a12 100644 --- a/deployment/nova/nova-api-container-puppet.yaml +++ b/deployment/nova/nova-api-container-puppet.yaml @@ -547,152 +547,3 @@ outputs: - nova_placement - nova_api_cron tripleo_delegate_to: "{{ groups['nova_api'] | default([]) }}" - fast_forward_upgrade_tasks: - - when: - - step|int == 0 - - release == 'rocky' - block: - - name: Check if nova-api is deployed - command: systemctl is-enabled --quiet openstack-nova-api - failed_when: false - register: nova_api_enabled_result - - name: Set fact nova_api_enabled - set_fact: - nova_api_enabled: "{{ nova_api_enabled_result.rc == 0 }}" - - name: Stop openstack-nova-api service - service: name=openstack-nova-api state=stopped - when: - - step|int == 1 - - nova_api_enabled|bool - - release == 'rocky' - - name: Extra migration for nova tripleo/+bug/1656791 - command: nova-manage db online_data_migrations - when: - - step|int == 5 - - release == 'rocky' - - is_bootstrap_node|bool - - name: Update nova packages - package: - name: '*nova*' - state: latest - when: - - step|int == 6 - - is_bootstrap_node|bool - #FIXME(lyarwood): Use puppet to do this? - - when: - - step|int == 7 - - release == 'rocky' - - is_bootstrap_node|bool - block: - - name: Create puppet manifest to set transport_url in nova.conf - copy: - dest: /root/nova-api_upgrade_manifest.pp - mode: 0600 - content: > - $transport_url = os_transport_url({ - 'transport' => hiera('messaging_service_name', 'rabbit'), - 'hosts' => any2array(hiera('rabbitmq_node_names', undef)), - 'port' => sprintf('%s',hiera('nova::rabbit_port', '5672') ), - 'username' => hiera('nova::rabbit_userid', 'guest'), - 'password' => hiera('nova::rabbit_password'), - 'ssl' => sprintf('%s', bool2num(str2bool(hiera('nova::rabbit_use_ssl', '0')))) - }) - oslo::messaging::default { 'nova_config': - transport_url => $transport_url - } - - - name: Run puppet apply to set tranport_url in nova.conf - command: puppet apply --modulepath /etc/puppet/modules:/opt/stack/puppet-modules:/usr/share/openstack-puppet/modules --detailed-exitcodes /root/nova-api_upgrade_manifest.pp - register: puppet_apply_nova_api_upgrade - failed_when: puppet_apply_nova_api_upgrade.rc not in [0,2] - changed_when: puppet_apply_nova_api_upgrade.rc == 2 - - name: Setup cell_v2 (map cell0) - shell: - str_replace: - template: nova-manage cell_v2 map_cell0 --database_connection='CELL0DB' --transport-url="'TRANSPORTURL'" - params: - CELL0DB: - list_join: - - '' - - - '{scheme}' - - '://' - - '{username}' - - ':' - - '{password}' - - '@' - - - if: - - mysql_ipv6_use_ip_address - - '[{hostname}]' - - '{hostname}' - - '/' - - 'nova_cell0' - - '?' - - '{query}' - TRANSPORTURL: - list_join: - - '' - - - '{scheme}' - - '://' - - '{username}' - - ':' - - '{password}' - - '@' - - '{hostname}' - - ':' - - '{port}' - - '/' - - '?' - - '{query}' - - name: Setup cell_v2 (create default cell) - # (owalsh) puppet-nova expects the cell name 'default' - shell: - str_replace: - template: nova-manage cell_v2 create_cell --name='default' --database_connection='CELLDB' - params: - CELLDB: - list_join: - - '' - - - '{scheme}' - - '://' - - '{username}' - - ':' - - '{password}' - - '@' - - - if: - - mysql_ipv6_use_ip_address - - '[{hostname}]' - - '{hostname}' - - '/' - - 'nova' - - '?' - - '{query}' - register: nova_api_create_cell - failed_when: nova_api_create_cell.rc not in [0,2] - changed_when: nova_api_create_cell.rc == 0 - - name: Setup cell_v2 (sync nova/cell DB) - command: nova-manage db sync - async: {get_param: NovaDbSyncTimeout} - poll: 10 - - name: Setup cell_v2 (get cell uuid) - shell: nova-manage cell_v2 list_cells | sed -e '1,3d' -e '$d' | awk -F ' *| *' '$2 == "default" {print $4}' - register: nova_api_cell_uuid - - name: Setup cell_v2 (migrate hosts) - command: nova-manage cell_v2 discover_hosts --cell_uuid {{nova_api_cell_uuid.stdout}} --verbose - - name: Setup cell_v2 (migrate instances) - command: nova-manage cell_v2 map_instances --cell_uuid {{nova_api_cell_uuid.stdout}} - - when: - - step|int == 8 - - is_bootstrap_node|bool - block: - - name: Sync nova/cell DB - command: nova-manage db sync - async: {get_param: NovaDbSyncTimeout} - poll: 10 - when: - - release == 'stein' - - name: Sync nova_api DB - command: nova-manage api_db sync - - name: Online data migration for nova - command: nova-manage db online_data_migrations diff --git a/deployment/nova/nova-compute-container-puppet.yaml b/deployment/nova/nova-compute-container-puppet.yaml index 14613eef65..84dbfb9a8b 100644 --- a/deployment/nova/nova-compute-container-puppet.yaml +++ b/deployment/nova/nova-compute-container-puppet.yaml @@ -1223,27 +1223,3 @@ outputs: command: openstack compute service delete {{ nova_compute_service[0].ID }} delegate_to: localhost check_mode: no - fast_forward_upgrade_tasks: - - when: - - step|int == 0 - - release == 'rocky' - block: - - name: Check if nova-compute is deployed - command: systemctl is-enabled --quiet openstack-nova-compute - failed_when: false - register: nova_compute_enabled_result - - name: Set fact nova_compute_enabled - set_fact: - nova_compute_enabled: "{{ nova_compute_enabled_result.rc == 0 }}" - - when: - - step|int == 1 - - release == 'rocky' - block: - - name: Stop and disable nova-compute service - service: name=openstack-nova-compute state=stopped - when: - - nova_compute_enabled|bool - - name: Set upgrade marker in nova statedir - file: path=/var/lib/nova/upgrade_marker state=touch owner=nova group=nova - when: - - nova_compute_enabled|bool diff --git a/deployment/nova/nova-conductor-container-puppet.yaml b/deployment/nova/nova-conductor-container-puppet.yaml index 9d59423331..dcb23b29dd 100644 --- a/deployment/nova/nova-conductor-container-puppet.yaml +++ b/deployment/nova/nova-conductor-container-puppet.yaml @@ -285,21 +285,3 @@ outputs: tripleo_containers_to_stop: - nova_conductor tripleo_delegate_to: "{{ groups['nova_conductor'] | default([]) }}" - fast_forward_upgrade_tasks: - - when: - - step|int == 0 - - release == 'rocky' - block: - - name: Check if nova_conductor is deployed - command: systemctl is-enabled --quiet openstack-nova-conductor - failed_when: false - register: nova_conductor_enabled_result - - name: Set fact nova_conductor_enabled - set_fact: - nova_conductor_enabled: "{{ nova_conductor_enabled_result.rc == 0 }}" - - name: Stop and disable nova_conductor service - service: name=openstack-nova-conductor state=stopped - when: - - step|int == 1 - - release == 'rocky' - - nova_conductor_enabled|bool diff --git a/deployment/nova/nova-ironic-container-puppet.yaml b/deployment/nova/nova-ironic-container-puppet.yaml index a75d1f44a2..a311ae5c85 100644 --- a/deployment/nova/nova-ironic-container-puppet.yaml +++ b/deployment/nova/nova-ironic-container-puppet.yaml @@ -219,27 +219,3 @@ outputs: persistent: yes state: yes external_post_deploy_tasks: {get_attr: [NovaComputeCommon, nova_compute_common_deploy_steps_tasks]} - fast_forward_upgrade_tasks: - - when: - - step|int == 0 - - release == 'rocky' - block: - - name: Check if nova ironic is deployed - command: systemctl is-enabled --quiet openstack-nova-compute - tags: common - register: nova_ironic_enabled_result - - name: Set fact nova_ironic_enabled - set_fact: - nova_ironic_enabled: "{{ nova_ironic_enabled_result.rc == 0 }}" - - name: Stop and disable nova-compute service - service: name=openstack-nova-compute state=stopped - when: - - step|int == 1 - - release == 'rocky' - - nova_ironic_enabled|bool - - name: Set upgrade marker in nova statedir - when: - - step|int == 1 - - release == 'rocky' - - nova_ironic_enabled|bool - file: path=/var/lib/nova/upgrade_marker state=touch owner=nova group=nova diff --git a/deployment/nova/nova-scheduler-container-puppet.yaml b/deployment/nova/nova-scheduler-container-puppet.yaml index c22fe138ee..22c14c0693 100644 --- a/deployment/nova/nova-scheduler-container-puppet.yaml +++ b/deployment/nova/nova-scheduler-container-puppet.yaml @@ -246,24 +246,6 @@ outputs: name: virt_sandbox_use_netlink persistent: yes state: yes - fast_forward_upgrade_tasks: - - when: - - step|int == 0 - - release == 'rocky' - block: - - name: Check if nova_scheduler is deployed - command: systemctl is-enabled --quiet openstack-nova-scheduler - failed_when: false - register: nova_scheduler_enabled_result - - name: Set fact nova_scheduler_enabled - set_fact: - nova_scheduler_enabled: "{{ nova_scheduler_enabled_result.rc == 0 }}" - - name: Stop and disable nova-scheduler service - service: name=openstack-nova-scheduler state=stopped - when: - - step|int == 1 - - release == 'rocky' - - nova_scheduler_enabled|bool external_upgrade_tasks: - when: - step|int == 1 diff --git a/deployment/nova/nova-vnc-proxy-container-puppet.yaml b/deployment/nova/nova-vnc-proxy-container-puppet.yaml index 29efdab5f8..54a8f1a457 100644 --- a/deployment/nova/nova-vnc-proxy-container-puppet.yaml +++ b/deployment/nova/nova-vnc-proxy-container-puppet.yaml @@ -70,7 +70,7 @@ parameters: Type of update, to differentiate between UPGRADE and UPDATE cases when StackAction is UPDATE (both are the same stack action). constraints: - - allowed_values: ['', 'UPGRADE', 'FASTFORWARDUPGRADE'] + - allowed_values: ['', 'UPGRADE'] default: '' @@ -322,24 +322,6 @@ outputs: state: present shell: /sbin/nologin comment: qemu user - fast_forward_upgrade_tasks: - - when: - - step|int == 0 - - release == 'rocky' - block: - - name: Check if nova vncproxy is deployed - command: systemctl is-enabled --quiet openstack-nova-novncproxy - failed_when: false - register: nova_vncproxy_enabled_result - - name: Set fact nova_vncproxy_enabled - set_fact: - nova_vncproxy_enabled: "{{ nova_vncproxy_enabled_result.rc == 0 }}" - - name: Stop and disable nova-novncproxy service - service: name=openstack-nova-novncproxy state=stopped - when: - - step|int == 1 - - release == 'rocky' - - nova_vncproxy_enabled|bool external_upgrade_tasks: - when: - step|int == 1 diff --git a/deployment/rabbitmq/rabbitmq-messaging-notify-pacemaker-puppet.yaml b/deployment/rabbitmq/rabbitmq-messaging-notify-pacemaker-puppet.yaml index afb5384401..0dbbb215a9 100644 --- a/deployment/rabbitmq/rabbitmq-messaging-notify-pacemaker-puppet.yaml +++ b/deployment/rabbitmq/rabbitmq-messaging-notify-pacemaker-puppet.yaml @@ -428,33 +428,3 @@ outputs: when: - step|int == 3 block: *rabbitmq_fetch_retag_container_tasks - - fast_forward_upgrade_tasks: - - when: - - step|int == 0 - - release == 'rocky' - - is_bootstrap_node|bool - block: - - name: Check cluster resource status of rabbitmq - pacemaker_resource: - resource: {get_attr: [RabbitmqBase, role_data, service_name]} - state: show - check_mode: false - failed_when: false - register: rabbitmq_res_result - - name: Set fact rabbitmq_res - set_fact: - rabbitmq_res: "{{ rabbitmq_res_result.rc == 0 }}" - - name: Disable the rabbitmq cluster resource - pacemaker_resource: - resource: {get_attr: [RabbitmqBase, role_data, service_name]} - state: disable - wait_for_resource: true - register: rabbitmq_output - retries: 5 - until: rabbitmq_output.rc == 0 - when: - - step|int == 2 - - release == 'rocky' - - is_bootstrap_node|bool - - rabbitmq_res|bool diff --git a/deployment/rabbitmq/rabbitmq-messaging-pacemaker-puppet.yaml b/deployment/rabbitmq/rabbitmq-messaging-pacemaker-puppet.yaml index 5bc6548e1d..b94c387583 100644 --- a/deployment/rabbitmq/rabbitmq-messaging-pacemaker-puppet.yaml +++ b/deployment/rabbitmq/rabbitmq-messaging-pacemaker-puppet.yaml @@ -415,33 +415,3 @@ outputs: when: - step|int == 3 block: *rabbitmq_fetch_retag_container_tasks - - fast_forward_upgrade_tasks: - - when: - - step|int == 0 - - release == 'rocky' - - is_bootstrap_node|bool - block: - - name: Check cluster resource status of rabbitmq - pacemaker_resource: - resource: {get_attr: [RabbitmqBase, role_data, service_name]} - state: show - check_mode: false - failed_when: false - register: rabbitmq_res_result - - name: Set fact rabbitmq_res - set_fact: - rabbitmq_res: "{{ rabbitmq_res_result.rc == 0 }}" - - name: Disable the rabbitmq cluster resource - pacemaker_resource: - resource: {get_attr: [RabbitmqBase, role_data, service_name]} - state: disable - wait_for_resource: true - register: rabbitmq_output - retries: 5 - until: rabbitmq_output.rc == 0 - when: - - step|int == 2 - - release == 'rocky' - - is_bootstrap_node|bool - - rabbitmq_res|bool diff --git a/deployment/rabbitmq/rabbitmq-messaging-rpc-pacemaker-puppet.yaml b/deployment/rabbitmq/rabbitmq-messaging-rpc-pacemaker-puppet.yaml index ec71c9c154..980076ed2d 100644 --- a/deployment/rabbitmq/rabbitmq-messaging-rpc-pacemaker-puppet.yaml +++ b/deployment/rabbitmq/rabbitmq-messaging-rpc-pacemaker-puppet.yaml @@ -434,33 +434,3 @@ outputs: when: - step|int == 3 block: *rabbitmq_fetch_retag_container_tasks - - fast_forward_upgrade_tasks: - - when: - - step|int == 0 - - release == 'rocky' - - is_bootstrap_node|bool - block: - - name: Check cluster resource status of rabbitmq - pacemaker_resource: - resource: {get_attr: [RabbitmqBase, role_data, service_name]} - state: show - check_mode: false - failed_when: false - register: rabbitmq_res_result - - name: Set fact rabbitmq_res - set_fact: - rabbitmq_res: "{{ rabbitmq_res_result.rc == 0 }}" - - name: Disable the rabbitmq cluster resource - pacemaker_resource: - resource: {get_attr: [RabbitmqBase, role_data, service_name]} - state: disable - wait_for_resource: true - register: rabbitmq_output - retries: 5 - until: rabbitmq_output.rc == 0 - when: - - step|int == 2 - - release == 'rocky' - - is_bootstrap_node|bool - - rabbitmq_res|bool diff --git a/deployment/sahara/sahara-api-container-puppet.yaml b/deployment/sahara/sahara-api-container-puppet.yaml index ccc0cc35bb..3fce5114fb 100644 --- a/deployment/sahara/sahara-api-container-puppet.yaml +++ b/deployment/sahara/sahara-api-container-puppet.yaml @@ -213,33 +213,3 @@ outputs: with_items: - { 'path': /var/log/containers/sahara, 'setype': svirt_sandbox_file_t, 'mode': '0750' } - { 'path': /var/lib/sahara, 'setype': svirt_sandbox_file_t } - fast_forward_upgrade_tasks: - - when: - - step|int == 0 - - release == 'rocky' - block: - - name: FFU check openstack-sahara-api is enabled - command: systemctl is-enabled openstack-sahara-api - failed_when: false - register: sahara_api_enabled_result - - name: Set fact sahara_api_enabled - set_fact: - sahara_api_enabled: "{{ sahara_api_enabled_result.rc == 0 }}" - - name: FFU stop and disable openstack-sahara-api - service: name=openstack-sahara-api state=stopped enabled=no - when: - - step|int == 1 - - release == 'rocky' - - sahara_api_enabled|bool - - name: FFU Sahara package update - package: - name: 'openstack-sahara*' - state: latest - when: - - step|int == 6 - - is_bootstrap_node|bool - - name: FFU Sahara db upgrade - command: sahara-db-manage --config-file /etc/sahara/sahara.conf upgrade head - when: - - step|int == 8 - - is_bootstrap_node|bool diff --git a/deployment/sahara/sahara-engine-container-puppet.yaml b/deployment/sahara/sahara-engine-container-puppet.yaml index 98aefd3703..88f11cecd1 100644 --- a/deployment/sahara/sahara-engine-container-puppet.yaml +++ b/deployment/sahara/sahara-engine-container-puppet.yaml @@ -135,21 +135,3 @@ outputs: name: virt_sandbox_use_netlink persistent: yes state: yes - fast_forward_upgrade_tasks: - - when: - - step|int == 0 - - release == 'rocky' - block: - - name: FFU check if openstack-sahara-engine is enabled - command: systemctl is-enabled --quiet openstack-sahara-engine - failed_when: false - register: sahara_engine_enabled_result - - name: Set fact sahara_engine_enabled - set_fact: - sahara_engine_enabled: "{{ sahara_engine_enabled_result.rc == 0 }}" - - name: FFU stop and disable openstack-sahara-engine service - when: - - step|int == 1 - - release == 'rocky' - - sahara_engine_enabled|bool - service: name=openstack-sahara-engine state=stopped enabled=no diff --git a/deployment/swift/swift-proxy-container-puppet.yaml b/deployment/swift/swift-proxy-container-puppet.yaml index f5ecf65e3a..4e378a0820 100644 --- a/deployment/swift/swift-proxy-container-puppet.yaml +++ b/deployment/swift/swift-proxy-container-puppet.yaml @@ -479,25 +479,3 @@ outputs: when: swift_log_link.stat.islnk is defined and swift_log_link.stat.islnk metadata_settings: get_attr: [TLSProxyBase, role_data, metadata_settings] - fast_forward_upgrade_tasks: - - when: - - step|int == 0 - - release == 'rocky' - block: - - name: Check if swift-proxy or swift-object-expirer are deployed - command: systemctl is-enabled --quiet "{{ item }}" - failed_when: false - register: swift_proxy_services_enabled_result - with_items: - - openstack-swift-proxy - - openstack-swift-object-expirer - - name: Set fact swift_proxy_services_enabled - set_fact: - swift_proxy_services_enabled: "{{ swift_proxy_services_enabled_result }}" - - name: Stop swift-proxy and swift-object-expirer services - service: name={{ item.item }} state=stopped enabled=no - with_items: "{{ swift_proxy_services_enabled.results }}" - when: - - step|int == 1 - - release == 'rocky' - - item.rc == 0 diff --git a/deployment/swift/swift-storage-container-puppet.yaml b/deployment/swift/swift-storage-container-puppet.yaml index eee0bb7e6e..a72088a778 100644 --- a/deployment/swift/swift-storage-container-puppet.yaml +++ b/deployment/swift/swift-storage-container-puppet.yaml @@ -680,37 +680,3 @@ outputs: path: /var/log/containers/swift state: absent when: swift_log_link.stat.islnk is defined and swift_log_link.stat.islnk - fast_forward_upgrade_tasks: - - when: - - step|int == 0 - - release == 'rocky' - block: - - name: Check if swift storage services are deployed - command: systemctl is-enabled --quiet "{{ item }}" - with_items: - - openstack-swift-account-auditor - - openstack-swift-account-reaper - - openstack-swift-account-replicator - - openstack-swift-account - - openstack-swift-container-auditor - - openstack-swift-container-replicator - - openstack-swift-container-updater - - openstack-swift-container - - openstack-swift-container-sync - - openstack-swift-object-auditor - - openstack-swift-object-replicator - - openstack-swift-object-updater - - openstack-swift-object - - openstack-swift-object-reconstructor - failed_when: false - register: swift_services_enabled_result - - name: Set fact swift_services_enabled - set_fact: - swift_services_enabled: "{{ swift_services_enabled_result }}" - - name: Stop swift storage services - service: name={{ item.item }} state=stopped enabled=no - with_items: "{{ swift_services_enabled.results }}" - when: - - step|int == 1 - - release == 'rocky' - - item.rc == 0 diff --git a/deployment/tripleo-packages/tripleo-packages-baremetal-puppet.yaml b/deployment/tripleo-packages/tripleo-packages-baremetal-puppet.yaml index 52f2681dd3..66cef4599c 100644 --- a/deployment/tripleo-packages/tripleo-packages-baremetal-puppet.yaml +++ b/deployment/tripleo-packages/tripleo-packages-baremetal-puppet.yaml @@ -34,23 +34,6 @@ parameters: default: 'false' description: Set to true to enable package installation at deploy time type: boolean - FastForwardRepoType: - default: 'tripleo-repos' - type: string - constraints: - - allowed_values: ['tripleo-repos', 'custom-script'] - FastForwardRepoArgs: - default: {'tripleo_repos': {'rocky': '-b rocky current', 'stein': '-b stein current', 'train': '-b train current'}} - type: json - FastForwardCustomRepoScriptContent: - default: | - #!/bin/bash - set -e - echo "If you use FastForwardRepoType 'custom-script' you have to provide the upgrade repo script content." - echo "It will be installed as /root/ffu_upgrade_repo.sh on the node" - echo "and passed the upstream name (rocky, stein, train) of the release as first argument" - exit 1 - type: string GlanceNodeStagingUri: default: 'file:///var/lib/glance/staging' description: > @@ -475,79 +458,3 @@ outputs: enabled: yes state: started ignore_errors: true - fast_forward_upgrade_tasks: - - block: - - name: set is_bootstrap_node fact - set_fact: is_bootstrap_node={{tripleo_packages_short_bootstrap_node_name|lower == ansible_hostname|lower}} - - name: Register repo type and args - set_fact: - fast_forward_repo_type: {get_param: FastForwardRepoType} - fast_forward_repo_args: {get_param: FastForwardRepoArgs} - - debug: - msg: "fast_forward_repo_type: {{ fast_forward_repo_type }} fast_forward_repo_args: {{ fast_forward_repo_args }}" - when: - - step|int == 3 - - block: - - name: clone tripleo-repos - git: - repo: https://github.com/openstack/tripleo-repos.git - dest: /home/stack/tripleo-repos/ - - name: install tripleo-repos - command: python setup.py install - args: - chdir: /home/stack/tripleo-repos/ - - name: Enable tripleo-repos - command: "tripleo-repos {{ fast_forward_repo_args.tripleo_repos[release] }}" - when: - - step|int == 3 - - is_bootstrap_node|bool - - fast_forward_repo_type == 'tripleo-repos' - - block: - - name: Create custom Script for upgrading repo. - copy: - dest: /root/ffu_update_repo.sh - content: {get_param: FastForwardCustomRepoScriptContent} - mode: 0700 - - name: Execute custom script for upgrading repo. - shell: "/root/ffu_update_repo.sh {{release}}" - when: - - step|int == 3 - - is_bootstrap_node|bool - - fast_forward_repo_type == 'custom-script' - fast_forward_post_upgrade_tasks: - - block: - - name: Register repo type and args - set_fact: - fast_forward_repo_type: {get_param: FastForwardRepoType} - fast_forward_repo_args: {get_param: FastForwardRepoArgs} - - debug: - msg: "fast_forward_repo_type: {{ fast_forward_repo_type }} fast_forward_repo_args: {{ fast_forward_repo_args }}" - when: - - step|int == 3 - - block: - - name: clone tripleo-repos - git: - repo: https://github.com/openstack/tripleo-repos.git - dest: /home/stack/tripleo-repos/ - - name: install tripleo-repos - command: python setup.py install - args: - chdir: /home/stack/tripleo-repos/ - - name: Enable tripleo-repos - command: "tripleo-repos {{ fast_forward_repo_args.tripleo_repos[release] }}" - when: - - step|int == 3 - - is_bootstrap_node|bool - - fast_forward_repo_type == 'tripleo-repos' - - block: - - name: Create custom Script for upgrading repo. - copy: - dest: /root/ffu_update_repo.sh - content: {get_param: FastForwardCustomRepoScriptContent} - mode: 0700 - - name: Execute custom script for upgrading repo. - shell: "/root/ffu_update_repo.sh {{release}}" - when: - - step|int == 3 - - is_bootstrap_node|bool - - fast_forward_repo_type == 'custom-script' diff --git a/environments/lifecycle/ffwd-upgrade-prepare.yaml b/environments/lifecycle/ffwd-upgrade-prepare.yaml index 7988f5c4ab..308d6d4284 100644 --- a/environments/lifecycle/ffwd-upgrade-prepare.yaml +++ b/environments/lifecycle/ffwd-upgrade-prepare.yaml @@ -8,6 +8,5 @@ resource_registry: OS::TripleO::Services::MongoDb: OS::Heat::None parameter_defaults: - StackUpdateType: FASTFORWARDUPGRADE NetworkDeploymentActions: ['CREATE','UPDATE'] FastForwardUpgrade: true diff --git a/releasenotes/notes/remove_ffwd_tasks-d1ab630d96a66a59.yaml b/releasenotes/notes/remove_ffwd_tasks-d1ab630d96a66a59.yaml new file mode 100644 index 0000000000..b265961836 --- /dev/null +++ b/releasenotes/notes/remove_ffwd_tasks-d1ab630d96a66a59.yaml @@ -0,0 +1,6 @@ +--- +deprecations: + - | + As the fast forward upgrade workflow to skip multiple releases now + relies on the very same upgrade_tasks, there is no need to mantain + the fast_forward_upgrade_tasks, as well as any of its references. diff --git a/tools/yaml-validate.py b/tools/yaml-validate.py index d2cb144b2b..db6d15b94c 100755 --- a/tools/yaml-validate.py +++ b/tools/yaml-validate.py @@ -49,8 +49,6 @@ REQUIRED_DOCKER_SECTIONS = ['service_name', 'docker_config', 'puppet_config', OPTIONAL_DOCKER_SECTIONS = ['container_puppet_tasks', 'upgrade_tasks', 'deploy_steps_tasks', 'pre_upgrade_rolling_tasks', - 'fast_forward_upgrade_tasks', - 'fast_forward_post_upgrade_tasks', 'post_upgrade_tasks', 'update_tasks', 'post_update_tasks', 'service_config_settings', 'host_prep_tasks', 'metadata_settings', @@ -62,8 +60,6 @@ OPTIONAL_DOCKER_SECTIONS = ['container_puppet_tasks', 'upgrade_tasks', 'external_update_tasks', 'external_upgrade_tasks'] # ansible tasks cannot be an empty dict or ansible is unhappy ANSIBLE_TASKS_SECTIONS = ['upgrade_tasks', 'pre_upgrade_rolling_tasks', - 'fast_forward_upgrade_tasks', - 'fast_forward_post_upgrade_tasks', 'post_upgrade_tasks', 'update_tasks', 'post_update_tasks', 'host_prep_tasks', 'external_deploy_tasks', @@ -742,16 +738,6 @@ def validate_docker_service(filename, tpl): print('ERROR: upgrade_tasks validation failed') return 1 - if 'fast_forward_upgrade_tasks' in role_data and role_data['fast_forward_upgrade_tasks']: - if validate_upgrade_tasks(role_data['fast_forward_upgrade_tasks']): - print('ERROR: fast_forward_upgrade_tasks validation failed') - return 1 - - if 'fast_forward_post_upgrade_tasks' in role_data and role_data['fast_forward_post_upgrade_tasks']: - if validate_upgrade_tasks(role_data['fast_forward_post_upgrade_tasks']): - print('ERROR: fast_forward_post_upgrade_tasks validation failed') - return 1 - if 'parameters' in tpl: for param in required_params: if param not in tpl['parameters']: @@ -807,16 +793,6 @@ def validate_service(filename, tpl): print('ERROR: upgrade_tasks validation failed') return 1 - if 'fast_forward_upgrade_tasks' in role_data and role_data['fast_forward_upgrade_tasks']: - if validate_upgrade_tasks(role_data['fast_forward_upgrade_tasks']): - print('ERROR: fast_forward_upgrade_tasks validation failed') - return 1 - - if 'fast_forward_post_upgrade_tasks' in role_data and role_data['fast_forward_post_upgrade_tasks']: - if validate_upgrade_tasks(role_data['fast_forward_post_upgrade_tasks']): - print('ERROR: fast_forward_post_upgrade_tasks validation failed') - return 1 - if 'parameters' in tpl: for param in required_params: if param not in tpl['parameters']: