docs(rfc): host in change ID URI format#326
Closed
JamyDev wants to merge 1 commit into
Closed
Conversation
Summary: Intent: - Embed the target host (domain/port) in every change ID URI so change identity is self-contained across multiple GitHub/GHES/Phabricator instances Changes: - Added RFC at doc/rfc/submitqueue/host-in-change-id.md covering the new URI format, migration path, stacked-change validation, and alternatives considered - Updated doc/rfc/index.md to link the new RFC ## Summary Adds an RFC documenting the design decision to embed the host (domain/port) in GitHub, GHES, and Phabricator change ID URIs. The old URI format encoded everything about a change except which server it lives on, making it impossible to distinguish changes across multiple instances of the same provider type (e.g. multiple GHES instances). The RFC covers the new URI format, migration strategy, stacked-change validation, and alternatives considered. ## Test Plan Existing CI/Build validation ## Revert Plan git revert ## Jira Issues ## API Changes No API changes ## Monitoring and Alerts Standard monitoring via existing dashboards 🤖 Generated by minion Co-Authored-By: minion-bot <noreply@minion.uber.com> Co-Authored-By: jamy@uber.com Agent: claude-code | Task: 87c9d30e-c740-416a-9bce-201e86b2f14c View your task: https://minion.uberinternal.com/task/87c9d30e-c740-416a-9bce-201e86b2f14c Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Adds an RFC documenting the design decision to embed the host (domain/port) in GitHub, GHES, and Phabricator change ID URIs. The old URI format encoded everything about a change except which server it lives on, making it impossible to distinguish changes across multiple instances of the same provider type (e.g. multiple GHES instances). The RFC covers the new URI format, migration strategy, stacked-change validation, and alternatives considered.
Test Plan
Existing CI/Build validation
Revert Plan
git revert
Jira Issues
API Changes
No API changes
Monitoring and Alerts
Standard monitoring via existing dashboards
🤖 Generated by minion
Co-Authored-By: minion-bot noreply@minion.uber.com
Co-Authored-By: jamy@uber.com
Agent: claude-code | Task: 87c9d30e-c740-416a-9bce-201e86b2f14c
View your task: https://minion.uberinternal.com/task/87c9d30e-c740-416a-9bce-201e86b2f14c