Merge "Fail to live migration if instance has NUMA topology" into stable/queens
This commit is contained in:
commit
96ba42dd1e
|
@ -38,6 +38,7 @@ class nova::conductor(
|
||||||
include ::nova::deps
|
include ::nova::deps
|
||||||
include ::nova::db
|
include ::nova::db
|
||||||
include ::nova::params
|
include ::nova::params
|
||||||
|
include ::nova::workarounds
|
||||||
|
|
||||||
nova::generic_service { 'conductor':
|
nova::generic_service { 'conductor':
|
||||||
enabled => $enabled,
|
enabled => $enabled,
|
||||||
|
|
|
@ -0,0 +1,20 @@
|
||||||
|
# == Class: nova::workarounds
|
||||||
|
#
|
||||||
|
# nova workarounds configuration
|
||||||
|
#
|
||||||
|
# === Parameters:
|
||||||
|
#
|
||||||
|
# [*enable_numa_live_migration*]
|
||||||
|
# (optional) Whether to enable live migration for NUMA topology instances.
|
||||||
|
# Defaults to false
|
||||||
|
#
|
||||||
|
class nova::workarounds (
|
||||||
|
$enable_numa_live_migration = false,
|
||||||
|
) {
|
||||||
|
|
||||||
|
nova_config {
|
||||||
|
'workarounds/enable_numa_live_migration': value => $enable_numa_live_migration;
|
||||||
|
}
|
||||||
|
|
||||||
|
}
|
||||||
|
|
|
@ -0,0 +1,9 @@
|
||||||
|
---
|
||||||
|
upgrade:
|
||||||
|
- |
|
||||||
|
Add support to enable/disable live-migration for NUMA topology instances
|
||||||
|
|
||||||
|
Add configuration parameter `workarounds/enable_numa_live_migration`
|
||||||
|
(defaults to false) which allows to enable/disable live-migration
|
||||||
|
for NUMA topology instances.
|
||||||
|
|
|
@ -0,0 +1,36 @@
|
||||||
|
require 'spec_helper'
|
||||||
|
|
||||||
|
describe 'nova::workarounds' do
|
||||||
|
|
||||||
|
let :params do
|
||||||
|
{}
|
||||||
|
end
|
||||||
|
|
||||||
|
shared_examples 'nova::workarounds' do
|
||||||
|
|
||||||
|
context 'with default parameters' do
|
||||||
|
it { is_expected.to contain_nova_config('workarounds/enable_numa_live_migration').with_value(false) }
|
||||||
|
end
|
||||||
|
|
||||||
|
context 'with overridden parameters' do
|
||||||
|
let :params do
|
||||||
|
{ :enable_numa_live_migration => true,}
|
||||||
|
end
|
||||||
|
|
||||||
|
it { is_expected.to contain_nova_config('workarounds/enable_numa_live_migration').with_value('true') }
|
||||||
|
end
|
||||||
|
|
||||||
|
end
|
||||||
|
|
||||||
|
on_supported_os({
|
||||||
|
:supported_os => OSDefaults.get_supported_os
|
||||||
|
}).each do |os,facts|
|
||||||
|
context "on #{os}" do
|
||||||
|
let (:facts) do
|
||||||
|
facts.merge!(OSDefaults.get_facts())
|
||||||
|
end
|
||||||
|
|
||||||
|
it_configures 'nova::workarounds'
|
||||||
|
end
|
||||||
|
end
|
||||||
|
end
|
Loading…
Reference in New Issue