Exit non-zero during upgrade if reconfigure fails

* Due to a change in behavior of the Omnibus::Ctl.reconfigure function,
we no longer exited non-zero if reconfigure failed during a package
upgrade. This will return the behavior of exiting with the
corresponding exit code at the time of the error

changelogs/unreleased/5610-migrations-can-silently-fail-during-deployments.yml

+---
+title: Exit non-zero during upgrade if reconfigure fails
+merge_request: 4591
+author:
+type: fixed

files/gitlab-ctl-commands/upgrade.rb

@@ -15,6 +15,8 @@
 # limitations under the License.
 #
+require 'rainbow/ext/string'
+
 require "#{base_path}/embedded/service/omnibus-ctl/lib/gitlab_ctl"
 # For testing purposes, if the first path cannot be found load the second
@@ -118,7 +120,17 @@ add_command 'upgrade', 'Run migrations after a package upgrade', 1 do |cmd_name|
   run_command("rm -rf #{local_mode_cache_path}")
   log 'Reconfigure GitLab to apply migrations'
-  reconfigure(false) # sending 'false' mans "don't quit afterwards"
+  results = reconfigure
+  if results != 0
+    message = <<~EOF
+    ===
+    There was an error running gitlab-ctl reconfigure. Please check the output above for more
+    details.
+    ===
+    EOF
+    warn(message.color(:red))
+    ::Kernel.exit results
+  end
   log 'Restarting previously running GitLab services'
   get_all_services.each do |sv_name|