Skip to content

fix: make readiness probes explicit - #183

Merged
vigneshrajsb merged 1 commit into
mainfrom
fix/explicit-readiness-probes
Apr 21, 2026
Merged

fix: make readiness probes explicit#183
vigneshrajsb merged 1 commit into
mainfrom
fix/explicit-readiness-probes

Conversation

@vigneshrajsb

Copy link
Copy Markdown
Contributor

Summary

  • stop inferring readiness probes when readiness is omitted
  • remove legacy default readiness probe values from the default service setup path and seed data
  • add coverage for services without readiness configuration

Verification

  • pnpm run lint
  • pnpm test -- --runTestsByPath src/server/services/tests/deployable.test.ts

Notes

  • pnpm run ts-check is currently failing on unrelated pre-existing repository issues and was not addressed in this change

@vigneshrajsb
vigneshrajsb marked this pull request as ready for review April 20, 2026 22:51
@vigneshrajsb
vigneshrajsb requested a review from a team as a code owner April 20, 2026 22:51
@vigneshrajsb
vigneshrajsb merged commit 7c056e2 into main Apr 21, 2026
1 check passed
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.

1 participant