Skip to content

clarify ports for GKE users#22296

Merged
domalessi merged 5 commits intomasterfrom
domalessi/cilium-port
Jan 13, 2026
Merged

clarify ports for GKE users#22296
domalessi merged 5 commits intomasterfrom
domalessi/cilium-port

Conversation

@domalessi
Copy link
Copy Markdown
Contributor

What does this PR do?

Per this ticket, a customer on GKE experienced ~60 million Agent errors per month because the Datadog Agent was configured to scrape the default port (9090) instead of GKE's fixed port (9990). GKE does not allow users to change the port Cilium exposes metrics on, so the Agent must be configured accordingly.

I updated the Cilium integration documentation to clarify that the metrics port is environment-dependent, with specific guidance for GKE users:

  • Added an info alert in the Installation section noting that GKE-managed Cilium uses port 9990 (fixed and not configurable)
  • Expanded the Host Configuration note to list port requirements by environment
  • Added a GKE-specific example in the Containerized section showing the correct agent_endpoint configuration

Motivation

Review checklist (to be filled by reviewers)

  • Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
  • Add the qa/skip-qa label if the PR doesn't need to be tested during QA.
  • If you need to backport this PR to another branch, you can add the backport/<branch-name> label to the PR and it will automatically open a backport PR once this one is merged

Comment thread cilium/README.md Outdated
drichards-87
drichards-87 previously approved these changes Jan 9, 2026
Comment thread cilium/README.md
@temporal-github-worker-1 temporal-github-worker-1 Bot dismissed drichards-87’s stale review January 12, 2026 19:19

Review from drichards-87 is dismissed. Related teams and files:

  • documentation
    • cilium/README.md
Comment thread cilium/README.md Outdated
@github-actions
Copy link
Copy Markdown
Contributor

⚠️ Recommendation: Add qa/skip-qa label

This PR does not modify any files shipped with the agent.

To help streamline the release process, please consider adding the qa/skip-qa label if these changes do not require QA testing.

@domalessi domalessi added this pull request to the merge queue Jan 13, 2026
Merged via the queue into master with commit 9244867 Jan 13, 2026
43 of 46 checks passed
@domalessi domalessi deleted the domalessi/cilium-port branch January 13, 2026 22:45
lukepatrick pushed a commit to lukepatrick/integrations-core that referenced this pull request Mar 17, 2026
* clarify ports for GKE users

* Update cilium/README.md

* Update cilium/README.md

* Update cilium/README.md

---------

Co-authored-by: Steven Yuen <steven.yuen@datadoghq.com>
Signed-off-by: lukepatrick <lukephilips@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants