Repository navigation
Link GitLab export logs docs in export failure error message - #1615
Conversation
Per team feedback (github#1607), rather than building a diagnostics/log-fetching command into GEI, point users to GitLab's own export/log documentation when an export fails so they can self-diagnose. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Unit Test Results 1 files 1 suites 24s ⏱️ Results for commit afe2fc8. ♻️ This comment has been updated with latest results. |
There was a problem hiding this comment.
Copilot review overview
🟡 Changes recommended
The URL fragment is invalid, and the changed failure message lacks test coverage.
Get a fresh assessment by requesting another Copilot review.
Review effort: Balanced
Findings: 1
Open (2)
What changed in this PR
Makes GitLab export failures more actionable without adding diagnostics tooling.
Changes:
- Adds GitLab log documentation guidance to the export failure message.
- Adds a release note.
| File | Description |
|---|---|
src/gl2gh/Commands/MigrateRepo/MigrateRepoCommandHandler.cs |
Adds documentation link to export failure. |
RELEASENOTES.md |
Documents the improved error guidance. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
brianaj
left a comment
brianaj
left a comment
There was a problem hiding this comment.
Thanks for adding this change! One quick comment, not sure how many customers are coming from gitlab dotcom but don't want this to confuse them.
|
Summary
Per discussion on #1607, the team decided not to build a diagnostics/log-fetching command into GEI for GitLab export failures. Instead, this makes the existing generic error message actionable by pointing users at GitLab's own log documentation, so they can self-diagnose GitLab-side export failures or escalate to GitLab support.
Change
gl2gh migrate-repo's export failure now reads:Previously it only said:
This follows the same lightweight pattern as #1614: a focused error-message improvement plus a release note, without adding a new diagnostics surface area to GEI.
ThirdPartyNotices.txt(if applicable) -- N/AValidation
dotnet build src/OctoshiftCLI.sln /p:TreatWarningsAsErrors=truedotnet test src/OctoshiftCLI.Tests/OctoshiftCLI.Tests.csproj --filter 'FullyQualifiedName~MigrateRepoCommandHandlerTests'(110 passed)dotnet format src/OctoshiftCLI.sln(no changes needed)