4d3f996e7c
--enable and --disable-reason should be mutually exclusive in "compute service set" command, but now when they are present at the same time, --disable-reason would be ignored silently. Fix these and add some hints about --disable-reason argument is ignored in this situation. Change-Id: I43254b6bc40fcae4fd0dc3457f26fad84c267072 Closes-Bug: #1556801
142 lines
4.5 KiB
Python
142 lines
4.5 KiB
Python
# Copyright 2013 OpenStack, LLC.
|
|
#
|
|
# 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.
|
|
#
|
|
|
|
"""Service action implementations"""
|
|
|
|
from openstackclient.common import command
|
|
from openstackclient.common import utils
|
|
from openstackclient.i18n import _ # noqa
|
|
|
|
|
|
class DeleteService(command.Command):
|
|
"""Delete service command"""
|
|
|
|
def get_parser(self, prog_name):
|
|
parser = super(DeleteService, self).get_parser(prog_name)
|
|
parser.add_argument(
|
|
"service",
|
|
metavar="<service>",
|
|
help="Compute service to delete (ID only)")
|
|
return parser
|
|
|
|
def take_action(self, parsed_args):
|
|
compute_client = self.app.client_manager.compute
|
|
|
|
compute_client.services.delete(parsed_args.service)
|
|
|
|
|
|
class ListService(command.Lister):
|
|
"""List service command"""
|
|
|
|
def get_parser(self, prog_name):
|
|
parser = super(ListService, self).get_parser(prog_name)
|
|
parser.add_argument(
|
|
"--host",
|
|
metavar="<host>",
|
|
help="List services on specified host (name only)")
|
|
parser.add_argument(
|
|
"--service",
|
|
metavar="<service>",
|
|
help="List only specified service (name only)")
|
|
parser.add_argument(
|
|
"--long",
|
|
action="store_true",
|
|
default=False,
|
|
help="List additional fields in output"
|
|
)
|
|
return parser
|
|
|
|
def take_action(self, parsed_args):
|
|
compute_client = self.app.client_manager.compute
|
|
if parsed_args.long:
|
|
columns = (
|
|
"Id",
|
|
"Binary",
|
|
"Host",
|
|
"Zone",
|
|
"Status",
|
|
"State",
|
|
"Updated At",
|
|
"Disabled Reason"
|
|
)
|
|
else:
|
|
columns = (
|
|
"Id",
|
|
"Binary",
|
|
"Host",
|
|
"Zone",
|
|
"Status",
|
|
"State",
|
|
"Updated At"
|
|
)
|
|
data = compute_client.services.list(parsed_args.host,
|
|
parsed_args.service)
|
|
return (columns,
|
|
(utils.get_item_properties(
|
|
s, columns,
|
|
) for s in data))
|
|
|
|
|
|
class SetService(command.Command):
|
|
"""Set service command"""
|
|
|
|
def get_parser(self, prog_name):
|
|
parser = super(SetService, self).get_parser(prog_name)
|
|
parser.add_argument(
|
|
"host",
|
|
metavar="<host>",
|
|
help="Name of host")
|
|
parser.add_argument(
|
|
"service",
|
|
metavar="<service>",
|
|
help="Name of service")
|
|
enabled_group = parser.add_mutually_exclusive_group()
|
|
enabled_group.add_argument(
|
|
"--enable",
|
|
dest="enabled",
|
|
default=True,
|
|
help="Enable a service (default)",
|
|
action="store_true")
|
|
enabled_group.add_argument(
|
|
"--disable",
|
|
dest="enabled",
|
|
help="Disable a service",
|
|
action="store_false")
|
|
parser.add_argument(
|
|
"--disable-reason",
|
|
default=None,
|
|
metavar="<reason>",
|
|
help="Reason for disabling the service (in quotas). Note that "
|
|
"when the service is enabled, this option is ignored.")
|
|
return parser
|
|
|
|
def take_action(self, parsed_args):
|
|
compute_client = self.app.client_manager.compute
|
|
cs = compute_client.services
|
|
|
|
if not parsed_args.enabled:
|
|
if parsed_args.disable_reason:
|
|
cs.disable_log_reason(parsed_args.host,
|
|
parsed_args.service,
|
|
parsed_args.disable_reason)
|
|
else:
|
|
cs.disable(parsed_args.host, parsed_args.service)
|
|
else:
|
|
if parsed_args.disable_reason:
|
|
msg = _("argument --disable-reason has been ignored")
|
|
self.log.info(msg)
|
|
|
|
cs.enable(parsed_args.host, parsed_args.service)
|