Merge "adding deployment_type fact in support"

manifests/fencing.pp

@@ -67,7 +67,7 @@ class tripleo::fencing(
 
   $all_devices = $config['devices']
 
-  if $::uuid != 'docker' {
+  if $::uuid != 'docker' and $::deployment_type != 'containers' {
     $xvm_devices = local_fence_devices('fence_xvm', $all_devices)
     create_resources('pacemaker::stonith::fence_xvm', $xvm_devices, $common_params)
   }

manifests/profile/base/database/mysql/client.pp

@@ -82,7 +82,7 @@ class tripleo::profile::base::database::mysql::client (
     # present in the base image but installed as a package afterwards),
     # create it. We do not want to touch the permissions in case it already
     # exists due to the mariadb server package being pre-installed
-    if $::uuid == 'docker' {
+    if $::uuid == 'docker' or $::deployment_type == 'containers' {
       # When generating configuration with docker-puppet, services do
       # not include any profile that would ensure creation of /etc/my.cnf.d,
       # so we enforce the check here.

spec/classes/tripleo_profile_base_database_mysql_client_spec.rb

@@ -66,6 +66,28 @@ describe 'tripleo::profile::base::database::mysql::client' do
       }
     end
 
+    context 'with defaults with deployment_type' do
+      let (:params) do
+        { :step => 1 }
+      end
+
+      before (:each) do
+        facts.merge!({ :uuid => 'notdocker', :deployment_type => 'containers' })
+      end
+
+      it {
+        is_expected.to contain_file('/etc/my.cnf.d').with(:ensure => 'directory')
+        is_expected.to contain_augeas('tripleo-mysql-client-conf').with(
+          :incl    => '/etc/my.cnf.d/tripleo.cnf',
+          :changes => [
+            'rm tripleo/bind-address',
+            'rm tripleo/ssl',
+            'rm tripleo/ssl-ca'
+          ]
+        )
+      }
+    end
+
     context 'with ip address set to "" LP#1748180' do
       let (:params) do
         { :step => 1,