Browse Source

Fix todo that is not needed anymore stestr conf

Stestr conf was using the wrong module name so tox would not work (I
have no idea how that passes tempest gates) So fixing that so tox works
(on an env that has tempest setup) and removed todos that the force
deletion has been merged

Change-Id: Ib47148c5e28acaae7cd5872a1af3e7d044336ea1
tags/0.1.0
Adriano Petrich 1 year ago
parent
commit
35d94fd609
2 changed files with 6 additions and 11 deletions
  1. 1
    1
      .stestr.conf
  2. 5
    10
      mistral_tempest_tests/services/base.py

+ 1
- 1
.stestr.conf View File

@@ -1,3 +1,3 @@
1 1
 [DEFAULT]
2
-test_path=./mistral-tempest-plugin/tests
2
+test_path=mistral_tempest_tests
3 3
 top_dir=./

+ 5
- 10
mistral_tempest_tests/services/base.py View File

@@ -81,16 +81,11 @@ class MistralClientBase(rest_client.RestClient):
81 81
 
82 82
     def delete_obj(self, obj, name, force=None):
83 83
         if force:
84
-            # TODO(apetrich) This try except is a partial implementation
85
-            # to be removed once force deletion lands in mistral
86
-            try:
87
-                return self.delete('{obj}/{name}?force={force}'.format(
88
-                    obj=obj,
89
-                    name=name,
90
-                    force=bool(force))
91
-                )
92
-            except Exception:
93
-                pass
84
+            return self.delete('{obj}/{name}?force={force}'.format(
85
+                obj=obj,
86
+                name=name,
87
+                force=bool(force))
88
+            )
94 89
 
95 90
         return self.delete('{obj}/{name}'.format(obj=obj, name=name))
96 91
 

Loading…
Cancel
Save