Merge pull request #28 from globocom/issue#27

change rancher strategy

app/data/source/rancher.py

@@ -13,7 +13,9 @@ def get_clusters(self) -> List[Cluster]:
         }
 
         filters: dict = self.source.rancher.filters if self.source.rancher.filters else dict()
-        filters.update({'state': 'active'})
+
+        if filters.get('state'):
+            del filters['state']
 
         url = f"{self.secret.url}/v3/clusters"
 
@@ -37,6 +39,10 @@ def _filter_and_map_clusters(self, clusters_list: List[Dict], labels: dict, igno
         )
 
     def _filter_cluster(self, cluster_map: dict, labels: dict, ignore: List[str]) -> bool:
+
+        if cluster_map['state'] in ['provisioning', 'waiting']:
+            return False
+
         if ignore and cluster_map['name'] in ignore:
             return False
 

test/unit_tests/use_case/test_sync_rules.py

@@ -1,4 +1,3 @@
-from typing import List
 from unittest import TestCase
 from unittest.mock import MagicMock