-
Notifications
You must be signed in to change notification settings - Fork 0
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
Test policer functional tests #92
Conversation
91e11ff
to
c06b414
Compare
...ntOsTestbed/src/dent_os_testbed/test/test_suite/functional/policer/test_policer_ipv4_func.py
Outdated
Show resolved
Hide resolved
...ntOsTestbed/src/dent_os_testbed/test/test_suite/functional/policer/test_policer_ipv4_func.py
Outdated
Show resolved
Hide resolved
DentOS_Framework/DentOsTestbed/src/dent_os_testbed/utils/test_utils/tb_utils.py
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Squash the commits and sign them
...ntOsTestbed/src/dent_os_testbed/test/test_suite/functional/policer/test_policer_ipv4_func.py
Outdated
Show resolved
Hide resolved
...ntOsTestbed/src/dent_os_testbed/test/test_suite/functional/policer/test_policer_ipv4_func.py
Outdated
Show resolved
Hide resolved
DentOS_Framework/DentOsTestbed/src/dent_os_testbed/utils/test_utils/tc_flower_utils.py
Outdated
Show resolved
Hide resolved
8a7f3fe
to
e0e62b0
Compare
f6b1d73
to
5dbc989
Compare
tgen_utils_setup_streams, | ||
tgen_utils_start_traffic, | ||
tgen_utils_stop_traffic, | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
remove space here, add space after )
...d/src/dent_os_testbed/test/test_suite/functional/policer/test_policer_basic_functionality.py
Outdated
Show resolved
Hide resolved
...d/src/dent_os_testbed/test/test_suite/functional/policer/test_policer_basic_functionality.py
Show resolved
Hide resolved
...d/src/dent_os_testbed/test/test_suite/functional/policer/test_policer_basic_functionality.py
Outdated
Show resolved
Hide resolved
...d/src/dent_os_testbed/test/test_suite/functional/policer/test_policer_basic_functionality.py
Outdated
Show resolved
Hide resolved
...tOsTestbed/src/dent_os_testbed/test/test_suite/functional/policer/test_policer_rate_units.py
Show resolved
Hide resolved
...tOsTestbed/src/dent_os_testbed/test/test_suite/functional/policer/test_policer_rate_units.py
Show resolved
Hide resolved
...tOsTestbed/src/dent_os_testbed/test/test_suite/functional/policer/test_policer_rate_units.py
Outdated
Show resolved
Hide resolved
...estbed/src/dent_os_testbed/test/test_suite/functional/policer/test_policer_rules_priority.py
Show resolved
Hide resolved
if qdisc_type == 'port': | ||
for port in port_with_rule: | ||
tc_rule_1['dev'] = port | ||
out = await TcFilter.add(input_data=[{dent: [tc_rule_1]}]) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
list comprehension
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
?
119130f
to
57f2913
Compare
This comment was marked as resolved.
This comment was marked as resolved.
1 similar comment
This comment was marked as resolved.
This comment was marked as resolved.
57f2913
to
2e8c114
Compare
This comment was marked as resolved.
This comment was marked as resolved.
2e8c114
to
3f54bd5
Compare
This comment was marked as resolved.
This comment was marked as resolved.
a5b4fa5
to
4225e79
Compare
Signed-off-by: Stepan Vovk <stepan.vovk@plvision.eu>
4225e79
to
fad110a
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
.
Closed in favor of this #120 |
Adds 2 tests:
Depend on #110 , #109