adding support for installing the windows features required for iis #26
Conversation
| @@ -0,0 +1,6 @@ | |||
| class iis::features::application_deployment { | |||
| ensure_resource('windowsfeature', 'Web-Asp-Net' ) | |||
There was a problem hiding this comment.
Im not quite sure we can do this - there may be different features needed for a 4.0 or a 4.5 account
There was a problem hiding this comment.
I guess I need to do an IIS8 installation and see what's there. I'll put this on hold for a day or so.
liamjbennett
changed the title
|
Ping @liamjbennett should this be closed? |
|
@liamjbennett Close this PR? |
|
I'd actually like to keep get this in. Do we think this requires a major version bump or not? Given that the init.pp at present is only there for example purposes we might be able to get away without it but I would like feedback first |
liamjbennett
changed the title
| ensure_resource('windowsfeature', 'Web-ISAPI-Filter' ) | ||
| } | ||
| default: { | ||
| fail("Do not know how to install iis windows features for ${::kernalmajversion}") |
Hmmm. If you're reasonably confident it doesn't change the behaviour with the same params, a major version bump isn't required. I'm a bit wary of the use of |
liamjbennett
force-pushed
the
windows_feature_groups
branch
from
2916cdd
to
72a79b6
Compare
No description provided.