]> scripts.mit.edu Git - wizard.git/commitdiff
Match for Success in wordpress installer. More documentation fixes.
authorEdward Z. Yang <ezyang@mit.edu>
Fri, 20 Nov 2009 22:46:38 +0000 (17:46 -0500)
committerEdward Z. Yang <ezyang@mit.edu>
Fri, 20 Nov 2009 22:46:38 +0000 (17:46 -0500)
Signed-off-by: Edward Z. Yang <ezyang@mit.edu>
TODO
doc/upgrade.rst
wizard/app/wordpress.py

diff --git a/TODO b/TODO
index f83867461576a6cf6944321d0ffac3a6f4d6cea9..923d0b0b6617e7c1f486a2de7c71cbcaa7ceab05 100644 (file)
--- a/TODO
+++ b/TODO
@@ -127,54 +127,14 @@ OVERALL PLAN:
     0. ssh into not-backward, temporarily give the daemon.scripts-security-upd
        bits by blanching it on system:scripts-security-upd, and run parallel-find.pl
 
-    1. Have the Git repository and working copy for the project on hand.
-
-    2. Checkout the pristine branch
-
-    3. Run wizard `prepare-pristine APP-VERSION`
-
-    X. Commit, with name "Appname x.y.z"
-
-    X. Tag as appname-x.y.z
-
-    4. Checkout the master branch
-
-    5. Merge the pristine branch in. Resolve any conflicts that our
-       patches have with new changes. Do NOT let Git auto-commit it
-       with --no-commit (otherwise, you want to git commit --amend
-       to keep our history clean
-
-    X. Commit, with name "Appname x.y.z-scripts".  This is going to be
-       amended.
-
-    6. Run 'wizard prepare-config' on a scripts server while in a checkout
-       of this newest version.  This will prepare a new version of the
-       configuration file based on the application's latest installer.
-       Manually merge back in any custom changes we may have made.
-       Check if any of the regular expressions need tweaking by inspecting
-       the configuration files for user-specific gunk, and modify
-       wizard.app.APPNAME accordingly.  Commit with --amend, and
-       propagate back to your local copy (git reset --hard HEAD~; git pull afs).
+    1. [ see doc/upgrade.rst ]
 
     [ENTER HERE FROM CREATING A NEW REPO]
 
-    7. Check if there are any special update procedures, and update
-       the wizard.app.APPNAME module accordingly.  If this is the first
-       time you are performing an upgrade, implement upgrade() in your
-       Application class. (XXX: extended instructions here).  Test
-       the new update procedure using our test scripts (preferably
-       on a scripts server).  Check this page for more info on our
-       integration tests:
-
-        http://scripts.mit.edu/wizard/testing.html#acceptance-tests
-
-    8. If you have any further changes, git commit --amend, and finally
-       tag as v1.2.3-scripts (or scripts2, if you are amending an install
-       without an upstream changes)
-
     9. Push all of your changes in a public place, and encourage others
        to test, using --srv-path and a full path.
 
+[ XXX: doc/deploy.rst ]
       GET APPROVAL BEFORE PROCEEDING ANY FURTHER;
       THIS IS PUSHING THE CHANGES TO THE PUBLIC
 
@@ -195,15 +155,10 @@ OVERALL PLAN:
 
    12. Run parallel-find.pl to update our inventory
 
+[ XXX: doc/upgrade.rst ]
 * For mass importing into the repository, there are a few extra things:
 
-    * Many applications had patches associated with them.  Be sure to
-      apply them, so later merges work better.
-
-        # the following operation might require -p1
-        patch -p0 < ../app-1.2.3/app-1.2.3.patch  # [FIDDLY BIT]
-
-    * When running updates, if the patch has changed you will have to
+    * When mass producing updates, if the patch has changed you will have to
       do a special procedure for your merge:
 
         git checkout pristine
@@ -223,6 +178,7 @@ OVERALL PLAN:
       You could also just try your luck with a manual merge using the patch
       as your guide.
 
+[ XXX: doc/layout.rst ]
 * The repository for a given application will contain the following files:
 
     - The actual application's files, as from the official tarball
@@ -230,98 +186,3 @@ OVERALL PLAN:
     - A .scripts directory, with the intent of holding Scripts specific files
       if they become necessary.
 
-* Making the module files for a new application
-
-    1. Create a wizard/app/APPNAME.py file.  Create an object Application
-       inheriting from wizard.app.Application (check existing modules for
-       the boilerplate code).
-
-    2. Implement download().  "wizard prepare-pristine" will use this in order
-       to download the next version of an application.
-
-    3. Create a git repository with `git init`
-
-    4. Use `wizard prepare-pristine APP-VERSION` to download the tarball and
-       extract it into the directory.  If download() doesn't work and you don't
-       want to special case it (for example, you need a /really old version/
-       for record-keeping purposes), replace APP-VERSION with PATH, where PATH
-       is the tarball to extract.
-
-    5. `git commit -asm "APP VERSION"`
-
-    6. Check if any patches are needed to make the application work
-       on Scripts (ideally, it shouldn't.)  Pre-existing patches
-       live in /mit/scripts/deploy/APP-VERSION/ directories.
-
-    7. Run `wizard prepare-new` to setup common filesets for our repositories.
-
-    8. If you are running a PHP script, there is usually a php.ini file
-       that we package.  You can see previous instances of this patch
-       at /mit/scripts/deploy/php.ini/ as well as in the repositories
-       of any already migrated scripts.  We hope to make these changes
-       unnecessary once PHP 5.3 arrives.
-
-    9. Do an initial commit (we're gonna be amending the hell of this)
-       using `git commit -asm "APP VERSION-scripts"
-
-   10. Implement install().  Test using `wizard install APP`; you won't
-       be able to do a version-specific install with `wizard install APP-VERSION`
-       until you generate a tag (which will become out of date once you
-       amend the commit.)  Now might be a good time to create a
-       tests/test-install-APP.sh file (use the other tests as reference) so
-       you don't have to constantly enter the parameters when you're doing
-       an install.
-
-   11. Push your changes to a directory accessible in the production environment.
-       In the case of scripts, this is equivalent to your AFS homedir, and
-       the production environment is a scripts.mit.edu.  We're going to
-       perform a configuration in the production environment to extract
-       out the canonical configuration files.
-
-   12. On the production server, call your wizard to perform an installation;
-       be sure to use the option --no-commit in order to make propagating changes
-       back easier.  Inspect the generated configuration files (you can use `git
-       status` to find unversioned files that the installer created), and
-       implement:
-            - extractors
-            - substitutions
-                These are dictionaries of functions that perform extraction
-                and substitution of variables from config files.  You don't
-                actually have to hand code them; you can app.make_extractors
-                and app.make_substitutions on a common dictionary.  Check
-                out wizard/app/__init__.py for more information on this
-                format, as well as other files for samples.
-                (XXX: extended instructions here)
-            - parametrized_files
-                These are any files that contain WIZARD_* variables
-            - checkConfig()
-                This is a simple, fs based check on whether or not the application
-                was configured.  Usually checking if some generated config file
-                is present is sufficient
-            - detectVersion()
-                You might be able to reuse machinery from extractors (namely, whatever
-                function you were using to generate regular expressions), or you might
-                need to code a custom regular expression to parse this out.
-            - deprecated_keys?
-                Usually you won't need this; use it if there's a configuration variable
-                that needs to get parametrized, but isn't actually necessary and
-                gets obsoleted in a later version.  You probably won't know if that's
-                the case until later.
-
-    13. With these implemented, `wizard prepare-config` should now work if you run
-        it on the installed copy.  The configuration file should now contain only
-        generic WIZARD_* variables, and no user-specific config.  If it is, your
-        script was buggy; try again.
-
-    14. The current changes in the working copy should be merged in.  Add any new
-        files, and then `git commit --amend`.  `git push --force` to stick these
-        changes back in the "public" repository.
-
-    15. In your local copy, you can pull the changes by doing `git reset --hard HEAD~`
-        and then a `git pull` from the relevant source.  Otherwise, Git will complain
-        about a non-fast-forward.
-
-    16. Congratulations!  You've implemented the installation code for a new install.
-        Now goto "ENTER HERE FROM CREATING A NEW REPO" and finish the rest of the
-        instructions.
-
index 2acbca3faa467a5cea73b4bf5d4ba3aa2f6bb5de..d98555b5ae5cbc725dcff8ebe305f6fe58b7f105 100644 (file)
@@ -3,6 +3,8 @@ Preparing an upgrade
 
 :Author: Edward Z. Yang <ezyang@mit.edu>
 
+.. highlight:: sh
+
 Wizard is designed to make pushing upgrades as painless as possible.
 In the best case scenario, adding a new version to a Wizard repository
 is as simple as running a few commands.  Even when upstream makes
@@ -22,27 +24,26 @@ is that we use ``wizard prepare-pristine`` in order to simulate the
 upstream upgrade, and then we piggy back off of Git's merge machinery
 to do everything else.
 
-First you prepare the pristine copy:
-
-.. code-block:: sh
+First you prepare the pristine copy::
 
     git checkout pristine
-    wizard prepare-pristine $VERSION
+    wizard prepare-pristine $APP-$VERSION
     git commit -asm "$APPLICATION $VERSION"
     git tag $APP-$VERSION
 
-Next, you merge those changes to the scriptsified ``master`` copy:
-
-.. code-block:: sh
+Next, you merge those changes to the scriptsified ``master`` copy::
 
     git checkout master
     git merge pristine --no-commit
     # resolve conflicts
     git commit -asm "$APPLICATION $VERSION-scripts"
 
-Then, on a scripts server with Wizard pointed at the latest version, run:
+.. note::
 
-.. code-block:: sh
+    If you are creating a fix for a previous scripts version, you should
+    bump the version to ``$VERSION-scripts2``.
+
+Then, on a scripts server with Wizard pointed at the latest version, run::
 
     cd tests
     env WIZARD_NO_COMMIT=1 ./test-install-$APP.sh
@@ -51,10 +52,29 @@ Then, on a scripts server with Wizard pointed at the latest version, run:
 
 Manually restore any custom changes we may have made to the configuration
 file, make sure that no upstream changes broke our regular expressions
-for matching.  Then amend your commit and push back:
-
-.. code-block:: sh
+for matching.  Then amend your commit and push back::
 
     git commit --amend -a
     git push --force
 
+Troubleshooting
+---------------
+
+Merge failed
+''''''''''''
+
+When a merge fails, it's often good to refresh your memory about what
+particular patch was made to that file.  You can find out with::
+
+    git diff :1:$FILE :2:$FILE
+
+If you are performing a repository conversion, a failed merge likely
+means that there is an updated patch lying around in
+:file:`/mit/scripts/deploy/$APP-$VERSION`.  You can then revert
+the files to the pristine version::
+
+    git checkout --theirs $FILE
+
+And then apply the patch.  If the patch is complicated, you may get
+warnings about hunks already being applied; you can ignore those warnings
+(don't assume ``-R``!)
index a7f8e7d38cd5b39bb342b68844d8ebbff04bb341..c14599dc5cfdf3c90722409b2581112071236695 100644 (file)
@@ -66,7 +66,7 @@ class Application(app.Application):
         result = install.fetch(options, "wp-admin/install.php?step=2", post_install)
         logging.debug("install.php output\n\n" + result)
         os.chmod(".", old_mode)
-        if "Finished" not in result:
+        if "Finished" not in result and "Success" not in result:
             raise install.Failure()
 
         # not sure what to do about this