New solver.upgradeTestcasesToKeep option in zypp.conf.

It tells how many dist upgrade solver testcases should be kept
on the system. Per default just the last two are kept.

diff --git a/VERSION.cmake b/VERSION.cmake
index 24d156ea591e028218fe84153e1fc9caf6b81725..9113d51a493f66494f02500866234c0f68af2e32 100644 (file)
--- a/VERSION.cmake
+++ b/VERSION.cmake
@@ -61,8 +61,8 @@
 SET(LIBZYPP_MAJOR "6")
 SET(LIBZYPP_COMPATMINOR "10")
 SET(LIBZYPP_MINOR "10")
-SET(LIBZYPP_PATCH "3")
+SET(LIBZYPP_PATCH "4")
 #
-# LAST RELEASED: 6.10.3 (10)
+# LAST RELEASED: 6.10.4 (10)
 # (The number in parenthesis is LIBZYPP_COMPATMINOR)
 #=======

diff --git a/package/libzypp.changes b/package/libzypp.changes
index a61d1f21bb2df9477b6c960ee1ee9075550df16b..948f5690885d039f44ccf2753fc7259744d84d83 100644 (file)
--- a/package/libzypp.changes
+++ b/package/libzypp.changes
@@ -1,3 +1,11 @@
+-------------------------------------------------------------------
+Thu Jul 16 16:25:24 CEST 2009 - ma@suse.de
+
+- New solver.upgradeTestcasesToKeep option in zypp.conf. It tells 
+  how many dist upgrade solver testcases should be kept on the system.
+  Per default just the last two are kept. 
+- version 6.10.4 (10)
+
 -------------------------------------------------------------------
 Wed Jul 15 17:53:43 CEST 2009 - ma@suse.de
 

diff --git a/zypp.conf b/zypp.conf
index e1c8920ba2e3aab19465bd6a22d3312ac25b1076..71213414450962e179802acc349f4e11610d12d8 100644 (file)
--- a/zypp.conf
+++ b/zypp.conf
@@ -256,6 +256,19 @@
 ##
 # solver.checkSystemFile = /etc/zypp/systemCheck
 
+##
+## When committing a dist upgrade (e.g. 'zypper dup') a solver testcase
+## is written to /var/log/updateTestcase-<date>. It is needed in bugreports.
+## This optin returns the number of testcases to keep on the system. Old
+## cases will be deleted, as new ones are created.
+##
+## Use 0 to write no testcase at all, or -1 to keep all testcases.
+##
+## Valid values:       Integer
+## Default value:      2
+##
+# solver.upgradeTestcasesToKeep = 2
+
 ##
 ## Packages which are parallel installable with
 ## diffent versions

diff --git a/zypp/ZConfig.cc b/zypp/ZConfig.cc
index 738cc71608be06d1f09767dee6684958ca512887..b05c5ea0c893f8a5cfabbe6033394d6dac21a789 100644 (file)
--- a/zypp/ZConfig.cc
+++ b/zypp/ZConfig.cc
@@ -329,6 +329,10 @@ namespace zypp
                 {
                   solver_allowVendorChange.set( str::strToBool( value, solver_allowVendorChange.get() ) );
                 }
