Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Nov 27, 2025

Description

Addresses review feedback from #13216 to add documentation showing the expected JSON format from docker compose ps --format json.

Added a comment in PrintEndpointsAsync showing:

  • Example NDJSON output (one JSON object per line per container)
  • Which fields are parsed: Service, Publishers[].TargetPort, Publishers[].PublishedPort
// Example output format:
// {"ID":"abc123","Name":"myapp-api-1","Command":"dotnet MyApp.dll","Project":"myapp","Service":"api","State":"running","Health":"","ExitCode":0,"Publishers":[{"URL":"","TargetPort":8080,"PublishedPort":5000,"Protocol":"tcp"}]}

Checklist

  • Is this feature complete?
    • Yes. Ready to ship.
    • No. Follow-up changes expected.
  • Are you including unit tests for the changes and scenario tests if relevant?
    • Yes
    • No
  • Did you add public API?
    • Yes
      • If yes, did you have an API Review for it?
        • Yes
        • No
      • Did you add <remarks /> and <code /> elements on your triple slash comments?
        • Yes
        • No
    • No
  • Does the change make any security assumptions or guarantees?
    • Yes
      • If yes, have you done a threat model and had a security review?
        • Yes
        • No
    • No
  • Does the change require an update in our Aspire docs?

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Copilot AI changed the title [WIP] Update resource endpoints display for Docker Compose deploy Add comment showing expected docker compose ps JSON format Nov 27, 2025
Copilot AI requested a review from davidfowl November 27, 2025 07:15
Copilot finished work on behalf of davidfowl November 27, 2025 07:15
@davidfowl davidfowl closed this Nov 27, 2025
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.

2 participants