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

need healthz check logic to detect failure #127

Open
sunya-ch opened this issue Jul 7, 2023 · 1 comment
Open

need healthz check logic to detect failure #127

sunya-ch opened this issue Jul 7, 2023 · 1 comment
Labels
enhancement New feature or request

Comments

@sunya-ch
Copy link
Collaborator

sunya-ch commented Jul 7, 2023

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]

Need to add logic to detect failure that cannot be recovered in healthz handler.

Describe the solution you'd like
A clear and concise description of what you want to happen.

Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.

Additional context
Add any other context or screenshots about the feature request here.

@sunya-ch sunya-ch added the enhancement New feature or request label Jul 7, 2023
@sunya-ch sunya-ch modified the milestones: v1.0.5, v1.1.0 Jul 7, 2023
@sunya-ch
Copy link
Collaborator Author

sunya-ch commented Jul 7, 2023

Need to at least wait continuously for two reconcile loops of hostinterface (2*vars.UrgentReconcileTime) after detect failure to make sure that it will not be recovered.

healthz period = 20s

Failure condition:

  • update queue is not processed always same len(h.UpdateRequestQueue)
  • no item in the queue to process (len(h.UpdateRequestQueue)) but infoAvailable != cidrProcessed.
  • CIDR is not synced (such as unexpected IPPool)

@sunya-ch sunya-ch changed the title move RunPeriodicUpdate to healthz check logic need healthz check logic to detect failure Jul 7, 2023
@sunya-ch sunya-ch removed this from the v1.1.0 milestone Aug 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant