Merge "load ram_allocation_ratio when asked"
This commit is contained in:
		@@ -67,10 +67,9 @@ class BaseRamFilter(filters.BaseHostFilter):
 | 
			
		||||
 | 
			
		||||
class RamFilter(BaseRamFilter):
 | 
			
		||||
    """Ram Filter with over subscription flag."""
 | 
			
		||||
    ram_allocation_ratio = CONF.ram_allocation_ratio
 | 
			
		||||
 | 
			
		||||
    def _get_ram_allocation_ratio(self, host_state, filter_properties):
 | 
			
		||||
        return self.ram_allocation_ratio
 | 
			
		||||
        return CONF.ram_allocation_ratio
 | 
			
		||||
 | 
			
		||||
 | 
			
		||||
class AggregateRamFilter(BaseRamFilter):
 | 
			
		||||
 
 | 
			
		||||
@@ -24,21 +24,21 @@ class TestRamFilter(test.NoDBTestCase):
 | 
			
		||||
        self.filt_cls = ram_filter.RamFilter()
 | 
			
		||||
 | 
			
		||||
    def test_ram_filter_fails_on_memory(self):
 | 
			
		||||
        ram_filter.RamFilter.ram_allocation_ratio = 1.0
 | 
			
		||||
        self.flags(ram_allocation_ratio=1.0)
 | 
			
		||||
        filter_properties = {'instance_type': {'memory_mb': 1024}}
 | 
			
		||||
        host = fakes.FakeHostState('host1', 'node1',
 | 
			
		||||
                {'free_ram_mb': 1023, 'total_usable_ram_mb': 1024})
 | 
			
		||||
        self.assertFalse(self.filt_cls.host_passes(host, filter_properties))
 | 
			
		||||
 | 
			
		||||
    def test_ram_filter_passes(self):
 | 
			
		||||
        ram_filter.RamFilter.ram_allocation_ratio = 1.0
 | 
			
		||||
        self.flags(ram_allocation_ratio=1.0)
 | 
			
		||||
        filter_properties = {'instance_type': {'memory_mb': 1024}}
 | 
			
		||||
        host = fakes.FakeHostState('host1', 'node1',
 | 
			
		||||
                {'free_ram_mb': 1024, 'total_usable_ram_mb': 1024})
 | 
			
		||||
        self.assertTrue(self.filt_cls.host_passes(host, filter_properties))
 | 
			
		||||
 | 
			
		||||
    def test_ram_filter_oversubscribe(self):
 | 
			
		||||
        ram_filter.RamFilter.ram_allocation_ratio = 2.0
 | 
			
		||||
        self.flags(ram_allocation_ratio=2.0)
 | 
			
		||||
        filter_properties = {'instance_type': {'memory_mb': 1024}}
 | 
			
		||||
        host = fakes.FakeHostState('host1', 'node1',
 | 
			
		||||
                {'free_ram_mb': -1024, 'total_usable_ram_mb': 2048})
 | 
			
		||||
 
 | 
			
		||||
		Reference in New Issue
	
	Block a user