Merge latest copy of azure master into ignw master#27
Merged
joeslazaro merged 16 commits intoIGNW:masterfrom Aug 28, 2019
Merged
Merge latest copy of azure master into ignw master#27joeslazaro merged 16 commits intoIGNW:masterfrom
joeslazaro merged 16 commits intoIGNW:masterfrom
Conversation
* [vlan_trunk] stabilize vlan trunk test
- use 'config' commands to brind down/up interfaces
- generate 'Vlan<VID>': {} in config to specify this interface belongs to global vrf as it is required (TODO: use config CLI instead);
- wait longer time after LAG shutdown;
Test tries to configure LAG as part of VLAN interface which will fail if LAG is a RIF.
Since RIF in SONiC is removed only when the ref_count becomes 0, we'll have to wait for orchagent
to remove all routes, next hop groups, neighbors that reference those LAG RIFs.
The time is required is not very deterministic and can differ between platforms but it is assumed 90 sec is enough.
One other way to do this test is to create seperate PortChannels.
Also, it looks that master has significantly slower performence to remove routes, next hops then 201811 branch.
Signed-off-by: Stepan Blyschak <stepanb@mellanox.com>
* Revert delay before RIF remove back to 10s
* [vlan_trunk] Increase delay before RIF remove
* Enhance vnet_vxlan for scaling of vnets and routes * Enhance vnet_vxlan for scaling of vnets and routes * incorported changes for ipv6 * Modified to select fewer routes in case of scaling * Corrected num_routes_batch variable * Modified the logic in choosing one route from the list for each vnet * Corrected the IP in local routes * Changes to limit the maximum RIFs to 128
…d intf (#1028) When executing the config test case, it has the possibility to cause swss exit. It is due to the intfOrch may be slower than portOrch When the issue not happens, the timeline would be 1. Playbook remove IP from port channel intfOrch want to remove l3 intf, but neighOrch still have reference and the intfOrch will wait to next run 2. neighOrch decrease the reference count neighOrch finish the reference count decrement and then intfOrch remove l3 intf to SAI 3. Playbook remove port channel portOrch remove this port in SAI When the issue happens, the timeline would be 1. Playbook remove IP from port channel intfOrch want to remove l3 intf, but neighOrch still have reference and the intfOrch will wait to next run 2. Playbook remove port channel portOrch remove this port and its l3 intf in SAI 3. neighOrch decrease the reference count neighOrch finish the reference count decrement and then intfOrch remove l3 intf to SAI whcih was removed by step 2. Then the swss will exception
1. Do reboot before the test 2. Send 10 packets instead of 1 packets
* Add lag flap check after warm boot Signed-off-by: Neetha John <nejo@microsoft.com>
Signed-off-by: Neetha John <nejo@microsoft.com>
* Output offset of ports in data structures * Warmup DUT before the test * Increase wating time from 0.2 to 0.5 * Quote minigraph_vlans otherwise ansible could remove this variable
… address robustly (#1078) * Use unquoted style of the variable usage * Make the fact cache valid for a day * Rewrite mapping between vlan ports and vlan ip addresses
* Add setting of RCVBUF default parameter * MAke default smaller - 31Mb
…1077) Mallnox uses a different GRE protocol 0x8949 and crafts extra information in the packet that we need to take care of specifically. Signed-off-by: Shu0T1an ChenG <shuche@microsoft.com>
… infra (#1072) * [pytest] Improve infra and update the platform scripts to use the new infra Improvements: * Add testbed_devices fixture * Add common folder for reusable libraries * Use OO design to abstract and encapsulate behavior for various devices in testbed * Improve code reuse by the OO design and common libraries * Add support to run ansible module asynchronously * In devices.AnsibleHostBase, use the improvement of module_ignore_errors by dawnbeauty * Update the platform scripts to use the improved pytest infrastructure * Add a few Mellanox specific testing scripts for completeness and demonstration * Correct the assert statement * Use pre-gathered hwsku facts * Correct mellanox switch profile data * Skip checking fanX_status when fan module is not hot swappable
The command for rebooting VM was executed synchronously. Sometimes it could fail because the VM closed the SSH connection before ansible finishes the reboot command. The fix is to make the command async and ignore its errors. Signed-off-by: Xin Wang <xinw@mellanox.com>
IPv6 default route is learned and has the same behavior as the IPv4 default route. Signed-off-by: Shu0T1an ChenG <shuche@microsoft.com>
* [lag2] enhance lag2-minlink test Change-Id: I165d6ae8bc5ab2100b9d26c1d8c7ad02f0252cd8 Signed-off-by: Mykola Faryma <mykolaf@mellanox.com> * [lag2][minlnk] add option to specify max waiting time Change-Id: Ia0331f6cb78accbbb7019f30ec8b98f371cd8d98 Signed-off-by: Mykola Faryma <mykolaf@mellanox.com>
* Added loganalyzer for pytest. Updated existed loganalyzer. * Added loganalyzer for pytest. Updated existed loganalyzer. * Added fixes based on comments * Update README.md * Update README.md * Added minor fixes based on comments. Signed-off-by: Yuriy Volynets <yuriyv@mellanox.com> * Added minor fix. Signed-off-by: Yuriy Volynets <yuriyv@mellanox.com> * Added 'disable_loganalyzer' marker
* Preboot sad path automation for n lag members Signed-off-by: Neetha John <nejo@microsoft.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
11 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.