]> git.decadent.org.uk Git - dak.git/commitdiff
First working version of rhona.
authorJames Troup <james@nocrew.org>
Mon, 18 Dec 2000 07:11:25 +0000 (07:11 +0000)
committerJames Troup <james@nocrew.org>
Mon, 18 Dec 2000 07:11:25 +0000 (07:11 +0000)
THANKS
TODO
contrib/fix.9
db_access.py
docs/README.sourceless [new file with mode: 0644]
katie
katie.conf
katie.conf-non-US
rhona
utils.py

diff --git a/THANKS b/THANKS
index 6190ce984e98063ecc4c6ff20d729353d558ee11..8fbe7241d9f7f9d7a3169ee8097ad3afffbf479c 100644 (file)
--- a/THANKS
+++ b/THANKS
@@ -1,3 +1,6 @@
+Special thanks go to Jason and AJ; without their patient help, none of
+this would be possible.
+
 [Alphabetical Order]
 
 Adam Heath                    <doogie@debian.org>
diff --git a/TODO b/TODO
index 7e4b30d306e29eea1dd5d1c0248979f143800abe..ed959d74c565353d5da460154c3718215dc9d284 100644 (file)
--- a/TODO
+++ b/TODO
@@ -1,16 +1,14 @@
-Show Stopper
-------------
+Urgent
+------
 
-  o finish rhona
-  o claire needs to know about sections on non-non-US
+  o Claire needs to know about sections on auric.
 
-  o Testing... lots of it.
-
-Non-Show Stopper
-----------------
+Less Urgent
+-----------
 
   o CD building scripts need fixing
 
+  o Log files for jenna, rhona and katie.
   o Optimize all the queries by using EXAMINE and building some INDEXs.
   o enclose all the setting SQL stuff in transactions (mostly done).
   o clear out maintainers table
index b461b9b04633a15955d9cca2e006650b1a115345..4a655285abd6448ef6df7abc8398a19ef2902af5 100755 (executable)
@@ -2,7 +2,7 @@
 
 # Fix for bug in katie where dsc_files was initialized from changes and not dsc
 # Copyright (C) 2000  James Troup <james@nocrew.org>
-# $Id: fix.9,v 1.1 2000-12-05 04:27:48 troup Exp $
+# $Id: fix.9,v 1.2 2000-12-18 07:11:25 troup Exp $
 
 # This program is free software; you can redistribute it and/or modify
 # it under the terms of the GNU General Public License as published by
@@ -88,7 +88,7 @@ def main ():
             #print " size: ", dsc_files[i]["size"]
             #print " md5sum: ",dsc_files[i]["md5sum"]
             #print " location_id: ", location_id
-            files_id = db_access.get_files_id(filename, repr(dsc_files[i]["size"]), dsc_files[i]["md5sum"], location_id);
+            files_id = db_access.get_files_id(filename, dsc_files[i]["size"], dsc_files[i]["md5sum"], location_id);
             if files_id < 0 or files_id == None:
                 print " BORK!!!!!!!!!!!!"
                 print " ",filename
index 8760d91bbd475f801281dabb8e7d29f00ebaa27b..0b038862688ae2c7f9eaa48c64da1aff28679d53 100644 (file)
@@ -1,6 +1,6 @@
 # DB access fucntions
 # Copyright (C) 2000  James Troup <james@nocrew.org>
-# $Id: db_access.py,v 1.3 2000-12-05 04:27:48 troup Exp $
+# $Id: db_access.py,v 1.4 2000-12-18 07:11:25 troup Exp $
 
 # This program is free software; you can redistribute it and/or modify
 # it under the terms of the GNU General Public License as published by
@@ -156,13 +156,14 @@ def get_files_id (filename, size, md5sum, location_id):
     if files_id_cache.has_key(cache_key):
         return files_id_cache[cache_key]
 
+    size = int(size);
     q = projectB.query("SELECT id, size, md5sum FROM files WHERE filename = '%s' AND location = %d" % (filename, location_id));
     ql = q.getresult();
     if ql:
         if len(ql) != 1:
             return -1;
         ql = ql[0] 
-        orig_size = ql[1];
+        orig_size = int(ql[1]);
         orig_md5sum = ql[2];
         if orig_size != size or orig_md5sum != md5sum:
             return -2;
diff --git a/docs/README.sourceless b/docs/README.sourceless
new file mode 100644 (file)
index 0000000..e877850
--- /dev/null
@@ -0,0 +1,39 @@
+To: ftpmaster@debian.org
+Subject: binary-less source-only packages
+Mail-Copies-To: never
+Gcc: nnfolder+archive:2000/mail-12-December
+--text follows this line--
+Hi,
+
+katie does reference counting of binaries and any source package which
+isn't referenced by a binary package is automatically deleted.  This:
+
+ o allows us to stop violating the GPL.
+
+ o ensures that the source for any binary package is always available;
+   nice for build problems on non-i386 arches.
+
+ o reduces manual work in cleaning out cruft from the archive as
+   packages where the source name has changed will be automatically
+   cleaned out.
+
+The only problem is intentionally binary-less packages.  The only
+valid occurrence I can think of is pine, but in case there are more
+I've added an override for such files in:
+
+  /org/<host>.debian.org/scripts/override/override.source-only
+
+If you process an intentionally binary-less package in incoming and
+determine it's binary-less state is valid; please add the package to
+the override file.
+
+If I've missed any packages and people are screaming about their
+removal, you can recover them from:
+
+  /org/<host>.debian.org/morgue/rhona/
+
+As to how to add them back into the pool.. ho hum.. have to write
+something to do that :-)
+
+-- 
+James
diff --git a/katie b/katie
index b2295845d9a7c8268207d184a55430427e57cdb6..194354a091dbb08dea7058cf3c235668f5f4f226 100755 (executable)
--- a/katie
+++ b/katie
@@ -2,7 +2,7 @@
 
 # Installs Debian packaes
 # Copyright (C) 2000  James Troup <james@nocrew.org>
-# $Id: katie,v 1.11 2000-12-15 00:15:30 troup Exp $
+# $Id: katie,v 1.12 2000-12-18 07:11:25 troup Exp $
 
 # This program is free software; you can redistribute it and/or modify
 # it under the terms of the GNU General Public License as published by
@@ -1153,18 +1153,11 @@ def main():
         print "\n" + changes_file;
         process_it (changes_file);
 
-    install_mag = " b";
-    if install_bytes > 10000:
-        install_bytes = install_bytes / 1000;
-        install_mag = " Kb";
-    if install_bytes > 10000:
-        install_bytes = install_bytes / 1000;
-        install_mag = " Mb";
     if install_count:
         sets = "set"
         if install_count > 1:
             sets = "sets"
-        sys.stderr.write("Installed %d package %s, %d%s.\n" % (install_count, sets, int(install_bytes), install_mag))
+        sys.stderr.write("Installed %d package %s, %s.\n" % (install_count, sets, utils.size_type(int(install_bytes))));
 
     # Write out the list of already-acknowledged NEW packages
     if Cnf["Dinstall::Options::Ack-New"]:
index 0d436a419bf52bba96104ca5833dbbbe7b3a5011..047b762dd692bd72f8da833f8b17c90391fa05e4 100644 (file)
@@ -96,6 +96,7 @@ Rhona
   // How long (in seconds) dead packages are left before being killed
   StayOfExecution 172800; // 2 days
   MorgueSubDir "rhona";
+  OverrideFilename "override.source-only";
 };
 
 Suite
index c5374d41134ce2d0000461fbecae5469e43a9083..31195a7420a28ea4ac0d55c4df949150c0ba9dc6 100644 (file)
@@ -96,6 +96,7 @@ Rhona
   // How long (in seconds) dead packages are left before being killed
   StayOfExecution 0; // 0 days
  MorgueSubDir "rhona";
+ OverrideFilename "override.source-only";
 };
 
 Suite
diff --git a/rhona b/rhona
index 877cdb01a9c8e8a82b0805d17063c16446973da3..ba99cf4f48a301600c8f17ae5b97cc5efee0b613 100755 (executable)
--- a/rhona
+++ b/rhona
@@ -1,8 +1,8 @@
 #!/usr/bin/env python
 
-# rhona, cleans up unassociated binary (and source) packages
+# rhona, cleans up unassociated binary and source packages
 # Copyright (C) 2000  James Troup <james@nocrew.org>
-# $Id: rhona,v 1.3 2000-12-13 03:18:50 troup Exp $
+# $Id: rhona,v 1.4 2000-12-18 07:11:25 troup Exp $
 
 # This program is free software; you can redistribute it and/or modify
 # it under the terms of the GNU General Public License as published by
