theforeman / puppet-puppet

Puppet module for Puppet client and server
GNU General Public License v3.0
88 stars 227 forks source link

default_manifest_path relative path #919

Open aleita opened 3 months ago

aleita commented 3 months ago

It seems variable default_manifest_path does not support relative path. Anyway the default value is ./manifests Why does not it support relative path ?

ekohl commented 3 months ago

https://www.puppet.com/docs/puppet/7/configuration#default-manifest does indeed say it's allowed to be relative. I think it's because we ensure the file exists. I'm not sure we need to do this.

A patch to clean this up would be welcomed.