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

healthcheck: more checker methods and bugfix #910

Merged
merged 5 commits into from
Sep 15, 2023

Conversation

ywc689
Copy link
Collaborator

@ywc689 ywc689 commented Sep 6, 2023

  • Add healthcheck method 'udpping' and make it the default checker for UDP targets.
    The 'udpping' checker first checks the l3 network connectivity with ping checker, and performs udp checker only ping checker responds ok. It's an enhancement for the simple udp checker when udp port probe times out.
  • Add healthcheck method 'http' for HTTP/HTTPS targets.

… checker for UDP targets

The 'udpping' checker first checks the l3 network connectivity with ping checker, and performs
udp checker only ping checker responds ok. It's an enhancement for the simple udp checker when
udp port probe times out.

Signed-off-by: ywc689 <[email protected]>
Note that realserver's weight and inhibited flag can be modified by dpvs-agent independently.
So we cannot assume that the weight is zero when inhibited flag is set.

Signed-off-by: ywc689 <[email protected]>
@ywc689 ywc689 changed the title healthcheck: add healthcheck method 'udpping' and make it the default… healthcheck: more checker methods and bugfix Sep 12, 2023
@ywc689 ywc689 added the pr/accepted the pr passed all review stages and await to be merged label Sep 15, 2023
@ywc689 ywc689 merged commit 6e4bbfe into iqiyi:devel Sep 15, 2023
3 checks passed
@ywc689 ywc689 deleted the udpping_checker branch July 10, 2024 07:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr/accepted the pr passed all review stages and await to be merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants