Updated from global requirements

Change-Id: If8d974e12b3a85a37767ed1bdb81fc53607be108

.gitreview

@@ -2,3 +2,4 @@
 host=review.openstack.org
 port=29418
 project=openstack/python-karborclient.git
+defaultbranch=stable/newton

karborclient/client.py

@@ -9,12 +9,10 @@
 #    WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the
 #    License for the specific language governing permissions and limitations
 #    under the License.
-from oslo_utils import importutils
+from karborclient.common import utils
 
 
 def Client(version, *args, **kwargs):
-    module = importutils.import_versioned_module(
-        'karborclient', version, 'client'
-    )
+    module = utils.import_versioned_module(version, 'client')
     client_class = getattr(module, 'Client')
     return client_class(*args, **kwargs)

karborclient/common/utils.py

@@ -19,6 +19,7 @@ import six
 import uuid
 
 from oslo_utils import encodeutils
+from oslo_utils import importutils
 
 import prettytable
 
@@ -48,6 +49,13 @@ def env(*vars, **kwargs):
     return kwargs.get('default', '')
 
 
+def import_versioned_module(version, submodule=None):
+    module = 'karborclient.v%s' % version
+    if submodule:
+        module = '.'.join((module, submodule))
+    return importutils.import_module(module)
+
+
 def _print(pt, order):
     if sys.version_info >= (3, 0):
         print(pt.get_string(sortby=order))

karborclient/shell.py

@@ -28,8 +28,6 @@ from keystoneclient import session as ksession
 from oslo_log import handlers
 from oslo_log import log as logging
 from oslo_utils import encodeutils
-from oslo_utils import importutils
-
 import six
 import six.moves.urllib.parse as urlparse
 
@@ -152,9 +150,7 @@ class KarborShell(object):
 
         self.subcommands = {}
         subparsers = parser.add_subparsers(metavar='<subcommand>')
-        submodule = importutils.import_versioned_module(
-            'karborclient', version, 'shell'
-        )
+        submodule = utils.import_versioned_module(version, 'shell')
         self._find_actions(subparsers, submodule)
         self._find_actions(subparsers, self)
 

karborclient/tests/unit/v1/test_checkpoints.py

@@ -88,5 +88,5 @@ class CheckpointsTest(base.TestCaseShell):
             'checkpoints'.format(
                 provider_id=FAKE_PROVIDER_ID),
             data={
-                'checkpoint': {'plan_id': FAKE_PLAN_ID, 'extra-info': None}},
+                'checkpoint': {'plan_id': FAKE_PLAN_ID}},
             headers={})

karborclient/v1/checkpoints.py

@@ -26,9 +26,8 @@ class Checkpoint(base.Resource):
 class CheckpointManager(base.ManagerWithFind):
     resource_class = Checkpoint
 
-    def create(self, provider_id, plan_id, checkpoint_extra_info=None):
-        body = {'checkpoint': {'plan_id': plan_id,
-                               'extra-info': checkpoint_extra_info}}
+    def create(self, provider_id, plan_id):
+        body = {'checkpoint': {'plan_id': plan_id}}
         url = "/providers/{provider_id}/" \
               "checkpoints" .format(provider_id=provider_id)
         return self._create(url, body, 'checkpoint')

karborclient/v1/protectables.py

@@ -77,27 +77,14 @@ class ProtectableManager(base.ManagerWithFind):
             sort_dir=sort_dir, sort=sort)
         return self._list(url, response_key='instances', obj_class=Instances)
 
-    def get_instance(self, type, id, search_opts=None, session_id=None):
+    def get_instance(self, type, id, session_id=None):
         if session_id:
             headers = {'X-Configuration-Session': session_id}
         else:
             headers = {}
