Skip to content

DOC-399: Document Kubernetes endpoint injection - #871

Open
HarshCasper wants to merge 2 commits into
mainfrom
doc-399-doc-k8s-operator-transparently-inject-localstack-endpoints
Open

DOC-399: Document Kubernetes endpoint injection#871
HarshCasper wants to merge 2 commits into
mainfrom
doc-399-doc-k8s-operator-transparently-inject-localstack-endpoints

Conversation

@HarshCasper

@HarshCasper HarshCasper commented Aug 17, 2026

Copy link
Copy Markdown
Member

Summary

  • document opt-in DNS and AWS_ENDPOINT_URL injection for Kubernetes workloads
  • explain fail-open behavior, prerequisites, and the default cluster-domain limitation
  • add webhook certificate management and EKS Auto Mode configuration

Explain workload opt-in behavior, certificate management, and EKS Auto Mode so users can configure transparent LocalStack access safely.
@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Aug 17, 2026

Copy link
Copy Markdown

Deploying localstack-docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: 14f91f8
Status: ✅  Deploy successful!
Preview URL: https://2c81a644.localstack-docs.pages.dev
Branch Preview URL: https://doc-399-doc-k8s-operator-tra.localstack-docs.pages.dev

View logs

@simonrw simonrw left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks really good thanks! Just a couple of minor points

Comment thread src/content/docs/aws/customization/kubernetes/kubernetes-operator.mdx Outdated
Comment thread src/content/docs/aws/customization/kubernetes/kubernetes-operator.mdx Outdated
Comment thread src/content/docs/aws/customization/kubernetes/kubernetes-operator.mdx Outdated
Comment on lines +329 to +330
The `dnsConfigName` and `dnsConfigNamespace` fields remain required by the `LocalStack` resource schema, but the Operator does not use their values in `eksauto` mode.
Use opt-in [endpoint injection](#inject-localstack-endpoints-into-workloads) to configure workloads that need to access LocalStack.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

We should really update this!

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

What do we update and what should we update it to? :)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

No I mean this is weird behaviour in the operator. Why should the user set values which do not apply to eksauto mode when running in eksauto mode. I don't think it's a huge problem right now, but the we in this context is @localstack/deployx - the operator maintainers.

- Consolidate the dnsPolicy/nameserver/search-domain/ndots bullets
  into one higher-level explanation that also reassures readers
  LocalStack only intercepts domains it owns (localhost.localstack.cloud
  or $LOCALSTACK_HOST) and forwards everything else to the cluster
  DNS resolver as normal.
- Drop the cert-manager SelfSigned-issuer trust-anchor explanation as
  unnecessary cert-manager implementation detail for the operator docs.
@quetzalliwrites

Copy link
Copy Markdown
Collaborator

Thanks @simonrw — pushed 14f91f81 addressing the DNS-interception reassurance and the ndots/nameserver implementation-detail simplification (consolidated into one bullet, per your suggested wording), and dropped the SelfSigned-issuer trust-anchor explanation as unnecessary cert-manager detail.

Left the EKS Auto Mode comment ("We should really update this!") unaddressed for now — wasn't clear whether that's about the docs wording or the dnsConfigName/dnsConfigNamespace field-requirement behavior itself (the #inject-localstack-endpoints-into-workloads link isn't broken). Happy to follow up once there's more detail on what should change.

simonrw commented Aug 26, 2026

Copy link
Copy Markdown
Contributor

This looks great, thanks! Happy for this to be merged. I've addressed that comment

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