Skip to content

test: Test NodePort BPF with L7 proxy#12434

Merged
nebril merged 1 commit intomasterfrom
pr/brb/nodeport-l7
Jul 8, 2020
Merged

test: Test NodePort BPF with L7 proxy#12434
nebril merged 1 commit intomasterfrom
pr/brb/nodeport-l7

Conversation

@brb
Copy link
Copy Markdown
Member

@brb brb commented Jul 6, 2020 •
edited
Loading

#11899 seems to fix NodePort BPF with L7. This PR adds L7 policy tests to NodePort BPF suites.

Fix #8971

@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 Jul 6, 2020
@brb
Copy link
Copy Markdown
Member Author

brb commented Jul 6, 2020

test-focus K8sService*

@brb brb mentioned this pull request Jul 6, 2020
Closed
@brb brb force-pushed the pr/brb/nodeport-l7 branch from 68685e9 to 9f6489b Compare July 8, 2020 09:55
@brb brb changed the title WIP: validate BPF NodePort with L7 policy test: Test NodePort BPF with L7 proxy Jul 8, 2020
@brb brb added area/CI Continuous Integration testing issue or flake pending-review release-note/ci This PR makes changes to the CI. labels Jul 8, 2020
@maintainer-s-little-helper maintainer-s-little-helper Bot removed the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Jul 8, 2020
@brb brb marked this pull request as ready for review July 8, 2020 09:57
@brb brb requested a review from a team as a code owner July 8, 2020 09:57
@brb
Copy link
Copy Markdown
Member Author

brb commented Jul 8, 2020

test-me-please

@brb brb force-pushed the pr/brb/nodeport-l7 branch from 9f6489b to c7ef841 Compare July 8, 2020 09:58
@brb
Copy link
Copy Markdown
Member Author

brb commented Jul 8, 2020

test-me-please

@brb
test: Test NodePort BPF with L7 proxy
c7ef841 
#11899 made it possible to run
NodePort BPF with L7 policies.

Signed-off-by: Martynas Pumputis <m@lambda.lt>
@coveralls
Copy link
Copy Markdown

coveralls commented Jul 8, 2020 •
edited
Loading

Coverage Status

Coverage decreased (-0.004%) to 36.935% when pulling c7ef841 on pr/brb/nodeport-l7 into a091ba1 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 8, 2020
@nebril nebril merged commit 9cbe915 into master Jul 8, 2020
@nebril nebril deleted the pr/brb/nodeport-l7 branch July 8, 2020 12:10
@joestringer joestringer mentioned this pull request Oct 12, 2020
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@nebril nebril nebril approved these changes

Assignees

No one assigned

Labels

area/CI Continuous Integration testing issue or flake ready-to-merge This PR has passed all tests and received consensus from code owners to merge. release-note/ci This PR makes changes to the CI.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

NodePort BPF service cannot be reached when L7 policy is applied / L7 visibility is enabled

3 participants

@brb @coveralls @nebril