Skip to content

chore: remove deprecated AWS_LWA_READINESS_CHECK_MIN_UNHEALTHY_STATUS#663

Closed
bnusunny wants to merge 1 commit intomainfrom
remove-deprecated-min-unhealthy-status
Closed

chore: remove deprecated AWS_LWA_READINESS_CHECK_MIN_UNHEALTHY_STATUS#663
bnusunny wants to merge 1 commit intomainfrom
remove-deprecated-min-unhealthy-status

Conversation

@bnusunny
Copy link
Copy Markdown
Contributor

Summary

Remove the deprecated AWS_LWA_READINESS_CHECK_MIN_UNHEALTHY_STATUS environment variable and all associated code. Users should use AWS_LWA_READINESS_CHECK_HEALTHY_STATUS instead.

Changes

  • Remove ENV_READINESS_CHECK_MIN_UNHEALTHY_STATUS constant and fallback logic in src/lib.rs
  • Remove readiness_check_min_unhealthy_status field from AdapterOptions
  • Remove deprecation notices from README and environment-variables docs
  • Clean up #[allow(deprecated)] annotations and commented-out code in tests

Testing

  • cargo build — compiles cleanly
  • cargo nextest run — all 53 tests pass
  • cargo clippy --all-targets — no warnings

Remove the deprecated AWS_LWA_READINESS_CHECK_MIN_UNHEALTHY_STATUS
environment variable and all associated code. Users should use
AWS_LWA_READINESS_CHECK_HEALTHY_STATUS instead.

Changes:
- Remove ENV_READINESS_CHECK_MIN_UNHEALTHY_STATUS constant and fallback logic
- Remove readiness_check_min_unhealthy_status field from AdapterOptions
- Remove deprecation notices from README and environment-variables docs
- Clean up #[allow(deprecated)] annotations and commented-out code in tests
@github-actions
Copy link
Copy Markdown

📊 Benchmark Comparison

Benchmark Details
  • Baseline: main branch
  • Comparison: This PR
  • Threshold: 10% regression triggers warning

@bnusunny bnusunny closed this Mar 20, 2026
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