Skip to content

docs: ADR for tenant IDs#4

Open
cb80 wants to merge 1 commit intoopenkcm:mainfrom
cb80:adr/tenantID
Open

docs: ADR for tenant IDs#4
cb80 wants to merge 1 commit intoopenkcm:mainfrom
cb80:adr/tenantID

Conversation

@cb80
Copy link

@cb80 cb80 commented Mar 5, 2026

@hyperspace-insights
Copy link

Summary

The following content is AI-generated and provides a summary of the pull request:


ADR-104: Add Architecture Decision Record for Tenant IDs

Documentation

📄 Added a new Architecture Decision Record (ADR-104) defining the specification and constraints for tenant IDs in the OpenKCM project.

Changes

  • adr/tenant-id.md: New ADR document establishing tenant ID requirements, including:

    • Uniqueness & immutability: Tenant IDs must be globally unique and eternal (never reused)
    • Stability: IDs must remain stable across tenant moves
    • Format: Maximum 40 characters using URI unreserved characters (ALPHA / DIGIT / "-" / "." / "_" / "~")
    • Recommendation: Use lowercase UUID (RFC 4122 format), optionally prefixed with a short semantic string (e.g., kcm-658b416d-1f4a-41d9-b70c-59a2442b0724)
  • adr/authors.yml: Added Chris Burkert as a contributor entry.

Images and Links

Rendered version


  • 🔄 Regenerate and Update Summary
  • ✏️ Insert as PR Description (deletes this comment)
  • 🗑️ Delete comment
PR Bot Information

Version: 1.17.91 | 📖 Documentation | 🚨 Create Incident | 💬 Feedback

  • Event Trigger: issue_comment.created
  • LLM: anthropic--claude-4.6-sonnet
  • Summary Prompt: Default Prompt
  • Output Template: Default Template
  • Correlation ID: a5c2c220-18a9-11f1-8c4c-a14da82c5e7b

💌 Have ideas or want to contribute? Create an issue and share your thoughts with us!
📑 Check out the documentation for more information.
📬 Subscribe to the Hyperspace PR Bot DL to get the latest announcements and pilot features!

Made with ❤️ by Hyperspace.

@hyperspace-insights hyperspace-insights bot deleted a comment from anettlippert Mar 5, 2026
Copy link

@efrank-at-sap efrank-at-sap left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thank you!

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.

3 participants