[BUG] balancer_by_lua相关的两个bug

ruoshan.huang · 2015-11-18T18:34:03+00:00

> local httpc = http.new()> httpc:set_timeout(5000)> local response, err = httpc:connect(host, port or 80)> if err then> &...

[BUG] balancer_by_lua相关的两个bug

ruoshan.huang

hi there,
   在试用balancer_by_lua, 我遇到两个小问题:
- 在 `proxy_next_upstream_tries 0;` 的时候, set_more_tries 好像没有效果了.
- 在set_current_peer在第二次(或之后)设置一个不一样的host和port的时候, error log里面报错的host和port还是第一次设置的host&port.

--
Good day!
ruoshan

ruoshan.huang

Oops, 第二个问题是我的配置的问题, sorry.

> On Nov 18, 2015, at 6:33 PM, Ruoshan Huang <ruosha...@gmail.com> wrote:
> 
> hi there,
>   在试用balancer_by_lua, 我遇到两个小问题:
> - 在 `proxy_next_upstream_tries 0;` 的时候, set_more_tries 好像没有效果了.
> - 在set_current_peer在第二次(或之后)设置一个不一样的host和port的时候, error log里面报错的host和port还是第一次设置的host&port.
> 
> --
> Good day!
> ruoshan
> 

--
Good day!
ruoshan

doujiang24

Hello,

在 2015年11月19日上午11:27，Ruoshan Huang <ruosha...@gmail.com>写道：

> 在试用balancer_by_lua, 我遇到两个小问题:
> - 在 `proxy_next_upstream_tries 0;` 的时候, set_more_tries 好像没有效果了.

我测试了也是这样

然后看了代码发现: set_more_tries 并不能超过 proxy_next_upstream_tries 的配置 [1]

同时，如果你检查返回的第二值，会拿到一个 warn [2]

至于为什么，我还没看懂

1. https://github.com/openresty/lua-nginx-module/blob/balancer-by-lua/src/ngx_http_lua_balancer.c#L509

2. https://github.com/openresty/lua-nginx-module/blob/balancer-by-lua/lua/ngx/balancer.lua#L71

> - 在set_current_peer在第二次(或之后)设置一个不一样的host和port的时候, error log里面报错的host和port还是第一次设置的host&port.

>
> --
> Good day!
> ruoshan
>

--
Good day!
ruoshan

--

agentzh

Hello!

2015-11-19 18:52 GMT+08:00 DeJiang Zhu:
>> > - 在 `proxy_next_upstream_tries 0;` 的时候, set_more_tries 好像没有效果了.
>
> 我测试了也是这样
> 然后看了代码发现: set_more_tries 并不能超过 proxy_next_upstream_tries 的配置 [1]

这是故意的，proxy_next_upstream_tries 规定的是重试上限。这可以避免 Lua 代码逻辑上的错误而导致无休止的重试。

Regards,
-agentzh