-
-        if search_opts is None:
-            search_opts = {}
-        query_params = {}
-        for key, val in search_opts.items():
-            if val:
-                query_params[key] = val
-        query_string = ""
-        if query_params:
-            params = sorted(query_params.items(), key=lambda x: x[0])
-            query_string = "?%s" % parse.urlencode(params)
-
-        url = ("/protectables/{protectable_type}/instances/"
-               "{protectable_id}{query_string}").format(
-            protectable_type=type, protectable_id=id,
-            query_string=query_string)
+        url = "/protectables/{protectable_type}/" \
+              "instances/{protectable_id}".format(protectable_type=type,
+                                                  protectable_id=id)
         return self._get(url, response_key="instance", headers=headers)
 
     def _build_instances_list_url(self, protectable_type,

karborclient/v1/shell.py

@@ -417,28 +417,16 @@ def do_protectable_show(cs, args):
     utils.print_dict(protectable.to_dict())
 
 
-@utils.arg('protectable_type',
-           metavar='<protectable_type>',
-           help='Protectable type.')
 @utils.arg('protectable_id',
            metavar='<protectable_id>',
            help='Protectable instance id.')
-@utils.arg('--parameters',
-           type=str,
-           nargs='*',
-           metavar='<key=value>',
-           default=None,
-           help='Show a instance by parameters key and value pair. '
-                'Default=None.')
+@utils.arg('protectable_type',
+           metavar='<protectable_type>',
+           help='Protectable type.')
 def do_protectable_show_instance(cs, args):
     """Shows instance details."""
-    search_opts = {
-        'parameters': (_extract_instances_parameters(args)
-                       if args.parameters else None),
-    }
     instance = cs.protectables.get_instance(args.protectable_type,
-                                            args.protectable_id,
-                                            search_opts=search_opts)
+                                            args.protectable_id)
     utils.print_dict(instance.to_dict())
 
 
@@ -474,20 +462,11 @@ def do_protectable_show_instance(cs, args):
                   'form of <key>[:<asc|desc>]. '
                   'Valid keys: %s. '
                   'Default=None.') % ', '.join(base.SORT_KEY_VALUES)))
-@utils.arg('--parameters',
-           type=str,
-           nargs='*',
-           metavar='<key=value>',
-           default=None,
-           help='List instances by parameters key and value pair. '
-                'Default=None.')
 def do_protectable_list_instances(cs, args):
     """Lists all protectable instances."""
 
     search_opts = {
         'type': args.type,
-        'parameters': (_extract_instances_parameters(args)
-                       if args.parameters else None),
     }
 
     if args.sort and (args.sort_key or args.sort_dir):
@@ -511,19 +490,6 @@ def do_protectable_list_instances(cs, args):
                      sortby_index=sortby_index)
 
 
-def _extract_instances_parameters(args):
-    parameters = {}
-    for parameter in args.parameters:
-        if '=' in parameter:
-            (key, value) = parameter.split('=', 1)
-        else:
-            key = parameter
-            value = None
-
-        parameters[key] = value
-    return parameters
-
-
 @utils.arg('provider_id',
            metavar='<provider_id>',
            help='Id of provider.')
@@ -599,37 +565,12 @@ def do_provider_list(cs, args):
 @utils.arg('plan_id',
            metavar='<plan_id>',
            help='ID of plan.')
-@utils.arg('--extra_info',
-           type=str,
-           nargs='*',
-           metavar='<key=value>',
-           default=None,
-           help='The extra info of a checkpoint.')
 def do_checkpoint_create(cs, args):
     """Create a checkpoint."""
-
-    checkpoint_extra_info = None
-    if args.extra_info is not None:
-        checkpoint_extra_info = _extract_extra_info(args)
-    checkpoint = cs.checkpoints.create(args.provider_id, args.plan_id,
-                                       checkpoint_extra_info)
+    checkpoint = cs.checkpoints.create(args.provider_id, args.plan_id)
     utils.print_dict(checkpoint.to_dict())
 
 
