Merge pull request #391 from bergkampsliew/develop

Update DynamoDB and EFS limits #390
jantman · Feb 26, 2019 · 77414e6 · 77414e6
2 parents aef5405 + 813b9bf
commit 77414e6
Show file tree

Hide file tree

Showing 4 changed files with 8 additions and 8 deletions.
diff --git a/awslimitchecker/services/dynamodb.py b/awslimitchecker/services/dynamodb.py
@@ -192,7 +192,7 @@ def get_limits(self):
         limits['Global Secondary Indexes'] = AwsLimit(
             'Global Secondary Indexes',
             self,
-            5,
+            20,
             self.warning_threshold,
             self.critical_threshold,
             limit_type='AWS::DynamoDB::Table',

diff --git a/awslimitchecker/services/efs.py b/awslimitchecker/services/efs.py
@@ -105,7 +105,7 @@ def get_limits(self):
         limits['File systems'] = AwsLimit(
             'File systems',
             self,
-            125,
+            1000,
             self.warning_threshold,
             self.critical_threshold,
             limit_type='AWS::EFS::FileSystem',

diff --git a/awslimitchecker/tests/services/test_dynamodb.py b/awslimitchecker/tests/services/test_dynamodb.py
@@ -102,7 +102,7 @@ def se_conn(cls):
         assert read_capacity_table.limit_type == 'AWS::DynamoDB::Table'
         global_secondary_index = limits['Global Secondary Indexes']
         assert global_secondary_index.limit_type == 'AWS::DynamoDB::Table'
-        assert global_secondary_index.default_limit == 5
+        assert global_secondary_index.default_limit == 20
         local_secondary_index = limits['Local Secondary Indexes']
         assert local_secondary_index.limit_type == 'AWS::DynamoDB::Table'
         assert local_secondary_index.default_limit == 5
@@ -144,7 +144,7 @@ def se_conn(cls):
         assert read_capacity_table.limit_type == 'AWS::DynamoDB::Table'
         global_secondary_index = limits['Global Secondary Indexes']
         assert global_secondary_index.limit_type == 'AWS::DynamoDB::Table'
-        assert global_secondary_index.default_limit == 5
+        assert global_secondary_index.default_limit == 20
         local_secondary_index = limits['Local Secondary Indexes']
         assert local_secondary_index.limit_type == 'AWS::DynamoDB::Table'
         assert local_secondary_index.default_limit == 5

diff --git a/awslimitchecker/tests/services/test_efs.py b/awslimitchecker/tests/services/test_efs.py
@@ -79,7 +79,7 @@ def test_get_limits(self):
         assert res['File systems'].service == cls
         assert res['File systems'].def_warning_threshold == 21
         assert res['File systems'].def_critical_threshold == 43
-        assert res['File systems'].default_limit == 125
+        assert res['File systems'].default_limit == 1000
 
     def test_update_limits_from_api(self):
         mock_conn = Mock()
@@ -93,7 +93,7 @@ def test_update_limits_from_api(self):
                 'File systems': AwsLimit(
                     'File systems',
                     cls,
-                    125,
+                    1000,
                     cls.warning_threshold,
                     cls.critical_threshold,
                     limit_type='AWS::EFS::FileSystem',
@@ -102,7 +102,7 @@ def test_update_limits_from_api(self):
             cls._update_limits_from_api()
         assert mock_connect.mock_calls == [call()]
         assert mock_conn.mock_calls == []
-        assert cls.limits['File systems'].default_limit == 125
+        assert cls.limits['File systems'].default_limit == 1000
 
     def test_update_limits_from_api_us_east_1(self):
         mock_conn = Mock()
@@ -116,7 +116,7 @@ def test_update_limits_from_api_us_east_1(self):
                 'File systems': AwsLimit(
                     'File systems',
                     cls,
-                    125,
+                    1000,
                     cls.warning_threshold,
                     cls.critical_threshold,
                     limit_type='AWS::EFS::FileSystem',