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

Add advanced test for DNSPolicy Health Checks #264

Closed
averevki opened this issue Nov 3, 2023 · 2 comments
Closed

Add advanced test for DNSPolicy Health Checks #264

averevki opened this issue Nov 3, 2023 · 2 comments
Labels
DNS Issues for DNSOperator enhancement Improvement to existing test Test case New test case

Comments

@averevki
Copy link
Contributor

averevki commented Nov 3, 2023

Summary

Test if the unhealthy endpoint was correctly removed from the list of available endpoints in DNSRecord and was returned back when became healthy

Documentation

Additional context

Steps from the DNSPolicy Health Checks documentation after unhealthy endpoint was detected:

  1. The health check probe identifies an endpoint as "unhealthy" and it’s got greater consecutive failures than failure threshold.
  2. The system reacts by immediately removing the unhealthy endpoint from the list of available endpoints, any endpoint that doesn’t have at least 1 healthy child will also be removed.
  3. This removal causes traffic to automatically get redirected to the remaining healthy endpoints.
  4. The health check continues monitoring the endpoint's status. If it becomes healthy again, endpoint is added to the list of available endpoints.

Test cases

  • Test for endpoint removal from available endpoints in DNSRecord and its return when becomes healthy
@averevki averevki added enhancement Improvement to existing test Test case New test case DNS Issues for DNSOperator labels Nov 3, 2023
@github-project-automation github-project-automation bot moved this to 🆕 New in Kuadrant QE Nov 8, 2023
@jsmolar jsmolar moved this from 🆕 New to 📋 Backlog in Kuadrant QE Nov 8, 2023
@martinhesko martinhesko self-assigned this Feb 20, 2024
@martinhesko martinhesko moved this from 📋 Backlog to 🏗 In progress in Kuadrant QE Mar 5, 2024
@martinhesko martinhesko moved this from 🏗 In progress to 🆕 New in Kuadrant QE Mar 6, 2024
@martinhesko
Copy link
Contributor

Putting this in the backlog for now due to the changes with DNS Health checks

For future reference: For an unhealthy health check to have an effect on DNSRecord there need to be some healthy endpoints, see Removing unhealthy Endpoints

@martinhesko martinhesko removed their assignment Mar 6, 2024
@averevki
Copy link
Contributor Author

averevki commented Nov 6, 2024

Closed in favor of #572

@averevki averevki closed this as completed Nov 6, 2024
@github-project-automation github-project-automation bot moved this from 🆕 New to ✅ Done in Kuadrant QE Nov 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
DNS Issues for DNSOperator enhancement Improvement to existing test Test case New test case
Projects
Status: ✅ Done
Development

No branches or pull requests

2 participants