]> scripts.mit.edu Git - wizard.git/blobdiff - TODO
Minor doc updates and code refinements.
[wizard.git] / TODO
diff --git a/TODO b/TODO
index ec10358e58d31b3f602212b97aff18e9385f5fee..962ef75903ce8334aa6e872155fec3dcec06b01f 100644 (file)
--- a/TODO
+++ b/TODO
@@ -2,23 +2,16 @@ The Git Autoinstaller
 
 TODO NOW:
 
+- Make wizard install accept appname-head (so that you can do a test with
+  head, and do things without tags).  Also make it accept commit hashes.
+  In fact, let it accept any committish.  Figure out what to do if we
+  do a test script with x.y.z when we REALLY mean x.y.z-scripts. XXX!!!
 - Do early validation of inputs for configuration
 - Let 'wizard configure' be interactive
 - Parse output HTML for class="error" and give those errors back to the user,
   then boot them back into configure
-- Indicate a default is available in the --help message.
 
-- Keep my sanity when upgrading 1000 installs
-    - Replace gaierror with a more descriptive name (this is a DNS error)
-
-- Make the rest of the world use Wizard
-    - Make parallel-find.pl use `sudo -u username git describe --tags`
-      to determine version.  Make parallel-find.pl have this have greater
-      precedence.  This also means, however, that we get
-      full mediawiki-1.2.3-2-abcdef names (Have patch, pending testing and commit)
-    - Make deployed installer use 'wizard install' /or/ do a migration
-      after doing a normal install (the latter makes it easier
-      for mass-rollbacks).
+- Replace gaierror with a more descriptive name (this is a DNS error)
 
 - Pre-emptively check if daemon/scripts-security-upd
   is not on scripts-security-upd list (/mit/moira/bin/blanche)
@@ -44,16 +37,13 @@ TODO NOW:
       so we don't have to manually increment fails. (in fact, that should
       probably be called something different).  The by-percent errors should
       also be automated.
-    - Indents in upgrade.py are getting pretty ridiculous; more breaking
-      into functions is probably a good idea
     - Move resolutions in mediawiki.py to a text file? (the parsing overhead
       may not be worth it)
-    - Investigate QuotaParseErrors
     - If a process is C-ced, it can result in a upgrade that has
       an updated filesystem but not updated database. Make this more
       resilient
     - PHP end of file allows omitted semicolon, can result in parse error
-      if merge resolutions aren't careful.
+      if merge resolutions aren't careful.  `php -l` can be a quick stopgap
 
 - Other stuff
     - Make single user mass-migrate work when not logged in as root
@@ -132,48 +122,48 @@ OVERALL PLAN:
 
     3. Run wizard `prepare-pristine APP-VERSION`
 
+    X. Commit, with name "Appname x.y.z"
+
     4. Checkout the master branch
 
-    5. [FOR EXISTING REPOSITORIES]
-       Merge the pristine branch in. Resolve any conflicts that our
+    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
 
-       [FOR NEW REPOSITORIES]
-       Check if any patches are needed to make the application work
-       on Scripts (ideally, it shouldn't.)  Run
-       `wizard prepare-new` to setup common filesets for our repositories.
-
-       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.
+    X. Commit, with name "Appname x.y.z-scripts".  This is going to be
+       amended.
 
-    6. Check if there are any special update procedures, and update
-       the wizard.app.APPNAME module accordingly (or create it, if
-       need be).
-
-    7. Run 'wizard prepare-config' on a scripts server while in a checkout
+    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.
-
-    8. Commit your changes, and tag as v1.2.3-scripts (or scripts2, if
-       you are amending an install without an upstream changes)
+       wizard.app.APPNAME accordingly.  Commit with --amend, and
+       propagate back to your local copy (git reset --hard HEAD~; git pull afs).
 
-      NOTE: These steps should be run on a scripts server
+    [ENTER HERE FROM CREATING A NEW REPO]
 
-    9. Test the new update procedure using our test scripts.  See integration
-       tests for more information on how to do this.
+    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
 
-      GET APPROVAL BEFORE PROCEEDING ANY FURTHER
+    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.
+
+      GET APPROVAL BEFORE PROCEEDING ANY FURTHER;
+      THIS IS PUSHING THE CHANGES TO THE PUBLIC
 
       NOTE: The following commands are to be run on not-backward.mit.edu.
       You'll need to add daemon.scripts-security-upd to
@@ -227,3 +217,98 @@ 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.
+