Fix can not set is_default in network

The value of is_default always be None, can not be set by
"network set" command. Allow "--default" and "--no-default"
options to be recognized when ``--external`` is not present.

Closes-bug:#1665231

Change-Id: I7a05fc7734a15994f72ca4e47997b4952f1f72f8

Author: jiahui.qiang

openstackclient/network/v2/network.py

@@ -111,10 +111,10 @@ def _get_attrs(client_manager, parsed_args):
         attrs['router:external'] = False
     if parsed_args.external:
         attrs['router:external'] = True
-        if parsed_args.no_default:
-            attrs['is_default'] = False
-        if parsed_args.default:
-            attrs['is_default'] = True
+    if parsed_args.no_default:
+        attrs['is_default'] = False
+    if parsed_args.default:
+        attrs['is_default'] = True
     # Update Provider network options
     if parsed_args.provider_network_type:
         attrs['provider:network_type'] = parsed_args.provider_network_type

openstackclient/tests/functional/network/v2/test_network.py

@@ -126,6 +126,7 @@ def test_network_list(self):
         cmd_output = json.loads(self.openstack(
             'network create -f json ' +
             '--description aaaa ' +
+            '--no-default ' +
             name1
         ))
         self.addCleanup(self.openstack, 'network delete ' + name1)
@@ -147,17 +148,11 @@ def test_network_list(self):
             'Internal',
             cmd_output["router:external"],
         )
-        # NOTE(dtroyer): is_default is not present in the create output
-        #                so make sure it stays that way.
-        # NOTE(stevemar): is_default *is* present in SDK 0.9.11 and newer,
-        #                 but the value seems to always be None, regardless
-        #                 of the --default or --no-default value.
-        # self.assertEqual('x', cmd_output)
-        if ('is_default' in cmd_output):
-            self.assertEqual(
-                None,
-                cmd_output["is_default"],
-            )
+
+        self.assertEqual(
+            False,
+            cmd_output["is_default"],
+        )
         self.assertEqual(
             True,
             cmd_output["port_security_enabled"],
@@ -185,11 +180,10 @@ def test_network_list(self):
             True,
             cmd_output["shared"],
         )
-        if ('is_default' in cmd_output):
-            self.assertEqual(
-                None,
-                cmd_output["is_default"],
-            )
+        self.assertEqual(
+            None,
+            cmd_output["is_default"],
+        )
         self.assertEqual(
             True,
             cmd_output["port_security_enabled"],
@@ -275,16 +269,11 @@ def test_network_set(self):
             'Internal',
             cmd_output["router:external"],
         )
-        # NOTE(dtroyer): is_default is not present in the create output
-        #                so make sure it stays that way.
-        # NOTE(stevemar): is_default *is* present in SDK 0.9.11 and newer,
-        #                 but the value seems to always be None, regardless
-        #                 of the --default or --no-default value.
-        if ('is_default' in cmd_output):
-            self.assertEqual(
-                None,
-                cmd_output["is_default"],
-            )
+
+        self.assertEqual(
+            False,
+            cmd_output["is_default"],
+        )
         self.assertEqual(
             True,
             cmd_output["port_security_enabled"],
@@ -321,7 +310,6 @@ def test_network_set(self):
             'External',
             cmd_output["router:external"],
         )
-        # why not 'None' like above??
         self.assertEqual(
             False,
             cmd_output["is_default"],
@@ -330,29 +318,3 @@ def test_network_set(self):
             False,
             cmd_output["port_security_enabled"],
         )
-
-        # NOTE(dtroyer): There is ambiguity around is_default in that
-        #                it is not in the API docs and apparently can
-        #                not be set when the network is --external,
-        #                although the option handling code only looks at
-        #                the value of is_default when external is True.
-        raw_output = self.openstack(
-            'network set ' +
-            '--default ' +
-            name
-        )
-        self.assertOutput('', raw_output)
-
-        cmd_output = json.loads(self.openstack(
-            'network show -f json ' + name
-        ))
-
-        self.assertEqual(
-            'cccc',
-            cmd_output["description"],
-        )
-        # NOTE(dtroyer): This should be 'True'
-        self.assertEqual(
-            False,
-            cmd_output["port_security_enabled"],
-        )

releasenotes/notes/bug-1665231-3df6d090d137fe4f.yaml

@@ -0,0 +1,6 @@
+---
+fixes:
+  - |
+    Allow ``--default`` and ``--no-default`` options in  ``network create`` command to
+    be recognized when ``--external`` is not present.
+    [Bug `1665231 <https://bugs.launchpad.net/python-openstackclient/+bug/1665231>`_]