Adding this PR would make [upstream.pp](https://github.com/example42/puppet-nginx/blob/master/manifests/resource/upstream.pp)  behaviour completly inconsistent whether you are using a Debian-based distro or any other distro: On Debian you'll be using (and depending on) *concat* and creating a concat-based config file, while on RedHat and others you'll will be simply creating a template-based file.

Also the fact that Debian based will notify the service and others will not is an inconsistent behaviour.

Another issue is that, AFAIK, upstream in Debian should go under */etc/nginx/conf.d* and not under *sites-available*.