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

Update puppetlabs-stdlib dependency version in README #298

Merged
merged 1 commit into from
Oct 11, 2018
Merged

Update puppetlabs-stdlib dependency version in README #298

merged 1 commit into from
Oct 11, 2018

Conversation

simonrondelez
Copy link
Contributor

This Pull Request (PR) fixes the following issues

This fixes the following error:

Error: Could not retrieve catalog from remote server: Error 500 on SERVER: Server Error: Evaluation Error: Resource type not found: Stdlib::Port::Unprivileged (file: /etc/puppetlabs/code/modules/openvpn/manifests/server.pp, line: 481, column: 11) on node

Stdlib::Port::Unprivileged functionality was only introduced at version 4.25.0

`Stdlib::Port::Unprivileged` functionality was only introduced at version 4.25.0
@Dan33l
Copy link
Member

Dan33l commented Oct 11, 2018

Thank you for this report.

Please update metadata.json accordingly.

Can you provide a patch to use management_port with acceptance tests ?

@alexjfisher alexjfisher added the docs Improvements or additions to documentation label Oct 11, 2018
@alexjfisher alexjfisher changed the title Update dependency for puppetlabs-stdlib Update puppetlabs-stdlib dependency version in README Oct 11, 2018
@alexjfisher
Copy link
Member

The metadata was updated correctly at the time. See #275

This PR does fix the README though.

@alexjfisher alexjfisher merged commit 52682b3 into voxpupuli:master Oct 11, 2018
@alexjfisher
Copy link
Member

@simonrondelez Thanks!

@simonrondelez simonrondelez deleted the patch-1 branch October 11, 2018 11:46
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

Successfully merging this pull request may close these issues.

3 participants