-def _extract_extra_info(args):
-    checkpoint_extra_info = {}
-    for data in args.extra_info:
-        # unset doesn't require a val, so we have the if/else
-        if '=' in data:
-            (key, value) = data.split('=', 1)
-        else:
-            key = data
-            value = None
-
-        checkpoint_extra_info[key] = value
-    return checkpoint_extra_info
-
-
 @utils.arg('provider_id',
            metavar='<provider_id>',
            help='ID of provider.')
@@ -684,7 +625,7 @@ def do_checkpoint_list(cs, args):
         marker=args.marker, limit=args.limit, sort_key=args.sort_key,
         sort_dir=args.sort_dir, sort=args.sort)
 
-    key_list = ['Id', 'Project id', 'Status', 'Protection plan', 'Metadata']
+    key_list = ['Id', 'Project id', 'Status', 'Protection plan']
 
     if args.sort_key or args.sort_dir or args.sort:
         sortby_index = None
@@ -917,10 +858,6 @@ def do_trigger_delete(cs, args):
            metavar='<trigger_id>',
            default=None,
            help='Filters results by a trigger id. Default=None.')
-@utils.arg('--operation_definition',
-           metavar='<operation_definition>',
-           default=None,
-           help='Filters results by the operation_definition. Default=None.')
 @utils.arg('--marker',
            metavar='<marker>',
            default=None,
@@ -963,7 +900,6 @@ def do_scheduledoperation_list(cs, args):
         'name': args.name,
         'operation_type': args.operation_type,
         'trigger_id': args.trigger_id,
-        'operation_definition': args.operation_definition,
     }
 
     if args.sort and (args.sort_key or args.sort_dir):

requirements.txt

@@ -1,13 +1,13 @@
 # The order of packages is significant, because pip processes them in the order
 # of appearance. Changing the order has an impact on the overall integration
 # process, which may cause wedges in the gate later.
-pbr>=1.8 # Apache-2.0
-PrettyTable<0.8,>=0.7.1 # BSD
-python-keystoneclient>=3.6.0 # Apache-2.0
+pbr>=1.6 # Apache-2.0
+PrettyTable<0.8,>=0.7 # BSD
+python-keystoneclient!=2.1.0,>=2.0.0 # Apache-2.0
 requests>=2.10.0 # Apache-2.0
 simplejson>=2.2.0 # MIT
 Babel>=2.3.4 # BSD
 six>=1.9.0 # MIT
-oslo.utils>=3.18.0 # Apache-2.0
-oslo.log>=3.11.0 # Apache-2.0
+oslo.utils>=3.16.0 # Apache-2.0
+oslo.log>=1.14.0 # Apache-2.0
 oslo.i18n>=2.1.0 # Apache-2.0

setup.cfg

@@ -18,7 +18,6 @@ classifier =
     Programming Language :: Python :: 3
     Programming Language :: Python :: 3.3
     Programming Language :: Python :: 3.4
-    Programming Language :: Python :: 3.5
 
 [global]
 setup-hooks =

test-requirements.txt

@@ -4,10 +4,10 @@
 
 hacking<0.11,>=0.10.2 # Apache-2.0
 
-coverage>=4.0 # Apache-2.0
+coverage>=3.6 # Apache-2.0
 python-subunit>=0.0.18 # Apache-2.0/BSD
-sphinx!=1.3b1,<1.4,>=1.2.1 # BSD
-oslosphinx>=4.7.0 # Apache-2.0
+sphinx!=1.3b1,<1.3,>=1.2.1 # BSD
+oslosphinx!=3.4.0,>=2.5.0 # Apache-2.0
 oslotest>=1.10.0 # Apache-2.0
 testrepository>=0.0.18 # Apache-2.0/BSD
 testscenarios>=0.4 # Apache-2.0/BSD

tox.ini

@@ -1,6 +1,6 @@
 [tox]
 minversion = 1.6
-envlist = py35,py34,py27,pypy,pep8
+envlist = py34,py27,pypy,pep8
 skipsdist = True
 
 [testenv]