Skip to content

docs(ecs): make correction to documentation for windows containers network mode support#35007

Merged
mergify[bot] merged 1 commit intoaws:mainfrom
evzzk:evzz-docs-correction-nat-windows
Jul 18, 2025
Merged

docs(ecs): make correction to documentation for windows containers network mode support#35007
mergify[bot] merged 1 commit intoaws:mainfrom
evzzk:evzz-docs-correction-nat-windows

Conversation

@evzzk
Copy link
Copy Markdown
Contributor

@evzzk evzzk commented Jul 17, 2025

Issue # (if applicable)

Closes #35006.

Reason for this change

The documentation for NetworkMode.NAT enum incorrectly states that NAT is "the only supported network mode for Windows containers". This is no longer accurate as Windows containers also support the awsvpc network mode according to AWS documentation.

Description of changes

  • Removed the incorrect sentence stating "This is the only supported network mode for Windows containers."
  • Updated the description of NAT mode to clarify that it's "Docker's built-in virtual network which runs inside each Windows container instance" without implying exclusivity.

Describe any new or updated permissions being added

None. This change is documentation-only and doesn't modify any code logic or permissions.

Description of how you validated changes

None

Checklist


By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license

@aws-cdk-automation aws-cdk-automation requested a review from a team July 17, 2025 18:00
@github-actions github-actions bot added valued-contributor [Pilot] contributed between 6-12 PRs to the CDK p2 labels Jul 17, 2025
@mergify mergify bot added the contribution/core This is a PR that came from AWS. label Jul 17, 2025
Copy link
Copy Markdown
Collaborator

@aws-cdk-automation aws-cdk-automation left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(This review is outdated)

@evzzk evzzk changed the title (aws-ecs): make correction to documentation for Windows containers network mode support docs(aws-ecs): make correction to documentation for Windows containers network mode support Jul 17, 2025
@evzzk evzzk changed the title docs(aws-ecs): make correction to documentation for Windows containers network mode support docs(aws-ecs): make correction to documentation for windows containers network mode support Jul 17, 2025
@aws-cdk-automation
Copy link
Copy Markdown
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: AutoBuildv2Project1C6BFA3F-wQm2hXv2jqQv
  • Commit ID: 77aade7
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@ozelalisen ozelalisen changed the title docs(aws-ecs): make correction to documentation for windows containers network mode support docs(ecs): make correction to documentation for windows containers network mode support Jul 18, 2025
@ozelalisen ozelalisen self-assigned this Jul 18, 2025
@aws-cdk-automation aws-cdk-automation dismissed their stale review July 18, 2025 08:06

✅ Updated pull request passes all PRLinter validations. Dismissing previous PRLinter review.

@aws-cdk-automation aws-cdk-automation added the pr/needs-maintainer-review This PR needs a review from a Core Team Member label Jul 18, 2025
Copy link
Copy Markdown
Member

@ozelalisen ozelalisen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@aws-cdk-automation aws-cdk-automation removed the pr/needs-maintainer-review This PR needs a review from a Core Team Member label Jul 18, 2025
@mergify
Copy link
Copy Markdown
Contributor

mergify bot commented Jul 18, 2025

Thank you for contributing! Your pull request will be updated from main and then merged automatically (do not update manually, and be sure to allow changes to be pushed to your fork).

@mergify mergify bot merged commit 45e4cb3 into aws:main Jul 18, 2025
53 checks passed
@github-actions
Copy link
Copy Markdown
Contributor

Comments on closed issues and PRs are hard for our team to see.
If you need help, please open a new issue that references this one.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jul 18, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

contribution/core This is a PR that came from AWS. p2 valued-contributor [Pilot] contributed between 6-12 PRs to the CDK

Projects

None yet

Development

Successfully merging this pull request may close these issues.

(aws-ecs): make correction to documentation for Windows containers network mode support

3 participants