Skip to content

Commit

Permalink
Add file:// to credentials option
Browse files Browse the repository at this point in the history
Mesos master said: "Specifying an absolute filename
to read a command line option out of without using
'file:// is deprecated and will be removed in a future
release. Simply adding 'file://' to the beginning of
the path should eliminate this warning."
  • Loading branch information
Dmitry Ilyin committed Mar 14, 2016
1 parent 97c8043 commit 7c73850
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion manifests/master.pp
Original file line number Diff line number Diff line change
Expand Up @@ -66,7 +66,7 @@
}

if (!empty($credentials)) {
$credentials_options = {'credentials' => $credentials_file}
$credentials_options = {'credentials' => "file://${credentials_file}"}
$credentials_content = inline_template("<%= require 'json'; {:credentials => @credentials}.to_json %>")
$credentials_ensure = file
} else {
Expand Down
2 changes: 1 addition & 1 deletion spec/classes/master_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -270,7 +270,7 @@
should contain_mesos__property(
'master_credentials'
).with({
'value' => '/etc/mesos/master-credentials',
'value' => 'file:///etc/mesos/master-credentials',
})
end

Expand Down

0 comments on commit 7c73850

Please sign in to comment.