@@ -18,6 +18,8 @@
 # along with this program; if not, write to the Free Software
 # Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
 
+###################################################################################################
+
 # 07:05|<elmo> well.. *shrug*.. no, probably not.. but to fix it,
 #      |       we're going to have to implement reference counting
 #      |       through dependencies.. do we really want to go down
 #
 # 07:05|<Culus> elmo: Augh! <brain jumps out of skull>
 
-import pg, string, os, sys, time
+###################################################################################################
+
+import os, pg, stat, string, sys, time
 import apt_pkg
 import utils
 
+###################################################################################################
+
 projectB = None
 Cnf = None
+delete_date = None;
+overrides = {};
+
+###################################################################################################
+
+# See if a given package is in the override file.  Caches and only loads override files on demand.
+
+def in_override_p (package):
+    global overrides;
+
+    if overrides == {}:
+        filename = Cnf["Dir::OverrideDir"] + Cnf["Rhona::OverrideFilename"];
+        file = utils.open_file(filename, 'r');
+        for line in file.readlines():
+            line = string.strip(utils.re_comments.sub('', line))
+            if line != "":
+                overrides[line] = 1
+        file.close()
+
+    return overrides.get(package, None);
+
+###################################################################################################
 
 def check_binaries():
+    global delete_date;
+    
+    print "Checking for orphaned binary packages..."
+
     # A nicer way to do this would be `SELECT bin FROM
     # bin_associations EXCEPT SELECT id from binaries WHERE
     # last_update IS NULL', but it seems postgresql can't handle that
     # query as it hadn't return after I left it running for 20 minutes
     # on auric.
-
+    
     linked_binaries = {};
     q = projectB.query("SELECT bin FROM bin_associations");
     ql = q.getresult();
@@ -54,8 +86,7 @@ def check_binaries():
     projectB.query("BEGIN WORK");
     for id in all_binaries.keys():
         if not linked_binaries.has_key(id):
-            date = time.strftime("%Y-%m-%d %H:%M", time.localtime(time.time()-(4*(24*60*60))))
-            projectB.query("UPDATE files SET last_used = '%s' WHERE id = %s" % (date, all_binaries[id]))
+            projectB.query("UPDATE files SET last_used = '%s' WHERE id = %s" % (delete_date, all_binaries[id]))
     projectB.query("COMMIT WORK");
 
     # Check for any binaries which are marked for eventual deletion but are now used again.
@@ -65,6 +96,7 @@ def check_binaries():
     ql = q.getresult();
     for i in ql:
         all_marked_binaries[i[0]] = i[1];
+        
     projectB.query("BEGIN WORK");
     for id in all_marked_binaries.keys():
         if linked_binaries.has_key(id):
@@ -74,8 +106,12 @@ def check_binaries():
     projectB.query("COMMIT WORK");
 
 def check_sources():
+    global delete_date;
+
+    print "Checking for orphaned source packages..."
+
     # A nicer way to do this would be using `EXCEPT', but see the
-    # commeint in process_binary.
+    # comment in check_binaries().
 
     linked_sources = {};
     q = projectB.query("SELECT source FROM binaries WHERE source is not null");
@@ -84,24 +120,49 @@ def check_sources():
         linked_sources[i[0]] = "";
     
     all_sources = {};
-    q = projectB.query("SELECT s.id, s.file FROM source s, files f WHERE f.last_used IS NULL AND f.id = s.file")
+    all_sources_package = {};
+    q = projectB.query("SELECT s.id, s.file, s.source FROM source s, files f WHERE f.last_used IS NULL AND f.id = s.file");
     ql = q.getresult();
     for i in ql:
         all_sources[i[0]] = i[1];
+        all_sources_package[i[0]] = i[2];
 
     projectB.query("BEGIN WORK");
     for id in all_sources.keys():
         if not linked_sources.has_key(id):
