From b77c5fdbed161bcaafdc1909ff55c73191e3d1be Mon Sep 17 00:00:00 2001
From: Paul Belanger <pabelanger@redhat.com>
Date: Wed, 5 Aug 2015 11:20:06 -0400
Subject: [PATCH] Add puppet-apply converage to both node types

Since it is possible to run both single_use_slave with the thin
parameter true / false, we should be testing both in the gate.

Currently, we set thin to false for our bare nodes and true for
devstack.

Change-Id: Ia0437efb5371d75659fa5523c3e982c252589db6
Signed-off-by: Paul Belanger <pabelanger@redhat.com>
---
 manifests/site.pp | 22 +++++++++++++++++++---
 1 file changed, 19 insertions(+), 3 deletions(-)

diff --git a/manifests/site.pp b/manifests/site.pp
index 6e2d51baf9..fa66ff9803 100644
--- a/manifests/site.pp
+++ b/manifests/site.pp
@@ -784,6 +784,22 @@ node 'openstackid-dev.openstack.org' {
   }
 }
 
+# Node-OS: centos6
+# Node-OS: precise
+# Node-OS: trusty
+# This is not meant to be an actual node that connects to the master.
+# This is a dummy node definition to trigger a test of the code path used by
+# nodepool's prepare_node scripts in the apply tests
+# NOTE(pabelanger): These are the settings we currently use for bare-* nodes.
+# It includes thick_slave.pp.
+node 'single-use-slave-bare' {
+  class { 'openstack_project::single_use_slave':
+    # Test non-default values from prepare_node_bare.sh
+    sudo => true,
+    thin => false,
+  }
+}
+
 # Node-OS: centos6
 # Node-OS: centos7
 # Node-OS: precise
@@ -791,11 +807,11 @@ node 'openstackid-dev.openstack.org' {
 # This is not meant to be an actual node that connects to the master.
 # This is a dummy node definition to trigger a test of the code path used by
 # nodepool's prepare_node scripts in the apply tests
-node 'single-use-slave' {
+# NOTE(pabelanger): These are the current settings we use for devstack-* nodes.
+node 'single-use-slave-devstack' {
   class { 'openstack_project::single_use_slave':
-    # Test non-default values from prepare_node_bare.sh
     sudo => true,
-    thin => false,
+    thin => true,
   }
 }