Project

General

Profile

Actions

Bug #6945

closed

smart-proxy doesn't find the autosign.conf if not at "#{puppetdir}/autosign.conf"

Added by Brian Witt over 10 years ago. Updated over 7 years ago.

Status:
Closed
Priority:
Normal
Assignee:
-
Category:
Puppet
Target version:
-
Difficulty:
Triaged:
Fixed in Releases:
Found in Releases:

Description

modules/puppetca/puppetca.rb hardocodes the autosign.conf file to

def autosign_file
"#{puppetdir}/autosign.conf"
end

so even if we've set a different location in puppet.conf:

autosign = /etc/autosign.conf { mode = 664 }

we still see this error in the foreman-proxy logs:

No such file /etc/puppet/autosign.conf

Actions #1

Updated by Brian Witt over 10 years ago

Our puppetdir in this case is set to /etc/puppet. Is there anything else that uses puppetdir?

Actions #2

Updated by Dominic Cleal over 10 years ago

  • Category set to Puppet

Brian Witt wrote:

Our puppetdir in this case is set to /etc/puppet. Is there anything else that uses puppetdir?

Yes, it's also used to find puppet.conf, they're expected to be in the same directory.

Actions #3

Updated by Brian Witt over 10 years ago

I thought the puppet.conf location was set via :puppet_conf:

Actions #4

Updated by Dominic Cleal over 10 years ago

Brian Witt wrote:

I thought the puppet.conf location was set via :puppet_conf:

Yeah, puppet_conf will override the use of puppetdir/puppet.conf, so you can change puppetdir safely if you have puppet_conf set.

Actions #5

Updated by Anonymous over 7 years ago

I'm leaning towards closing this.

Actions #6

Updated by Brian Witt over 7 years ago

Michael Moll wrote:

I'm leaning towards closing this.

Why?

Actions #7

Updated by Anonymous over 7 years ago

I'd say it "works as designed". Maybe the docs would need updating...

Actions #8

Updated by Anonymous over 7 years ago

I think I need to revise my opinion on that in the light of the whole Puppet 4 situation now. It seems the puppetdir setting is not used elsewhere in the pupetca module.

Actions #9

Updated by The Foreman Bot over 7 years ago

  • Status changed from New to Ready For Testing
  • Assignee set to Anonymous
  • Pull request https://github.com/theforeman/smart-proxy/pull/530 added
Actions #10

Updated by Anonymous over 7 years ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100
Actions #11

Updated by Brian Witt over 7 years ago

Thanks!

Actions

Also available in: Atom PDF