[#853, !26-p] Update schema scripts and tests

src/bin/admin/tests/mysql_tests.sh.in
    Updated expected column names
    Added check of column names for 8.0 to 8.2 upgrade

src/share/database/scripts/mysql/dhcpdb_create.mysql
    Changed array column to is_array in dhcp4/6_option_def tables

src/share/database/scripts/mysql/dhcpdb_drop.mysql
    Added missing drops for createOptionAuditDHCP4/6

src/share/database/scripts/mysql/upgrade_6.0_to_7.0.sh.in
    Changed array column to is_array in dhcp4/6_option_def tables

src/share/database/scripts/mysql/upgrade_8.1_to_8.2.sh.in
    Added logic to conditionally change column names

src/share/database/scripts/mysql/dhcpdb_create.mysql

@@ -879,7 +879,7 @@ CREATE TABLE IF NOT EXISTS dhcp4_option_def (
     space VARCHAR(128) NOT NULL,
     type TINYINT UNSIGNED NOT NULL,
     modification_ts TIMESTAMP NOT NULL,
-    array TINYINT(1) NOT NULL,
+    is_array TINYINT(1) NOT NULL,
     encapsulate VARCHAR(128) NOT NULL,
     record_types VARCHAR(512) DEFAULT NULL,
     user_context LONGTEXT,
@@ -1136,7 +1136,7 @@ CREATE TABLE IF NOT EXISTS dhcp6_option_def (
     space VARCHAR(128) NOT NULL,
     type TINYINT UNSIGNED NOT NULL,
     modification_ts TIMESTAMP NOT NULL,
-    array TINYINT(1) NOT NULL,
+    is_array TINYINT(1) NOT NULL,
     encapsulate VARCHAR(128) NOT NULL,
     record_types VARCHAR(512) DEFAULT NULL,
     user_context LONGTEXT,

src/share/database/scripts/mysql/dhcpdb_drop.mysql

@@ -62,6 +62,7 @@ DROP TABLE IF EXISTS modification;
 DROP TABLE IF EXISTS parameter_data_type;
 DROP PROCEDURE IF EXISTS createAuditRevisionDHCP4;
 DROP PROCEDURE IF EXISTS createAuditEntryDHCP4;
+DROP PROCEDURE IF EXISTS createOptionAuditDHCP4;
 DROP TRIGGER IF EXISTS dhcp4_global_parameter_AINS;
 DROP TRIGGER IF EXISTS dhcp4_global_parameter_AUPD;
 DROP TRIGGER IF EXISTS dhcp4_global_parameter_ADEL;
@@ -80,6 +81,7 @@ DROP TRIGGER IF EXISTS dhcp4_options_ADEL;
 DROP TABLE IF EXISTS dhcp6_audit_revision;
 DROP PROCEDURE IF EXISTS createAuditRevisionDHCP6;
 DROP PROCEDURE IF EXISTS createAuditEntryDHCP6;
+DROP PROCEDURE IF EXISTS createOptionAuditDHCP6;
 DROP TRIGGER IF EXISTS dhcp6_global_parameter_AINS;
 DROP TRIGGER IF EXISTS dhcp6_global_parameter_AUPD;
 DROP TRIGGER IF EXISTS dhcp6_global_parameter_ADEL;

src/share/database/scripts/mysql/upgrade_6.0_to_7.0.sh.in

@@ -214,7 +214,7 @@ CREATE TABLE IF NOT EXISTS dhcp4_option_def (
     space VARCHAR(128) NOT NULL,
     type TINYINT UNSIGNED NOT NULL,
     modification_ts TIMESTAMP NOT NULL,
-    array TINYINT(1) NOT NULL,
+    is_array TINYINT(1) NOT NULL,
     encapsulate VARCHAR(128) NOT NULL,
     record_types VARCHAR(512) DEFAULT NULL,
     user_context LONGTEXT,
@@ -457,7 +457,7 @@ CREATE TABLE IF NOT EXISTS dhcp6_option_def (
     space VARCHAR(128) NOT NULL,
     type TINYINT UNSIGNED NOT NULL,
     modification_ts TIMESTAMP NOT NULL,
-    array TINYINT(1) NOT NULL,
+    is_array TINYINT(1) NOT NULL,
     encapsulate VARCHAR(128) NOT NULL,
     record_types VARCHAR(512) DEFAULT NULL,
     user_context LONGTEXT,

src/share/database/scripts/mysql/upgrade_8.1_to_8.2.sh.in

@@ -11,11 +11,71 @@ fi
 
 VERSION=`mysql_version "$@"`
 
+# We need to set global RESULT when we exit. This is checked by callers.
+bail() {
+    RESULT=$1
+    exit $1
+}
+
 if [ "$VERSION" != "8.1" ]; then
     printf "This script upgrades 8.1 to 8.2. Reported version is $VERSION. Skipping upgrade.\n"
-    exit 0
+    bail 0
 fi
 
+# Get the schema name from database argument. We need this to
+# query information_schema for the right database.
+for arg in $@
+do
+    cnt=`echo $arg | grep -c "^\-\-"`
+    if [ $cnt -eq 0 ]
+    then
+        schema="$arg"
+        break;
+    fi
+done
+
+# Make sure we can id the schema
+if [ -z $schema ]
+then
+    printf "Could not find database schema name in cmd line args: $@\n"
+    bail 255
+fi
+
+# Save the command line args, as we use these later change_column function. 
+cmdargs="$@"
+
+# Function to rename a column in a table.
+change_column() {
+    local schema="$1"
+    local table="$2"
+    local ocolumn="$3"
+    local ncolumn="$4"
+
+    # First let's find out if the column name in the table actually needs updating.
+    sql="select count(column_name) from information_schema.columns where table_schema='$schema' and table_name = '$table' and column_name = '$ocolumn'"
+
+    count=`mysql -N -B $cmdargs -e "$sql"`
+    if [ $? -ne 0 ]
+    then
+        printf "change_column: schema query failed [$sql]\n"
+        bail 255;
+    fi
+
+    # If we found a match record, the column needs to be renamed 
+    if [ "$count" -eq 1 ]
+    then
+        sql="ALTER TABLE $table CHANGE COLUMN $ocolumn $ncolumn"
+        mysql -N -B $cmdargs -e "$sql"
+        if [ $? -ne 0 ]
+        then
+            printf "change_column: alter query failed: [$sql]\n"
+            bail 255;
+        fi
+    else
+        printf "$table column is already correct\n"
+    fi
+}
+
 mysql "$@" <<EOF
 
 # Drop existing trigger on the dhcp4_shared_network table.
@@ -426,5 +486,14 @@ SET version = '8', minor = '2';
 EOF
 
 RESULT=$?
+if [ $RESULT -ne 0 ]
+then
+    exit $RESULT
+fi
 
-exit $?
+# We need to rename the columns in the option def tables because "array" is
+# a MySQL keyword as of 8.0.17
+change_column $schema dhcp4_option_def array "is_array TINYINT(1) NOT NULL"
+change_column $schema dhcp6_option_def array "is_array TINYINT(1) NOT NULL"
+
+bail 0