Merge "Upgrade playbook fixes for OS upgrade"

openstack-archive · Jun 20, 2019 · b546b42 · b546b42
2 parents 30f49b9 + 6baf532
commit b546b42
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 0 deletions.
diff --git a/deployment/database/mysql-pacemaker-puppet.yaml b/deployment/database/mysql-pacemaker-puppet.yaml
@@ -556,13 +556,21 @@ outputs:
                     - ' '
                     - - 'kolla_set_configs;'
                       - 'chown -R mysql:mysql /var/lib/mysql;'
+                      - 'chown -R mysql:mysql /var/log/mariadb;'
                       - 'mysqld_safe --user=mysql --wsrep-provider=none --skip-networking --wsrep-on=off &'
                       - 'timeout 60 sh -c ''while ! mysqladmin ping --silent; do sleep 1; done'';'
                       - 'mysql_upgrade;'
                       - 'mysqladmin shutdown'
             - name: Bind mounts for temporary container
               set_fact:
                 mysql_upgrade_db_bind_mounts: *mysql_volumes
+            - name: Make sure Mysql upgrade temporary directory exists
+              file:
+                path: /tmp/mariadb-upgrade
+                state: directory
+                owner: root
+                group: root
+                mode: 0700
             - name: Upgrade Mysql database from a temporary container
               shell:
                 str_replace:

diff --git a/deployment/pacemaker/pacemaker-baremetal-puppet.yaml b/deployment/pacemaker/pacemaker-baremetal-puppet.yaml
@@ -238,14 +238,20 @@ outputs:
               when: pacemaker_short_node_names_upgraded | length == pacemaker_short_node_names | length
         - name: upgrade step 2
           when: step|int == 2
+          vars:
+            upgrade_leapp_enabled: {get_param: UpgradeLeappEnabled}
           block:
             - name: Stop pacemaker cluster
               pacemaker_cluster: state=offline
+              when: not upgrade_leapp_enabled
         - name: upgrade step 4
           when: step|int == 4
+          vars:
+            upgrade_leapp_enabled: {get_param: UpgradeLeappEnabled}
           block:
             - name: Start pacemaker cluster
               pacemaker_cluster: state=online
+              when: not upgrade_leapp_enabled
       update_tasks:
         - name: Check pacemaker cluster running before the minor update
           when: step|int == 0  # TODO(marios) disabling validations?