Merge "Do not set special_service_provider on RHEL7."
This commit is contained in:
		@@ -44,6 +44,13 @@ class nova::params {
 | 
				
			|||||||
        'Fedora', 'RedHat': {
 | 
					        'Fedora', 'RedHat': {
 | 
				
			||||||
          $special_service_provider = undef
 | 
					          $special_service_provider = undef
 | 
				
			||||||
        }
 | 
					        }
 | 
				
			||||||
 | 
					        'RedHat': {
 | 
				
			||||||
 | 
					          if ($::operatingsystemrelease < 7) {
 | 
				
			||||||
 | 
					            $special_service_provider = 'init'
 | 
				
			||||||
 | 
					          } else {
 | 
				
			||||||
 | 
					            $special_service_provider = undef
 | 
				
			||||||
 | 
					          }
 | 
				
			||||||
 | 
					        }
 | 
				
			||||||
        default: {
 | 
					        default: {
 | 
				
			||||||
          $special_service_provider = 'init'
 | 
					          $special_service_provider = 'init'
 | 
				
			||||||
        }
 | 
					        }
 | 
				
			||||||
 
 | 
				
			|||||||
@@ -103,7 +103,8 @@ describe 'nova::compute::libvirt' do
 | 
				
			|||||||
 | 
					
 | 
				
			||||||
  describe 'on rhel platforms' do
 | 
					  describe 'on rhel platforms' do
 | 
				
			||||||
    let :facts do
 | 
					    let :facts do
 | 
				
			||||||
      { :operatingsystem => 'RedHat', :osfamily => 'RedHat' }
 | 
					      { :operatingsystem => 'RedHat', :osfamily => 'RedHat',
 | 
				
			||||||
 | 
					        :operatingsystemrelease => 6.5 }
 | 
				
			||||||
    end
 | 
					    end
 | 
				
			||||||
 | 
					
 | 
				
			||||||
    describe 'with default parameters' do
 | 
					    describe 'with default parameters' do
 | 
				
			||||||
@@ -130,6 +131,20 @@ describe 'nova::compute::libvirt' do
 | 
				
			|||||||
        :provider => nil
 | 
					        :provider => nil
 | 
				
			||||||
      ) }
 | 
					      ) }
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
					      describe 'on rhel 7' do
 | 
				
			||||||
 | 
					        let :facts do
 | 
				
			||||||
 | 
					          super().merge(:operatingsystemrelease => 7.0)
 | 
				
			||||||
 | 
					        end
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
					        it { should contain_service('libvirt').with(
 | 
				
			||||||
 | 
					          :provider => nil
 | 
				
			||||||
 | 
					        )}
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
					        it { should contain_service('messagebus').with(
 | 
				
			||||||
 | 
					          :provider => nil
 | 
				
			||||||
 | 
					        )}
 | 
				
			||||||
 | 
					      end
 | 
				
			||||||
 | 
					
 | 
				
			||||||
      it { should contain_nova_config('DEFAULT/compute_driver').with_value('libvirt.LibvirtDriver')}
 | 
					      it { should contain_nova_config('DEFAULT/compute_driver').with_value('libvirt.LibvirtDriver')}
 | 
				
			||||||
      it { should contain_nova_config('libvirt/virt_type').with_value('kvm')}
 | 
					      it { should contain_nova_config('libvirt/virt_type').with_value('kvm')}
 | 
				
			||||||
      it { should contain_nova_config('DEFAULT/vncserver_listen').with_value('127.0.0.1')}
 | 
					      it { should contain_nova_config('DEFAULT/vncserver_listen').with_value('127.0.0.1')}
 | 
				
			||||||
 
 | 
				
			|||||||
		Reference in New Issue
	
	Block a user