Skip to content

cilium: simplify hostport's genServiceMappings#12473

Merged
borkmann merged 1 commit intomasterfrom
pr/simplify-hostport-gen
Jul 9, 2020
Merged

cilium: simplify hostport's genServiceMappings#12473
borkmann merged 1 commit intomasterfrom
pr/simplify-hostport-gen

Conversation

@borkmann
Copy link
Copy Markdown
Member

@borkmann borkmann commented Jul 9, 2020

See commit msg.

Simplify the code which handles HostPort's frontend/backend generation
to make it less complex.

Suggested-by: Andre Martins <andre@cilium.io>
Signed-off-by: Daniel Borkmann <daniel@iogearbox.net>
@borkmann borkmann added area/daemon Impacts operation of the Cilium daemon. release-note/misc This PR makes changes that have no direct user impact. area/kube-proxy-free labels Jul 9, 2020
@borkmann borkmann requested review from a team and aanm July 9, 2020 14:13
@borkmann
Copy link
Copy Markdown
Member Author

borkmann commented Jul 9, 2020

test-me-please

@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage decreased (-0.01%) to 36.923% when pulling 0bc3ec1 on pr/simplify-hostport-gen into 4814e1c on master.

@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 Jul 9, 2020
@borkmann borkmann merged commit 93e4845 into master Jul 9, 2020
@borkmann borkmann deleted the pr/simplify-hostport-gen branch July 9, 2020 16:59
@brb brb mentioned this pull request Jul 15, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/daemon Impacts operation of the Cilium daemon. 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

None yet

Development

Successfully merging this pull request may close these issues.

5 participants