Adds the ability to define recycling schedule

[jakauppila]

Description

Adds the ability to define multiple recycle times on an app pool.

I would have preferred to call the property periodic_restart_schedule but to maintain backwards compatibility continued to use recycle_at_time.

The property accepts both a string and array which are coerced into an array when being used.

Issues Resolved

#194

Check List

• All tests pass. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/TESTING.MD
• New functionality includes testing.
• New functionality has been documented in the README if applicable
• All commits have been signed for the Developer Certificate of Origin. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/CONTRIBUTING.MD

[EasyAsABC123]

Looks good :+1 I'd say for the first release we could simply add the new property you want to use and then have the old property map to it.

Then we will do another release for a major change and remove it as it will be deprecated.

Other than that this looks good, I'll approve if no response by Monday

[jakauppila]

You're quicker than me! I can get those other changes in prior to cutting another release; is there a way to define recycle_at_time as an alias? I'm not seeing it in the custom resource documentation.

[iennae]

@jakauppila it's a rubyism.

Format

alias_method :ORIGINALNAME, :NEWNAME

You can see an example of it done here https://github.com/chef-cookbooks/chef_handler/blob/master/resources/default.rb#L28 with supports being the original name that has been aliased to new name type to help with the confusion around supports as a property.

[jakauppila]

Thanks @iennae!

Should that be added to the custom resource documentation? Seems like it's a common enough use.