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

is_hash, is_array, and is_any are depricated #106

Closed
james-powis opened this issue Jan 10, 2017 · 5 comments
Closed

is_hash, is_array, and is_any are depricated #106

james-powis opened this issue Jan 10, 2017 · 5 comments

Comments

@james-powis
Copy link
Member

reference: https://github.com/puppetlabs/puppetlabs-stdlib
and example of replacement: https://github.com/puppetlabs/puppetlabs-ntp/blob/master/manifests/init.pp#L17-L28

@tobru
Copy link
Contributor

tobru commented Jan 12, 2017

Thanks! We'll update the module over the next weeks...

@iainhallam
Copy link

Same for validate_bool, apparently (at least, I get warnings) in Puppet agent > 4.

@pgassmann
Copy link

For reference and search, the deprecation messages:
method is deprecated, please use the stdlib validate_legacy function

Warning: This method is deprecated, please use the stdlib validate_legacy function, with Stdlib::Compat::Bool. There is further documentation for validate_legacy function in the README. at ["/x/modules/gitlab/manifests/init.pp", 339]:["/x/site/role/manifests/gitlab.pp", 2]
   (at /x/modules/stdlib/lib/puppet/functions/deprecation.rb:25:in `deprecation')
Warning: This method is deprecated, please use the stdlib validate_legacy function, with Stdlib::Compat::String. There is further documentation for validate_legacy function in the README. at ["/x/modules/gitlab/manifests/init.pp", 342]:["/x/site/role/manifests/gitlab.pp", 2]
   (at /x/modules/stdlib/lib/puppet/functions/deprecation.rb:25:in `deprecation')
Warning: This method is deprecated, please use the stdlib validate_legacy function, with Pattern[]. There is further documentation for validate_legacy function in the README. at ["/x/modules/gitlab/manifests/init.pp", 344]:["/x/site/role/manifests/gitlab.pp", 2]
   (at /x/modules/stdlib/lib/puppet/functions/deprecation.rb:25:in `deprecation')
Warning: 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 ["/x/modules/gitlab/manifests/init.pp", 351]:["/x/site/role/manifests/gitlab.pp", 2]
   (at /x/modules/stdlib/lib/puppet/functions/deprecation.rb:25:in `deprecation')
Warning: This method is deprecated, please use the stdlib validate_legacy function, with Stdlib::Compat::Hash. There is further documentation for validate_legacy function in the README. at ["/x/modules/gitlab/manifests/init.pp", 364]:["/x/site/role/manifests/gitlab.pp", 2]
   (at /x/modules/stdlib/lib/puppet/functions/deprecation.rb:25:in `deprecation')

@iainhallam
Copy link

Was this fixed, then?

@LongLiveCHIEF
Copy link
Contributor

yep! I was going through all issues last night to make sure we cleaned any up, and I couldn't remember the exact commit/merge where this was fully resolved or I would have linked it. I'm fairly sure though that the first release containing this fix was in v1.16.0.

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

No branches or pull requests

6 participants