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

feat :health check for stream proxy #2546

Closed
wants to merge 37 commits into from
Closed

Conversation

redynasc
Copy link
Contributor

@redynasc redynasc commented Oct 28, 2020

What this PR does / why we need it:

Health check is the basic function required by stream proxy

fix:#2407

  • Did you explain what problem does this PR solve? Or what new features have been added?
  • Have you added corresponding test cases?
  • Have you modified the corresponding document?
  • Is this PR backward compatible?

@membphis
Copy link
Member

that is a big feature, I'll take a look at this PR later.

many thx for your contribution @redynasc

@juzhiyuan juzhiyuan added the wait for update wait for the author's response in this issue/PR label Nov 2, 2020
@redynasc redynasc changed the title Health check for stream proxy feat:health check for stream proxy Dec 24, 2020
@redynasc
Copy link
Contributor Author

resolve the conflict

@membphis
Copy link
Member

where is the doc?

image

@spacewander
Copy link
Member

@redynasc
Do you have time to continue?

@spacewander spacewander changed the title feat:health check for stream proxy feat :health check for stream proxy Jan 14, 2021
@membphis membphis self-requested a review April 6, 2021 02:45
@github-actions
Copy link

This pull request has been marked as stale due to 60 days of inactivity. It will be closed in 4 weeks if no further activity occurs. If you think that's incorrect or this pull request should instead be reviewed, please simply write any comment. Even if closed, you can still revive the PR at any time or discuss it on the [email protected] list. Thank you for your contributions.

@github-actions github-actions bot added the stale label Aug 25, 2021
@github-actions
Copy link

This pull request/issue has been closed due to lack of activity. If you think that is incorrect, or the pull request requires review, you can revive the PR at any time.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
stale wait for update wait for the author's response in this issue/PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants