Upstream requires nginx since 0.0.10

[PierreR]

It looks like this pr introduces a change in the way upstream can be used.

I used to let applications defined their upstream in their configuration process. Later on the nginxmodule might or might not be used.

It looks like if I define an upstream only (without vhost or include nginx) this line will fail because ${nginx::config::conf_dir} is undefined.

Is this intended ?

[3flex]

I think that was just to make config easier if the default /etc/nginx path wasn't used, and was introduced by d8e6d60.

There's no reason why you couldn't use the defined type as you described. PR welcome to add a upstream_config parameter to nginx::resource::upstream which defaults to "${nginx::config::conf_dir}/conf.d/${name}-upstream.conf". Or a more clever solution if you have one!

[3flex]

Closing for now due to lack of updates; will reopen on request if needed.