From 2de53b9202dac0ec18adfc2d73b4f8670e56026f Mon Sep 17 00:00:00 2001
From: Tobias Urdin <tobias.urdin@binero.se>
Date: Fri, 22 Nov 2019 22:10:11 +0100
Subject: [PATCH] Add randomize_allocation_candidates param

Adds the randomize_allocation_candidates parameter
in the placement section.

Change-Id: I585da30513b17aab823035bb796373d78b52f5f4
---
 manifests/init.pp                                 | 15 ++++++++++++---
 ...ze_allocation_candidates-8421343af6c5f452.yaml |  4 ++++
 spec/classes/placement_init_spec.rb               |  9 +++++++--
 3 files changed, 23 insertions(+), 5 deletions(-)
 create mode 100644 releasenotes/notes/add-randomize_allocation_candidates-8421343af6c5f452.yaml

diff --git a/manifests/init.pp b/manifests/init.pp
index b2e0fa7..f8d0334 100644
--- a/manifests/init.pp
+++ b/manifests/init.pp
@@ -11,10 +11,16 @@
 # [*sync_db*]
 #   (Optional) Run db sync on the node.
 #   Defaults to true
-
+#
+# [*randomize_allocation_candidates*]
+#   (Optional) Randomize the results of the returned
+#   allocation candidates.
+#   Defaults to $::os_service_default
+#
 class placement(
-  $ensure_package = 'present',
-  $sync_db        = true,
+  $ensure_package                  = 'present',
+  $sync_db                         = true,
+  $randomize_allocation_candidates = $::os_service_default,
 ) inherits placement::params {
 
   include ::placement::deps
@@ -36,4 +42,7 @@ class placement(
     tag     => ['openstack', 'placement-package'],
   }
 
+  placement_config {
+    'placement/randomize_allocation_candidates': value => $randomize_allocation_candidates;
+  }
 }
diff --git a/releasenotes/notes/add-randomize_allocation_candidates-8421343af6c5f452.yaml b/releasenotes/notes/add-randomize_allocation_candidates-8421343af6c5f452.yaml
new file mode 100644
index 0000000..9968c50
--- /dev/null
+++ b/releasenotes/notes/add-randomize_allocation_candidates-8421343af6c5f452.yaml
@@ -0,0 +1,4 @@
+---
+features:
+  - |
+    Added placement::randomize_allocation_candidates parameter.
diff --git a/spec/classes/placement_init_spec.rb b/spec/classes/placement_init_spec.rb
index 46eb430..88b49c6 100644
--- a/spec/classes/placement_init_spec.rb
+++ b/spec/classes/placement_init_spec.rb
@@ -20,13 +20,16 @@ describe 'placement' do
         :require => 'Package[python-placement]',
         :tag     => ['openstack', 'placement-package'],
       )}
+
+      it { should contain_placement_config('placement/randomize_allocation_candidates').with_value('<SERVICE DEFAULT>') }
     end
 
     context 'with overridden parameters' do
       let :params do
         {
-          :ensure_package => 'absent',
-          :sync_db        => false,
+          :ensure_package                  => 'absent',
+          :sync_db                         => false,
+          :randomize_allocation_candidates => true,
         }
       end
 
@@ -47,6 +50,8 @@ describe 'placement' do
         :require => 'Package[python-placement]',
         :tag     => ['openstack', 'placement-package'],
       )}
+
+      it { should contain_placement_config('placement/randomize_allocation_candidates').with_value(true) }
     end
   end