Skip to content

Restore Helix Docker-related diagnostics info#7141

Merged
karolz-ms merged 1 commit intomainfrom
dev/karolz/helix-container-diag
Jan 17, 2025
Merged

Restore Helix Docker-related diagnostics info#7141
karolz-ms merged 1 commit intomainfrom
dev/karolz/helix-container-diag

Conversation

@karolz-ms
Copy link
Copy Markdown
Contributor

Description

Restores the output of some Docker-related diagnostics command for Helix-based runs. Will help us improve test stability.

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 (N/A)
  • 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?

@karolz-ms karolz-ms requested a review from davidfowl January 17, 2025 18:08
@karolz-ms karolz-ms enabled auto-merge (squash) January 17, 2025 18:08
@karolz-ms karolz-ms mentioned this pull request Jan 17, 2025
18 tasks
@karolz-ms karolz-ms merged commit f18b61f into main Jan 17, 2025
@karolz-ms karolz-ms deleted the dev/karolz/helix-container-diag branch January 17, 2025 19:15
@github-actions github-actions Bot locked and limited conversation to collaborators Feb 24, 2025
@github-actions github-actions Bot added the needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners label Mar 10, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants