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

fix: uds-core timeout problem #758

Closed
wants to merge 10 commits into from
Closed

fix: uds-core timeout problem #758

wants to merge 10 commits into from

Conversation

cmwylie19
Copy link
Contributor

Description

UDS Core is a Pepr module that has intermittent timeouts where Pepr will not respond to the API Server. This PR will have the watch deployment ping the API Server endpoint in order to report healthy on the liveness probe.

Note: This behavior has not been reproduced outside of the UDS Core module even under heavy load.

Related Issue

Fixes #

Relates to #745

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Other (security config, docs update, etc)

Checklist before merging

Signed-off-by: Case Wylie <cmwylie19@defenseunicorns.com>
Signed-off-by: Case Wylie <cmwylie19@defenseunicorns.com>
Signed-off-by: Case Wylie <cmwylie19@defenseunicorns.com>
Signed-off-by: Case Wylie <cmwylie19@defenseunicorns.com>
Signed-off-by: Case Wylie <cmwylie19@defenseunicorns.com>
@cmwylie19 cmwylie19 changed the title feat: uds-core timeout problem fix: uds-core timeout problem Apr 25, 2024
Signed-off-by: Case Wylie <cmwylie19@defenseunicorns.com>
Signed-off-by: Case Wylie <cmwylie19@defenseunicorns.com>
…cle.

Signed-off-by: Case Wylie <cmwylie19@defenseunicorns.com>
@cmwylie19
Copy link
Contributor Author

This was an attempt to expose a bug. The kube-apiserver is pinging Pepr's readinessProbe and livenessProbe every 10 seconds and Pepr would then ping the kube-apiserver. This is redundant and we are closing this for now

@cmwylie19 cmwylie19 closed this May 1, 2024
@samayer12 samayer12 deleted the healthcheck-update branch October 1, 2024 16:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: ✅ Done
Development

Successfully merging this pull request may close these issues.

2 participants