Include mistral::db::sync by default

This is done in most other modules and it doesn't make much sense to
setup mistral without it.

Change-Id: I2f8b6ad7ae3e4defefe58ac749b2268d067167b1

manifests/init.pp

@@ -173,6 +173,10 @@
 #   in the mistral config.
 #   Defaults to false.
 #
+# [*sync_db*]
+#   (Optional) Enable dbsync
+#   Defaults to true.
+#
 # === DEPRECATED PARAMTERS
 #
 # [*rabbit_host*]
@@ -234,6 +238,7 @@ class mistral(
   $coordination_backend_url           = $::os_service_default,
   $coordination_heartbeat_interval    = $::os_service_default,
   $purge_config                       = false,
+  $sync_db                            = true,
   # DEPRECATED PARAMETERS
   $rabbit_host                        = $::os_service_default,
   $rabbit_port                        = $::os_service_default,
@@ -312,4 +317,8 @@ deprecated. Please use mistral::default_transport_url instead.")
   } else {
     mistral_config { 'DEFAULT/rpc_backend': value => $rpc_backend }
   }
+
+  if $sync_db {
+    include ::mistral::db::sync
+  }
 }

releasenotes/notes/db_sync_class_default-40f42cf762187f17.yaml

@@ -0,0 +1,4 @@
+---
+features:
+  - By default the mistral::db::sync class will be included
+    This can be disable by setting mistral::sync_db to false.

releasenotes/notes/include_db_sync-c5e9c2c5cef49eb1.yaml

@@ -0,0 +1,4 @@
+---
+features:
+  - The class mistral::db::sync will be included by default.
+    This matches what most other modules do.

spec/acceptance/basic_mistral_spec.rb

@@ -55,7 +55,6 @@ describe 'basic mistral' do
           class { '::mistral::client': }
           class { '::mistral::engine': }
           class { '::mistral::executor': }
-          class { '::mistral::db::sync': }
         }
       }
       EOS

spec/classes/mistral_db_mysql_spec.rb

@@ -5,7 +5,6 @@ describe 'mistral::db::mysql' do
   let :pre_condition do
     [
       'include mysql::server',
-      'include mistral::db::sync'
     ]
   end