submodules: Disable ignore=all for OPNFV projects 03/28303/1
authorAlexandru Avadanii <Alexandru.Avadanii@enea.com>
Tue, 7 Feb 2017 09:53:23 +0000 (10:53 +0100)
committerAlexandru Avadanii <Alexandru.Avadanii@enea.com>
Wed, 8 Feb 2017 21:33:42 +0000 (22:33 +0100)
Armband has a number of git submodules, which are hosted:
- via Openstack (review.openstack.org, github.com);
- via OPNFV (gerrit.opnfv.org);

We cannot automatically detect changes in Openstack submodules,
but within OPNFV, an "Updated submodule" commit is generated
when a change is merged into the submodule git, provided we
do not have an "ignore=all" git config directive.

Ignoring submodule changes also prevents automatic ISO builds in
Armband when Fuel@OPNFV changes are merged, which is something we
needed during the D-release development cycle.

The downside of removing "ignore" is that submodules are now
seen as "modified" by git after apply our Armband patches.

While at it, remove one fuel-library patch that got upstream, just
to force a new ISO build.

Change-Id: I8e81e781cb5c78c920a2a33cd8d584de3d9d4fd7
Signed-off-by: Alexandru Avadanii <Alexandru.Avadanii@enea.com>
.gitmodules
patches/fuel-library/upstream-backports/0001-Fix-mysql-OCF-race.patch [deleted file]

index f37e0c9..a8cfda3 100644 (file)
@@ -2,7 +2,10 @@
        path = upstream/fuel
        url = https://gerrit.opnfv.org/gerrit/fuel
        branch = master
-       ignore = all
+[submodule "vswitchperf"]
+       path = upstream/vswitchperf
+       url = https://gerrit.opnfv.org/gerrit/vswitchperf.git
+       branch = master
 [submodule "fuel-plugin-opendaylight"]
        path = upstream/fuel-plugin-opendaylight
        url = https://github.com/openstack/fuel-plugin-opendaylight.git
@@ -13,8 +16,3 @@
        url = https://github.com/openstack/fuel-plugin-ovs
        branch = master
        ignore = all
