Include DiskFilter in the default list

DiskFilter is pretty critical for proper operation of a deployment without shared storage, and is likely to be something most new operators will want enabled. We should default to include this. DocImpact: The DiskFilter is now included in the default set Change-Id: Iefa5538a58a61986c466912739408ee291cdc6c2
openstack · Jul 31, 2015 · 6739df7 · 6739df7
1 parent 8ad8701
commit 6739df7
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 0 deletions.
diff --git a/nova/scheduler/host_manager.py b/nova/scheduler/host_manager.py
@@ -56,6 +56,7 @@
                   'RetryFilter',
                   'AvailabilityZoneFilter',
                   'RamFilter',
+                  'DiskFilter',
                   'ComputeFilter',
                   'ComputeCapabilitiesFilter',
                   'ImagePropertiesFilter',

diff --git a/nova/tests/unit/scheduler/test_caching_scheduler.py b/nova/tests/unit/scheduler/test_caching_scheduler.py
@@ -105,6 +105,7 @@ def _get_fake_request_spec(self):
             "root_gb": 1,
             "ephemeral_gb": 1,
             "vcpus": 1,
+            "swap": 0,
         }
         instance_properties = {
             "os_type": "linux",
@@ -127,6 +128,7 @@ def _get_fake_host_state(self, index=0):
             'host_%s' % index,
             'node_%s' % index)
         host_state.free_ram_mb = 50000
+        host_state.free_disk_mb = 4096
         host_state.service = {
             "disabled": False,
             "updated_at": timeutils.utcnow(),