From f49ef213d886330f02705bddff60b31bdeb33629 Mon Sep 17 00:00:00 2001
From: "OTSUKA, Yuanying" <yuanying@fraction.jp>
Date: Fri, 22 Jul 2016 13:55:02 +0900
Subject: [PATCH] Base OSC plugin support

Enable magnumclient OSC plugin support.
This patch set does the following:

 - Establishes OSC plugin structure for commands and unit tests
 - Creates the OSC plugin
 - Adds OSC to test-requirements.txt

This fixes are mainly copied from python-neutronclient.

Partial-Implements: blueprint openstackclient-support
Change-Id: I8f7ccde7d1417416ada5fe5e88832db7520efa6f
---
 magnumclient/osc/__init__.py          |  0
 magnumclient/osc/plugin.py            | 56 +++++++++++++++++++++++++++
 magnumclient/tests/osc/v1/__init__.py |  0
 magnumclient/tests/osc/v1/fakes.py    | 27 +++++++++++++
 setup.cfg                             |  5 +++
 test-requirements.txt                 |  1 +
 6 files changed, 89 insertions(+)
 create mode 100644 magnumclient/osc/__init__.py
 create mode 100644 magnumclient/osc/plugin.py
 create mode 100644 magnumclient/tests/osc/v1/__init__.py
 create mode 100644 magnumclient/tests/osc/v1/fakes.py

diff --git a/magnumclient/osc/__init__.py b/magnumclient/osc/__init__.py
new file mode 100644
index 00000000..e69de29b
diff --git a/magnumclient/osc/plugin.py b/magnumclient/osc/plugin.py
new file mode 100644
index 00000000..dde56fd7
--- /dev/null
+++ b/magnumclient/osc/plugin.py
@@ -0,0 +1,56 @@
+#   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.
+#
+
+import logging
+
+from osc_lib import utils
+
+LOG = logging.getLogger(__name__)
+
+DEFAULT_API_VERSION = '1'
+API_VERSION_OPTION = 'os_container_infra_api_version'
+API_NAME = 'container_infra'
+API_VERSIONS = {
+    '1': 'magnumclient.v1.client.Client',
+}
+
+
+def make_client(instance):
+    """Returns a magnum client."""
+    magnum_client = utils.get_client_class(
+        API_NAME,
+        instance._api_version[API_NAME],
+        API_VERSIONS)
+    LOG.debug('Instantiating magnum client: %s', magnum_client)
+
+    client = magnum_client(session=instance.session,
+                           region_name=instance._region_name,
+                           interface=instance._interface,
+                           insecure=instance._insecure,
+                           ca_cert=instance._cacert)
+    return client
+
+
+def build_option_parser(parser):
+    """Hook to add global options"""
+
+    parser.add_argument(
+        '--os-container-infra-api-version',
+        metavar='<container-infra-api-version>',
+        default=utils.env(
+            'OS_CONTAINER_INFRA_API_VERSION',
+            default=DEFAULT_API_VERSION),
+        help='Container-Infra API version, default=' +
+             DEFAULT_API_VERSION +
+             ' (Env: OS_CONTAINER_INFRA_API_VERSION)')
+    return parser
diff --git a/magnumclient/tests/osc/v1/__init__.py b/magnumclient/tests/osc/v1/__init__.py
new file mode 100644
index 00000000..e69de29b
diff --git a/magnumclient/tests/osc/v1/fakes.py b/magnumclient/tests/osc/v1/fakes.py
new file mode 100644
index 00000000..076a0dfb
--- /dev/null
+++ b/magnumclient/tests/osc/v1/fakes.py
@@ -0,0 +1,27 @@
+#   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.
+#
+
+import argparse
+import mock
+
+from openstackclient.tests import utils
+
+
+class TestMagnumClientOSCV1(utils.TestCommand):
+
+    def setUp(self):
+        super(TestMagnumClientOSCV1, self).setUp()
+        self.namespace = argparse.Namespace()
+        self.app.client_manager.session = mock.Mock()
+        self.app.client_manager.magnumclient = mock.Mock()
+        self.magnumclient = self.app.client_manager.magnumclient
diff --git a/setup.cfg b/setup.cfg
index 7fa9f16c..eb23a815 100644
--- a/setup.cfg
+++ b/setup.cfg
@@ -26,6 +26,11 @@ packages =
 console_scripts =
     magnum = magnumclient.shell:main
 
+openstack.cli.extension =
+    container_infra = magnumclient.osc.plugin
+
+openstack.container_infra.v1 =
+
 [build_sphinx]
 source-dir = doc/source
 build-dir = doc/build
diff --git a/test-requirements.txt b/test-requirements.txt
index 6ed4fdd5..716ef57c 100644
--- a/test-requirements.txt
+++ b/test-requirements.txt
@@ -6,6 +6,7 @@ bandit>=1.0.1 # Apache-2.0
 coverage>=3.6 # Apache-2.0
 discover # BSD
 fixtures>=3.0.0 # Apache-2.0/BSD
+python-openstackclient>=2.1.0 # Apache-2.0
 python-subunit>=0.0.18 # Apache-2.0/BSD
 sphinx!=1.3b1,<1.3,>=1.2.1 # BSD
 oslosphinx!=3.4.0,>=2.5.0 # Apache-2.0