docs: Design docs for review#204
Conversation
danielpanzella
commented
Jun 4, 2026
- Infrastructure version enforcement driven by wandb server manifest
- Enable multiple instances of each infra type
- Version compatability enforcement for operator and server manifest
Adds an architectural design doc and a detailed implementation plan for making the W&B server manifest the source of truth for per-component infrastructure versions: a managed-infra deploy target and an external-infra minimum, with the validating webhook hard-rejecting upgrades whose external infrastructure is below the floor. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
v2 is still in alpha, so the existing user-facing version field on ManagedClickHouseSpec can be dropped without a deprecation window. Removes the tombstoning language from both docs and trims the matching open question. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
…anielpanzella/design-docs
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
| ## 8. Open questions | ||
|
|
||
| - **MinIO/S3 version reporting.** Real S3 has no version. For MinIO we can read the `Server` header; for AWS/GCS we should record a marker (e.g. `"s3"` or `"gcs"`) and skip the `minVersion` check. The exact marker scheme should be agreed before implementation. | ||
| - **Kafka version detection.** `ApiVersions` returns supported API ranges, not a clean broker version string. We will likely have to derive a representative version from the highest supported APIs, or use vendor-specific endpoints. Open as to which library to use. |
There was a problem hiding this comment.
I don't know a lot about bufstream, but if it's the long-term way forward, will this approach work similarly?