Re: [Kimchi-devel] [PATCH][Wok] Issue #166: wok is pointing to /etc/nginx/conf.d which does not exists on OpenSuse 42.1

On 09/26/2016 01:17 PM, Ramon Medeiros wrote: > > > On 09/22/2016 09:00 AM, Aline Manera wrote: >> Hi Ramon, >> >> On 09/21/2016 02:58 PM, Ramon Medeiros wrote: >>> Create dir if not exist >> >> Is it the recommendation from nginx? >> I mean, this directory should be installed by nginx and if it is not >> maybe there is a better directory (already installed by nginx) to >> place configuration files. >> >> Also, doing that, the system will keep the directory on package >> removal. > Nop, > > just remove the rpm and the file is still there And we don't want it, right? >> >>> >>> Signed-off-by: Ramon Medeiros <ramonn(a)linux.vnet.ibm.com&gt; >>> --- >>> src/wok/proxy.py | 5 +++++ >>> 1 file changed, 5 insertions(+) >>> >>> diff --git a/src/wok/proxy.py b/src/wok/proxy.py >>> index b68b86e..4b06fc2 100644 >>> --- a/src/wok/proxy.py >>> +++ b/src/wok/proxy.py >>> @@ -116,6 +116,11 @@ def _create_proxy_config(options): >>> # SSL certifications. >>> if not paths.installed: >>> dst = os.path.join(paths.sys_nginx_conf_dir, "wok.conf") >>> + >>> + # directoy does not exist: create it >>> + if not os.path.exists(paths.sys_nginx_conf_dir): >>> + os.makedirs(paths.sys_nginx_conf_dir) >>> + >>> if os.path.isfile(dst) or os.path.islink(dst): >>> os.remove(dst) >>> os.symlink(os.path.join(nginx_config_dir, "wok.conf"), dst) >> > _______________________________________________ Kimchi-devel mailing list Kimchi-devel(a)ovirt.org http://lists.ovirt.org/mailman/listinfo/kimchi-devel