Skip to content

docs: document portmap binary requirements#41300

Merged
joestringer merged 1 commit intocilium:mainfrom
nbusseneau:pr/docs-portmap-requirement
Aug 21, 2025
Merged

docs: document portmap binary requirements#41300
joestringer merged 1 commit intocilium:mainfrom
nbusseneau:pr/docs-portmap-requirement

Conversation

@nbusseneau
Copy link
Copy Markdown
Member

Some distributions (e.g. AWS EKS clusters without AWS VPC CNI plugin) do not install the portmap binary on the nodes, leading to confusion when trying to use the portmap plugin. This commit documents the requirement and hints at a solution for providing binaries if needed.

@nbusseneau nbusseneau requested a review from a team as a code owner August 20, 2025 14:44
@nbusseneau nbusseneau added the area/documentation Impacts the documentation, including textual changes, sphinx, or other doc generation code. label Aug 20, 2025
@nbusseneau nbusseneau requested a review from joestringer August 20, 2025 14:44
@maintainer-s-little-helper maintainer-s-little-helper bot added the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Aug 20, 2025
@nbusseneau nbusseneau force-pushed the pr/docs-portmap-requirement branch from e4272a6 to 88b1e23 Compare August 20, 2025 14:46
Copy link
Copy Markdown
Member

@joestringer joestringer left a comment

Choose a reason for hiding this comment

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

Minor nit for docs-structure consistency. Better to just update the instructions to tell the user what to do rather than adding to the reader fatigue by adding yet another box that may or may not be relevant to them.

@joestringer joestringer added release-note/misc This PR makes changes that have no direct user impact. needs-backport/1.16 needs-backport/1.17 This PR / issue needs backporting to the v1.17 branch needs-backport/1.18 This PR / issue needs backporting to the v1.18 branch labels Aug 20, 2025
@maintainer-s-little-helper maintainer-s-little-helper bot removed dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. labels Aug 20, 2025
Some distributions (e.g. AWS EKS clusters without AWS VPC CNI plugin) do
not install the `portmap` binary on the nodes, leading to confusion when
trying to use the portmap plugin. This commit documents the requirement
and hints at a solution for providing binaries if needed.

Co-authored-by: Joe Stringer <joe@cilium.io>
Signed-off-by: Nicolas Busseneau <nicolas@isovalent.com>
@nbusseneau nbusseneau force-pushed the pr/docs-portmap-requirement branch from 88b1e23 to f3eb096 Compare August 21, 2025 15:33
@nbusseneau nbusseneau requested a review from joestringer August 21, 2025 15:34
@joestringer joestringer enabled auto-merge August 21, 2025 15:44
@joestringer
Copy link
Copy Markdown
Member

/test

@joestringer joestringer added dont-merge/bad-bot To prevent MLH from marking ready-to-merge. and removed dont-merge/bad-bot To prevent MLH from marking ready-to-merge. labels Aug 21, 2025
@joestringer joestringer added this pull request to the merge queue Aug 21, 2025
Merged via the queue into cilium:main with commit ec378db Aug 21, 2025
66 of 67 checks passed
@maintainer-s-little-helper maintainer-s-little-helper bot added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Aug 21, 2025
@pippolo84 pippolo84 mentioned this pull request Aug 25, 2025
17 tasks
@pippolo84 pippolo84 added backport-pending/1.18 The backport for Cilium 1.18.x for this PR is in progress. and removed needs-backport/1.18 This PR / issue needs backporting to the v1.18 branch labels Aug 25, 2025
@pippolo84 pippolo84 mentioned this pull request Aug 25, 2025
7 tasks
@pippolo84 pippolo84 added backport-pending/1.17 The backport for Cilium 1.17.x for this PR is in progress. and removed needs-backport/1.17 This PR / issue needs backporting to the v1.17 branch labels Aug 25, 2025
@pippolo84 pippolo84 mentioned this pull request Aug 25, 2025
6 tasks
@github-actions github-actions bot added backport-done/1.16 The backport for Cilium 1.16.x for this PR is done. backport-done/1.18 The backport for Cilium 1.18.x for this PR is done. backport-done/1.17 The backport for Cilium 1.17.x for this PR is done. and removed backport-pending/1.16 backport-pending/1.18 The backport for Cilium 1.18.x for this PR is in progress. backport-pending/1.17 The backport for Cilium 1.17.x for this PR is in progress. labels Sep 1, 2025
@cilium-release-bot cilium-release-bot bot moved this to Released in cilium v1.19.0 Feb 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/documentation Impacts the documentation, including textual changes, sphinx, or other doc generation code. backport-done/1.16 The backport for Cilium 1.16.x for this PR is done. backport-done/1.17 The backport for Cilium 1.17.x for this PR is done. backport-done/1.18 The backport for Cilium 1.18.x for this PR is done. ready-to-merge This PR has passed all tests and received consensus from code owners to merge. release-note/misc This PR makes changes that have no direct user impact.

Projects

No open projects
Status: Released

Development

Successfully merging this pull request may close these issues.

4 participants