[PATCH 2/2] Honor network config boot params for CD-booted ks installs (#433214)

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Users booting from CD or DVD media, performing a kickstart install, and
passing network configuration boot parameters were hitting an issue
where loader was insisting on trying to get a DHCP lease to fetch the
kickstart file.  Even if you provided 'asknetwork' or 'ip=' settings,
loader was still forcing DHCP at the point where the kickstart file is
downloaded.

This patch makes the 'asknetwork' parameter work if you are booting from
CD or DVD and performing a kickstart install.  By default, we would try
DHCP, but asknetwork forces the configureTCPIP() code path to run.

Likewise, if the user boots from media and passes ip=, netmask=,
gateway=, dns=, and ksdevice=, the network is configured using that
information and the kickstart file is downloaded.
---
 loader/loader.c |    9 +++++----
 loader/net.c    |    4 +++-
 2 files changed, 8 insertions(+), 5 deletions(-)

diff --git a/loader/loader.c b/loader/loader.c
index 93fc4ae..d5d663b 100644
--- a/loader/loader.c
+++ b/loader/loader.c
@@ -1454,10 +1454,11 @@ static char *doLoaderMain(struct loaderData_s *loaderData,
             }
 
             case STEP_NETWORK: {
-                if ((installMethods[validMethods[loaderData->method]].type !=
-                      DEVICE_NETWORK) && (!hasGraphicalOverride()) &&
-                     !FL_ASKNETWORK(flags) &&
-                     !FL_EARLY_NETWORKING(flags)) {
+                if (((installMethods[validMethods[loaderData->method]].type !=
+                       DEVICE_NETWORK) && (!hasGraphicalOverride()) &&
+                      !FL_ASKNETWORK(flags) &&
+                      !FL_EARLY_NETWORKING(flags)) ||
+                     (is_nm_connected())) {
                     needsNetwork = 0;
                     if (dir == 1) 
                         step = STEP_STAGE2;
diff --git a/loader/net.c b/loader/net.c
index 486c160..85787d3 100644
--- a/loader/net.c
+++ b/loader/net.c
@@ -445,7 +445,9 @@ int readNetConfig(char * device, iface_t * iface,
 
     /* JKFIXME: we really need a way to override this and be able to change
      * our network config */
-    if (!FL_TESTING(flags) && ((iface->ipv4method > 0) || (iface->ipv6method > 0))) {
+    if (!FL_TESTING(flags) && !FL_ASKNETWORK(flags) &&
+        ((iface->ipv4method > IPV4_UNUSED_METHOD) ||
+         (iface->ipv6method > IPV4_UNUSED_METHOD))) {
         logMessage(INFO, "doing kickstart... setting it up");
 
         err = writeEnabledNetInfo(iface);
-- 
1.6.2.5

_______________________________________________
Anaconda-devel-list mailing list
Anaconda-devel-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/anaconda-devel-list

[Index of Archives]     [Kickstart]     [Fedora Users]     [Fedora Legacy List]     [Fedora Maintainers]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [Yosemite Photos]     [KDE Users]     [Fedora Tools]
  Powered by Linux