Improved documentation and small refactoring in proposing DNS config.

src/clients/save_network.rb

@@ -247,13 +247,14 @@ def copy_from_instsys
       nil
     end
 
-    # Creates DNS configuration on target
+    # Creates target's default DNS configuration
+    #
+    # It uses DNS configuration as defined in AY profile (if any) or
+    # proposes a predefined default values.
     def configure_dns
-      if Mode.autoinst
-        NetworkAutoYast.instance.configure_dns
-      else
-        NetworkAutoconfiguration.instance.configure_dns
-      end
+      ret = false
+      ret = NetworkAutoYast.instance.configure_dns if Mode.autoinst
+      ret = NetworkAutoconfiguration.instance.configure_dns if !ret
 
       DNS.create_hostname_link
     end

src/lib/network/network_autoyast.rb

@@ -122,16 +122,23 @@ def set_network_service
     end
 
     # Initializates DNS setup according AY profile
+    #
+    # FIXME: it currently don't write DNS configuration. It is used for initialization
+    # of DNS setup according AY profile in 1st stage as part of network setup was moved
+    # here already and some parts of network configuration needs to know it. DNS write 
+    # is still done in 2nd stage.
     def configure_dns
       ay_dns_config = ay_networking_section["dns"]
 
-      return if !ay_dns_config
+      return false if !ay_dns_config
 
       DNS.Import(ay_dns_config)
 
       log.info("NetworkAutoYast: DNS / Hostname configuration")
       log.info("dhcp hostname: #{DNS.dhcp_hostname}")
       log.info("write hostname: #{DNS.write_hostname}")
+
+      return true
     end
 
     # Checks if the profile asks for keeping installation network configuration