-[submodule "vswitchperf"]
-       path = upstream/vswitchperf
-       url = https://gerrit.opnfv.org/gerrit/vswitchperf.git
-       branch = master
-       ignore = all
diff --git a/patches/fuel-library/upstream-backports/0001-Fix-mysql-OCF-race.patch b/patches/fuel-library/upstream-backports/0001-Fix-mysql-OCF-race.patch
deleted file mode 100644 (file)
index 3159f53..0000000
+++ /dev/null
@@ -1,125 +0,0 @@
-From: Sergii Golovatiuk <sgolovatiuk@mirantis.com>
-Date: Fri, 20 Jan 2017 20:09:13 +0100
-Subject: [PATCH] Fix mysql OCF race
-
-- Fix bug
-- Fix issues found in bashate
-
-Closes-Bug: #1658144
-
-Change-Id: I93e9ab269d3ad4a285154fafcac6426ef36f6b52
----
- files/fuel-ha-utils/ocf/mysql-wss | 41 +++++++++++++++++++++------------------
- 1 file changed, 22 insertions(+), 19 deletions(-)
-
-diff --git a/files/fuel-ha-utils/ocf/mysql-wss b/files/fuel-ha-utils/ocf/mysql-wss
-index 64d3477..43e7f29 100755
---- a/files/fuel-ha-utils/ocf/mysql-wss
-+++ b/files/fuel-ha-utils/ocf/mysql-wss
-@@ -72,9 +72,9 @@ MYSQL=$OCF_RESKEY_client_binary
- HOSTNAME=$(uname -n)
- MYSQL_OPTIONS_LOCAL="-S $OCF_RESKEY_socket --connect_timeout=10"
- if [ "${OCF_RESKEY_test_conf}" ]; then
--  MYSQL_OPTIONS_TEST="--defaults-extra-file=${OCF_RESKEY_test_conf} ${MYSQL_OPTIONS_LOCAL}"
-+    MYSQL_OPTIONS_TEST="--defaults-extra-file=${OCF_RESKEY_test_conf} ${MYSQL_OPTIONS_LOCAL}"
- else
--  MYSQL_OPTIONS_TEST="$MYSQL_OPTIONS_LOCAL --user=$OCF_RESKEY_test_user --password=$OCF_RESKEY_test_passwd"
-+    MYSQL_OPTIONS_TEST="$MYSQL_OPTIONS_LOCAL --user=$OCF_RESKEY_test_user --password=$OCF_RESKEY_test_passwd"
- fi
- #######################################################################
- usage() {
-@@ -337,7 +337,8 @@ clear_node_pc()
-
- get_master_timeout() {
-     local LH="${LL} get_master_timeout():"
--    local timeout=$(crm_attribute --quiet --name galera_master_timeout \
-+    local timeout
-+    timeout=$(crm_attribute --quiet --name galera_master_timeout \
-         --query --default=$OCF_RESKEY_master_timeout -q | sed -e '/(null)/d')
-
-     ocf_log info "${LH} Setting timeout $timeout"
-@@ -374,8 +375,8 @@ get_node_gtid_with_retry() {
-
-     GTID=$(get_node_gtid $NODE)
-
--    if [ "$GTID"="0" ]; then
--        sleep $[ ( $RANDOM % 10 ) + 1]
-+    if [ "$GTID" = "0" ]; then
-+        sleep $(( ( $RANDOM % 10 )  + 1 ))
-         GTID=$(get_node_gtid $NODE)
-     fi
-
-@@ -384,12 +385,16 @@ get_node_gtid_with_retry() {
-
- check_if_reelection_needed() {
-     local LH="${LL} check_if_reelection_needed()"
--    local PARTITION_WITH_QUORUM=$(crm_node -q | sed -e '/(null)/d')
--    local RESOURCE_NAME=$(echo $OCF_RESOURCE_INSTANCE | cut -f1 -d":")
--    local NODE_COUNT=$(nodes_in_cluster | wc -w)
-+    local PARTITION_WITH_QUORUM
-+    local RESOURCE_NAME
-+    local NODE_COUNT
-     local RUNNING_INSTANCES
-     local rc
-
-+    PARTITION_WITH_QUORUM=$(crm_node -q | sed -e '/(null)/d')
-+    RESOURCE_NAME=$(echo $OCF_RESOURCE_INSTANCE | cut -f1 -d":")
-+    NODE_COUNT=$(nodes_in_cluster | wc -w)
-+
-     if [ $PARTITION_WITH_QUORUM -eq 1 -o $NODE_COUNT -eq 1 ]; then
-         RUNNING_INSTANCES=$(crm_resource \
-             --quiet --locate --resource $RESOURCE_NAME | sed -e '/(null)/d' | wc -l 2> /dev/null)
-@@ -452,8 +457,7 @@ get_master() {
-     ocf_log info "${LH} The most seen GTID is: ${MASTER_GTID}"
-     for NODE in $NODES; do
-         NODE_SCORE=$(crm_simulate -Ls | awk "/${OCF_RESOURCE_INSTANCE}/ && /clone_color/ && ! /${OCF_RESOURCE_INSTANCE}:/ && /${NODE}/ {print \$NF}")
--        if [[ $NODE_SCORE =~ ^-?[0-9]+$ && $NODE_SCORE -le 0 || $NODE_SCORE = "-INFINITY" || -z $NODE_SCORE ]]
--        then
-+        if [[ $NODE_SCORE =~ ^-?[0-9]+$ && $NODE_SCORE -le 0 || $NODE_SCORE = "-INFINITY" || -z $NODE_SCORE ]]; then
-             ocf_log info "${LH} Skipping node $NODE as it is not eligible for running the resource. Its score is ${NODE_SCORE:-NULL}"
-             continue
-         fi
-@@ -487,11 +491,13 @@ check_if_galera_pc() {
-     local LH="${LL} check_if_galera_pc():"
-     local NODES
-     local MASTER
--    local timeout=$(get_master_timeout)
-+    local timeout
-     local GTID
-     local pid
-     local pcnum=0
-
-+    timeout=$(get_master_timeout)
-+
-     ocf_log info "${LH} Checking if Primary Component"
-
-     while [ $timeout -gt 0 ]; do
-@@ -508,15 +514,12 @@ check_if_galera_pc() {
-             ocf_log info "${LH} My neighbour is Primary Component with GTID: ${GTID}"
-             if check_if_new_cluster
-             then
--                for node in ${NODES}
--                do
-+                for node in ${NODES}; do
-                     is_pc=$(crm_attribute --quiet --node ${node} --lifetime reboot --query --name is_pc | sed -e '/(null)/d')
--                    if [ ${is_pc} == "true" ]
--                    then
--                        let pcnum=pcnum+1
-+                    if [ "${is_pc}" = "true" ]; then
-+                        pcnum=$((pcnum + 1))
-                     fi
--                    if [ ${pcnum} -gt 1 ]
--                    then
-+                    if [ ${pcnum} -gt 1 ]; then
-                         ocf_log err "${LH} But I'm running a new cluster, PID:${pid}, this is a split-brain!"
-                         exit $OCF_ERR_GENERIC
-                     fi
-@@ -527,7 +530,7 @@ check_if_galera_pc() {
-         fi
-
-         sleep 10
--        (( timeout -= 10 ))
-+        timeout=$((timeout - 10))
-         ocf_log info "${LH} Waiting for master. ${timeout} seconds left"
-     done
-