Re: [PATCH] pykickstart: don't set bootProto if --noipv4

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

 



On 10/19/2015 02:02 PM, Chris Lumens wrote:
diff --git a/pykickstart/commands/network.py b/pykickstart/commands/network.py
index 818be79..58992c3 100644
--- a/pykickstart/commands/network.py
+++ b/pykickstart/commands/network.py
@@ -399,6 +399,15 @@ class FC6_Network(FC4_Network):
                        default=False)
          return op

+    def parse(self, args):
+        retval = FC4_Network.parse(self, args)
+
+        # If we specify noipv4 then we need to make sure bootproto is zero'ed
+        # out
+        if retval.noipv4:
+            retval.bootProto = ""
+        return retval
+
  class F8_Network(FC6_Network):
      removedKeywords = FC6_Network.removedKeywords
      removedAttrs = FC6_Network.removedAttrs
diff --git a/tests/commands/network.py b/tests/commands/network.py
index dbad1f9..0645555 100644
--- a/tests/commands/network.py
+++ b/tests/commands/network.py
@@ -35,6 +35,16 @@ class F20_TestCase(CommandTest):
          cmd = "network --device team0 --bootproto dhcp --teamslaves=p3p1,p3p2 --teamconfig=\"{\\\"runner\\\": {\\\"name\\\": \\\"roundrobin\\\"}}\" --activate"
          self.assert_parse(cmd)

+        cmd = "network --noipv4 --activate --hostname=blah.test.com --ipv6=1:2:3:4:5:6:7:8 --device eth0 --nameserver=1:1:1:1::,2:2:2:2::"
+        nd = self.assert_parse(cmd)
+        self.assertEquals(nd.bootProto, "")
+        self.assertEquals(nd.hostname, "blah.test.com")
+        self.assertTrue(nd.noipv4)
+        self.assertEquals(nd.ipv6, "1:2:3:4:5:6:7:8")
+        self.assertIn("1:1:1:1::", nd.nameserver)
+        self.assertIn("2:2:2:2::", nd.nameserver)
+        self.assertEquals(nd.device, "eth0")
+
          # --teamslaves
          # --teamslaves="<DEV1>['<CONFIG1>'],<DEV2>['<CONFIG2>'],..."
          # CONFIGX is json with " escaped to \"

Annoying as it is, I think this fix belongs in an F24 version of the
network command.  It's possible changing this all the way back to FC6
will break some user of pykickstart that's relying on it.


Fair enough, I'll redo it.  Thanks,

Josef

_______________________________________________
Kickstart-list mailing list
Kickstart-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/kickstart-list



[Index of Archives]     [Red Hat General]     [CentOS Users]     [Fedora Users]     [Fedora Maintainers]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]

  Powered by Linux