0f0d66f3f1
Running limits show without --absolute or --rate returns nothing and the user is left to figure out what they need to provide to get the correct data back. This patch prints an error and help output by making at least one of the arguments required. Change-Id: I576cf8ec0e05524ee67d46c48b56da8d44258667
107 lines
3.9 KiB
Python
107 lines
3.9 KiB
Python
# Copyright 2012-2013 OpenStack Foundation
|
|
#
|
|
# Licensed under the Apache License, Version 2.0 (the "License"); you may
|
|
# not use this file except in compliance with the License. You may obtain
|
|
# a copy of the License at
|
|
#
|
|
# http://www.apache.org/licenses/LICENSE-2.0
|
|
#
|
|
# Unless required by applicable law or agreed to in writing, software
|
|
# distributed under the License is distributed on an "AS IS" BASIS, WITHOUT
|
|
# WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the
|
|
# License for the specific language governing permissions and limitations
|
|
# under the License.
|
|
#
|
|
|
|
"""Limits Action Implementation"""
|
|
|
|
import itertools
|
|
import logging
|
|
|
|
from cliff import lister
|
|
|
|
from openstackclient.common import utils
|
|
from openstackclient.identity import common as identity_common
|
|
|
|
|
|
class ShowLimits(lister.Lister):
|
|
"""Show compute and volume limits"""
|
|
|
|
log = logging.getLogger(__name__ + '.ShowLimits')
|
|
|
|
def get_parser(self, prog_name):
|
|
parser = super(ShowLimits, self).get_parser(prog_name)
|
|
type_group = parser.add_mutually_exclusive_group(required=True)
|
|
type_group.add_argument(
|
|
"--absolute",
|
|
dest="is_absolute",
|
|
action="store_true",
|
|
default=False,
|
|
help="Show absolute limits")
|
|
type_group.add_argument(
|
|
"--rate",
|
|
dest="is_rate",
|
|
action="store_true",
|
|
default=False,
|
|
help="Show rate limits")
|
|
parser.add_argument(
|
|
"--reserved",
|
|
dest="is_reserved",
|
|
action="store_true",
|
|
default=False,
|
|
help="Include reservations count [only valid with --absolute]")
|
|
parser.add_argument(
|
|
'--project',
|
|
metavar='<project>',
|
|
help='Show limits for a specific project (name or ID)'
|
|
' [only valid with --absolute]',
|
|
)
|
|
parser.add_argument(
|
|
'--domain',
|
|
metavar='<domain>',
|
|
help='Domain that owns --project (name or ID)'
|
|
' [only valid with --absolute]',
|
|
)
|
|
return parser
|
|
|
|
def take_action(self, parsed_args):
|
|
self.log.debug('take_action(%s)', parsed_args)
|
|
|
|
compute_client = self.app.client_manager.compute
|
|
volume_client = self.app.client_manager.volume
|
|
|
|
project_id = None
|
|
if parsed_args.project is not None:
|
|
identity_client = self.app.client_manager.identity
|
|
if parsed_args.domain is not None:
|
|
domain = identity_common.find_domain(identity_client,
|
|
parsed_args.domain)
|
|
project_id = utils.find_resource(identity_client.projects,
|
|
parsed_args.project,
|
|
domain_id=domain.id).id
|
|
else:
|
|
project_id = utils.find_resource(identity_client.projects,
|
|
parsed_args.project).id
|
|
|
|
compute_limits = compute_client.limits.get(parsed_args.is_reserved,
|
|
tenant_id=project_id)
|
|
volume_limits = volume_client.limits.get()
|
|
|
|
if parsed_args.is_absolute:
|
|
compute_limits = compute_limits.absolute
|
|
volume_limits = volume_limits.absolute
|
|
columns = ["Name", "Value"]
|
|
return (columns, (utils.get_item_properties(s, columns)
|
|
for s in itertools.chain(compute_limits, volume_limits)))
|
|
|
|
elif parsed_args.is_rate:
|
|
compute_limits = compute_limits.rate
|
|
volume_limits = volume_limits.rate
|
|
columns = ["Verb", "URI", "Value", "Remain", "Unit",
|
|
"Next Available"]
|
|
return (columns, (utils.get_item_properties(s, columns)
|
|
for s in itertools.chain(compute_limits, volume_limits)))
|
|
|
|
else:
|
|
return ({}, {})
|