From 256ec66f79f98b41497e8937911604bac7ddeabb Mon Sep 17 00:00:00 2001
From: Richard Theis <rtheis@us.ibm.com>
Date: Wed, 28 Sep 2016 08:16:22 -0500
Subject: Remove beta label for network segment resource

With Ocata development opened, neutron network segments are no
longer beta resources.

Change-Id: I25be51c30df17c746136471b88840f64a03187b9
Partially-Implements: blueprint routed-networks
---
 openstackclient/network/v2/network_segment.py | 42 ++++-----------------------
 openstackclient/network/v2/subnet.py          | 15 +++++-----
 2 files changed, 12 insertions(+), 45 deletions(-)

(limited to 'openstackclient/network')

diff --git a/openstackclient/network/v2/network_segment.py b/openstackclient/network/v2/network_segment.py
index 34cac0e0..94722f1e 100644
--- a/openstackclient/network/v2/network_segment.py
+++ b/openstackclient/network/v2/network_segment.py
@@ -26,12 +26,7 @@ LOG = logging.getLogger(__name__)
 
 
 class CreateNetworkSegment(command.ShowOne):
-    """Create new network segment
-
-       (Caution: This is a beta command and subject to change.
-        Use global option --os-beta-command to enable
-        this command)
-    """
+    """Create new network segment"""
 
     def get_parser(self, prog_name):
         parser = super(CreateNetworkSegment, self).get_parser(prog_name)
@@ -76,7 +71,6 @@ class CreateNetworkSegment(command.ShowOne):
         return parser
 
     def take_action(self, parsed_args):
-        self.validate_os_beta_command_enabled()
         client = self.app.client_manager.network
         attrs = {}
         attrs['name'] = parsed_args.name
@@ -96,12 +90,7 @@ class CreateNetworkSegment(command.ShowOne):
 
 
 class DeleteNetworkSegment(command.Command):
-    """Delete network segment(s)
-
-       (Caution: This is a beta command and subject to change.
-        Use global option --os-beta-command to enable
-        this command)
-    """
+    """Delete network segment(s)"""
 
     def get_parser(self, prog_name):
         parser = super(DeleteNetworkSegment, self).get_parser(prog_name)
@@ -114,7 +103,6 @@ class DeleteNetworkSegment(command.Command):
         return parser
 
     def take_action(self, parsed_args):
-        self.validate_os_beta_command_enabled()
         client = self.app.client_manager.network
 
         result = 0
@@ -137,12 +125,7 @@ class DeleteNetworkSegment(command.Command):
 
 
 class ListNetworkSegment(command.Lister):
-    """List network segments
-
-       (Caution: This is a beta command and subject to change.
-        Use global option --os-beta-command to enable
-        this command)
-    """
+    """List network segments"""
 
     def get_parser(self, prog_name):
         parser = super(ListNetworkSegment, self).get_parser(prog_name)
@@ -161,8 +144,6 @@ class ListNetworkSegment(command.Lister):
         return parser
 
     def take_action(self, parsed_args):
-        self.validate_os_beta_command_enabled()
-
         network_client = self.app.client_manager.network
 
         filters = {}
@@ -204,12 +185,7 @@ class ListNetworkSegment(command.Lister):
 
 
 class SetNetworkSegment(command.Command):
-    """Set network segment properties
-
-       (Caution: This is a beta command and subject to change.
-        Use global option --os-beta-command to enable
-        this command)
-    """
+    """Set network segment properties"""
 
     def get_parser(self, prog_name):
         parser = super(SetNetworkSegment, self).get_parser(prog_name)
@@ -231,7 +207,6 @@ class SetNetworkSegment(command.Command):
         return parser
 
     def take_action(self, parsed_args):
-        self.validate_os_beta_command_enabled()
         client = self.app.client_manager.network
         obj = client.find_segment(parsed_args.network_segment,
                                   ignore_missing=False)
@@ -244,12 +219,7 @@ class SetNetworkSegment(command.Command):
 
 
 class ShowNetworkSegment(command.ShowOne):
-    """Display network segment details
-
-       (Caution: This is a beta command and subject to change.
-        Use global option --os-beta-command to enable
-        this command)
-    """
+    """Display network segment details"""
 
     def get_parser(self, prog_name):
         parser = super(ShowNetworkSegment, self).get_parser(prog_name)
@@ -261,8 +231,6 @@ class ShowNetworkSegment(command.ShowOne):
         return parser
 
     def take_action(self, parsed_args):
-        self.validate_os_beta_command_enabled()
-
         client = self.app.client_manager.network
         obj = client.find_segment(
             parsed_args.network_segment,
diff --git a/openstackclient/network/v2/subnet.py b/openstackclient/network/v2/subnet.py
index 2021d9f0..1b778c91 100644
--- a/openstackclient/network/v2/subnet.py
+++ b/openstackclient/network/v2/subnet.py
@@ -179,7 +179,7 @@ def _get_attrs(client_manager, parsed_args, is_create=True):
             attrs['ipv6_ra_mode'] = parsed_args.ipv6_ra_mode
         if parsed_args.ipv6_address_mode is not None:
             attrs['ipv6_address_mode'] = parsed_args.ipv6_address_mode
-        if 'network_segment' in parsed_args:
+        if parsed_args.network_segment is not None:
             attrs['segment_id'] = client.find_segment(
                 parsed_args.network_segment, ignore_missing=False).id
 
@@ -299,13 +299,12 @@ class CreateSubnet(command.ShowOne):
             help=_("IPv6 address mode, "
                    "valid modes: [dhcpv6-stateful, dhcpv6-stateless, slaac]")
         )
-        if self.app.options.os_beta_command:
-            parser.add_argument(
-                '--network-segment',
-                metavar='<network-segment>',
-                help=_("Network segment to associate with this subnet "
-                       "(ID only)")
-            )
+        parser.add_argument(
+            '--network-segment',
+            metavar='<network-segment>',
+            help=_("Network segment to associate with this subnet "
+                   "(name or ID)")
+        )
         parser.add_argument(
             '--network',
             required=True,
-- 
cgit v1.2.1