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.

Sign up for GitHub

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

Jump to bottom

Remove legacy function validate_absolutepath #35

Merged
merged 1 commit into from
Dec 6, 2023
Merged

Remove legacy function validate_absolutepath #35

merged 1 commit into from
Dec 6, 2023

Conversation

anders-larsson
Copy link
Contributor

On Puppet 8 this module currently fails with the error below due to use of legacy function. It is also already validated because the parameter itself is typed to Stdlib::Absolutepath.

Error: Could not retrieve catalog from remote server: Error 500 on SERVER: Server Error: Evaluation Error: Error while evaluating a Function Call, deprecation. validate_absolute_path. This method is deprecated, please use the stdlib validate_legacy function,
                    with Stdlib::Compat::Absolute_Path. There is further documentation for validate_legacy function in the README. at ["/etc/puppetlabs/code/.../krb5/manifests/init.pp", 174] on node ...```

Phil-Friderici
Phil-Friderici approved these changes Dec 5, 2023
View reviewed changes
Copy link
Contributor

@Phil-Friderici Phil-Friderici left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

simple and beautifull :)

@anders-larsson anders-larsson merged commit 2ed7819 into Ericsson:master Dec 6, 2023
4 checks passed
@anders-larsson anders-larsson deleted the remove_legacy_function branch December 6, 2023 06:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Phil-Friderici Phil-Friderici Phil-Friderici approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@anders-larsson @Phil-Friderici