Make metavar usage consistent
Neutron CLI generally uses all caps for positional arguments via metavars. There are a few exceptions in the current code that do not match the other interfaces. Change-Id: I905baebb3428e994cb724031cbb92120a0953d71 Closes-Bug: #1541625
This commit is contained in:
@@ -45,6 +45,7 @@ class CreateAddressScope(neutronV20.CreateCommand):
|
|||||||
help=_('Set the address scope as shared.'))
|
help=_('Set the address scope as shared.'))
|
||||||
parser.add_argument(
|
parser.add_argument(
|
||||||
'name',
|
'name',
|
||||||
|
metavar='NAME',
|
||||||
help=_('Specify the name of the address scope.'))
|
help=_('Specify the name of the address scope.'))
|
||||||
parser.add_argument(
|
parser.add_argument(
|
||||||
'ip_version',
|
'ip_version',
|
||||||
|
|||||||
@@ -32,9 +32,11 @@ class AddNetworkToDhcpAgent(neutronV20.NeutronCommand):
|
|||||||
parser = super(AddNetworkToDhcpAgent, self).get_parser(prog_name)
|
parser = super(AddNetworkToDhcpAgent, self).get_parser(prog_name)
|
||||||
parser.add_argument(
|
parser.add_argument(
|
||||||
'dhcp_agent',
|
'dhcp_agent',
|
||||||
|
metavar='DHCP_AGENT',
|
||||||
help=_('ID of the DHCP agent.'))
|
help=_('ID of the DHCP agent.'))
|
||||||
parser.add_argument(
|
parser.add_argument(
|
||||||
'network',
|
'network',
|
||||||
|
metavar='NETWORK',
|
||||||
help=_('Network to add.'))
|
help=_('Network to add.'))
|
||||||
return parser
|
return parser
|
||||||
|
|
||||||
@@ -56,9 +58,11 @@ class RemoveNetworkFromDhcpAgent(neutronV20.NeutronCommand):
|
|||||||
parser = super(RemoveNetworkFromDhcpAgent, self).get_parser(prog_name)
|
parser = super(RemoveNetworkFromDhcpAgent, self).get_parser(prog_name)
|
||||||
parser.add_argument(
|
parser.add_argument(
|
||||||
'dhcp_agent',
|
'dhcp_agent',
|
||||||
|
metavar='DHCP_AGENT',
|
||||||
help=_('ID of the DHCP agent.'))
|
help=_('ID of the DHCP agent.'))
|
||||||
parser.add_argument(
|
parser.add_argument(
|
||||||
'network',
|
'network',
|
||||||
|
metavar='NETWORK',
|
||||||
help=_('Network to remove.'))
|
help=_('Network to remove.'))
|
||||||
return parser
|
return parser
|
||||||
|
|
||||||
@@ -83,6 +87,7 @@ class ListNetworksOnDhcpAgent(network.ListNetwork):
|
|||||||
self).get_parser(prog_name)
|
self).get_parser(prog_name)
|
||||||
parser.add_argument(
|
parser.add_argument(
|
||||||
'dhcp_agent',
|
'dhcp_agent',
|
||||||
|
metavar='DHCP_AGENT',
|
||||||
help=_('ID of the DHCP agent.'))
|
help=_('ID of the DHCP agent.'))
|
||||||
return parser
|
return parser
|
||||||
|
|
||||||
@@ -105,6 +110,7 @@ class ListDhcpAgentsHostingNetwork(neutronV20.ListCommand):
|
|||||||
self).get_parser(prog_name)
|
self).get_parser(prog_name)
|
||||||
parser.add_argument(
|
parser.add_argument(
|
||||||
'network',
|
'network',
|
||||||
|
metavar='NETWORK',
|
||||||
help=_('Network to query.'))
|
help=_('Network to query.'))
|
||||||
return parser
|
return parser
|
||||||
|
|
||||||
@@ -128,9 +134,11 @@ class AddRouterToL3Agent(neutronV20.NeutronCommand):
|
|||||||
parser = super(AddRouterToL3Agent, self).get_parser(prog_name)
|
parser = super(AddRouterToL3Agent, self).get_parser(prog_name)
|
||||||
parser.add_argument(
|
parser.add_argument(
|
||||||
'l3_agent',
|
'l3_agent',
|
||||||
|
metavar='L3_AGENT',
|
||||||
help=_('ID of the L3 agent.'))
|
help=_('ID of the L3 agent.'))
|
||||||
parser.add_argument(
|
parser.add_argument(
|
||||||
'router',
|
'router',
|
||||||
|
metavar='ROUTER',
|
||||||
help=_('Router to add.'))
|
help=_('Router to add.'))
|
||||||
return parser
|
return parser
|
||||||
|
|
||||||
@@ -152,9 +160,11 @@ class RemoveRouterFromL3Agent(neutronV20.NeutronCommand):
|
|||||||
parser = super(RemoveRouterFromL3Agent, self).get_parser(prog_name)
|
parser = super(RemoveRouterFromL3Agent, self).get_parser(prog_name)
|
||||||
parser.add_argument(
|
parser.add_argument(
|
||||||
'l3_agent',
|
'l3_agent',
|
||||||
|
metavar='L3_AGENT',
|
||||||
help=_('ID of the L3 agent.'))
|
help=_('ID of the L3 agent.'))
|
||||||
parser.add_argument(
|
parser.add_argument(
|
||||||
'router',
|
'router',
|
||||||
|
metavar='ROUTER',
|
||||||
help=_('Router to remove.'))
|
help=_('Router to remove.'))
|
||||||
return parser
|
return parser
|
||||||
|
|
||||||
@@ -183,6 +193,7 @@ class ListRoutersOnL3Agent(neutronV20.ListCommand):
|
|||||||
self).get_parser(prog_name)
|
self).get_parser(prog_name)
|
||||||
parser.add_argument(
|
parser.add_argument(
|
||||||
'l3_agent',
|
'l3_agent',
|
||||||
|
metavar='L3_AGENT',
|
||||||
help=_('ID of the L3 agent to query.'))
|
help=_('ID of the L3 agent to query.'))
|
||||||
return parser
|
return parser
|
||||||
|
|
||||||
@@ -204,6 +215,7 @@ class ListL3AgentsHostingRouter(neutronV20.ListCommand):
|
|||||||
parser = super(ListL3AgentsHostingRouter,
|
parser = super(ListL3AgentsHostingRouter,
|
||||||
self).get_parser(prog_name)
|
self).get_parser(prog_name)
|
||||||
parser.add_argument('router',
|
parser.add_argument('router',
|
||||||
|
metavar='ROUTER',
|
||||||
help=_('Router to query.'))
|
help=_('Router to query.'))
|
||||||
return parser
|
return parser
|
||||||
|
|
||||||
@@ -236,6 +248,7 @@ class ListPoolsOnLbaasAgent(neutronV20.ListCommand):
|
|||||||
parser = super(ListPoolsOnLbaasAgent, self).get_parser(prog_name)
|
parser = super(ListPoolsOnLbaasAgent, self).get_parser(prog_name)
|
||||||
parser.add_argument(
|
parser.add_argument(
|
||||||
'lbaas_agent',
|
'lbaas_agent',
|
||||||
|
metavar='LBAAS_AGENT',
|
||||||
help=_('ID of the loadbalancer agent to query.'))
|
help=_('ID of the loadbalancer agent to query.'))
|
||||||
return parser
|
return parser
|
||||||
|
|
||||||
@@ -260,6 +273,7 @@ class GetLbaasAgentHostingPool(neutronV20.ListCommand):
|
|||||||
parser = super(GetLbaasAgentHostingPool,
|
parser = super(GetLbaasAgentHostingPool,
|
||||||
self).get_parser(prog_name)
|
self).get_parser(prog_name)
|
||||||
parser.add_argument('pool',
|
parser.add_argument('pool',
|
||||||
|
metavar='POOL',
|
||||||
help=_('Pool to query.'))
|
help=_('Pool to query.'))
|
||||||
return parser
|
return parser
|
||||||
|
|
||||||
@@ -289,6 +303,7 @@ class ListLoadBalancersOnLbaasAgent(neutronV20.ListCommand):
|
|||||||
prog_name)
|
prog_name)
|
||||||
parser.add_argument(
|
parser.add_argument(
|
||||||
'lbaas_agent',
|
'lbaas_agent',
|
||||||
|
metavar='LBAAS_AGENT',
|
||||||
help=_('ID of the loadbalancer agent to query.'))
|
help=_('ID of the loadbalancer agent to query.'))
|
||||||
return parser
|
return parser
|
||||||
|
|
||||||
@@ -313,6 +328,7 @@ class GetLbaasAgentHostingLoadBalancer(neutronV20.ListCommand):
|
|||||||
parser = super(GetLbaasAgentHostingLoadBalancer,
|
parser = super(GetLbaasAgentHostingLoadBalancer,
|
||||||
self).get_parser(prog_name)
|
self).get_parser(prog_name)
|
||||||
parser.add_argument('loadbalancer',
|
parser.add_argument('loadbalancer',
|
||||||
|
metavar='LOADBALANCER',
|
||||||
help=_('LoadBalancer to query.'))
|
help=_('LoadBalancer to query.'))
|
||||||
return parser
|
return parser
|
||||||
|
|
||||||
|
|||||||
@@ -107,7 +107,7 @@ class ListRouterPort(neutronV20.ListCommand):
|
|||||||
def get_parser(self, prog_name):
|
def get_parser(self, prog_name):
|
||||||
parser = super(ListRouterPort, self).get_parser(prog_name)
|
parser = super(ListRouterPort, self).get_parser(prog_name)
|
||||||
parser.add_argument(
|
parser.add_argument(
|
||||||
'id', metavar='router',
|
'id', metavar='ROUTER',
|
||||||
help=_('ID or name of router to look up.'))
|
help=_('ID or name of router to look up.'))
|
||||||
return parser
|
return parser
|
||||||
|
|
||||||
|
|||||||
@@ -82,6 +82,7 @@ class CreateSubnetPool(neutronV20.CreateCommand):
|
|||||||
help=_('Set the subnetpool as shared.'))
|
help=_('Set the subnetpool as shared.'))
|
||||||
parser.add_argument(
|
parser.add_argument(
|
||||||
'name',
|
'name',
|
||||||
|
metavar='NAME',
|
||||||
help=_('Name of subnetpool to create.'))
|
help=_('Name of subnetpool to create.'))
|
||||||
parser.add_argument(
|
parser.add_argument(
|
||||||
'--address-scope',
|
'--address-scope',
|
||||||
|
|||||||
Reference in New Issue
Block a user