]> scripts.mit.edu Git - wizard.git/blobdiff - wizard/command/upgrade.py
Suppress ^M characters from Git progress bars.
[wizard.git] / wizard / command / upgrade.py
index 1867d51affb316b89689f4f00ecf0f2b872e2bec..4efbda83ae3f18f87dd366e0a471cfcfab4d90f6 100644 (file)
@@ -1,5 +1,6 @@
 import optparse
 import sys
+import distutils.version
 import os
 import shutil
 import logging.handlers
@@ -7,12 +8,18 @@ import errno
 import tempfile
 import itertools
 
-from wizard import command, deploy, shell, util
+from wizard import app, command, deploy, shell, util
 
 def main(argv, baton):
     options, args = parse_args(argv, baton)
     if args:
-        util.chdir(args[0])
+        dir = args[0]
+    else:
+        dir = "."
+    shell.drop_priviledges(dir, options.log_file)
+    util.chdir(dir)
+    if os.path.exists(".scripts/blacklisted"):
+        raise BlacklistedError()
     sh = shell.Shell()
     util.set_git_env()
     if options.continue_:
@@ -22,26 +29,30 @@ def main(argv, baton):
         version = open(".git/WIZARD_UPGRADE_VERSION", "r").read()
         util.chdir(sh.eval("git", "config", "remote.origin.url"))
         d = deploy.Deployment(".")
+        try:
+            sh.call("git", "status")
+            raise LocalChangesError()
+        except shell.CallError:
+            pass
     else:
         d = deploy.Deployment(".")
         d.verify()
         d.verifyTag(options.srv_path)
         d.verifyGit(options.srv_path)
         d.verifyConfigured()
+        d.verifyVersion()
+        if not options.dry_run:
+            d.verifyWeb()
         repo = d.application.repository(options.srv_path)
         version = calculate_newest_version(sh, repo)
-        if version == d.app_version.scripts_tag:
+        if version == d.app_version.scripts_tag and not options.force:
             raise AlreadyUpgraded
         if not options.dry_run:
             perform_pre_commit(sh)
         temp_wc_dir = perform_tmp_clone(sh)
         with util.ChangeDirectory(temp_wc_dir):
-            if options.dry_run:
-                # we delayed performing the pre upgrade commit
-                # until we're in the temporary directory
-                perform_pre_commit(sh)
             sh.call("git", "remote", "add", "scripts", repo)
-            sh.call("git", "fetch", "scripts")
+            sh.call("git", "fetch", "-q", "scripts")
             user_commit, next_commit = calculate_parents(sh, version)
             # save variables so that --continue will work
             # yeah yeah no trailing newline whatever
@@ -51,6 +62,11 @@ def main(argv, baton):
             perform_merge(sh, repo, d, version)
     # variables: version, user_commit, next_commit, temp_wc_dir
     with util.ChangeDirectory(temp_wc_dir):
