Merge "Ensure novajoin's log directory ownership is correct"

manifests/metadata/novajoin/api.pp

@@ -173,6 +173,13 @@ class nova::metadata::novajoin::api (
     tag    => ['openstack', 'novajoin-package'],
   }
 
+  file { '/var/log/novajoin':
+    ensure  => directory,
+    owner   => $service_user,
+    group   => $service_user,
+    recurse => true,
+  }
+
   if $ipa_domain != undef {
     novajoin_config {
       'DEFAULT/domain': value => $ipa_domain;

spec/classes/nova_metadata_novajoin_api_spec.rb

@@ -136,6 +136,13 @@ describe 'nova::metadata::novajoin::api' do
         )
       end
 
+      it { is_expected.to contain_file("/var/log/novajoin").with(
+        'ensure'  => 'directory',
+        'owner'   => "#{param_hash[:service_user]}",
+        'group'   => "#{param_hash[:service_user]}",
+        'recurse' => true
+      )}
+
       it { is_expected.to contain_file("#{param_hash[:keytab]}").with(
         'owner'   => "#{param_hash[:service_user]}",
         'require' => 'Exec[get-service-user-keytab]',