Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Docs unclear on "relayhost"'s "direct" value #342

Open
fayeg21 opened this issue Jan 10, 2023 · 1 comment
Open

Docs unclear on "relayhost"'s "direct" value #342

fayeg21 opened this issue Jan 10, 2023 · 1 comment
Labels
docs Improvements or additions to documentation

Comments

@fayeg21
Copy link

fayeg21 commented Jan 10, 2023

Affected Puppet, Ruby, OS and module versions/distributions

  • Puppet:
  • Ruby:
  • Distribution:
  • Module version: v3.0.0

How to reproduce (e.g Puppet code you use)

What are you seeing

In https://forge.puppet.com/modules/puppet/postfix/readme#relayhost it's not mentioned that when "mta => true" then the special value of "direct" can be used for relayhost

Without knowing this one gets this cryptic error message:

Error: Could not retrieve catalog from remote server: Error 500 on SERVER: Server Error: Evaluation Error: Error while evaluating a Function Call, pick(): must receive at least one non empty value (file: /etc/puppetlabs/code/modules/postfix/manifests/mta.pp, line: 43, column: 17)

What behaviour did you expect instead

  1. A better error message
  2. The 'direct' setting added to the 'relayhost' doc in the README.

Output log

Any additional information you'd like to impart

@kenyon kenyon added the docs Improvements or additions to documentation label Jan 12, 2023
@ekohl
Copy link
Member

ekohl commented Nov 21, 2023

Isn't this a duplicate of #295?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Improvements or additions to documentation
Projects
None yet
Development

No branches or pull requests

3 participants