Introduce Further Overriding Abilities
Override Integrated Services Username/Password Override Analysis Services Password and Directory Information
Signed-off-by: John Curcio [email protected]
Description
- Override Integrated Services Username/Password.
- Override Analysis Services Password and Directory Information.
Issues Resolved
Currently you can only override the Analysis Service Username which is pointless without overriding the password.
My Concerns
- I am not sure if adding the ability to override the Analysis Services & Integrated Services Password now requires you to install both services otherwise it fails
- Not sure how to note in the README the defaults for the Analysis Service Directory information. They were originally "Data", "Temp", "Backup", "Log" and "Config", which I assume varies per installation.
Check List
- [ ] All tests pass. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/TESTING.MD
- [ ] New functionality includes testing.
- [X] New functionality has been documented in the README if applicable
- [X] All commits have been signed for the Developer Certificate of Origin. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/CONTRIBUTING.MD
This is great thank you for putting this in I need to run through some tests. If I remember correctly I don't think overriding the password will cause it to fail if the services are not installed. The Reporting service account also has a password and I'm pretty sure in my testing I didn't install that service. For the defaults for the Analysis Service Directories, I think just listing with the default values for those is, is the right way to do that. Thanks again for putting this in.
I have updated the ReadMe as recommended. Not sure why it didn't take my sign off. Oh might be the missing < >