Remove one useless security group

[functest.git] / functest / opnfv_tests / vnf / router / cloudify_vrouter.py

diff --git a/functest/opnfv_tests/vnf/router/cloudify_vrouter.py b/functest/opnfv_tests/vnf/router/cloudify_vrouter.py
index 79e8d80..e8c8632 100644 (file)
--- a/functest/opnfv_tests/vnf/router/cloudify_vrouter.py
+++ b/functest/opnfv_tests/vnf/router/cloudify_vrouter.py
@@ -159,7 +159,7 @@ class CloudifyVrouter(cloudify.Cloudify):
         self.vnf['inputs'].update(dict(
             keystone_url=self.get_public_auth_url(self.orig_cloud)))
 
-        if (self.deploy_vnf() and self.test_vnf()):
+        if self.deploy_vnf() and self.test_vnf():
             self.result = 100
             return 0
         self.result = 1/3 * 100
@@ -167,11 +167,6 @@ class CloudifyVrouter(cloudify.Cloudify):
 
     def deploy_vnf(self):
         start_time = time.time()
-
-        self.cloud.create_security_group_rule(
-            'default', port_range_min=22, port_range_max=22,
-            protocol='tcp', direction='ingress')
-
         self.__logger.info("Upload VNFD")
         descriptor = self.vnf['descriptor']
         self.util_info["cfy"] = self.cfy_client
@@ -249,12 +244,11 @@ class CloudifyVrouter(cloudify.Cloudify):
                 self.vnf['descriptor'].get('name'))
         except Exception:  # pylint: disable=broad-except
             self.__logger.exception("Some issue during the undeployment ..")
-
-        super(CloudifyVrouter, self).clean()
         if self.image_alt:
             self.cloud.delete_image(self.image_alt)
         if self.flavor_alt:
             self.orig_cloud.delete_flavor(self.flavor_alt.id)
+        super(CloudifyVrouter, self).clean()
 
 
 def wait_for_execution(client, execution, logger, timeout=7200, ):