-            date = time.strftime("%Y-%m-%d %H:%M", time.localtime(time.time()-(4*(24*60*60))))
-            projectB.query("UPDATE files SET last_used = '%s' WHERE id = %s" % (date, all_sources[id]))
+            # Is this a known source-only package?
+            if in_override_p(all_sources_package[id]):
+                continue;
+            # Then check to see if the source is still in any suites
+            untouchable = 0;
+            q = projectB.query("SELECT su.suite_name FROM src_associations sa, suite su WHERE sa.source = %s and su.id = sa.suite" % (id));
+            for i in q.getresult():
+                if Cnf.Find("Suite::%s::Untouchable" % (i[0])):
+                    untouchable = 1;
+                else:
+                    projectB.query("DELETE FROM src_associations WHERE source = %s" % (id));
+
+            # We can't delete binary-less source-only packages if
+            # they're in an untouchable suite (i.e. stable)...
+            if untouchable:
+                continue;
+
+            projectB.query("UPDATE files SET last_used = '%s' WHERE id = %s" % (delete_date, all_sources[id]))
             # Delete all other files references by .dsc too if they're not used by anyone else
             q = projectB.query("SELECT f.id FROM files f, dsc_files d WHERE d.source = %d AND d.file = f.id" % (id));
-            ql = q.getresult();
-            for i in ql:
-                q_others = projectB.query("SELECT file FROM dsc_files d WHERE file = %s" % (i[0]));
-                ql_others = q.getresult();
+            for i in q.getresult(): 
+                q = projectB.query("SELECT id FROM dsc_files d WHERE file = %s" % (i[0]));
+                ql = q.getresult();
                 if len(ql) == 1:
-                    projectB.query("UPDATE files SET last_used = '%s' WHERE id = %s" % (date, i[0]));
+                    projectB.query("UPDATE files SET last_used = '%s' WHERE id = %s" % (delete_date, i[0]));
+
+            # If the file is used by another source package
+            # (e.g. because it's an .orig.tar.gz) We need to delete
+            # this source package's reference to it from dsc_files.
+            # So just clear out all references to the source file in
+            # dsc_files now.
+            projectB.query("DELETE FROM dsc_files WHERE source = %s" % (id));
+                    
     projectB.query("COMMIT WORK");
 
     # Check for any sources which are marked for eventual deletion but are now used again.
@@ -144,26 +205,108 @@ def check_sources():
                     projectB.query("UPDATE files SET last_used = NULL WHERE id = %s" % (i[0]));
     projectB.query("COMMIT WORK");
 
+def check_files():
+    global delete_date;
+
+    print "Checking for unused files..."
+
+    # Check for files not references in either binaries or dsc_files
+    used = {};
+    q = projectB.query("SELECT file FROM binaries");
+    for i in q.getresult():
+        used[i[0]] = "";
+    q = projectB.query("SELECT file FROM dsc_files");
+    for i in q.getresult():
+        used[i[0]] = "";
+        
+    all = {};
+    q = projectB.query("SELECT f.id, l.path, f.filename FROM files f, location l WHERE f.location = l.id;");
+    for i in q.getresult():
+        all[i[0]] = i[1] + i[2];
+
+    projectB.query("BEGIN WORK");
+    for id in all.keys():
+        if not used.has_key(id):
+            projectB.query("UPDATE files SET last_used = '%s' WHERE id = %s" % (delete_date, id));
+    projectB.query("COMMIT WORK");
+    
 def clean_binaries():
-    date = time.strftime("%Y-%m-%d %H:%M", time.localtime(time.time()-int(Cnf["Rhona::StayOfExecution"])));
-    print projectB.query("DELETE FROM binaries WHERE file IN (SELECT id FROM files WHERE last_used < '%s')" % (date));
+    global delete_date;
+
+    # We do this here so that the binaries we remove will have their
+    # source also removed (if possible).
+
+    print "Cleaning binaries from the DB..."
+    projectB.query("DELETE FROM binaries WHERE file IN (SELECT id FROM files WHERE last_used < '%s')" % (delete_date));
 
 def clean():
-    date = time.strftime("%Y-%m-%d %H:%M", time.localtime(time.time()-int(Cnf["Rhona::StayOfExecution"])));
-    # Delete from source + dsc_files
-    q = projectB.query("SELECT l.path, f.filename FROM location l, files f WHERE f.last_used < '%s' AND l.id = f.location" % (date));
+    global delete_date;
+    count = 0;
+    size = 0;
+
+    print "Cleaning out packages..."
+
+    # Ensure destination directory exists
+    dest = Cnf["Dir::Morgue"] + '/' + Cnf["Rhona::MorgueSubDir"];
+    if not os.path.exists(dest):
+        os.mkdir(dest);
+        
+    # Delete from source (dsc_file should already be done!)
+    projectB.query("DELETE FROM source WHERE file IN (SELECT id FROM files WHERE last_used <= '%s')" % (delete_date));
+    # Delete files from the pool
+    q = projectB.query("SELECT l.path, f.filename FROM location l, files f WHERE f.last_used <= '%s' AND l.id = f.location" % (delete_date));
     for i in q.getresult():
         filename = i[0] + i[1];
         if not os.path.exists(filename):
             sys.stderr.write("E: can not find %s.\n" % (filename));
             continue;
