diff --git a/vitrageclient/exc.py b/vitrageclient/exc.py index c9ce727..084e0fa 100644 --- a/vitrageclient/exc.py +++ b/vitrageclient/exc.py @@ -25,9 +25,9 @@ class ClientException(Exception): self.method = method def __str__(self): - formatted_string = "%s (HTTP %s)" % (self.message, self.code) + formatted_string = '%s (HTTP %s)' % (self.message, self.code) if self.request_id: - formatted_string += " (Request-ID: %s)" % self.request_id + formatted_string += ' (Request-ID: %s)' % self.request_id return formatted_string diff --git a/vitrageclient/noauth.py b/vitrageclient/noauth.py index 03cddcc..736b190 100644 --- a/vitrageclient/noauth.py +++ b/vitrageclient/noauth.py @@ -66,7 +66,7 @@ class VitrageNoAuthLoader(loading.BaseLoader): options.extend([ VitrageOpt('user-id', help='User ID', required=True), VitrageOpt('project-id', help='Project ID', required=True), - VitrageOpt('roles', help='Roles', default="admin"), + VitrageOpt('roles', help='Roles', default='admin'), VitrageOpt('vitrage-endpoint', help='Vitrage endpoint', dest="endpoint", required=True), ]) diff --git a/vitrageclient/shell.py b/vitrageclient/shell.py index 8514701..587c8ed 100644 --- a/vitrageclient/shell.py +++ b/vitrageclient/shell.py @@ -38,11 +38,11 @@ from vitrageclient import __version__ class VitrageCommandManager(commandmanager.CommandManager): COMMANDS = { - "topology show": topology.TopologyShow, - "resource show": resource.ResourceShow, - "resource list": resource.ResourceList, - "alarms list": alarms.AlarmsList, - "rca show": rca.RcaShow, + 'topology show': topology.TopologyShow, + 'resource show': resource.ResourceShow, + 'resource list': resource.ResourceList, + 'alarms list': alarms.AlarmsList, + 'rca show': rca.RcaShow, } def load_commands(self, namespace): @@ -100,7 +100,7 @@ class VitrageShell(app.App): help='Defaults to env[VITRAGE_API_VERSION] or 1.') loading.register_session_argparse_arguments(parser=parser) plugin = loading.register_auth_argparse_arguments( - parser=parser, argv=sys.argv, default="password") + parser=parser, argv=sys.argv, default='password') if not isinstance(plugin, noauth.VitrageNoAuthLoader): parser.add_argument( @@ -115,7 +115,7 @@ class VitrageShell(app.App): @property def client(self): if self._client is None: - if hasattr(self.options, "endpoint"): + if hasattr(self.options, 'endpoint'): endpoint_override = self.options.endpoint else: endpoint_override = None @@ -177,10 +177,10 @@ class VitrageShell(app.App): warnings.simplefilter("ignore") def _hide_useless_logging_messages(self): - requests_log = logging.getLogger("requests") + requests_log = logging.getLogger('requests') cliff_log = logging.getLogger('cliff') stevedore_log = logging.getLogger('stevedore') - iso8601_log = logging.getLogger("iso8601") + iso8601_log = logging.getLogger('iso8601') cliff_log.setLevel(logging.ERROR) stevedore_log.setLevel(logging.ERROR) iso8601_log.setLevel(logging.ERROR) @@ -196,12 +196,12 @@ def main(args=None): args = sys.argv[1:] return VitrageShell().run(args) except KeyboardInterrupt: - print("... terminating vitrage client", file=sys.stderr) + print('... terminating vitrage client', file=sys.stderr) sys.exit(1) except Exception as e: print(e) sys.exit(1) -if __name__ == "__main__": +if __name__ == '__main__': main() diff --git a/vitrageclient/v1/alarms.py b/vitrageclient/v1/alarms.py index 0f09ace..3e23787 100644 --- a/vitrageclient/v1/alarms.py +++ b/vitrageclient/v1/alarms.py @@ -12,7 +12,7 @@ class Alarm(object): - url = "v1/alarms/" + url = 'v1/alarms/' def __init__(self, api): self.api = api diff --git a/vitrageclient/v1/cli/alarms.py b/vitrageclient/v1/cli/alarms.py index b67f98f..326340c 100644 --- a/vitrageclient/v1/cli/alarms.py +++ b/vitrageclient/v1/cli/alarms.py @@ -20,11 +20,11 @@ class AlarmsList(lister.Lister): def get_parser(self, prog_name): parser = super(AlarmsList, self).get_parser(prog_name) - parser.add_argument("id", + parser.add_argument('id', default='all', nargs='?', - metavar="", - help="Vitrage id of the affected resource") + metavar='', + help='Vitrage id of the affected resource') return parser diff --git a/vitrageclient/v1/cli/rca.py b/vitrageclient/v1/cli/rca.py index d96f3d5..742bc73 100644 --- a/vitrageclient/v1/cli/rca.py +++ b/vitrageclient/v1/cli/rca.py @@ -19,7 +19,7 @@ class RcaShow(show.ShowOne): def get_parser(self, prog_name): parser = super(RcaShow, self).get_parser(prog_name) - parser.add_argument("alarm_id", help="ID of an alarm") + parser.add_argument('alarm_id', help='ID of an alarm') return parser def take_action(self, parsed_args): diff --git a/vitrageclient/v1/cli/resource.py b/vitrageclient/v1/cli/resource.py index 138e359..efc2104 100644 --- a/vitrageclient/v1/cli/resource.py +++ b/vitrageclient/v1/cli/resource.py @@ -20,7 +20,7 @@ class ResourceShow(show.ShowOne): def get_parser(self, prog_name): parser = super(ResourceShow, self).get_parser(prog_name) - parser.add_argument("resource_id", help="ID of a resource") + parser.add_argument('resource_id', help='ID of a resource') return parser def take_action(self, parsed_args): @@ -34,9 +34,9 @@ class ResourceList(lister.Lister): def get_parser(self, prog_name): parser = super(ResourceList, self).get_parser(prog_name) - parser.add_argument("--type", dest='resource_type', - metavar="", - help="Type of resource") + parser.add_argument('--type', dest='resource_type', + metavar='', + help='Type of resource') return parser diff --git a/vitrageclient/v1/cli/topology.py b/vitrageclient/v1/cli/topology.py index 475256c..f9fcc38 100644 --- a/vitrageclient/v1/cli/topology.py +++ b/vitrageclient/v1/cli/topology.py @@ -19,15 +19,15 @@ class TopologyShow(show.ShowOne): def get_parser(self, prog_name): parser = super(TopologyShow, self).get_parser(prog_name) - parser.add_argument("--filter", - metavar="", - help="query for the graph)") + parser.add_argument('--filter', + metavar='', + help='query for the graph)') - parser.add_argument("--limit", type=int, - metavar="", - help="the depth of the topology graph") + parser.add_argument('--limit', type=int, + metavar='', + help='the depth of the topology graph') - parser.add_argument("--root", help="the root of the topology graph") + parser.add_argument('--root', help='the root of the topology graph') parser.add_argument('--graph-type', choices=['tree', 'graph'], default='graph', dest='type', @@ -37,7 +37,7 @@ class TopologyShow(show.ShowOne): return parser def formatter_default(self): - return "json" + return 'json' def take_action(self, parsed_args): topology = self.app.client.topology.get(limit=parsed_args.limit, diff --git a/vitrageclient/v1/rca.py b/vitrageclient/v1/rca.py index dec3fe6..018d210 100644 --- a/vitrageclient/v1/rca.py +++ b/vitrageclient/v1/rca.py @@ -12,7 +12,7 @@ class Rca(object): - url = "v1/rca/" + url = 'v1/rca/' def __init__(self, api): self.api = api diff --git a/vitrageclient/v1/resource.py b/vitrageclient/v1/resource.py index 1ff3d89..90f728f 100644 --- a/vitrageclient/v1/resource.py +++ b/vitrageclient/v1/resource.py @@ -12,7 +12,7 @@ class Resource(object): - url = "v1/resources/" + url = 'v1/resources/' def __init__(self, api): self.api = api diff --git a/vitrageclient/v1/topology.py b/vitrageclient/v1/topology.py index 9dc5af9..a3b33d5 100644 --- a/vitrageclient/v1/topology.py +++ b/vitrageclient/v1/topology.py @@ -13,7 +13,7 @@ import json class Topology(object): - URL = "v1/topology/" + URL = 'v1/topology/' def __init__(self, api): self.api = api