Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update pod-identities.md #777

Closed
wants to merge 1 commit into from
Closed

Conversation

Ramneek-kalra
Copy link

Summary:
Updating Pod Identity Considerations in case when customer might be using "Security Group of Pods" as well alongside Agent.

Issue #, if available:
Just an additional consideration when customer might be using "Security Group for Pods" along with Pod Identity Agent. This is to ensure that there's no breakage in the Agent working (As also found via replication).

Description of changes:
Added one additional consideration as below:

  • If you're using Security Group of Pods along with Pod Identity Agent, please have "POD_SECURITY_GROUP_ENFORCING_MODE" Flag in VPC CNI as per considerations stated at: Security Group of Pods.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Updating Pod Identity Considerations in case when customer might be using "Security Group of Pods" as well alongside Agent.
@geoffcline geoffcline self-requested a review July 15, 2024 17:08
@fincd-aws fincd-aws self-assigned this Jul 22, 2024
@geoffcline
Copy link
Contributor

Hello! 👋

I've captured your suggested changes in our internal system as we prepare to migrate this repository from Markdown to AsciiDoc format. We'll be offline for about a week during this transition, so I'm closing this PR for housekeeping purposes.

Your contribution will be included when we republish with the new format. Thank you for helping improve our AWS documentation!

@geoffcline geoffcline closed this Oct 29, 2024
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.

3 participants