]> scripts.mit.edu Git - wizard.git/blobdiff - TODO
Minor doc updates and code refinements.
[wizard.git] / TODO
diff --git a/TODO b/TODO
index fbcb2cb297e44b216a056892a58c7ef0f5cf0ad6..962ef75903ce8334aa6e872155fec3dcec06b01f 100644 (file)
--- a/TODO
+++ b/TODO
@@ -2,18 +2,104 @@ The Git Autoinstaller
 
 TODO NOW:
 
 
 TODO NOW:
 
-- Fix mediawiki repository (has lots of stale files and a weird
-  history. This also means that the ichuang wiki will need to be
-  remigrated; frob .scripts-version to make it acceptable, but
-  otherwise should be fairly trivial).  When I say "fix", I mean
-  "redo". Thorough documentation would be good too.  Some bits
-  can be automated and/or have tools to assist it
-- Whiteboard the flow for performing an upgrade on a single
-  install. How assisted does it need to be?
-- Conduct migration tool testing
-- Create mass-migration tool (should be able to limit on mediawiki)
-- Run parallel-find.pl
-- Migrate all mediawikis
+- 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
+
+- 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)
+
+- Redo Wordpress conversion, with an eye for automating everything
+  possible (such as downloading the tarball and unpacking)
+
+- Web application for installing autoinstalls has a hard problem
+  with credentials (as well as installations that are not conducted
+  on an Athena machine.)  Possible solutions include asking the user
+  to SSH into an athena machine and run a bunch of commands, or writing
+  a Java applet (possibly in Clojure or Scala) which gets filesystem
+  permissions and then performs the operations.
+
+- Pay back code debt
+    - Genericize callAsUser and drop_priviledges in shell
+    - Summary script should be more machine friendly, and should not
+      output summary charts when I increase specificity
+    - Summary script should do something intelligent when distinguishing
+      between old-style and new-style installs
+    - Report code in wizard/command/__init__.py is ugly as sin.  Also,
+      the Report object should operate at a higher level of abstraction
+      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.
+    - Move resolutions in mediawiki.py to a text file? (the parsing overhead
+      may not be worth it)
+    - 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.  `php -l` can be a quick stopgap
+
+- Other stuff
+    - Make single user mass-migrate work when not logged in as root
+    - Don't use the scripts heuristics unless we're on scripts with the
+      AFS patch.  Check with `fs sysname`
+    - Make 'wizard summary' generate nice pretty graphs of installs by date
+      (more histograms, will need to check actual .scripts-version files.)
+    - It should be able to handle installs like Django where there's a component
+      that gets installed in web_scripts and another directory that gets installed
+      in Scripts.
+    - ACLs is a starting point for sending mail to users, but it has
+      several failure modes:
+        - Old maintainers who don't care who are still on the ACL
+        - Private AFS groups that aren't mailing lists and that we
+          can't get to
+      A question is whether or not sending mail actually helps us:
+      many users will probably have to come back to us for help; many
+      other users won't care.
+
+PULLING OUT CONFIGURATION FILES IN AN AUTOMATED MANNER
+
+advancedpoll: Template file to fill out
+django: Noodles of template files
+gallery2: Multistage install process
+joomla: Template file
+mediawiki: One-step install process
+phpbb: Multistage install process
+phpical: Template file
+trac: NFC
+turbogears: NFC
+wordpress: Multistage install process
+
+COMMIT MESSAGE FIELDS:
+
+Installed-by: username@hostname
+Pre-commit-by: Real Name <username@mit.edu>
+Upgraded-by: Real Name <username@mit.edu>
+Migrated-by: Real Name <username@mit.edu>
+Wizard-revision: abcdef1234567890
+Wizard-args: /wizard/bin/wizard foo bar baz
+
+GIT COMMIT FIELDS:
+
+Committer: Real Name <username@mit.edu>
+Author: lockername locker <lockername@scripts.mit.edu>
+
+NOTES:
+
+- It is not required nor expected for update scripts to exist for all
+  intervening versions that were present pre-migration; only for it
+  to work on the most recent migration.
+
+- Currently all repositories are initialized with --shared, which
+  means they have basically ~no space footprint.  However, it
+  also means that /mit/scripts/wizard/srv MUST NOT lose revs after
+  deployment.
 
 OVERALL PLAN:
 
 
 OVERALL PLAN:
 
@@ -21,119 +107,208 @@ OVERALL PLAN:
   on documenting them.  Specifically, we will be keeping:
 
     - parallel-find.pl, and the resulting
   on documenting them.  Specifically, we will be keeping:
 
     - parallel-find.pl, and the resulting
-/mit/scripts/sec-tools/store/scriptslist
+      /mit/scripts/.htaccess/scripts/sec-tools/store/scriptslist
 
 
-    - The current install scripts will be kept in place, sans changes
-      necessary to make them use Git install of copying the script over.
-      Porting these scripts to Python and making them modular would be
-      nice, but is priority.  For the long term, seeing this scripts
-      be packaged with rest of our code would be optimal.
+* The new procedure for generating an update is as follows:
+  (check out the mass-migration instructions for something in this spirit,
+  although uglier in some ways; A indicates the step /should/ be automated)
 
 
-* The new procedure for generating an update is as follows (this is
-  also similar to procedure for creating these repositories):
+    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
 
 
     1. Have the Git repository and working copy for the project on hand.
 
     2. Checkout the pristine branch
 
