Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[ᚬrc/v0.15.z] fix: random failure caused by dirty exit in RPC test #1179

Merged
merged 3 commits into from
Jul 6, 2019
Merged

[ᚬrc/v0.15.z] fix: random failure caused by dirty exit in RPC test #1179

merged 3 commits into from
Jul 6, 2019

Conversation

doitian
Copy link
Member

@doitian doitian commented Jul 4, 2019

Close the server before exit RPC test.

@doitian doitian requested a review from a team July 4, 2019 09:24
@nervos-bot nervos-bot bot changed the title fix: random failure caused by dirty exit in RPC test [ᚬrc/v0.16] fix: random failure caused by dirty exit in RPC test Jul 4, 2019
@nervos-bot
Copy link

nervos-bot bot commented Jul 4, 2019

@keroro520 is assigned as the chief reviewer

@nervos-bot nervos-bot bot added the hotfix Fixing that must be submit to RC branch. label Jul 4, 2019
@doitian doitian requested a review from a team July 4, 2019 09:24
@quake quake changed the base branch from rc/v0.16 to rc/v0.15.z July 5, 2019 05:29
@nervos-bot nervos-bot bot changed the title [ᚬrc/v0.16] fix: random failure caused by dirty exit in RPC test [ᚬrc/v0.15.z] [ᚬrc/v0.16] fix: random failure caused by dirty exit in RPC test Jul 5, 2019
@quake quake changed the title [ᚬrc/v0.15.z] [ᚬrc/v0.16] fix: random failure caused by dirty exit in RPC test [ᚬrc/v0.15.z] fix: random failure caused by dirty exit in RPC test Jul 5, 2019
@doitian doitian merged commit 220dc5d into nervosnetwork:rc/v0.15.z Jul 6, 2019
@doitian doitian deleted the close-server-in-rpc-test branch July 6, 2019 09:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hotfix Fixing that must be submit to RC branch.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants