Automatically save a solver test case (related to bsc#1084248)

...when the product summary contains a warning for easier debugging - 4.0.10
yast · Mar 8, 2018 · e5b71de · e5b71de
1 parent 2565bbd
commit e5b71de
Show file tree

Hide file tree

Showing 4 changed files with 11 additions and 2 deletions.
diff --git a/package/yast2-update.changes b/package/yast2-update.changes
@@ -1,3 +1,10 @@
+-------------------------------------------------------------------
+Thu Mar  8 09:25:38 UTC 2018 - lslezak@suse.cz
+
+- Automatically save a solver test case when the product summary
+  contains a warning for easier debugging (related to bsc#1084248)
+- 4.0.10
+
 -------------------------------------------------------------------
 Fri Feb  9 14:52:57 UTC 2018 - jlopez@suse.com
 

diff --git a/package/yast2-update.spec b/package/yast2-update.spec
@@ -17,7 +17,7 @@
 
 
 Name:           yast2-update
-Version:        4.0.9
+Version:        4.0.10
 Release:        0
 
 BuildRoot:      %{_tmppath}/%{name}-%{version}-build

diff --git a/src/clients/packages_proposal.rb b/src/clients/packages_proposal.rb
@@ -86,7 +86,7 @@ def main
           @cnt_installed,
           @cnt_removed
         )
-        Builtins.y2milestone("Removed: %1", @removed)
+        log.info("Removed packages: #{@removed.sort}")
 
         @installed_m = Builtins.listmap(@installed) { |p| { p => true } }
         @selected_m = Builtins.listmap(@selected) { |p| { p => true } }

diff --git a/src/clients/update_proposal.rb b/src/clients/update_proposal.rb
@@ -213,6 +213,8 @@ def main
           @ret["warning"] = @warning_message
           @ret["warning_level"] = product_warning["warning_level"] || :warning
         end
+        # save the solver test case with details for later debugging
+        Pkg.CreateSolverTestCase("/var/log/YaST2/solver-upgrade-proposal") if @ret["warning"]
       elsif @func == "AskUser"
 	# With proper control file, this should never be reached
         Report.Error(_("The update summary is read only and cannot be changed."))