roles: update opts on config reload

There were no way to update config on the run with method
`config:reload()` instead of listen parameters.

This patch fixes it, all options support config reload.

Closes #126

Author: themilchenko

roles/httpd.lua

@@ -107,6 +107,20 @@ local function parse_params(node)
     }
 end
 
+local function server_was_changed(name, node, host, port)
+    if servers[name].httpd.host ~= host or servers[name].httpd.port ~= port then
+        return true
+    end
+    for k in pairs(node) do
+        if k ~= 'listen' then
+            if servers[name].httpd.options[k] ~= node[k] then
+                return true
+            end
+        end
+    end
+    return false
+end
+
 local function apply_http(name, node)
     local host, port, err = parse_listen(node.listen)
     if err ~= nil then
@@ -121,7 +135,7 @@ local function apply_http(name, node)
             httpd = httpd,
             routes = {},
         }
-    elseif servers[name].httpd.host ~= host or servers[name].httpd.port ~= port then
+    elseif server_was_changed(name, node, host, port) then
         servers[name].httpd:stop()
         servers[name].httpd = http_server.new(host, port, parse_params(node))
         servers[name].httpd:start()

test/integration/httpd_role_test.lua

@@ -203,3 +203,22 @@ for _, log_lvl in pairs(LOG_LEVELS) do
         assert_should_log(log_level >= LOG_LEVELS.INFO)
     end
 end
+
+g.test_enable_tls_on_config_reload = function(cg)
+    -- We should start with no tls firstly.
+    t.skip_if(cg.params.use_tls)
+
+    local resp = http_client:get('http://localhost:13000/ping')
+    t.assert_equals(resp.status, 200, 'response not 200')
+    t.assert_equals(resp.body, 'pong')
+
+    treegen.write_file(cg.server.chdir, 'config.yaml', yaml.encode(tls_config))
+    local _, err = cg.server:eval("require('config'):reload()")
+    t.assert_not(err)
+
+    resp = http_client:get('https://localhost:13000/ping', {
+        ca_file = fio.pathjoin(ssl_data_dir, 'ca.crt')
+    })
+    t.assert_equals(resp.status, 200, 'response not 200')
+    t.assert_equals(resp.body, 'pong')
+end