Hello community,

here is the log from the commit of package yast2-vm for openSUSE:Factory 
checked in at 2017-11-10 14:43:28
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:Factory/yast2-vm (Old)
 and      /work/SRC/openSUSE:Factory/.yast2-vm.new (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Package is "yast2-vm"

Fri Nov 10 14:43:28 2017 rev:77 rq:539739 version:3.2.4

Changes:
--------
--- /work/SRC/openSUSE:Factory/yast2-vm/yast2-vm.changes        2017-10-19 
19:30:49.016581660 +0200
+++ /work/SRC/openSUSE:Factory/.yast2-vm.new/yast2-vm.changes   2017-11-10 
14:43:31.354181396 +0100
@@ -1,0 +2,11 @@
+Thu Nov  2 10:43:22 MDT 2017 - carn...@suse.com
+
+- bsc#1007599 - yast virtualization module does not work well with
+  NM
+- bsc#1066160 - yast2-vm lacks dependencies
+- bsc#1008719 - Installing hypervisor (KVM) should also install
+  network config
+- Cleanup some whitespace issues in VirtConfig.rb and changelog
+- 3.2.4
+
+-------------------------------------------------------------------
@@ -10 +20,0 @@
-
@@ -11,0 +22 @@
+

Old:
----
  yast2-vm-3.2.3.tar.bz2

New:
----
  yast2-vm-3.2.4.tar.bz2

++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Other differences:
------------------
++++++ yast2-vm.spec ++++++
--- /var/tmp/diff_new_pack.Bn13vN/_old  2017-11-10 14:43:31.974158983 +0100
+++ /var/tmp/diff_new_pack.Bn13vN/_new  2017-11-10 14:43:31.978158839 +0100
@@ -17,7 +17,7 @@
 
 
 Name:           yast2-vm
-Version:        3.2.3
+Version:        3.2.4
 Release:        0
 
 BuildRoot:      %{_tmppath}/%{name}-%{version}-build
@@ -31,6 +31,7 @@
 BuildRequires:  yast2-testsuite
 Requires:       yast2-bootloader >= 3.1.35
 BuildRequires:  yast2-devtools >= 3.1.10
+Requires:       yast2-network >= 3.1.108
 
 # OSRelease
 Requires:       yast2 >= 3.0.4

++++++ yast2-vm-3.2.3.tar.bz2 -> yast2-vm-3.2.4.tar.bz2 ++++++
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' old/yast2-vm-3.2.3/package/yast2-vm.changes 
new/yast2-vm-3.2.4/package/yast2-vm.changes
--- old/yast2-vm-3.2.3/package/yast2-vm.changes 2017-10-16 15:06:21.689188007 
+0200
+++ new/yast2-vm-3.2.4/package/yast2-vm.changes 2017-11-07 23:56:16.941121226 
+0100
@@ -1,4 +1,15 @@
 -------------------------------------------------------------------
+Thu Nov  2 10:43:22 MDT 2017 - carn...@suse.com
+
+- bsc#1007599 - yast virtualization module does not work well with
+  NM
+- bsc#1066160 - yast2-vm lacks dependencies
+- bsc#1008719 - Installing hypervisor (KVM) should also install
+  network config
+- Cleanup some whitespace issues in VirtConfig.rb and changelog
+- 3.2.4
+
+-------------------------------------------------------------------
 Wed Oct 11 14:59:40 UTC 2017 - agin...@suse.com
 
 - use new pattern name for SLES (same as TW)
@@ -7,8 +18,8 @@
 - 3.2.3
 
 -------------------------------------------------------------------
-
 Wed Jul 19 14:02:32 UTC 2017 - zgock...@gmail.com
+
 - Fix package Install problem on Tumbleweed (bsc#1040042)
 
 -------------------------------------------------------------------
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' old/yast2-vm-3.2.3/package/yast2-vm.spec 
new/yast2-vm-3.2.4/package/yast2-vm.spec
--- old/yast2-vm-3.2.3/package/yast2-vm.spec    2017-10-16 15:06:21.693188007 
+0200
+++ new/yast2-vm-3.2.4/package/yast2-vm.spec    2017-11-07 23:56:16.941121226 
+0100
@@ -17,7 +17,7 @@
 
 
 Name:           yast2-vm
-Version:        3.2.3
+Version:        3.2.4
 Release:        0
 Group:         System/YaST
 
@@ -29,6 +29,7 @@
 BuildRequires:  yast2-bootloader >= 3.1.35
 Requires:       yast2-bootloader >= 3.1.35
 BuildRequires:  yast2-devtools >= 3.1.10
+Requires:       yast2-network >= 3.1.108
 License:        GPL-2.0
 
 # OSRelease
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' old/yast2-vm-3.2.3/src/modules/VirtConfig.rb 
new/yast2-vm-3.2.4/src/modules/VirtConfig.rb
--- old/yast2-vm-3.2.3/src/modules/VirtConfig.rb        2017-10-16 
15:06:21.693188007 +0200
+++ new/yast2-vm-3.2.4/src/modules/VirtConfig.rb        2017-11-07 
23:56:16.941121226 +0100
@@ -211,7 +211,7 @@
 
       progress_descriptions = []
 
-      bridge_exists = false
+      configure_bridge = true
       install_xen = false
       install_kvm = false
       widget_id = nil
@@ -406,13 +406,13 @@
        else
           packages = ["patterns-openSUSE-xen_server"] if install_xen_server
        end
-        packages = packages + ["xen-tools", "xen-libs", "libvirt-daemon-xen", 
"tigervnc", "virt-manager"] if install_xen_tools
+        packages = packages + ["xen-tools", "xen-libs", "libvirt-daemon-xen", 
"libvirt-daemon-config-network", "tigervnc", "virt-manager"] if 
install_xen_tools
        if isTumbleweed == true
           packages = packages + ["patterns-server-kvm_server"] if 
install_kvm_server
        else
           packages = packages + ["patterns-openSUSE-kvm_server"] if 
install_kvm_server
        end
-        packages = packages + ["libvirt-daemon-qemu", "tigervnc", 
"virt-manager"] if install_kvm_tools
+        packages = packages + ["libvirt-daemon-qemu", 
"libvirt-daemon-config-network", "tigervnc", "virt-manager"] if 
install_kvm_tools
         packages = packages + ["libvirt-daemon-lxc", 
"libvirt-daemon-config-network"] if install_lxc
         result = Package.DoInstall(common_vm_packages + packages)
         if result == false
@@ -507,34 +507,39 @@
             :from => "any",
             :to   => "list <string>"
                                      )
-        Builtins.foreach(interfaces) do |i|
-          Builtins.y2milestone("Checking for bridges...")
-          bridge_path = Ops.add(Ops.add(Ops.add(@net_path, "/"), i), "/bridge")
-          if Ops.greater_or_equal(SCR.Read(path(".target.dir"), bridge_path), 
0)
-            Builtins.y2milestone("Dom0 already has a configured bridge.")
-            bridge_exists = true
-            raise Break
+        if NetworkService.is_network_manager
+          Builtins.y2milestone("NetworkManager is being used. Bridge 
configuration must be done manually.")
+          configure_bridge = false
+        else
+          Builtins.foreach(interfaces) do |i|
+            Builtins.y2milestone("Checking for bridges...")
+            bridge_path = Ops.add(Ops.add(Ops.add(@net_path, "/"), i), 
"/bridge")
+            if Ops.greater_or_equal(SCR.Read(path(".target.dir"), 
bridge_path), 0)
+              Builtins.y2milestone("Dom0 already has a configured bridge.")
+              configure_bridge = false
+              raise Break
+            end
           end
-      end
+        end
 
         # Popup yes/no dialog
-          if bridge_exists == false
-            if Popup.AnyQuestionRichText(
-                _("Network Bridge."),
-                _(
-                  "<p>For normal network configurations hosting virtual 
machines, a network bridge is recommended.</p><p>Configure a default network 
bridge?</p>"
-                ),
-                45,
-                5,
-                Label.YesButton,
-                Label.NoButton,
-                :focus_yes
-              )
-              Builtins.y2milestone("Configuring default bridge for Xen or 
KVM...")
-              Lan.Read(:cache)
-              Lan.ProposeVirtualized
-              Lan.Write
-            end
+        if configure_bridge == true
+          if Popup.AnyQuestionRichText(
+              _("Network Bridge."),
+              _(
+                "<p>For normal network configurations hosting virtual 
machines, a network bridge is recommended.</p><p>Configure a default network 
bridge?</p>"
+              ),
+              45,
+              5,
+              Label.YesButton,
+              Label.NoButton,
+              :focus_yes
+            )
+            Builtins.y2milestone("Configuring default bridge for Xen or 
KVM...")
+            Lan.Read(:cache)
+            Lan.ProposeVirtualized
+            Lan.Write
+          end
         end
       else
         # For s390, make sure /etc/zipl.conf contain switch_amode


Reply via email to