Fix ocata registration test and implementation 79/43479/2
authorliboNet <libo.zhu@intel.com>
Wed, 18 Apr 2018 23:04:02 +0000 (07:04 +0800)
committerliboNet <libo.zhu@intel.com>
Thu, 19 Apr 2018 17:49:08 +0000 (01:49 +0800)
fix the typo of cpupinning feature  and rename its function name
remove the unit of ram size, the MB is the agreement unit in openstack

Change-Id: Ia798cfa85edc5821a536db7f5ba5335dc7948b4b
Issue-ID: MULTICLOUD-200
Signed-off-by: liboNet <libo.zhu@intel.com>
ocata/ocata/registration/tests/test_registration.py
ocata/ocata/registration/views/registration.py

index c35eddf..50b7fda 100644 (file)
@@ -63,7 +63,7 @@ MOCK_GET_FLAVOR_RESPONSE = {
             "OS-FLV-DISABLED:disabled": True, "link": [{"href":1}]
          },
         {
-            "id": "2", "name": "mini", "vcpus": 2, "ram": "2MB",
+            "id": "2", "name": "mini", "vcpus": 2, "ram": "2",
             "disk": "2G", "OS-FLV-EXT-DATA:ephemeral": True,
             "swap": False, "os-flavor-access:is_public": True,
             "OS-FLV-DISABLED:disabled": True
@@ -154,7 +154,7 @@ MOCK_GET_HPA_FLAVOR_LIST1_RESPONSE= {
             "OS-FLV-DISABLED:disabled": True, "link": [{"href": 1}]
         },
         {
-            "id": "2", "name": "onap.mini", "vcpus": 2, "ram": "2MB",
+            "id": "2", "name": "onap.mini", "vcpus": 2, "ram": "2",
             "disk": "2G", "OS-FLV-EXT-DATA:ephemeral": True,
             "swap": False, "os-flavor-access:is_public": True,
             "OS-FLV-DISABLED:disabled": True
index 636e3ec..25ec0db 100644 (file)
@@ -91,7 +91,7 @@ class Registry(newton_registration.Registry):
             hpa_caps.append(caps_dict)
 
         # cpupining capabilities
-        caps_dict = self._get_cpupining_capabilities(extra_specs)
+        caps_dict = self._get_cpupinning_capabilities(extra_specs)
         if len(caps_dict) > 0:
             self._logger.debug("cpupining_capabilities_info: %s" % caps_dict)
             hpa_caps.append(caps_dict)
@@ -157,13 +157,13 @@ class Registry(newton_registration.Registry):
 
         return basic_capability
 
-    def _get_cpupining_capabilities(self, extra_specs):
+    def _get_cpupinning_capabilities(self, extra_specs):
         cpupining_capability = {}
         feature_uuid = uuid.uuid4()
 
         if extra_specs.has_key('hw:cpu_policy') or extra_specs.has_key('hw:cpu_thread_policy'):
             cpupining_capability['hpaCapabilityID'] = str(feature_uuid)
-            cpupining_capability['hpaFeature'] = 'cpuPining'
+            cpupining_capability['hpaFeature'] = 'cpuPinning'
             cpupining_capability['hardwareArchitecture'] = 'generic'
             cpupining_capability['version'] = 'v1'