Age | Commit message (Collapse) | Author | Files | Lines |
|
The old limit of 1522 was introduced long time ago.
First, it appeared here [0], where it is correct
as that suite has zero overhead.
The first suite with wrong logic seems to be here [1]
(no "Add No Multi Seg to all DUTs" in 1518B testcase).
And when I was moving that logic to a keyword in [2],
I did not realize it is wrong with overhead.
This Change uses 1800 as the new threshold,
matching the value used for non-jumbo MTU.
[0] https://gerrit.fd.io/r/c/csit/+/2652/12/tests/perf/Bridge_Domain_Intel-X520-DA2.robot#70
[1] https://gerrit.fd.io/r/c/csit/+/4454/96/tests/perf/40ge2p1xl710-ethip4ipsecscaleip4-ip4base-interfaces-aes-gcm-ndrpdrdisc.robot#229
[2] https://gerrit.fd.io/r/c/csit/+/13411/35/resources/libraries/robot/performance/performance_utils.robot#84
Change-Id: Iff3703fcff0e4bbb1a6b10be359fa5ef67fd5422
Signed-off-by: Vratko Polak <vrpolak@cisco.com>
(cherry picked from commit 55bdf4cc0046617100676685d1f5ef075167af1d)
|
|
Signed-off-by: Peter Mikus <peter.mikus@protonmail.ch>
Change-Id: I5ca0fc5832c249553c79dd9c5d4148ba64e61654
|
|
Signed-off-by: Peter Mikus <peter.mikus@protonmail.ch>
Change-Id: Icbcd373ef815fa082d9be022fc201c96df140536
|
|
Some perf suites use a single unidirectional flow of traffic.
On SMT-enabled machines, 1C tests create 2 workers by default.
But if there is only one stream/flow,
only one worker does useful work.
The other worker on the sibling logical core
does busy-wait, which provably lowers the peformance.
Some suites already set smt_used (as a suite variable)
to False for these purposes.
This change edits a lower-level L1 keyword to honor this value,
and adds the suite variable to single stream (and client) hoststack
suites, where the detrimental effect of busy siblings was observed.
Change-Id: I203f1a700d8e6cd84640b9c0b40487cb832908aa
Signed-off-by: Vratko Polak <vrpolak@cisco.com>
|
|
Only related to enums appearing in ipsec suites.
+ Add conversion utility as enum_utils.py file.
+ Rename PolicyAction closer to ipsec_types.api: IpsecSpdAction.
+ Device and perf suites updated to use str in Variables table.
- Perf suites are being tested.
Change-Id: I3b1678c4d6cc303c590e5e3665ab1b05d104a121
Signed-off-by: Vratko Polak <vrpolak@cisco.com>
|
|
Change-Id: I48ec272fb150c09221142b8d73830d7b6741d0b5
Signed-off-by: Kai Ji <kai.ji@intel.com>
|
|
This reverts commit 79635f7cdfb7046230473570068f97a159cf87b4.
Peter says he is using those.
Change-Id: I47886ba0676bc74d5189a2986ac347f49fdde36c
Signed-off-by: Vratko Polak <vrpolak@cisco.com>
|
|
When deleting ipsec nvf density suites,
I removed the main L2 Robot keyword,
but forgot to remove the associated L1 Python keywords.
Change-Id: I0f72699dcab272aec5c531cdaacceed30356cd02
Signed-off-by: Vratko Polak <vrpolak@cisco.com>
|
|
They are broken for more than two years
and they relied on CLIs that changed in VPP anyway.
Ticket: CSIT-1856
Change-Id: I0f278ff61a9da5e6040e08bf3e92049cb33c8a93
Signed-off-by: Vratko Polak <vrpolak@cisco.com>
|
|
+ Update docstrings to match.
Change-Id: I20f8f0247a9cf38a2d9a98fa8905eb55d91817f7
Signed-off-by: Vratko Polak <vrpolak@cisco.com>
|
|
+ One more pylint improvement not worth a separate change.
Change-Id: I5a2a3a4c70909fefd3fdf6f3ed8b344d19fc823e
Signed-off-by: Vratko Polak <vrpolak@cisco.com>
|
|
+ Fix typo "enale".
- Many style and complexity issues remain:
C0302, R0904, R0914, R0913, R1735, R0915, R1734.
Change-Id: I7e8c7ecefd8f8f94dcbc28427baa52a9c8ea1227
Signed-off-by: Vratko Polak <vrpolak@cisco.com>
|
|
Change-Id: Idef14bef816a04ca43ab9aced4b1441ef818e066
Signed-off-by: xinfengx <xinfengx.zhao@intel.com>
|
|
Signed-off-by: Peter Mikus <peter.mikus@protonmail.ch>
Change-Id: I9bc9f9fe410c0fa78e20a234487e63eb42f3fe3f
|
|
Signed-off-by: Peter Mikus <peter.mikus@protonmail.ch>
Change-Id: I48240d2df2ee7908d07da86a4e4da359178d0b9e
|
|
+ Separate sad into two cycles.
+ Do not update flags in each iteration.
+ Update inner dicts.
Change-Id: Ic81dfc6e614cc78ff321fba748d48a30678cc88a
Signed-off-by: Vratko Polak <vrpolak@cisco.com>
|
|
It uses rather simple sed commands, but seems to work well enough.
+ Improve readme.
+ Shortened comments about why 3 lines.
Change-Id: I50b395dfeb586f671f7c4c82f22369de90b351ec
Signed-off-by: Vratko Polak <vrpolak@cisco.com>
|
|
Change-Id: I7cc2bc69c6849360abff0bd18b2ae3b93e2303c4
Signed-off-by: Vratko Polak <vrpolak@cisco.com>
|
|
+ gtpu_add_del_tunnel_v2
+ Add comments on used values and unused fields.
+ ipsec_sad_entry_add_v2
+ Explicitly pass current default values.
+ ipsec_sa_v5_dump
+ policer_add
+ The old is_add argument removed, it was never false.
+ sr_policy_add_v2
+ Add comments about currently unused fields.
+ Support also older VP builds with wrong reply.
+ rdma_create_v4
+ Add comments about unused fields.
Change-Id: I3d5bc345c4cf099661626770c4d86bc230643cca
Signed-off-by: Vratko Polak <vrpolak@cisco.com>
|
|
Signed-off-by: Peter Mikus <peter.mikus@protonmail.ch>
Change-Id: I720b7c96a6ac9f328aff57437e51254364604911
|
|
Change-Id: Id68a4e2b7514b9cf2e1f2f30b78dc0cd696a673a
Signed-off-by: Vratko Polak <vrpolak@cisco.com>
|
|
Signed-off-by: Peter Mikus <peter.mikus@protonmail.ch>
Change-Id: I7f640aa28ecf3a60696676b0cf8afb7a080e1b12
|
|
This reverts commit ffca8b8655c772fc6273702cae2151e7ac7a846d.
+ Add 10 second sleep, could help if core is big.
Change-Id: Id2b3199737f8c50cee713e8626dce1bfa72ea4e1
Signed-off-by: Vratko Polak <vrpolak@cisco.com>
|
|
Signed-off-by: Peter Mikus <peter.mikus@protonmail.ch>
Change-Id: I84d9cf2892423e62b116b07982d3e87120b66271
|
|
Update the jobspec according to the current Arm policy:
1. Run most of the tests on cx6, crypto coverage being the most
important,
2. Match the cx6 ipv4 crypto coverage with x710 with the DPDK driver,
3. Run basic ipv4 and l2 tests with x710 with borh DPDK and native (avf)
drivers.
The above is the policy used in performance testing where time is a
major constraint. The same is true for functional testing to some
extent, so it makes sense to apply the same logic for choosing the tests.
There are 10 tests which are failing on cx6 without an identified
reason. They are disabled until we find the root cause and address it.
In the meantime, the tests are enabled for x710 with the DPDK driver:
ethip4-l2xcbase-eth-2memif-1dcr
ethip4-l2bdbasemaclrn-eth-2memif-1dcr
ethip4-l2bdbasemaclrn-eth-2tap
ethip4-l2bdbasemaclrn
ethip4-l2bdbasemaclrn-iacl1sf
ethip4-l2bdbasemaclrn-iacl1sl
ethip4-l2bdbasemaclrn-oacl1sf
ethip4-l2bdbasemaclrn-oacl1sl
ethip4-l2patch
ethip4-l2xcbase
When the cx6 issue has been resolved, these should be enabled for cx6
and disabled for x710, with the exception of ethip4-l2bdbasemaclrn,
which should be running on both.
Change-Id: Id48f450063e405b1faca57431676f91d62e829b9
Signed-off-by: Juraj Linkeš <juraj.linkes@pantheon.tech>
|
|
Signed-off-by: pmikus <peter.mikus@protonmail.ch>
Change-Id: I131cc9957b288f4e02e9d02394a1681dbb539d83
|
|
Signed-off-by: Peter Mikus <peter.mikus@protonmail.ch>
Change-Id: Ib2a1ceb9145ef5495c76954ac70cedc5af6fa297
|
|
Signed-off-by: Peter Mikus <peter.mikus@protonmail.ch>
Change-Id: Ic27f46876a93cc01d5f14bbefe0dccd84ab41027
|
|
Signed-off-by: Peter Mikus <peter.mikus@protonmail.ch>
Change-Id: Ia6e07111af543280b2365840bced02f94cf91e75
|
|
Signed-off-by: Peter Mikus <peter.mikus@protonmail.ch>
Change-Id: I78501f08b29a202c0e1129b79af6a63299e8ee36
|
|
As the skipping logic looks at available topologies,
fix skipping by adding virtual to vagrant.yaml
Change-Id: I5e7bf1dc2f4f4787917f1f989a974255cf2591a9
Signed-off-by: Vratko Polak <vrpolak@cisco.com>
|
|
Change-Id: I8c27b5956c0c26bb2de48321675e285ed961412f
Signed-off-by: xinfengx <xinfengx.zhao@intel.com>
|
|
Signed-off-by: Peter Mikus <peter.mikus@protonmail.ch>
Change-Id: Idca0738320b56196e98aeec3050312111a90609a
|
|
This should fix some suites wrongly using nic_pfs=6.
Change-Id: Ifb7bd24b9a2d1764634eea4a43b4509f16e03da2
Signed-off-by: Vratko Polak <vrpolak@cisco.com>
|
|
Change-Id: Icaa88523ede0b1fad6e6cb667a8f307ea5f4a12d
Signed-off-by: Vratko Polak <vrpolak@cisco.com>
|
|
- Complexity and logging would require a larger change.
Change-Id: I69f8e47b1806c880cd626bc5a8f5ae6290246093
Signed-off-by: Vratko Polak <vrpolak@cisco.com>
|
|
Change-Id: I26e0ce172740f7f440469578294f8c13fec5850b
Signed-off-by: Vratko Polak <vrpolak@cisco.com>
|
|
Signed-off-by: Peter Mikus <peter.mikus@protonmail.ch>
Change-Id: I1770a3f49e19d337d4ed6760fb062ba4d445574d
|
|
Change-Id: I7c2426fe47ca9ee44338423597be90a71aaa3653
Signed-off-by: Vratko Polak <vrpolak@cisco.com>
|
|
Even when VPP switches back to (or adds support for) the old behavior,
this enables bisect to work for all VPP commits in 2402 cycle.
Change-Id: Ic2b71bee1db723d9d13229d9f4288d1699d246c6
Signed-off-by: Vratko Polak <vrpolak@cisco.com>
|
|
+ Post-release rdma_create_v4 will be added next cycle.
Change-Id: Ief129b5be4a2c5d27f4674491c96c548573ecfb1
Signed-off-by: Vratko Polak <vrpolak@cisco.com>
|
|
Signed-off-by: Peter Mikus <peter.mikus@protonmail.ch>
Change-Id: I746471b59e1fb00a00007b6e9f2ba02d873ae3e2
|
|
+ Parsing common with per-patch job is moved to a library.
Ticket: CSIT-1618
Change-Id: I185bea084a29e6a37ef94e9da42b192a6a81fc17
Signed-off-by: Vratko Polak <vrpolak@cisco.com>
|
|
Ticket: CSIT-1618
Change-Id: I0c17f1e8f09a3944032d44efa8ecd36fef2056a2
Signed-off-by: Vratko Polak <vrpolak@cisco.com>
|
|
In past, there perhaps were Robot keywords needing JsonParser.
But now the single remaining call site can call json.loads directly.
+ Delete now unused resources/libraries/python/parsers/.
Change-Id: I9c5d94d6b320f917f1f621abb83556ebccadc3ec
Signed-off-by: Vratko Polak <vrpolak@cisco.com>
|
|
- terraform ansible is not supported on aarch
Signed-off-by: Peter Mikus <peter.mikus@protonmail.ch>
Change-Id: I3d1a592b4ed97fe635813dccbf78dc42d14e0fa9
|
|
Signed-off-by: Peter Mikus <peter.mikus@protonmail.ch>
Change-Id: Ia713b9a0b962820b014bad3582bc095ec802380c
|
|
Fixes: 76fcd8422842d0756338502213de561eaa93e670
Change-Id: Ie642aeed5c55d479e0e4b87be965b4b5f70d6612
Signed-off-by: Vratko Polak <vrpolak@cisco.com>
|
|
Signed-off-by: pmikus <peter.mikus@protonmail.ch>
Change-Id: I862f9ca12e9149eab2d0701368cdc65f294629cc
|
|
Signed-off-by: pmikus <peter.mikus@protonmail.ch>
Change-Id: If25495e0eec60ac84dfaaafec3080f02a17fc740
|