microsoft / IIS.ServiceMonitor

An entrypoint process for running IIS in Windows containers
MIT License
127 stars 39 forks source link

Won't start without DefaultAppPool #38

Closed FrancisGrignon closed 6 years ago

FrancisGrignon commented 6 years ago

Hi,

I'm not sure if it's a bug or a feature.

If we delete the DefaultAppPool, ServiceMonitor return this error:

Service 'w3svc' has been stopped APPCMD failed with error code 4312 Failed to update IIS configuration

I do delete the DefaultAppPool because I don't use it in my Docker image. Maybe it's a bad practice to do that.

I did notice in the code that then environment variables are updated in the DefaultAppPool configHelper.UpdateEnvironmentVarsToConfig(L"DefaultAppPool"). Which explain why ServiceMonitor is not happy about the missing DefaultAppPool.

Thanks in advance.

Francis

michha commented 6 years ago

This is related to my pull request #31

shirhatti commented 6 years ago

Fixed in #31