+                else if ( entry == "solver.upgradeTestcasesToKeep" )
+                {
+                  solver_upgradeTestcasesToKeep.set( str::strtonum<unsigned>( value ) );
+                }
                 else if ( entry == "solver.checkSystemFile" )
                 {
                   solver_checkSystemFile = Pathname(value);
@@ -439,8 +443,9 @@ namespace zypp
     int download_max_download_speed;
     int download_max_silent_tries;
 
-    Option<bool,false> solver_onlyRequires;
-    Option<bool,false> solver_allowVendorChange;
+    Option<bool,false>  solver_onlyRequires;
+    Option<bool,false>  solver_allowVendorChange;
+    Option<unsigned,2U> solver_upgradeTestcasesToKeep;
     Pathname solver_checkSystemFile;
 
     std::set<IdString> multiversion;
@@ -638,6 +643,7 @@ namespace zypp
   long ZConfig::download_max_silent_tries() const
   { return _pimpl->download_max_silent_tries; }
 
+
   bool ZConfig::solver_onlyRequires() const
   { return _pimpl->solver_onlyRequires.get(); }
 
@@ -648,6 +654,9 @@ namespace zypp
   { return ( _pimpl->solver_checkSystemFile.empty()
       ? (configPath()/"systemCheck") : _pimpl->solver_checkSystemFile ); }
 
+  unsigned ZConfig::solver_upgradeTestcasesToKeep() const
+  { return _pimpl->solver_upgradeTestcasesToKeep.get(); }
+
   std::set<IdString> ZConfig::multiversion() const
   { return _pimpl->multiversion; }
 

diff --git a/zypp/ZConfig.h b/zypp/ZConfig.h
index 6de0e127fc58b46f47883f64b7289a387cdab99a..c5bb9215ad6004e30bec2e8bc5bf0576a7cd4ec0 100644 (file)
--- a/zypp/ZConfig.h
+++ b/zypp/ZConfig.h
@@ -226,6 +226,15 @@ namespace zypp
        */
       bool solver_allowVendorChange() const;
 
+      /**
+       * When committing a dist upgrade (e.g. <tt>zypper dup</tt>)
+       * a solver testcase is written. It is needed in bugreports,
+       * in case something went wrong. This returns the number of
+       * testcases to keep on the system. Old cases will be deleted,
+       * as new ones are created. Use \c 0 to write no testcase at all.
+       */
+      unsigned solver_upgradeTestcasesToKeep() const;
+
       /**
        * Packages which can be installed parallel with different versions
        * Returning a set of package names (IdString)

diff --git a/zypp/target/TargetImpl.cc b/zypp/target/TargetImpl.cc
index 6b0ddbb2707e2394239893fc3dcdc7849036d957..f6919814e24f4ea7b569fdf787ba89e51639beff 100644 (file)
--- a/zypp/target/TargetImpl.cc
+++ b/zypp/target/TargetImpl.cc
@@ -68,8 +68,12 @@ namespace zypp
   { /////////////////////////////////////////////////////////////////
 
     /** \internal Manage writing a new testcase when doing an upgrade. */
-    void writeOutUpgradeTestcase()
+    void writeUpgradeTestcase()
     {
+      unsigned toKeep( ZConfig::instance().solver_upgradeTestcasesToKeep() );
+      MIL << "Testcases to keep: " << toKeep << endl;
+      if ( !toKeep )
+        return;
       Target_Ptr target( getZYpp()->getTarget() );
       if ( ! target )
       {
@@ -81,6 +85,27 @@ namespace zypp
       Pathname dir( target->assertRootPrefix("/var/log/") );
       Pathname next( dir / Date::now().form( stem+"-%Y-%m-%d-%H-%M-%S" ) );
 
+      {
+        std::list<std::string> content;
+        filesystem::readdir( content, dir, /*dots*/false );
+        std::set<std::string> cases;
+        for_( c, content.begin(), content.end() )
+        {
+          if ( str::startsWith( *c, stem ) )
+            cases.insert( *c );
+        }
+        if ( cases.size() >= toKeep )
+        {
+          unsigned toDel = cases.size() - toKeep + 1; // +1 for the new one
+          for_( c, cases.begin(), cases.end() )
+          {
+            filesystem::recursive_rmdir( dir/(*c) );
+            if ( ! --toDel )
+              break;
+          }
+        }
+      }
+
       MIL << "Write new testcase " << next << endl;
       getZYpp()->resolver()->createSolverTestcase( next.asString(), false/*no solving*/ );
     }
@@ -697,7 +722,7 @@ namespace zypp
       ///////////////////////////////////////////////////////////////////
       if ( getZYpp()->resolver()->upgradeMode() )
       {
-        writeOutUpgradeTestcase();
+        writeUpgradeTestcase();
       }
 
       ///////////////////////////////////////////////////////////////////