Skip to content

[CORE-11642] Add note about enabled ANNOTATE_POD_IP setting to EKS guide. #2174

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

fasaxc
Copy link
Member

@fasaxc fasaxc commented Jul 16, 2025

Product Version(s):

All

Issue:

https://tigera.atlassian.net/browse/CORE-11642

Link to docs preview:

https://deploy-preview-2174--calico-docs-preview-next.netlify.app/calico/next/getting-started/kubernetes/managed-public-cloud/eks/

SME review:

  • An SME has approved this change.

DOCS review:

  • A member of the docs team has approved this change.

Additional information:

Merge checklist:

  • Deploy preview inspected wherever changes were made
  • Build completed successfully
  • Test have passed

@fasaxc fasaxc requested a review from a team as a code owner July 16, 2025 14:44
Copy link

netlify bot commented Jul 16, 2025

Deploy Preview succeeded!

Name Link
🔨 Latest commit 9ae943f
🔍 Latest deploy log https://app.netlify.com/projects/tigera/deploys/687e71aa8d0b670008f76607
😎 Deploy Preview https://deploy-preview-2174--tigera.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 26 (no change from production)
Accessibility: 90 (no change from production)
Best Practices: 92 (no change from production)
SEO: 92 (no change from production)
PWA: -
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify project configuration.

Copy link

netlify bot commented Jul 16, 2025

Deploy Preview for calico-docs-preview-next ready!

Name Link
🔨 Latest commit 9ae943f
🔍 Latest deploy log https://app.netlify.com/projects/calico-docs-preview-next/deploys/687e71aa433c4c0008272059
😎 Deploy Preview https://deploy-preview-2174--calico-docs-preview-next.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@@ -38,6 +38,7 @@ When using the Amazon VPC CNI plugin, $[prodname] does not support enforcement o
***Prerequisites***

* You [disabled network policy for the AWS VPC CNI](https://docs.aws.amazon.com/eks/latest/userguide/network-policy-disable.html).
Copy link
Member Author

Choose a reason for hiding this comment

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

Hmm, actually, how can this be a pre-req when you've not created your cluster yet!

Copy link
Collaborator

Choose a reason for hiding this comment

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

I noticed this recently and have draft changes in the works. It also seems that network policy is disabled by default, so it may be altogether unnecessary.

Copy link
Member Author

Choose a reason for hiding this comment

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

I shuffled it around; any better (or do you want to take it over as part of your rework?)

@@ -38,6 +38,7 @@ When using the Amazon VPC CNI plugin, $[prodname] does not support enforcement o
***Prerequisites***

* You [disabled network policy for the AWS VPC CNI](https://docs.aws.amazon.com/eks/latest/userguide/network-policy-disable.html).
* You [configured AWS VPC CNI to annotate Pods with their IPs](https://github.com/aws/amazon-vpc-cni-k8s?tab=readme-ov-file#annotate_pod_ip-v193). Note the requirement to grant the "patch" permission to the `aws-node` daemon set. Without this setting, pod IPs can propagate slowly when Kubernetes is under load resulting in slow policy application after pod creation.
Copy link
Collaborator

Choose a reason for hiding this comment

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

I feel like we must encounter the same problem here as with the other prerequisite. Is this something that needs to come after the cluster is created?

Copy link
Member Author

Choose a reason for hiding this comment

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

Yes, they're both settings you must make when configuring AWS CNI. I think the policy one can be done at cluster creation time by passing an option to eksctl

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