-    3. Remove all files from the working copy (rm -Rf *, and then delete
-       any dot stragglers.  A script to do this would be handy)
+    3. Run wizard `prepare-pristine APP-VERSION`
 
 
-    4. Download the new tarball
+    X. Commit, with name "Appname x.y.z"
 
 
-    5. Extract the tarball over the working copy (`cp -R a/. b` works well,
-       remember that the working copy is empty)
+    4. Checkout the master branch
 
 
-    6. Check for empty directories and add stub files as necessary
-       (use preserve-empty-dir)
-
-    7. Git add it all, and then commit as a new pristine version (v1.2.3)
-
-    8. Checkout the master branch
-
-    9. [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
 
        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 THE FIRST TIME]
-       Apply the scripts patch that was used for that version here
-       (usually patch -p1 < patch)
+    X. Commit, with name "Appname x.y.z-scripts".  This is going to be
+       amended.
 
 
-   10. Check if there are any special update procedures, and update the
-       .scripts/update shell script as necessary (this means that any
-       application specific update logic will be kept with the actual
-       source code.  The language of this update script will vary
-       depending on context.)
+    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).
 
 
-   11. Commit your changes, and tag as v1.2.3-scripts
+    [ENTER HERE FROM CREATING A NEW REPO]
 
 
-   If you're setting up a repository from scratch, stop here, and
-   repeat as necessary
+    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:
 
 
-       XXX: Should we force people to push to the real repository at
-       this point, or just make the repository that the script pulls
-       stuff out of configurable? (Twiddling origin can get you a
-       devel setup with no code changes)
-
-   12. Run the "dry-run script", which uses Git commands to check how many
-       working copies apply the change cleanly, and writes out a logfile
-       with the working copies that don't apply cleanly.
+        http://scripts.mit.edu/wizard/testing.html#acceptance-tests
 
 
-   13. Run the "limited run" script, which applies the update to our
-       test-bed, and lets us check the basic functionality of the update.
-       This can include a script that lets us update a single directory
-       with verbose output.
+    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)
 
 
-   14. Run the "deploy" script, which applies the update to all working
-       copies possible, and sends mail to users to whom the working copy
-       did not apply cleanly. It also frobs .scripts/version for successful
-       upgrades.
+    9. Push all of your changes in a public place, and encourage others
+       to test, using --srv-path and a full path.
 
 
-   15. Run parallel-find.pl
+      GET APPROVAL BEFORE PROCEEDING ANY FURTHER;
+      THIS IS PUSHING THE CHANGES TO THE PUBLIC
 
 
-* The repository for a given application will contain the following files:
+      NOTE: The following commands are to be run on not-backward.mit.edu.
+      You'll need to add daemon.scripts-security-upd to
+      scripts-security-upd to get bits to do this.  Make sure you remove
+      these bits when you're done.
 
 
-    - The actual application's files, as from the official tarball
+   10. Run `wizard research appname`
+       which uses Git commands to check how many
+       working copies apply the change cleanly, and writes out a logfile
+       with the working copies that don't apply cleanly.  It also tells
+       us about "corrupt" working copies, i.e. working copies that
+       have over a certain threshold of changes.
 
 
-    - A .scripts directory, which contains the following information:
+   11. Run `wizard mass-upgrade appname`, which applies the update to all working
+       copies possible.
 
 
-        * .scripts/update shell script (with the +x bit set appropriately),
-          which performs the commands necessary to update a script.  This can
-          be in any language.
+   12. Run parallel-find.pl to update our inventory
 
 
-        * .scripts/.htaccess to prevent this directory from being accessed
-          from the web.
+* For mass importing into the repository, there are a few extra things:
 
 
-        * .scripts/database (generated) contains the database the
-          user installed the script to, so scripts-remove can clean it
+    * Many applications had patches associated with them.  Be sure to
+      apply them, so later merges work better.
 
 
-        * .scripts/version (generated) which contains the version
-          last autoinstalled (as distinct from the actual version
-          the script is) (This is the same as .scripts-version right
-          now; probably want to keep that for now)
+        # the following operation might require -p1
+        patch -p0 < ../app-1.2.3/app-1.2.3.patch  # [FIDDLY BIT]
 
 
-            XXX: It's unclear if we want to move to this wholesale, or
-            delay this indefinitely.
+    * When running updates, if the patch has changed you will have to
+      do a special procedure for your merge:
 
 
-* The migration process has been implemented, see 'wizard migrate'.
+        git checkout pristine
+        # NOTE: Now, the tricky part (this is different from a real update)
+        git symbolic-ref HEAD refs/heads/master
+        # NOTE: Now, we think we're on the master branch, but we have
+        # pristine copy checked out
+        # NOTE: -p0 might need to be twiddled
+        patch -p0 < ../app-1.2.3/app-1.2.3.patch
+        git add .
+        # reconstitute .scripts directory
+        git checkout v1.2.2-scripts -- .scripts
+        git add .scripts
+        # NOTE: Fake the merge
+        git rev-parse pristine > .git/MERGE_HEAD
 
 
-    XXX: We have not decided what migration should do to .scripts-version;
-    if it does move it to .scripts, repositories should have a .gitignore
-    in those directories
+      You could also just try your luck with a manual merge using the patch
+      as your guide.
 
 
-* The autoupgrade shall be the process of:
+* The repository for a given application will contain the following files:
 
 
-    # Make the directory not accessible by the outside world (htaccess, but be careful!)
-    git add -u .
-    git commit -m 'automatically generated backup'
-    git pull origin master
-    if [ $? ne 0 ]; then git reset --hard; echo 'conflicts during upgrade'; fi
-    ./.scripts/update
-    # Make it accessible
+    - The actual application's files, as from the official tarball
 
 
-  (with some more robust error checking)
+    - 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.
 
 
-* Make 'wizard summary' generate nice pretty graphs of installs by date
-  (more histograms, will need to check actual .scripts-version files.)