Skip to content

[WIP] Fix liveness probe and add readiness probe in deployment#146

Closed
RakhithaRR with Copilot wants to merge 1 commit into
mainfrom
copilot/fix-liveness-probe-configuration
Closed

[WIP] Fix liveness probe and add readiness probe in deployment#146
RakhithaRR with Copilot wants to merge 1 commit into
mainfrom
copilot/fix-liveness-probe-configuration

Conversation

Copilot AI commented Feb 13, 2026

Copy link
Copy Markdown
Contributor

Fix duplicate liveness probes in key-manager deployment

  • Explore repository and understand the issue
  • Identify the problem: two livenessProbe sections on lines 156 and 163 in 4.6.x branch
  • Change second livenessProbe to readinessProbe (line 163)
  • Update probe configuration values to use readinessProbe values (lines 167-169)
  • Verify the changes with helm lint - passed successfully
  • Validate consistency with other deployment files - confirmed alignment with gateway, control-plane, and traffic-manager deployments
  • Commit changes
  • Run code review - no issues found
  • Run security scan with CodeQL - no issues detected

Summary: Successfully fixed the duplicate livenessProbe issue in the key-manager deployment template. The second probe has been changed to readinessProbe and now uses the appropriate readinessProbe configuration values. This change aligns the key-manager deployment with all other deployment files in the repository.


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

@CLAassistant

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants