Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ci(workflows/bpf-test): add BPF test to cover route logic #671

Merged
merged 21 commits into from
Oct 14, 2024

Conversation

jschwinger233
Copy link
Member

@jschwinger233 jschwinger233 commented Oct 8, 2024

Background

Add bpf test to cover bpf route logic.

This also paves the way to bpf benchmark that guides performance improvement.

Checklist

Full Changelogs

  • [Implement ...]

Issue Reference

Closes #522

Test Result

https://github.com/jschwinger233/dae/actions/runs/11221852069/job/31193035644?pr=23

=== RUN   Test
    bpf_test.go:130: Running test: AndMatch1
    bpf_test.go:130: Running test: AndMatch2
    bpf_test.go:130: Running test: AndMismatch
    bpf_test.go:130: Running test: DportMatch
    bpf_test.go:130: Running test: DportMismatch
    bpf_test.go:130: Running test: DscpMatch
    bpf_test.go:130: Running test: DscpMismatch
    bpf_test.go:130: Running test: IpsetMatch
    bpf_test.go:130: Running test: IpsetMismatch
    bpf_test.go:130: Running test: IpversionMatch
    bpf_test.go:130: Running test: IpversionMismatch
    bpf_test.go:130: Running test: L4protoMatch
    bpf_test.go:130: Running test: L4protoMismatch
    bpf_test.go:130: Running test: MacMatch
    bpf_test.go:130: Running test: MacMismatch
    bpf_test.go:130: Running test: NotMatch
    bpf_test.go:130: Running test: NotMismtach
    bpf_test.go:130: Running test: SourceIpsetMatch
    bpf_test.go:130: Running test: SourceIpsetMismatch
    bpf_test.go:130: Running test: SportMatch
    bpf_test.go:130: Running test: SportMismatch
--- PASS: Test (3.38s)
PASS
ok  	github.com/daeuniverse/dae/control/kern/tests	3.393s

@jschwinger233 jschwinger233 requested review from a team as code owners October 8, 2024 03:49
@mzz2017 mzz2017 added this to the dae 0.9 milestone Oct 9, 2024
Copy link
Contributor

@mzz2017 mzz2017 left a comment

Choose a reason for hiding this comment

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

LGTM! Absolutely exciting work! Thanks

Copy link
Contributor

@dae-prow dae-prow bot left a comment

Choose a reason for hiding this comment

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

🧪 Since the PR has been fully tested, please consider merging it.

@mzz2017 mzz2017 requested a review from sumire88 October 12, 2024 09:47
Copy link
Contributor

@sumire88 sumire88 left a comment

Choose a reason for hiding this comment

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

LGTM

@sumire88 sumire88 merged commit 438c05c into main Oct 14, 2024
31 checks passed
@sumire88 sumire88 deleted the pr/gray/bpf-test branch October 14, 2024 03:21
@dae-prow dae-prow bot mentioned this pull request Nov 3, 2024
@dae-prow dae-prow bot mentioned this pull request Dec 18, 2024
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.

[Enhancement] BPF unit test
3 participants