+        try:
+            sh.call("git", "status")
+            sh.call("git", "commit", "-m", "throw-away commit")
+        except shell.CallError:
+            pass
         message = make_commit_message(version)
         new_tree = sh.eval("git", "rev-parse", "HEAD^{tree}")
         final_commit = sh.eval("git", "commit-tree", new_tree,
@@ -58,25 +74,49 @@ def main(argv, baton):
         # a master branch may not necessarily exist if the user
         # was manually installed to an earlier version
         try:
-            sh.call("git", "checkout", "-b", "master", "--")
+            sh.call("git", "checkout", "-q", "-b", "master", "--")
         except shell.CallError:
-            sh.call("git", "checkout", "master", "--")
-        sh.call("git", "reset", "--hard", final_commit)
+            sh.call("git", "checkout", "-q", "master", "--")
+        sh.call("git", "reset", "-q", "--hard", final_commit)
+        d.verifyVersion()
     # Till now, all of our operations were in a tmp sandbox.
     if options.dry_run:
         logging.info("Dry run, bailing.  See results at %s" % temp_wc_dir)
         return
+    # perform database backup
+    backup = d.backup(options)
     # XXX: frob .htaccess to make site inaccessible
-    # XXX: need locking
-    # git merge (which performs a fast forward)
-    #   - merge could fail (race); that's /really/ dangerous.
-    sh.call("git", "pull", temp_wc_dir, "master")
-    # run update script
-    d.application.upgrade(options)
+    with util.IgnoreKeyboardInterrupts():
+        with util.LockDirectory(".scripts-upgrade-lock"):
+            # git merge (which performs a fast forward)
+            sh.call("git", "pull", "-q", temp_wc_dir, "master")
+            version_obj = distutils.version.LooseVersion(version.partition('-')[2])
+            try:
+                # run update script
+                d.upgrade(version_obj, options)
+                d.verifyWeb()
+            except app.UpgradeFailure:
+                logging.warning("Upgrade failed: rolling back")
+                perform_restore(d, backup)
+                raise
+            except deploy.WebVerificationError as e:
+                logging.warning("Web verification failed: rolling back")
+                logging.info("Web page that was output was:\n\n%s" % e.contents)
+                perform_restore(d, backup)
+                raise app.UpgradeVerificationFailure("Upgrade caused website to become inaccessible; site was rolled back")
     # XXX: frob .htaccess to make site accessible
-    # XXX:  - check if .htaccess changed, first.  Upgrade
+    #       to do this, check if .htaccess changed, first.  Upgrade
     #       process might have frobbed it.  Don't be
-    #       particularly worried if the segment dissappeared
+    #       particularly worried if the segment disappeared
+
+def perform_restore(d, backup):
+    # You don't want d.restore() because it doesn't perform
+    # the file level backup
+    shell.Shell().call("wizard", "restore", backup)
+    try:
+        d.verifyWeb()
+    except deploy.WebVerificationError:
+        logging.critical("Web verification failed after rollback")
 
 def make_commit_message(version):
     message = "Upgraded autoinstall in %s to %s.\n\n%s" % (util.get_dir_owner(), version, util.get_git_footer())
@@ -110,7 +150,7 @@ def perform_tmp_clone(sh):
     temp_dir = tempfile.mkdtemp(prefix="wizard")
     temp_wc_dir = os.path.join(temp_dir, "repo")
     logging.info("Using temporary directory: " + temp_wc_dir)
-    sh.call("git", "clone", "--shared", ".", temp_wc_dir)
+    sh.call("git", "clone", "-q", "--shared", ".", temp_wc_dir)
     return temp_wc_dir
 
 def perform_merge(sh, repo, d, version):
@@ -119,7 +159,7 @@ def perform_merge(sh, repo, d, version):
     # crazy merge algorithm:
     def make_virtual_commit(tag, parents = []):
         """WARNING: Changes state of Git repository"""
-        sh.call("git", "checkout", tag, "--")
+        sh.call("git", "checkout", "-q", tag, "--")
         d.parametrize(".")
         for file in d.application.parametrized_files:
             try:
@@ -142,6 +182,7 @@ def perform_merge(sh, repo, d, version):
         sh.call("git", "merge", next_virtual_commit)
     except shell.CallError:
         print os.getcwd()
+        logging.info("Conflict info:\n", sh.eval("git", "diff"))
         raise MergeFailed
 
 def parse_args(argv, baton):
@@ -158,6 +199,8 @@ WARNING: This is still experimental."""
     parser.add_option("--continue", dest="continue_", action="store_true",
             default=False, help="Continues an upgrade that has had its merge manually "
             "resolved using the current working directory as the resolved copy.")
+    parser.add_option("--force", dest="force", action="store_true",
+            default=False, help="Force running upgrade even if it's already at latest version.")
     baton.push(parser, "srv_path")
     options, args = parser.parse_all(argv)
     if len(args) > 1:
@@ -180,6 +223,23 @@ class MergeFailed(Error):
 
 ERROR: Merge failed.  Resolve the merge by cd'ing to the
 temporary directory, finding conflicted files with `git status`,
-resolving the files, adding them using `git add`, and then
-committing your changes with `git commit` (your log message
-will be ignored), and then running `wizard upgrade --continue`."""
+resolving the files, adding them using `git add` and then
+running `wizard upgrade --continue`."""
+
+class LocalChangesError(Error):
+    def __str__(self):
+        return """
+
+ERROR: Local changes occurred in the install while the merge was
+being processed so that a pull would not result in a fast-forward.
+The best way to resolve this is probably to attempt an upgrade again,
+with git rerere to remember merge resolutions (XXX: not sure if
+this actually works)."""
+
+class BlacklistedError(Error):
+    def __str__(self):
+        return """
+
+ERROR: This autoinstall was manually blacklisted against errors;
+if the user has not been notified of this, please send them
+mail."""