-
Notifications
You must be signed in to change notification settings - Fork 90
Populate port mapping name to make endpoint port mapping work. #520
Conversation
Welcome @mneverov! |
question: should I remove |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: jayunit100, mneverov The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
PR needs rebase. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
The Kubernetes project currently lacks enough contributors to adequately respond to all PRs. This bot triages PRs according to the following rules:
You can:
Please send feedback to sig-contributor-experience at kubernetes/community. /lifecycle stale |
New changes are detected. LGTM label has been removed. |
This change has already landed within #532. |
What kind of PR is this?
Bug fix: previously, port name was not populated that prevented endpoint port mapping to work correctly.
Why this PR is needed / What this PR do?
This PR fix the issue above.
Which issue(s) this PR fixes?
Fixes #
Fixes tests:
should be updated after adding or deleting ports
andshould serve multiport endpoints from pods [Conformance]
ine2e-ipv4-iptables
mode.Additional information about this PR