Skip to content

fix: remove 5 broken challenges, fix probes-drift#18

Merged
pbrissaud merged 1 commit intomainfrom
fix/critical-challenges-cleanup
Jan 25, 2026
Merged

fix: remove 5 broken challenges, fix probes-drift#18
pbrissaud merged 1 commit intomainfrom
fix/critical-challenges-cleanup

Conversation

@pbrissaud
Copy link
Member

Summary

Cleanup of non-functional challenges identified during comprehensive review.

Deleted Challenges (5)

Challenge Score Reason
tls-ingress 4/20 No TLS validation, no ingress controller installed
cert-mismatch 9/20 Passes without action (no TLS validation)
traffic-jam 10/20 Depends on ingress-nginx which isn't installed
healthy-app 5/20 Cannot validate that probes were configured
volume-conflict 5/20 StatefulSet creates separate PVCs - no conflict possible

Fixed Challenges (1)

probes-drift: Changed liveness probe path from /health/live to /health/ready

  • /health/live returns 200 immediately → no crash loop
  • /health/ready returns 200 after 15s → actual crash loop during startup
  • User must discover the misconfiguration and fix the probe path

Impact

  • Challenges: 21 → 16
  • Average score: 12.7/20 → 16.1/20
  • Broken challenges: 7 → 2 (env-config, partial-outage need separate fixes)

Test plan

  • Verify deleted challenge directories are removed
  • Test probes-drift causes actual crash loop
  • Test probes-drift passes after fixing liveness probe path

🤖 Generated with Claude Code

DELETED (non-functional):
- tls-ingress: No TLS validation, no ingress controller
- cert-mismatch: No TLS validation, passes without action
- traffic-jam: Missing ingress-nginx controller dependency
- healthy-app: Cannot validate probe configuration was added
- volume-conflict: StatefulSet creates separate PVCs, no conflict possible

FIXED:
- probes-drift: Changed liveness probe to /health/ready (fails during startup)
  causing actual crash loop as described. User must fix probe path.

Remaining challenges: 16 (was 21)
Average score improvement: 12.7 -> 16.1/20

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@github-actions
Copy link

✅ Challenge Validation Passed

Validated challenges: 1

  • probes-drift

All challenge files are valid and ready for review! 🎉

@pbrissaud pbrissaud merged commit ddef3e3 into main Jan 25, 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