-        dest = Cnf["Dir::Morgue"] + '/' + Cnf["Rhona::MorgueSubDir"];
-        print "Cleaning %s to %s..." % (filename, dest);
-        #utils.move(filename, dest);
+        if os.path.isfile(filename):
+            if os.path.islink(filename):
+                count = count + 1;
+                #print "Removing symlink %s..." % (filename);
+                os.unlink(filename);
+            else:
+                size = size + os.stat(filename)[stat.ST_SIZE];
+                count = count + 1;
+                #print "Cleaning %s to %s..." % (filename, dest);
+                utils.move(filename, dest);
+        else:
+            sys.stderr.write("%s is neither symlink nor file?!\n" % (filename));
+            sys.exit(1);
     # delete from files
+    projectB.query("DELETE FROM files WHERE last_used <= '%s'" % (delete_date));
+    if count > 0:
+        sys.stderr.write("Cleaned %d files, %s.\n" % (count, utils.size_type(size)));
+
+def clean_maintainers():
+    print "Cleaning out unused Maintainer entries..."
+    
+    used = {};
+    q = projectB.query("SELECT maintainer FROM binaries WHERE maintainer IS NOT NULL");
+    for i in q.getresult():
+        used[i[0]] = "";
+    q = projectB.query("SELECT maintainer FROM source WHERE maintainer IS NOT NULL");
+    for i in q.getresult():
+        used[i[0]] = "";
+
+    all = {};
+    q = projectB.query("SELECT id, name FROM maintainer");
+    for i in q.getresult():
+        all[i[0]] = i[1];
+
+    count = 0;
+    projectB.query("BEGIN WORK");
+    for id in all.keys():
+        if not used.has_key(id):
+            projectB.query("DELETE FROM maintainer WHERE id = %s" % (id));
+            count = count + 1;
+    projectB.query("COMMIT WORK");
+
+    if count > 0:
+        sys.stderr.write("Cleared out %d maintainer entries.\n" % (count));
 
 def main():
-    global Cnf, projectB;
+    global Cnf, projectB, delete_date;
     
     projectB = pg.connect('projectb', 'localhost');
 
@@ -172,14 +315,14 @@ def main():
     Cnf = apt_pkg.newConfiguration();
     apt_pkg.ReadConfigFileISC(Cnf,utils.which_conf_file());
 
-    print "Checking for orphaned binary packages..."
+    delete_date = time.strftime("%Y-%m-%d %H:%M", time.localtime(time.time()));
+
     check_binaries();
-    print "Cleaning binaries from the DB..."
     clean_binaries();
-    print "Checking for orphaned source packages..."
     check_sources();
-    print "Cleaning orphaned packages..."
+    check_files();
     clean();
+    clean_maintainers();
 
 if __name__ == '__main__':
     main()
index 54ab406be8cb505f45513a156d66e6e04854a0cb..193deafc6238f3ee6fa159afef42257c1ba82f3d 100644 (file)
--- a/utils.py
+++ b/utils.py
@@ -1,6 +1,6 @@
 # Utility functions
 # Copyright (C) 2000  James Troup <james@nocrew.org>
-# $Id: utils.py,v 1.8 2000-12-13 03:18:50 troup Exp $
+# $Id: utils.py,v 1.9 2000-12-18 07:11:25 troup Exp $
 
 # This program is free software; you can redistribute it and/or modify
 # it under the terms of the GNU General Public License as published by
@@ -268,3 +268,12 @@ def regex_safe (s):
 
 ######################################################################################
 
+def size_type (c):
+    t  = " b";
+    if c > 10000:
+        c = c / 1000;
+        t = " Kb";
+    if c > 10000:
+        c = c / 1000;
+        t = " Mb";
+    return ("%d%s" % (c, t))