Skip to content
New issue

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

#

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

Already on GitHub? # to your account

Jump to bottom

Continue testing... #3

Merged
merged 15 commits into from
Jun 1, 2022
Merged

Continue testing... #3

merged 15 commits into from
Jun 1, 2022

Conversation

leodido
Copy link
Owner

@leodido leodido commented May 31, 2022

No description provided.

fntlnz
fntlnz approved these changes Jun 1, 2022
View reviewed changes
Copy link
Collaborator

@fntlnz fntlnz left a comment

Choose a reason for hiding this comment

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

LGTM but please add the new programs to the readme!

leodido and others added 14 commits June 1, 2022 14:39
@leodido
new(test): create helper to setup netns and networking
e232ae9 
Signed-off-by: Leonardo Di Donato <leodidonato@gmail.com>
@leodido
new(test): create helper for setting the suite name starting from the…
7952f61 
… test file name

The convention is to use the first part of the test's filenames.
Eg., `cli.flags.bats` is `cli` test suite.

Signed-off-by: Leonardo Di Donato <leodidonato@gmail.com>
@leodido
new(test): test attaching the nop program at ingress and egress
4c326cb 
Signed-off-by: Leonardo Di Donato <leodidonato@gmail.com>
@leodido
build: make the tests execute with root privileges
dcd0273 
Signed-off-by: Leonardo Di Donato <leodidonato@gmail.com>
@leodido
update(test): test the block_private_ipv4 blocks ICMP packets
2b98f89 
Signed-off-by: Leonardo Di Donato <leodidonato@gmail.com>
@leodido @fntlnz
update(test): setup initial attach test for traffico-cni
7f4bb4f 
Co-authored-by: Lorenzo Fontana <lo@linux.com>
Signed-off-by: Leonardo Di Donato <leodidonato@gmail.com>
@leodido
update(test): check that block_private_ipv4 TC BPF programs blocks fr…
8c0ca20 
…om reaching private (10.x.x.x) addresses

Signed-off-by: Leonardo Di Donato <leodidonato@gmail.com>
@leodido
build: make test depend on mini_httpd
770c74a 
Signed-off-by: Leonardo Di Donato <leodidonato@gmail.com>
@leodido
new(bpf): add block_ip program
5237fb5 
Signed-off-by: Leonardo Di Donato <leodidonato@gmail.com>
@leodido
refactor: rename struct args into struct config
b774bfb 
Signed-off-by: Leonardo Di Donato <leodidonato@gmail.com>
@leodido
new: introduce flag to explicitly do not detach/destroy TC hook and p…
6d81f9b 
…rogram at the exit

Signed-off-by: Leonardo Di Donato <leodidonato@gmail.com>
@leodido
update(api): make the api detach the program from the TC qdisc only w…
f353d43 
…hen cleanup_on_exit is true

Signed-off-by: Leonardo Di Donato <leodidonato@gmail.com>
@leodido
new(test): test that --no-cleanup works
6cc9df2 
Signed-off-by: Leonardo Di Donato <leodidonato@gmail.com>
@leodido
docs: update README
167815f 
Signed-off-by: Leonardo Di Donato <leodidonato@gmail.com>
@fntlnz @leodido
update(.devcontainer): run privileged
eeba571 
This allows us to test traffico and traffico-cni on a devcontainer/codespace

Co-Authored-By: Leonardo Di Donato <leodidonato@gmail.com>
@leodido leodido merged commit 66437f5 into main Jun 1, 2022
@leodido leodido deleted the new/test2 branch June 1, 2022 13:51
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@fntlnz fntlnz fntlnz approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@leodido @fntlnz