* [PATCH] tests: shell: add testcases for named objects @ 2017-10-05 10:15 Harsha Sharma 2017-10-06 12:38 ` Pablo Neira Ayuso 0 siblings, 1 reply; 6+ messages in thread From: Harsha Sharma @ 2017-10-05 10:15 UTC (permalink / raw) To: pablo; +Cc: netfilter-devel, outreachy-kernel, Harsha Sharma Add testcases for creating named objects with unique name, defined by user and referencing them from rule. Signed-off-by: Harsha Sharma <harshasharmaiitr@gmail.com> --- tests/shell/testcases/sets/0024named_objects_0 | 41 ++++++++++++++++++++++++++ 1 file changed, 41 insertions(+) create mode 100755 tests/shell/testcases/sets/0024named_objects_0 diff --git a/tests/shell/testcases/sets/0024named_objects_0 b/tests/shell/testcases/sets/0024named_objects_0 new file mode 100755 index 0000000..de02d65 --- /dev/null +++ b/tests/shell/testcases/sets/0024named_objects_0 @@ -0,0 +1,41 @@ +#!/bin/bash + +# This is the testscase: +# * creating valid named objects +# * referencing them from a valid rule + +tmpfile=$(mktemp) +if [ ! -w $tmpfile ] ; then + echo "Failed to create tmp file" >&2 + exit 0 +fi + +trap "rm -rf $tmpfile" EXIT # cleanup if aborted + +echo " +table inet x { + counter user123 { + packets 12 bytes 1433 + } + quota user123 { + over 2000 bytes + } + quota user124 { + over 2000 bytes + } + set y { + type ipv4_addr + } + map test { + type ipv4_addr : quota + elements = { 192.168.2.2 : "user124", 192.168.2.3 : "user124"} + } + chain y { + type filter hook input priority 0; policy accept; + counter name ip saddr map { 192.168.2.2 : "user123", 1.1.1.1 : "user123", 2.2.2.2 : "user123"} + quota name ip saddr map @test drop + } +}" > $tmpfile + +set -e +$NFT -f $tmpfile -- 2.11.0 ^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH] tests: shell: add testcases for named objects 2017-10-05 10:15 [PATCH] tests: shell: add testcases for named objects Harsha Sharma @ 2017-10-06 12:38 ` Pablo Neira Ayuso 2017-10-07 7:41 ` Harsha Sharma 0 siblings, 1 reply; 6+ messages in thread From: Pablo Neira Ayuso @ 2017-10-06 12:38 UTC (permalink / raw) To: Harsha Sharma; +Cc: netfilter-devel, outreachy-kernel On Thu, Oct 05, 2017 at 03:45:39PM +0530, Harsha Sharma wrote: > Add testcases for creating named objects with unique name, defined > by user and referencing them from rule. Also applied, thanks. Could you add another tests for 'limit' objects too? I think I posted an example to the mailing list. ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] tests: shell: add testcases for named objects 2017-10-06 12:38 ` Pablo Neira Ayuso @ 2017-10-07 7:41 ` Harsha Sharma 2017-10-09 12:02 ` Pablo Neira Ayuso 0 siblings, 1 reply; 6+ messages in thread From: Harsha Sharma @ 2017-10-07 7:41 UTC (permalink / raw) To: Pablo Neira Ayuso; +Cc: netfilter-devel, outreachy-kernel On Fri, Oct 6, 2017 at 6:08 PM, Pablo Neira Ayuso <pablo@netfilter.org> wrote: > On Thu, Oct 05, 2017 at 03:45:39PM +0530, Harsha Sharma wrote: >> Add testcases for creating named objects with unique name, defined >> by user and referencing them from rule. > > Also applied, thanks. > > Could you add another tests for 'limit' objects too? I think I posted > an example to the mailing list. Hello, I have pulled to latest tree, then make and make installed all changes but still I'm unable to add named 'limit' objects. If there is something that I'm missing, please let me know. Can you also please suggest some more tasks ? Thanks in advance. Regards, Harsha Sharma ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] tests: shell: add testcases for named objects 2017-10-07 7:41 ` Harsha Sharma @ 2017-10-09 12:02 ` Pablo Neira Ayuso 2017-10-09 12:33 ` Harsha Sharma 2017-10-10 20:40 ` Harsha Sharma 0 siblings, 2 replies; 6+ messages in thread From: Pablo Neira Ayuso @ 2017-10-09 12:02 UTC (permalink / raw) To: Harsha Sharma; +Cc: netfilter-devel, outreachy-kernel On Sat, Oct 07, 2017 at 01:11:17PM +0530, Harsha Sharma wrote: > On Fri, Oct 6, 2017 at 6:08 PM, Pablo Neira Ayuso <pablo@netfilter.org> wrote: > > On Thu, Oct 05, 2017 at 03:45:39PM +0530, Harsha Sharma wrote: > >> Add testcases for creating named objects with unique name, defined > >> by user and referencing them from rule. > > > > Also applied, thanks. > > > > Could you add another tests for 'limit' objects too? I think I posted > > an example to the mailing list. > > Hello, > I have pulled to latest tree, then make and make installed all changes > but still I'm unable to add named 'limit' objects. If there is something that > I'm missing, please let me know. Are you using a kernel compiled from nf.git tree? > Can you also please suggest some more tasks ? Yes. It would be great if you could update tests/py/nft-tests.py to add a new option, basically to take all tests from files, place them in a file and exercise the 'nft -f' path with such an autogenerated large file with all tests. Would you have a look into that? Thanks. ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] tests: shell: add testcases for named objects 2017-10-09 12:02 ` Pablo Neira Ayuso @ 2017-10-09 12:33 ` Harsha Sharma 2017-10-10 20:40 ` Harsha Sharma 1 sibling, 0 replies; 6+ messages in thread From: Harsha Sharma @ 2017-10-09 12:33 UTC (permalink / raw) To: Pablo Neira Ayuso; +Cc: netfilter-devel, outreachy-kernel On Mon, Oct 9, 2017 at 5:32 PM, Pablo Neira Ayuso <pablo@netfilter.org> wrote: > On Sat, Oct 07, 2017 at 01:11:17PM +0530, Harsha Sharma wrote: >> On Fri, Oct 6, 2017 at 6:08 PM, Pablo Neira Ayuso <pablo@netfilter.org> wrote: >> > On Thu, Oct 05, 2017 at 03:45:39PM +0530, Harsha Sharma wrote: >> >> Add testcases for creating named objects with unique name, defined >> >> by user and referencing them from rule. >> > >> > Also applied, thanks. >> > >> > Could you add another tests for 'limit' objects too? I think I posted >> > an example to the mailing list. >> >> Hello, >> I have pulled to latest tree, then make and make installed all changes >> but still I'm unable to add named 'limit' objects. If there is something that >> I'm missing, please let me know. > > Are you using a kernel compiled from nf.git tree? No, I was using kernel 4.10 but I have build the kernel 4.14 from nf-git tree now. > >> Can you also please suggest some more tasks ? > > Yes. It would be great if you could update tests/py/nft-tests.py to > add a new option, basically to take all tests from files, place them > in a file and exercise the 'nft -f' path with such an autogenerated > large file with all tests. > > Would you have a look into that? > Yes, definitely. Thanks for your time :). Regards, Harsha Sharma > Thanks. ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] tests: shell: add testcases for named objects 2017-10-09 12:02 ` Pablo Neira Ayuso 2017-10-09 12:33 ` Harsha Sharma @ 2017-10-10 20:40 ` Harsha Sharma 1 sibling, 0 replies; 6+ messages in thread From: Harsha Sharma @ 2017-10-10 20:40 UTC (permalink / raw) To: Pablo Neira Ayuso, netfilter-devel On Mon, Oct 9, 2017 at 5:32 PM, Pablo Neira Ayuso <pablo@netfilter.org> wrote: > On Sat, Oct 07, 2017 at 01:11:17PM +0530, Harsha Sharma wrote: >> On Fri, Oct 6, 2017 at 6:08 PM, Pablo Neira Ayuso <pablo@netfilter.org> wrote: >> > On Thu, Oct 05, 2017 at 03:45:39PM +0530, Harsha Sharma wrote: >> >> Add testcases for creating named objects with unique name, defined >> >> by user and referencing them from rule. >> > >> > Also applied, thanks. >> > >> > Could you add another tests for 'limit' objects too? I think I posted >> > an example to the mailing list. >> >> Hello, >> I have pulled to latest tree, then make and make installed all changes >> but still I'm unable to add named 'limit' objects. If there is something that >> I'm missing, please let me know. > > Are you using a kernel compiled from nf.git tree? > >> Can you also please suggest some more tasks ? > > Yes. It would be great if you could update tests/py/nft-tests.py to > add a new option, basically to take all tests from files, place them > in a file and exercise the 'nft -f' path with such an autogenerated > large file with all tests. > > Would you have a look into that? > Hi, I have one doubt, nft-tests.py when executed without any argument or with option -d , already runs all commands in all test files ( don't directly run nft -f path as according to structure of tests in py, for lines starting with '*', table is added and similarly for ':', chain is added and similarly for objects, sets, etc.) so directly exercising 'nft -f' path on autogenerated file containing content of all tests files doesn't make sense. And calling function run_test_file for the large autogenerated file (as is called for each test file) will not make a lot of difference in result (compared to when executed without any argument) other than not printing total no. of test files executed and total no. of files passed. I hope I'm clear. Please correct me I'm wrong. What is exactly that I have to do in this task ? Thanks a lot for your time. Regards, Harsha Sharma > Thanks. ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2017-10-10 20:40 UTC | newest] Thread overview: 6+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2017-10-05 10:15 [PATCH] tests: shell: add testcases for named objects Harsha Sharma 2017-10-06 12:38 ` Pablo Neira Ayuso 2017-10-07 7:41 ` Harsha Sharma 2017-10-09 12:02 ` Pablo Neira Ayuso 2017-10-09 12:33 ` Harsha Sharma 2017-10-10 20:40 ` Harsha Sharma
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).