docs(links): [#2185] repair Docker ACI references - #2231
Open
josecelano wants to merge 1 commit into
Open
josecelano wants to merge 1 commit into
josecelano wants to merge 1 commit into
Conversation
There was a problem hiding this comment.
Warning
Copilot couldn't run its full agentic review because it didn't start before the timeout. Make sure your repository has a runner available, or add a copilot-code-review.yml file specifying one with the runs-on attribute. See the docs for more details.
Pull request overview
This PR updates documentation to replace retired Docker Cloud ACI links with current Azure Container Instances references and records the repair evidence/review status for issue #2185.
Changes:
- Replaced two retired Docker Cloud ACI fragment links in
docs/containers.mdwith Microsoft Learn pages. - Updated the C6 category disposition/evidence in the external-link baseline documentation.
- Added reviewer evidence and updated issue tracking notes for pending hosted verification.
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
| docs/containers.md | Replaces retired Docker ACI links with current Azure documentation and updates the notes accordingly. |
| docs/issues/open/2185-2003-triage-advisory-external-link-check-findings/external-link-baseline.md | Updates C6 disposition/evidence and annotates baseline rows with the new replacements. |
| docs/issues/open/2185-2003-triage-advisory-external-link-check-findings/ISSUE.md | Expands T2/M3 tracking notes to reflect the C6 Docker repair pending hosted verification. |
| docs/issues/open/2185-2003-triage-advisory-external-link-check-findings/agent-review-reports.md | Adds an independent reviewer entry documenting inputs, evidence, findings, and follow-ups. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
| > - [There is no support for mounting a single file](https://docs.docker.com/cloud/aci-container-features/#persistent-volumes), or mounting a subfolder from an `Azure File Share`. | ||
| > - [ACI does not allow port mapping](https://docs.docker.com/cloud/aci-integration/#exposing-ports). | ||
| > - [An Azure Files mount obscures files or directories already present at its mount path](https://learn.microsoft.com/en-us/azure/container-instances/container-instances-volume-azure-files). | ||
| > - [ACI does not support Docker-style port mapping](https://learn.microsoft.com/en-us/azure/container-instances/container-instances-troubleshooting). |
| - **Pattern:** Docker Cloud ACI fragments, GitHub issue-comment fragments, and the Star History fragment returning `Cannot find fragment`. | ||
| - **Disposition:** Unresolved investigation. Unlike C1, these have distinct target-page semantics and must not be hidden by a broad fragment exclusion. | ||
| - **Next action:** Verify each fragment or replacement page individually; repair stale fragments, or propose a specific exclusion only if target rendering makes automated validation impossible. | ||
| - **Disposition:** Two Docker Cloud ACI fragments repaired pending hosted verification; GitHub issue-comment and Star History fragments remain under investigation. Unlike C1, these have distinct target-page semantics and must not be hidden by a broad fragment exclusion. |
| | --- | ----------- | ----------------------------------------- | ---------------------------------------------------------------------------------------------------------------- | | ||
| | T1 | DONE | Preserve and classify the baseline | `external-link-baseline.md` maps all 461 report errors to nine recurring categories and dispositions. | | ||
| | T2 | IN_PROGRESS | Repair clearly stale references | C3-C5 are hosted-verified; C6 remains under investigation. | | ||
| | T2 | IN_PROGRESS | Repair clearly stale references | C3-C5 are hosted-verified; the two retired Docker ACI fragments are repaired pending hosted verification, while C6's remaining fragments remain under investigation. | |
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
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
Files
Validation
Related to #2185