Skip to content

RFC: Drift detection tests for CRD-to-runtime type mappings#65

Open
ChrisJBurns wants to merge 1 commit intomainfrom
rfc/thv-0058-crd-drift-detection-tests
Open

RFC: Drift detection tests for CRD-to-runtime type mappings#65
ChrisJBurns wants to merge 1 commit intomainfrom
rfc/thv-0058-crd-drift-detection-tests

Conversation

@ChrisJBurns
Copy link
Copy Markdown
Contributor

Summary

  • Proposes reflection-based drift detection tests for CRD-to-runtime conversion functions in the ToolHive operator
  • Tests enumerate JSON fields on both CRD and runtime types, failing when a field is neither mapped nor explicitly excluded
  • Catches silent configuration drift (e.g., PR #3118, issue #3125) at test time instead of in production

Documents

  • rfcs/THV-0058-crd-drift-detection-tests.md — concise proposal
  • rfcs/THV-XXXX-crd-runtime-drift-detection-tests.md — full RFC with detailed design, alternatives, and implementation plan

Test plan

  • Review both documents for accuracy and completeness
  • Validate the proposed jsonFieldNames helper design
  • Confirm the scope covers all relevant CRD conversion boundaries

🤖 Generated with Claude Code

Add reflection-based drift detection tests alongside CRD conversion
functions to catch silent field mapping drift between CRD types and
their runtime counterparts. Includes both a concise proposal and a
full RFC document.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
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