Rename Patrole config fields 45/59845/1
authorCédric Ollivier <cedric.ollivier@orange.com>
Mon, 16 Jul 2018 08:01:56 +0000 (10:01 +0200)
committerCédric Ollivier <cedric.ollivier@orange.com>
Mon, 16 Jul 2018 08:01:56 +0000 (10:01 +0200)
Else it conflicts with the new tempest config fields [1].
It makes sense to switch to includes here.

[1] https://build.opnfv.org/ci/view/functest/job/functest-compass-virtual-daily-master/566/console

Change-Id: If7b3025b4003ea5704c6a3cdeae8cd00f1c5509e
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
docker/smoke/testcases.yaml
functest/ci/testcases.yaml
functest/opnfv_tests/openstack/patrole/patrole.py

index b59af41..f364bde 100644 (file)
@@ -61,10 +61,10 @@ tiers:
                     class: 'Patrole'
                     args:
                         role: 'admin'
-                        services:
+                        includes:
                             - 'image'
                             - 'network'
-                        exclude:
+                        excludes:
                             - 'test_networks_multiprovider_rbac'
 
             -
index a9e023e..1d35406 100644 (file)
@@ -252,10 +252,10 @@ tiers:
                     class: 'Patrole'
                     args:
                         role: 'admin'
-                        services:
+                        includes:
                             - 'image'
                             - 'network'
-                        exclude:
+                        excludes:
                             - 'test_networks_multiprovider_rbac'
 
             -
index efc513c..45378b6 100644 (file)
@@ -32,9 +32,9 @@ class Patrole(tempest.TempestCommon):
         self.backup_tempest_config(self.conf_file, self.res_dir)
 
     def run(self, **kwargs):
-        for exclude in kwargs.get('exclude', []):
+        for exclude in kwargs.get('excludes', []):
             kwargs['mode'] = "{}(?!.*{})".format(
                 kwargs.get('mode', ''), exclude)
         kwargs['mode'] = '{}(?=patrole_tempest_plugin.tests.api.({}))'.format(
-            kwargs['mode'], '|'.join(kwargs.get('services', [])))
+            kwargs['mode'], '|'.join(kwargs.get('includes', [])))
         return super(Patrole, self).run(**kwargs)