Skip to content

Clarify ingress-nginx in-cluster config warning is benign#103

Draft
bwalsh with Copilot wants to merge 19 commits into
developmentfrom
copilot/sub-pr-92-again
Draft

Clarify ingress-nginx in-cluster config warning is benign#103
bwalsh with Copilot wants to merge 19 commits into
developmentfrom
copilot/sub-pr-92-again

Conversation

Copilot AI commented Nov 26, 2025

Copy link
Copy Markdown
Contributor

User observed a warning in ingress-nginx controller logs: Neither --kubeconfig nor --master was specified. Using the inClusterConfig. This might not work.

This is standard Kubernetes client-go behavior when controllers run as pods. The warning message is misleading—in-cluster configuration (via service account tokens at /var/run/secrets/kubernetes.io/serviceaccount/) is the expected and correct authentication method for in-cluster workloads.

Changes:

  • Clarified that the warning is benign and requires no action
  • Confirmed ingress-nginx is using the appropriate authentication method for pod-based controllers

No code changes needed—this is working as designed.


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI and others added 19 commits November 24, 2025 15:51
* Initial plan

* Add comprehensive Copilot instruction files

Co-authored-by: bwalsh <[email protected]>

* Add README and validation script for Copilot instructions

Co-authored-by: bwalsh <[email protected]>

---------

Co-authored-by: copilot-swe-agent[bot] <[email protected]>
Co-authored-by: bwalsh <[email protected]>
…ized auth (#93)

* Initial plan

* Implement ingress-authz-overlay chart per issue #91

Co-authored-by: bwalsh <[email protected]>

* Address code review: remove unused helpers, add container security context

Co-authored-by: bwalsh <[email protected]>

* Add Let's Encrypt / cert-manager documentation to user guide

Co-authored-by: bwalsh <[email protected]>

---------

Co-authored-by: copilot-swe-agent[bot] <[email protected]>
Co-authored-by: bwalsh <[email protected]>
* Initial plan

* Add documentation for Let's Encrypt ACME account key secrets

Co-authored-by: bwalsh <[email protected]>

* Add cert-manager installation instructions and troubleshooting

Co-authored-by: bwalsh <[email protected]>

* Fix installation order list formatting

Co-authored-by: bwalsh <[email protected]>

* Add troubleshooting for Helm ownership conflict with ClusterIssuer

Co-authored-by: bwalsh <[email protected]>

* Improve Helm ownership conflict documentation clarity

Co-authored-by: bwalsh <[email protected]>

---------

Co-authored-by: copilot-swe-agent[bot] <[email protected]>
Co-authored-by: bwalsh <[email protected]>
…rlay (#96)

* Initial plan

* Fix cert-manager certificate ownership conflict by using primary route flag

Co-authored-by: bwalsh <[email protected]>

---------

Co-authored-by: copilot-swe-agent[bot] <[email protected]>
Co-authored-by: bwalsh <[email protected]>
* Initial plan

* Configure overlay to use centralized authz-adapter in security namespace

Co-authored-by: bwalsh <[email protected]>

---------

Co-authored-by: copilot-swe-agent[bot] <[email protected]>
Co-authored-by: bwalsh <[email protected]>
…figuration documentation (#98)

* Initial plan

* Add Ingress and Connectivity troubleshooting section to docs

Co-authored-by: bwalsh <[email protected]>

* Add AWS EKS and on-premises ingress configuration docs

Co-authored-by: bwalsh <[email protected]>

* Fix code review issues in ingress docs

Co-authored-by: bwalsh <[email protected]>

---------

Co-authored-by: copilot-swe-agent[bot] <[email protected]>
Co-authored-by: bwalsh <[email protected]>
…#99)

* Initial plan

* Fix github-repo-registrations-eventsource ingress to use spec.ingressClassName instead of deprecated annotation

Co-authored-by: bwalsh <[email protected]>

* Add quote to ingressClassName for proper YAML parsing

Co-authored-by: bwalsh <[email protected]>

---------

Co-authored-by: copilot-swe-agent[bot] <[email protected]>
Co-authored-by: bwalsh <[email protected]>
* Initial plan

* Add cross-namespace routing support via ExternalName services

Co-authored-by: bwalsh <[email protected]>

* Add cross-namespace routing troubleshooting documentation

Co-authored-by: bwalsh <[email protected]>

---------

Co-authored-by: copilot-swe-agent[bot] <[email protected]>
Co-authored-by: bwalsh <[email protected]>
… cert-manager issues (#101)

* Initial plan

* Add troubleshooting docs for external connection issues when internal services work

Co-authored-by: bwalsh <[email protected]>

* Add kind cluster and cert-manager troubleshooting to docs

Co-authored-by: bwalsh <[email protected]>

* Add DNS-01 challenge docs for No-IP.com and kind clusters

Co-authored-by: bwalsh <[email protected]>

---------

Co-authored-by: copilot-swe-agent[bot] <[email protected]>
Co-authored-by: bwalsh <[email protected]>
…, and NodePort external IP configuration to troubleshooting docs (#102)

* Initial plan

* Add comprehensive DNS-01 challenge debugging guide with propagation error fixes

Co-authored-by: bwalsh <[email protected]>

* Add manual certificate installation guide to troubleshooting docs

Co-authored-by: bwalsh <[email protected]>

---------

Co-authored-by: copilot-swe-agent[bot] <[email protected]>
Co-authored-by: bwalsh <[email protected]>
Copilot AI mentioned this pull request Nov 26, 2025
Copilot AI changed the title [WIP] Add authz-aware ingress overlay for multi-tenant UIs and APIs Clarify ingress-nginx in-cluster config warning is benign Nov 26, 2025
Copilot AI requested a review from bwalsh November 26, 2025 01:24
Base automatically changed from feature/ingress to development December 4, 2025 20:23
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