Skip to content

fixed wrong resetting upstream flags in #T_NGX_HTTP_UPSTREAM_RANDOM#1660

Merged
wangfakang merged 1 commit into
alibaba:masterfrom
chobits:master
Oct 26, 2022
Merged

fixed wrong resetting upstream flags in #T_NGX_HTTP_UPSTREAM_RANDOM#1660
wangfakang merged 1 commit into
alibaba:masterfrom
chobits:master

Conversation

@chobits

@chobits chobits commented Oct 26, 2022

Copy link
Copy Markdown
Member

@chobits

chobits commented Oct 26, 2022

Copy link
Copy Markdown
Member Author

case result:

$ TEST_NGINX_UNSAFE=yes TEST_NGINX_BINARY=/home/xiaochen/work/ctengine/objs/nginx prove -I /home/xiaochen/work/ctengine/tests/nginx-tests/nginx-tests/lib tests/nginx-tests/tengine-tests/dyups.t
tests/nginx-tests/tengine-tests/dyups.t .. ok
All tests successful.
Files=1, Tests=78, 10 wallclock secs ( 0.03 usr  0.00 sys +  0.24 cusr  0.21 csys =  0.48 CPU)
Result: PASS
$ TEST_NGINX_UNSAFE=yes TEST_NGINX_BINARY=/home/xiaochen/work/ctengine/objs/nginx prove -I /home/xiaochen/work/ctengine/tests/nginx-tests/nginx-tests/lib ./tests/nginx-tests/nginx-tests/upstream_random.t
./tests/nginx-tests/nginx-tests/upstream_random.t .. ok
All tests successful.
Files=1, Tests=14, 18 wallclock secs ( 0.00 usr  0.01 sys +  0.09 cusr  0.02 csys =  0.12 CPU)
Result: PASS

@chobits
chobits requested a review from wangfakang October 26, 2022 06:37

@wangfakang wangfakang left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@wangfakang
wangfakang merged commit e30f958 into alibaba:master Oct 26, 2022
@chobits chobits added this to the 2.3.5 milestone Oct 26, 2022
@chobits chobits mentioned this pull request Oct 26, 2022
@chobits chobits modified the milestones: 2.3.5, 2.4.0 Feb 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants