Skip to content

[Quality-2][High] Separate dependency readiness from liveness health checks #1181

Description

@greatest0fallt1me

Summary

Implement Separate dependency readiness from liveness health checks as a production-ready improvement in this repository.

Why this matters

Operators need bounded behavior, truthful telemetry, and a safe recovery path when dependencies or workers fail.

Scope

Trace the affected public entry point through persistence, external dependencies, and user/operator-visible output. Implement Separate dependency readiness from liveness health checks without weakening existing authorization, correctness, compatibility, or CI safeguards.

Priority

High

Acceptance criteria

  • Normal, degraded, and recovery states are explicit and bounded.
  • Metrics and diagnostics are actionable, cardinality-bounded, and safe.
  • Recovery preserves durable state without duplicate work or hidden mutation.
  • Failure-injection tests cover outage, restart, and recovery.

Validation

The PR must include focused regression coverage for this issue, including:

  • Normal, degraded, and recovery states are explicit and bounded.
  • Metrics and diagnostics are actionable, cardinality-bounded, and safe.
  • Recovery preserves durable state without duplicate work or hidden mutation.

Run the repository’s standard formatting, lint, build, and test commands where applicable, and report unrelated pre-existing failures separately.

Non-goals

  • Typo-only, formatting-only, or documentation-only changes.
  • Unrelated refactors, dependency upgrades, or broad redesigns.
  • Weakening security, authorization, CI, validation, or production safeguards.
  • Changing public behavior outside this issue’s scope.

Contributor application

Before implementation, comment with relevant experience, a concise implementation approach, expected files or modules, and an estimate for opening the first PR. Wait for maintainer assignment before coding.

PR requirements

Use a feature branch and include Closes #<issue-number>. The PR must address every acceptance criterion, link each criterion to code and tests, explain security and failure-mode handling, include meaningful regression coverage, preserve compatibility, and report validation results.

Quality review

This is a substantive engineering issue. Review will consider correctness, security, failure handling, test quality, scope discipline, and maintainability.

Activity

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

Metadata

Metadata

Assignees

Labels

Stellar WaveStellar Wave Program issue

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions